Re: [PATCH net] net: broadcom/bcmsysport: Fix signedness in bcm_sysport_probe()

2019-09-25 Thread Florian Fainelli
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-of

[PATCH net] net: broadcom/bcmsysport: Fix signedness in bcm_sysport_probe()

2019-09-25 Thread Dan Carpenter
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 --- drivers/net/ethernet/broadcom/bcms