On 7/5/2020 1:36 PM, Andrew Lunn wrote:
> 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 <and...@lunn.ch>
Likewise for this patch, the subject should be:
net: dsa: bcm_sf2: Pass GENMASK() signed bits
(not the typo on bmc_sf2 vs. bcm_sf2), with that fixed:
Reviewed-by: Florian Fainelli <f.faine...@gmail.com>
--
Florian
