diff options
author | Carolyn Wyborny <carolyn.wyborny@intel.com> | 2014-04-10 21:45:23 -0400 |
---|---|---|
committer | Jeff Kirsher <jeffrey.t.kirsher@intel.com> | 2014-04-23 04:30:15 -0400 |
commit | d34a15abfe370252de83e14e763cf7fcb8c84585 (patch) | |
tree | 65c391f00fff7be098d9344522c72d6f599808bf /drivers/net | |
parent | c75c4edfc38da8235d110a8f28b596193de787ab (diff) |
igb: Cleanups to fix braces location warnings
This patch fixes WARNING:BRACES and ERROR:OPEN_BRACE from
checkpatch file check.
Signed-off-by: Carolyn Wyborny <carolyn.wyborny@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/net')
-rw-r--r-- | drivers/net/ethernet/intel/igb/e1000_82575.c | 8 | ||||
-rw-r--r-- | drivers/net/ethernet/intel/igb/igb_ethtool.c | 4 | ||||
-rw-r--r-- | drivers/net/ethernet/intel/igb/igb_main.c | 10 |
3 files changed, 9 insertions, 13 deletions
diff --git a/drivers/net/ethernet/intel/igb/e1000_82575.c b/drivers/net/ethernet/intel/igb/e1000_82575.c index 7b7d08c3461d..b1a759c337fb 100644 --- a/drivers/net/ethernet/intel/igb/e1000_82575.c +++ b/drivers/net/ethernet/intel/igb/e1000_82575.c | |||
@@ -73,9 +73,8 @@ static s32 igb_validate_nvm_checksum_82580(struct e1000_hw *hw); | |||
73 | static s32 igb_update_nvm_checksum_82580(struct e1000_hw *hw); | 73 | static s32 igb_update_nvm_checksum_82580(struct e1000_hw *hw); |
74 | static s32 igb_validate_nvm_checksum_i350(struct e1000_hw *hw); | 74 | static s32 igb_validate_nvm_checksum_i350(struct e1000_hw *hw); |
75 | static s32 igb_update_nvm_checksum_i350(struct e1000_hw *hw); | 75 | static s32 igb_update_nvm_checksum_i350(struct e1000_hw *hw); |
76 | static const u16 e1000_82580_rxpbs_table[] = | 76 | static const u16 e1000_82580_rxpbs_table[] = { |
77 | { 36, 72, 144, 1, 2, 4, 8, 16, | 77 | 36, 72, 144, 1, 2, 4, 8, 16, 35, 70, 140 }; |
78 | 35, 70, 140 }; | ||
79 | 78 | ||
80 | /** | 79 | /** |
81 | * igb_sgmii_uses_mdio_82575 - Determine if I2C pins are for external MDIO | 80 | * igb_sgmii_uses_mdio_82575 - Determine if I2C pins are for external MDIO |
@@ -1436,9 +1435,8 @@ static s32 igb_reset_hw_82575(struct e1000_hw *hw) | |||
1436 | 1435 | ||
1437 | /* set the completion timeout for interface */ | 1436 | /* set the completion timeout for interface */ |
1438 | ret_val = igb_set_pcie_completion_timeout(hw); | 1437 | ret_val = igb_set_pcie_completion_timeout(hw); |
1439 | if (ret_val) { | 1438 | if (ret_val) |
1440 | hw_dbg("PCI-E Set completion timeout has failed.\n"); | 1439 | hw_dbg("PCI-E Set completion timeout has failed.\n"); |
1441 | } | ||
1442 | 1440 | ||
1443 | hw_dbg("Masking off all interrupts\n"); | 1441 | hw_dbg("Masking off all interrupts\n"); |
1444 | wr32(E1000_IMC, 0xffffffff); | 1442 | wr32(E1000_IMC, 0xffffffff); |
diff --git a/drivers/net/ethernet/intel/igb/igb_ethtool.c b/drivers/net/ethernet/intel/igb/igb_ethtool.c index e5570acbeea8..1615226d1456 100644 --- a/drivers/net/ethernet/intel/igb/igb_ethtool.c +++ b/drivers/net/ethernet/intel/igb/igb_ethtool.c | |||
@@ -1196,8 +1196,8 @@ static bool reg_pattern_test(struct igb_adapter *adapter, u64 *data, | |||
1196 | { | 1196 | { |
1197 | struct e1000_hw *hw = &adapter->hw; | 1197 | struct e1000_hw *hw = &adapter->hw; |
1198 | u32 pat, val; | 1198 | u32 pat, val; |
1199 | static const u32 _test[] = | 1199 | static const u32 _test[] = { |
1200 | {0x5A5A5A5A, 0xA5A5A5A5, 0x00000000, 0xFFFFFFFF}; | 1200 | 0x5A5A5A5A, 0xA5A5A5A5, 0x00000000, 0xFFFFFFFF}; |
1201 | for (pat = 0; pat < ARRAY_SIZE(_test); pat++) { | 1201 | for (pat = 0; pat < ARRAY_SIZE(_test); pat++) { |
1202 | wr32(reg, (_test[pat] & write)); | 1202 | wr32(reg, (_test[pat] & write)); |
1203 | val = rd32(reg) & mask; | 1203 | val = rd32(reg) & mask; |
diff --git a/drivers/net/ethernet/intel/igb/igb_main.c b/drivers/net/ethernet/intel/igb/igb_main.c index 13edca95e56d..e6e4f4ba0b78 100644 --- a/drivers/net/ethernet/intel/igb/igb_main.c +++ b/drivers/net/ethernet/intel/igb/igb_main.c | |||
@@ -4235,9 +4235,8 @@ static void igb_watchdog_task(struct work_struct *work) | |||
4235 | 4235 | ||
4236 | /* check for thermal sensor event */ | 4236 | /* check for thermal sensor event */ |
4237 | if (igb_thermal_sensor_event(hw, | 4237 | if (igb_thermal_sensor_event(hw, |
4238 | E1000_THSTAT_LINK_THROTTLE)) { | 4238 | E1000_THSTAT_LINK_THROTTLE)) |
4239 | netdev_info(netdev, "The network adapter link speed was downshifted because it overheated\n"); | 4239 | netdev_info(netdev, "The network adapter link speed was downshifted because it overheated\n"); |
4240 | } | ||
4241 | 4240 | ||
4242 | /* adjust timeout factor according to speed/duplex */ | 4241 | /* adjust timeout factor according to speed/duplex */ |
4243 | adapter->tx_timeout_factor = 1; | 4242 | adapter->tx_timeout_factor = 1; |
@@ -4473,13 +4472,12 @@ static void igb_update_itr(struct igb_q_vector *q_vector, | |||
4473 | case low_latency: /* 50 usec aka 20000 ints/s */ | 4472 | case low_latency: /* 50 usec aka 20000 ints/s */ |
4474 | if (bytes > 10000) { | 4473 | if (bytes > 10000) { |
4475 | /* this if handles the TSO accounting */ | 4474 | /* this if handles the TSO accounting */ |
4476 | if (bytes/packets > 8000) { | 4475 | if (bytes/packets > 8000) |
4477 | itrval = bulk_latency; | 4476 | itrval = bulk_latency; |
4478 | } else if ((packets < 10) || ((bytes/packets) > 1200)) { | 4477 | else if ((packets < 10) || ((bytes/packets) > 1200)) |
4479 | itrval = bulk_latency; | 4478 | itrval = bulk_latency; |
4480 | } else if ((packets > 35)) { | 4479 | else if ((packets > 35)) |
4481 | itrval = lowest_latency; | 4480 | itrval = lowest_latency; |
4482 | } | ||
4483 | } else if (bytes/packets > 2000) { | 4481 | } else if (bytes/packets > 2000) { |
4484 | itrval = bulk_latency; | 4482 | itrval = bulk_latency; |
4485 | } else if (packets <= 2 && bytes < 512) { | 4483 | } else if (packets <= 2 && bytes < 512) { |