On 9/25/2019 3:56 AM, Dan Carpenter wrote: > The "priv->phy_interface" variable is an enum and in this context GCC > will treat it as unsigned so the error handling will never be > triggered. > > Fixes: 80105befdb4b ("net: systemport: add Broadcom SYSTEMPORT Ethernet MAC driver") > Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> Acked-by: Florian Fainelli <f.fainelli@xxxxxxxxx> -- Florian