projects / platform / kernel / linux-starfive.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: f76b6ef) | patch
net: dsa: bcm_sf2: Pass GENMASK() signed bits
authorAndrew Lunn <andrew@lunn.ch>
Sun, 5 Jul 2020 20:36:25 +0000 (22:36 +0200)
committerDavid S. Miller <davem@davemloft.net>
Sun, 5 Jul 2020 22:45:38 +0000 (15:45 -0700)
commitc226e2716ee25ab8e6b868559be00a1c30282486
tree958bb6416b97e780b1086ed93b6b2015b9391bd9tree | snapshot
parentf76b6ef11c5723e840c1fc49587d267297352e9ecommit | diff
net: dsa: bcm_sf2: Pass GENMASK() signed bits

Oddly, GENMASK() requires signed bit numbers, so that it can compare
them for < 0. If passed an unsigned type, we get warnings about the
test never being true. There is no danger of overflow here, udf is
always a u8, so there is plenty of space when expanding to an int.

Signed-off-by: Andrew Lunn <andrew@lunn.ch>
Reviewed-by: Florian Fainelli <f.fainelli@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/dsa/bcm_sf2_cfp.c diff | blob | history
Domain: System / Kernel;