Re: [PATCH] net: dsa: sja1105: fix speed setting for 10 MBPS

2020-05-02 Thread Sergei Shtylyov
Hello! On 01.05.2020 16:43, Colin King wrote: From: Colin Ian King The current logic for speed checking will never set the speed to 10 MBPS because bmcr & BMCR_SPEED10 is always 0 since BMCR_SPEED10 is 0. Also the erroneous setting where BMCR_SPEED1000 and BMCR_SPEED100 are both set causes th

Re: [PATCH] net: dsa: sja1105: fix speed setting for 10 MBPS

2020-05-01 Thread Russell King - ARM Linux admin
On Fri, May 01, 2020 at 06:00:52PM +, Walter Harms wrote: > IMHO it would be better to use switch case here to improve readability. > > switch (bmcr & mask) { > > case BMCR_SPEED1000: > speed = SPEED_1000; > break; > case BMC

AW: [PATCH] net: dsa: sja1105: fix speed setting for 10 MBPS

2020-05-01 Thread Walter Harms
.org; netdev@vger.kernel.org Betreff: [PATCH] net: dsa: sja1105: fix speed setting for 10 MBPS From: Colin Ian King The current logic for speed checking will never set the speed to 10 MBPS because bmcr & BMCR_SPEED10 is always 0 since BMCR_SPEED10 is 0. Also the erroneous setting where BMCR_SPEED

[PATCH] net: dsa: sja1105: fix speed setting for 10 MBPS

2020-05-01 Thread Colin King
From: Colin Ian King The current logic for speed checking will never set the speed to 10 MBPS because bmcr & BMCR_SPEED10 is always 0 since BMCR_SPEED10 is 0. Also the erroneous setting where BMCR_SPEED1000 and BMCR_SPEED100 are both set causes the speed to be 1000 MBS. Fix this by masking bps a