* [patch -next] cavium/liquidio: fix some error handling in lio_set_phys_id()
@ 2015-06-24 14:47 Dan Carpenter
2015-06-25 9:13 ` David Miller
0 siblings, 1 reply; 2+ messages in thread
From: Dan Carpenter @ 2015-06-24 14:47 UTC (permalink / raw
To: Derek Chickles
Cc: Satanand Burla, Felix Manlunas, Raghu Vatsavayi, Sunil Goutham,
Robert Richter, netdev, kernel-janitors
There was a missing assignment so the "if (ret)" on the next line is
never true.
Fixes: f21fb3ed364b ('Add support of Cavium Liquidio ethernet adapters')
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
diff --git a/drivers/net/ethernet/cavium/liquidio/lio_ethtool.c b/drivers/net/ethernet/cavium/liquidio/lio_ethtool.c
index 160f807..29f3308 100644
--- a/drivers/net/ethernet/cavium/liquidio/lio_ethtool.c
+++ b/drivers/net/ethernet/cavium/liquidio/lio_ethtool.c
@@ -434,8 +434,9 @@ static int lio_set_phys_id(struct net_device *netdev,
if (ret)
return ret;
- octnet_mdio45_access(lio, 1, LIO68XX_LED_BEACON_ADDR,
- &lio->phy_beacon_val);
+ ret = octnet_mdio45_access(lio, 1,
+ LIO68XX_LED_BEACON_ADDR,
+ &lio->phy_beacon_val);
if (ret)
return ret;
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [patch -next] cavium/liquidio: fix some error handling in lio_set_phys_id()
2015-06-24 14:47 [patch -next] cavium/liquidio: fix some error handling in lio_set_phys_id() Dan Carpenter
@ 2015-06-25 9:13 ` David Miller
0 siblings, 0 replies; 2+ messages in thread
From: David Miller @ 2015-06-25 9:13 UTC (permalink / raw
To: dan.carpenter
Cc: derek.chickles, satananda.burla, felix.manlunas, raghu.vatsavayi,
sgoutham, rric, netdev, kernel-janitors
From: Dan Carpenter <dan.carpenter@oracle.com>
Date: Wed, 24 Jun 2015 17:47:02 +0300
> There was a missing assignment so the "if (ret)" on the next line is
> never true.
>
> Fixes: f21fb3ed364b ('Add support of Cavium Liquidio ethernet adapters')
> Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Applied.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2015-06-25 9:01 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-06-24 14:47 [patch -next] cavium/liquidio: fix some error handling in lio_set_phys_id() Dan Carpenter
2015-06-25 9:13 ` David Miller
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).