Re: [patch 2/2] netxen: reversed condition in netxen_nic_set_link_parameters()

From: David Miller
Date: Sat May 07 2016 - 15:16:35 EST

From: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
Date: Thu, 5 May 2016 16:19:44 +0300

> My static checker complains that we are using "autoneg" without
> initializing it. The problem is the ->phy_read() condition is reversed
> so we only set this on error instead of success.
> Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>