On 7/5/2020 1:36 PM, Andrew Lunn wrote: > A __be16 variable should be initialised with a __be16 value. So add a > htons(). In this case it is pointless, given the value being assigned > is 0xffff, but it stops sparse from warnings. > > Signed-off-by: Andrew Lunn <and...@lunn.ch> The subject should be: net: dsa: bcm_sf2: Initialize __be16 with a __be16 value to be consistent with other commits done to the same file, with that: Reviewed-by: Florian Fainelli <f.faine...@gmail.com> -- Florian
- [PATCH net-next 0/3] dsa: b53/sf2 Andrew Lunn
- [PATCH net-next 3/3] dsa: bmc_sf2: Pass GENMASK() si... Andrew Lunn
- Re: [PATCH net-next 3/3] dsa: bmc_sf2: Pass GENM... Florian Fainelli
- [PATCH net-next 2/3] dsa: bcm_sf2: Initialize __be16... Andrew Lunn
- Re: [PATCH net-next 2/3] dsa: bcm_sf2: Initializ... Florian Fainelli
- [PATCH net-next 1/3] net: dsa: b53: Fixup endianness... Andrew Lunn
- Re: [PATCH net-next 1/3] net: dsa: b53: Fixup en... Florian Fainelli
- Re: [PATCH net-next 0/3] dsa: b53/sf2 David Miller