From 7a341066117f9eb5eb9385f61176ef6a08b90163 Mon Sep 17 00:00:00 2001 From: Wolfgang Denk Date: Fri, 28 Oct 2011 07:37:04 +0200 Subject: e1000: fix bugs from recent commits Commit 114d7fc0 "e1000: Rewrite EEPROM checksum error to give more information" failed to initialize the checksum variable which should result in random results. Fix that. Commit 2326a94d caused a ton of "unused variable 'x'" warnings. Fix these. While we are at it, remove some bogus parens. Signed-off-by: Wolfgang Denk Cc: Kyle Moffett Acked-by: Mike Frysinger Tested-by: Kyle Moffett --- drivers/net/e1000.c | 1 + 1 file changed, 1 insertion(+) (limited to 'drivers/net/e1000.c') diff --git a/drivers/net/e1000.c b/drivers/net/e1000.c index c86bf0a0e..6eab7b2cf 100644 --- a/drivers/net/e1000.c +++ b/drivers/net/e1000.c @@ -884,6 +884,7 @@ static int e1000_validate_eeprom_checksum(struct e1000_hw *hw) } /* Compute the checksum */ + checksum = 0; for (i = 0; i < EEPROM_CHECKSUM_REG; i++) checksum += buf[i]; checksum = ((uint16_t)EEPROM_SUM) - checksum; -- cgit v1.2.3-70-g09d2