From: Eric Sesterhenn 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 Signed-off-by: Alexey Dobriyan Cc: Stephen Hemminger Cc: "David S. Miller" Signed-off-by: Andrew Morton --- drivers/net/bonding/bond_main.c | 2 +- 1 files changed, 1 insertion(+), 1 deletion(-) diff -puN drivers/net/bonding/bond_main.c~bonding-fix-get_settings-error-checking drivers/net/bonding/bond_main.c --- devel/drivers/net/bonding/bond_main.c~bonding-fix-get_settings-error-checking 2006-01-22 02:58:57.000000000 -0800 +++ devel-akpm/drivers/net/bonding/bond_main.c 2006-01-22 02:58:57.000000000 -0800 @@ -576,7 +576,7 @@ static int bond_update_speed_duplex(stru slave->duplex = DUPLEX_FULL; if (slave_dev->ethtool_ops) { - u32 res; + int res; if (!slave_dev->ethtool_ops->get_settings) { return -1; _