diff options
author | Roel Kluin <roel.kluin@gmail.com> | 2009-12-01 10:54:24 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2009-12-02 03:35:56 -0500 |
commit | 8528b016d64a7c1c97ba6540ab53a538348d5026 (patch) | |
tree | 34d777a9e8848b07ccd34fc4db0dc44bc831a018 /drivers/net/e1000e | |
parent | 07f025e6bad7292af3904b8f49cd33e7b5fe3467 (diff) |
e1000e: Use sizeof struct rather than pointer in e1000_get_eeprom()
Don't use the sizeof the pointer to clear the result
Signed-off-by: Roel Kluin <roel.kluin@gmail.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/e1000e')
-rw-r--r-- | drivers/net/e1000e/ethtool.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/net/e1000e/ethtool.c b/drivers/net/e1000e/ethtool.c index b6243cad3103..0aa50c229c79 100644 --- a/drivers/net/e1000e/ethtool.c +++ b/drivers/net/e1000e/ethtool.c | |||
@@ -535,7 +535,8 @@ static int e1000_get_eeprom(struct net_device *netdev, | |||
535 | 535 | ||
536 | if (ret_val) { | 536 | if (ret_val) { |
537 | /* a read error occurred, throw away the result */ | 537 | /* a read error occurred, throw away the result */ |
538 | memset(eeprom_buff, 0xff, sizeof(eeprom_buff)); | 538 | memset(eeprom_buff, 0xff, sizeof(u16) * |
539 | (last_word - first_word + 1)); | ||
539 | } else { | 540 | } else { |
540 | /* Device's eeprom is always little-endian, word addressable */ | 541 | /* Device's eeprom is always little-endian, word addressable */ |
541 | for (i = 0; i < last_word - first_word + 1; i++) | 542 | for (i = 0; i < last_word - first_word + 1; i++) |