diff options
author | Dan Carpenter <error27@gmail.com> | 2010-12-17 03:25:43 +0000 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-12-23 10:21:12 -0800 |
commit | 0e214ad81545a35c5e62a4c2d6cf7275fd7a545f (patch) | |
tree | 8327c0394729760d43ba7ba1548d9afbfba9670c | |
parent | fdac1e0697356ac212259f2147aa60c72e334861 (diff) | |
download | op-kernel-dev-0e214ad81545a35c5e62a4c2d6cf7275fd7a545f.zip op-kernel-dev-0e214ad81545a35c5e62a4c2d6cf7275fd7a545f.tar.gz |
USB: mcs7830: return negative if auto negotiate fails
The original code returns 0 on success and 1 on failure. In fact, at
this point, "ret" is already either zero or a negative error code so
we can just return it directly.
Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | drivers/net/usb/mcs7830.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/usb/mcs7830.c b/drivers/net/usb/mcs7830.c index b701f59..2b79139 100644 --- a/drivers/net/usb/mcs7830.c +++ b/drivers/net/usb/mcs7830.c @@ -355,7 +355,7 @@ static int mcs7830_set_autoneg(struct usbnet *dev, int ptrUserPhyMode) if (!ret) ret = mcs7830_write_phy(dev, MII_BMCR, BMCR_ANENABLE | BMCR_ANRESTART ); - return ret < 0 ? : 0; + return ret; } |