From: Eric Sesterhenn <snakebyte@gmx.de>
Date: Fri, 20 Jan 2006 20:30:01 +0000 (+0300)
Subject: [PATCH] bonding: fix ->get_settings error checking
X-Git-Tag: v3.12-rc1~39014^2~10
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=6a986ce45d45b099ddf676c340267765e76db91e;p=kernel%2Fkernel-generic.git

[PATCH] bonding: fix ->get_settings error checking

Since get_settings() returns a signed int and it gets checked
for < 0 to catch an error, res should be a signed int too.

Signed-off-by: Eric Sesterhenn <snakebyte@gmx.de>
Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com>
Signed-off-by: Jeff Garzik <jgarzik@pobox.com>
---

diff --git a/drivers/net/bonding/bond_main.c b/drivers/net/bonding/bond_main.c
index 2582d98..4ff006c 100644
--- a/drivers/net/bonding/bond_main.c
+++ b/drivers/net/bonding/bond_main.c
@@ -576,7 +576,7 @@ static int bond_update_speed_duplex(struct slave *slave)
 	slave->duplex = DUPLEX_FULL;
 
 	if (slave_dev->ethtool_ops) {
-		u32 res;
+		int res;
 
 		if (!slave_dev->ethtool_ops->get_settings) {
 			return -1;