commit | c226e2716ee25ab8e6b868559be00a1c30282486 | [log] [tgz] |
---|---|---|
author | Andrew Lunn <andrew@lunn.ch> | Sun Jul 05 22:36:25 2020 +0200 |
committer | David S. Miller <davem@davemloft.net> | Sun Jul 05 15:45:38 2020 -0700 |
tree | 958bb6416b97e780b1086ed93b6b2015b9391bd9 | |
parent | f76b6ef11c5723e840c1fc49587d267297352e9e [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>