Re: [PATCH net-next v2 1/7] net: phy: at803x: Avoid comparison is always false warning

[Florian Fainelli]

On 7/6/2020 6:49 PM, Andrew Lunn wrote:
> By placing the GENMASK value into an unsigned int and then passing it
> to PREF_FIELD, the type is reduces down from ULL. Given the reduced
> size of the type, the range checks in PREP_FAIL() are always true, and
> -Wtype-limits then gives a warning.
> 
> By skipping the intermediate variable, the warning can be avoided.
> 
> Reviewed-by: Jakub Kicinski <k...@kernel.org>
> Signed-off-by: Andrew Lunn <and...@lunn.ch>

Reviwed-by: Florian Fainelli <f.faine...@gmail.com>
-- 
Florian