diff options
author | Bruce Allan <bruce.w.allan@intel.com> | 2011-07-22 02:21:41 -0400 |
---|---|---|
committer | Jeff Kirsher <jeffrey.t.kirsher@intel.com> | 2011-08-04 08:00:21 -0400 |
commit | b9e06f70dc186f8353cc593f2b4609383b3be7a9 (patch) | |
tree | ba35a9357e7060d45a1822c853c46c498bb381d7 /drivers | |
parent | 945a51517cc0bd9e461f2018624dfc1faef9ddee (diff) |
e1000e: remove unnecessary check for NULL pointer
The array shadow_ram is never NULL.
Signed-off-by: Bruce Allan <bruce.w.allan@intel.com>
Tested-by: Aaron Brown <aaron.f.brown@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/net/e1000e/ich8lan.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/net/e1000e/ich8lan.c b/drivers/net/e1000e/ich8lan.c index f7a75c1531ad..4e36978b8fd8 100644 --- a/drivers/net/e1000e/ich8lan.c +++ b/drivers/net/e1000e/ich8lan.c | |||
@@ -2137,8 +2137,7 @@ static s32 e1000_read_nvm_ich8lan(struct e1000_hw *hw, u16 offset, u16 words, | |||
2137 | 2137 | ||
2138 | ret_val = 0; | 2138 | ret_val = 0; |
2139 | for (i = 0; i < words; i++) { | 2139 | for (i = 0; i < words; i++) { |
2140 | if ((dev_spec->shadow_ram) && | 2140 | if (dev_spec->shadow_ram[offset+i].modified) { |
2141 | (dev_spec->shadow_ram[offset+i].modified)) { | ||
2142 | data[i] = dev_spec->shadow_ram[offset+i].value; | 2141 | data[i] = dev_spec->shadow_ram[offset+i].value; |
2143 | } else { | 2142 | } else { |
2144 | ret_val = e1000_read_flash_word_ich8lan(hw, | 2143 | ret_val = e1000_read_flash_word_ich8lan(hw, |