diff options
author | Bruce Allan <bruce.w.allan@intel.com> | 2012-01-31 01:37:32 -0500 |
---|---|---|
committer | Jeff Kirsher <jeffrey.t.kirsher@intel.com> | 2012-02-07 07:16:54 -0500 |
commit | e885d762b7f73e811432f67608a6f0affa861029 (patch) | |
tree | eedaee62600e7685cf6dac1fb0ce0864331da490 | |
parent | a2a5b3235d0dce0feaadf4e76727adea1caa6869 (diff) |
e1000e: fix sparse warnings with -D__CHECK_ENDIAN__
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>
-rw-r--r-- | drivers/net/ethernet/intel/e1000e/ethtool.c | 2 | ||||
-rw-r--r-- | drivers/net/ethernet/intel/e1000e/netdev.c | 15 |
2 files changed, 9 insertions, 8 deletions
diff --git a/drivers/net/ethernet/intel/e1000e/ethtool.c b/drivers/net/ethernet/intel/e1000e/ethtool.c index fd2ba3818d3c..f4dc0fa9d9b5 100644 --- a/drivers/net/ethernet/intel/e1000e/ethtool.c +++ b/drivers/net/ethernet/intel/e1000e/ethtool.c | |||
@@ -553,7 +553,7 @@ static int e1000_set_eeprom(struct net_device *netdev, | |||
553 | memcpy(ptr, bytes, eeprom->len); | 553 | memcpy(ptr, bytes, eeprom->len); |
554 | 554 | ||
555 | for (i = 0; i < last_word - first_word + 1; i++) | 555 | for (i = 0; i < last_word - first_word + 1; i++) |
556 | eeprom_buff[i] = cpu_to_le16(eeprom_buff[i]); | 556 | cpu_to_le16s(&eeprom_buff[i]); |
557 | 557 | ||
558 | ret_val = e1000_write_nvm(hw, first_word, | 558 | ret_val = e1000_write_nvm(hw, first_word, |
559 | last_word - first_word + 1, eeprom_buff); | 559 | last_word - first_word + 1, eeprom_buff); |
diff --git a/drivers/net/ethernet/intel/e1000e/netdev.c b/drivers/net/ethernet/intel/e1000e/netdev.c index a9f25fcfbfe3..2ac1164d04e5 100644 --- a/drivers/net/ethernet/intel/e1000e/netdev.c +++ b/drivers/net/ethernet/intel/e1000e/netdev.c | |||
@@ -183,18 +183,18 @@ static void e1000e_dump(struct e1000_adapter *adapter) | |||
183 | struct e1000_ring *tx_ring = adapter->tx_ring; | 183 | struct e1000_ring *tx_ring = adapter->tx_ring; |
184 | struct e1000_tx_desc *tx_desc; | 184 | struct e1000_tx_desc *tx_desc; |
185 | struct my_u0 { | 185 | struct my_u0 { |
186 | u64 a; | 186 | __le64 a; |
187 | u64 b; | 187 | __le64 b; |
188 | } *u0; | 188 | } *u0; |
189 | struct e1000_buffer *buffer_info; | 189 | struct e1000_buffer *buffer_info; |
190 | struct e1000_ring *rx_ring = adapter->rx_ring; | 190 | struct e1000_ring *rx_ring = adapter->rx_ring; |
191 | union e1000_rx_desc_packet_split *rx_desc_ps; | 191 | union e1000_rx_desc_packet_split *rx_desc_ps; |
192 | union e1000_rx_desc_extended *rx_desc; | 192 | union e1000_rx_desc_extended *rx_desc; |
193 | struct my_u1 { | 193 | struct my_u1 { |
194 | u64 a; | 194 | __le64 a; |
195 | u64 b; | 195 | __le64 b; |
196 | u64 c; | 196 | __le64 c; |
197 | u64 d; | 197 | __le64 d; |
198 | } *u1; | 198 | } *u1; |
199 | u32 staterr; | 199 | u32 staterr; |
200 | int i = 0; | 200 | int i = 0; |
@@ -5963,7 +5963,8 @@ static void e1000_eeprom_checks(struct e1000_adapter *adapter) | |||
5963 | return; | 5963 | return; |
5964 | 5964 | ||
5965 | ret_val = e1000_read_nvm(hw, NVM_INIT_CONTROL2_REG, 1, &buf); | 5965 | ret_val = e1000_read_nvm(hw, NVM_INIT_CONTROL2_REG, 1, &buf); |
5966 | if (!ret_val && (!(le16_to_cpu(buf) & (1 << 0)))) { | 5966 | le16_to_cpus(&buf); |
5967 | if (!ret_val && (!(buf & (1 << 0)))) { | ||
5967 | /* Deep Smart Power Down (DSPD) */ | 5968 | /* Deep Smart Power Down (DSPD) */ |
5968 | dev_warn(&adapter->pdev->dev, | 5969 | dev_warn(&adapter->pdev->dev, |
5969 | "Warning: detected DSPD enabled in EEPROM\n"); | 5970 | "Warning: detected DSPD enabled in EEPROM\n"); |