Re: [PATCH 1/2] net: dsa: microchip: set the correct number of ports in dsa_switch

2020-07-01 Thread Codrin.Ciubotariu
On 01.07.2020 20:09, Andrew Lunn wrote: > EXTERNAL EMAIL: Do not click links or open attachments unless you know the > content is safe > > On Wed, Jul 01, 2020 at 07:51:27PM +0300, Codrin Ciubotariu wrote: >> The number of ports is incorrectly set to the maximum available for a DSA >> switch. Eve

[PATCH 1/2] net: dsa: microchip: set the correct number of ports in dsa_switch

2020-07-01 Thread Codrin Ciubotariu
The number of ports is incorrectly set to the maximum available for a DSA switch. Even if the extra ports are not used, this causes some functions to be called later, like port_disable() and port_stp_state_set(). If the driver doesn't check the port index, it will end up modifying unknown registers

Re: [PATCH 1/2] net: dsa: microchip: set the correct number of ports in dsa_switch

2020-07-01 Thread Andrew Lunn
On Wed, Jul 01, 2020 at 07:51:27PM +0300, Codrin Ciubotariu wrote: > The number of ports is incorrectly set to the maximum available for a DSA > switch. Even if the extra ports are not used, this causes some functions > to be called later, like port_disable() and port_stp_state_set(). If the > driv