Skip to content

L
linux
Commit c75c4edf authored by Carolyn Wyborny's avatar Carolyn Wyborny Committed by Jeff Kirsher
Browse files
Options

igb: Cleanups for messaging 

This patch fixes WARNING:PREFER_PR_LEVEL and WARNING:SPLIT_STRING
from checkpatch file check.
Signed-off-by: default avatarCarolyn Wyborny <carolyn.wyborny@intel.com>
Tested-by: default avatarAaron Brown <aaron.f.brown@intel.com>
Signed-off-by: default avatarJeff Kirsher <jeffrey.t.kirsher@intel.com>
parent 0e8e842b
Hide whitespace changes
Inline Side-by-side
Showing with 18 additions and 33 deletions
drivers/net/ethernet/intel/igb/e1000_82575.c
View file @ c75c4edf
...@@ -1676,7 +1676,7 @@ static s32 igb_setup_serdes_link_82575(struct e1000_hw *hw) ...@@ -1676,7 +1676,7 @@ static s32 igb_setup_serdes_link_82575(struct e1000_hw *hw)
hw->mac.type == e1000_82576) { hw->mac.type == e1000_82576) {
ret_val = hw->nvm.ops.read(hw, NVM_COMPAT, 1, &data); ret_val = hw->nvm.ops.read(hw, NVM_COMPAT, 1, &data);
if (ret_val) { if (ret_val) {
printk(KERN_DEBUG "NVM Read Error\n\n"); hw_dbg(KERN_DEBUG "NVM Read Error\n\n");
return ret_val; return ret_val;
} }
...@@ -2436,8 +2436,7 @@ static s32 igb_update_nvm_checksum_82580(struct e1000_hw *hw) ...@@ -2436,8 +2436,7 @@ static s32 igb_update_nvm_checksum_82580(struct e1000_hw *hw)
ret_val = hw->nvm.ops.read(hw, NVM_COMPATIBILITY_REG_3, 1, &nvm_data); ret_val = hw->nvm.ops.read(hw, NVM_COMPATIBILITY_REG_3, 1, &nvm_data);
if (ret_val) { if (ret_val) {
hw_dbg("NVM Read Error while updating checksum" hw_dbg("NVM Read Error while updating checksum compatibility bit.\n");
" compatibility bit.\n");
goto out; goto out;
} }
...@@ -2447,8 +2446,7 @@ static s32 igb_update_nvm_checksum_82580(struct e1000_hw *hw) ...@@ -2447,8 +2446,7 @@ static s32 igb_update_nvm_checksum_82580(struct e1000_hw *hw)
ret_val = hw->nvm.ops.write(hw, NVM_COMPATIBILITY_REG_3, 1, ret_val = hw->nvm.ops.write(hw, NVM_COMPATIBILITY_REG_3, 1,
&nvm_data); &nvm_data);
if (ret_val) { if (ret_val) {
hw_dbg("NVM Write Error while updating checksum" hw_dbg("NVM Write Error while updating checksum compatibility bit.\n");
" compatibility bit.\n");
goto out; goto out;
} }
} }
......
drivers/net/ethernet/intel/igb/e1000_mac.c
View file @ c75c4edf
...@@ -866,8 +866,7 @@ s32 igb_config_fc_after_link_up(struct e1000_hw *hw) ...@@ -866,8 +866,7 @@ s32 igb_config_fc_after_link_up(struct e1000_hw *hw)
goto out; goto out;
if (!(mii_status_reg & MII_SR_AUTONEG_COMPLETE)) { if (!(mii_status_reg & MII_SR_AUTONEG_COMPLETE)) {
hw_dbg("Copper PHY and Auto Neg " hw_dbg("Copper PHY and Auto Neg has not completed.\n");
"has not completed.\n");
goto out; goto out;
} }
...@@ -932,8 +931,7 @@ s32 igb_config_fc_after_link_up(struct e1000_hw *hw) ...@@ -932,8 +931,7 @@ s32 igb_config_fc_after_link_up(struct e1000_hw *hw)
hw_dbg("Flow Control = FULL.\r\n"); hw_dbg("Flow Control = FULL.\r\n");
} else { } else {
hw->fc.current_mode = e1000_fc_rx_pause; hw->fc.current_mode = e1000_fc_rx_pause;
hw_dbg("Flow Control = " hw_dbg("Flow Control = RX PAUSE frames only.\r\n");
"RX PAUSE frames only.\r\n");
} }
} }
/* For receiving PAUSE frames ONLY. /* For receiving PAUSE frames ONLY.
......
drivers/net/ethernet/intel/igb/e1000_phy.c
View file @ c75c4edf
...@@ -924,8 +924,7 @@ static s32 igb_copper_link_autoneg(struct e1000_hw *hw) ...@@ -924,8 +924,7 @@ static s32 igb_copper_link_autoneg(struct e1000_hw *hw)
if (phy->autoneg_wait_to_complete) { if (phy->autoneg_wait_to_complete) {
ret_val = igb_wait_autoneg(hw); ret_val = igb_wait_autoneg(hw);
if (ret_val) { if (ret_val) {
hw_dbg("Error while waiting for " hw_dbg("Error while waiting for autoneg to complete\n");
"autoneg to complete\n");
goto out; goto out;
} }
} }
......
drivers/net/ethernet/intel/igb/igb_main.c
View file @ c75c4edf
...@@ -217,8 +217,7 @@ static void igb_netpoll(struct net_device *); ...@@ -217,8 +217,7 @@ static void igb_netpoll(struct net_device *);
#ifdef CONFIG_PCI_IOV #ifdef CONFIG_PCI_IOV
static unsigned int max_vfs = 0; static unsigned int max_vfs = 0;
module_param(max_vfs, uint, 0); module_param(max_vfs, uint, 0);
MODULE_PARM_DESC(max_vfs, "Maximum number of virtual functions to allocate " MODULE_PARM_DESC(max_vfs, "Maximum number of virtual functions to allocate per physical function");
"per physical function");
#endif /* CONFIG_PCI_IOV */ #endif /* CONFIG_PCI_IOV */
static pci_ers_result_t igb_io_error_detected(struct pci_dev *, static pci_ers_result_t igb_io_error_detected(struct pci_dev *,
...@@ -384,8 +383,7 @@ static void igb_dump(struct igb_adapter *adapter) ...@@ -384,8 +383,7 @@ static void igb_dump(struct igb_adapter *adapter)
/* Print netdevice Info */ /* Print netdevice Info */
if (netdev) { if (netdev) {
dev_info(&adapter->pdev->dev, "Net device Info\n"); dev_info(&adapter->pdev->dev, "Net device Info\n");
pr_info("Device Name state trans_start " pr_info("Device Name state trans_start last_rx\n");
"last_rx\n");
pr_info("%-15s %016lX %016lX %016lX\n", netdev->name, pr_info("%-15s %016lX %016lX %016lX\n", netdev->name,
netdev->state, netdev->trans_start, netdev->last_rx); netdev->state, netdev->trans_start, netdev->last_rx);
} }
...@@ -438,9 +436,7 @@ static void igb_dump(struct igb_adapter *adapter) ...@@ -438,9 +436,7 @@ static void igb_dump(struct igb_adapter *adapter)
pr_info("------------------------------------\n"); pr_info("------------------------------------\n");
pr_info("TX QUEUE INDEX = %d\n", tx_ring->queue_index); pr_info("TX QUEUE INDEX = %d\n", tx_ring->queue_index);
pr_info("------------------------------------\n"); pr_info("------------------------------------\n");
pr_info("T [desc] [address 63:0 ] [PlPOCIStDDM Ln] " pr_info("T [desc] [address 63:0 ] [PlPOCIStDDM Ln] [bi->dma ] leng ntw timestamp bi->skb\n");
"[bi->dma ] leng ntw timestamp "
"bi->skb\n");
for (i = 0; tx_ring->desc && (i < tx_ring->count); i++) { for (i = 0; tx_ring->desc && (i < tx_ring->count); i++) {
const char *next_desc; const char *next_desc;
...@@ -458,9 +454,8 @@ static void igb_dump(struct igb_adapter *adapter) ...@@ -458,9 +454,8 @@ static void igb_dump(struct igb_adapter *adapter)
else else
next_desc = ""; next_desc = "";
pr_info("T [0x%03X] %016llX %016llX %016llX" pr_info("T [0x%03X] %016llX %016llX %016llX %04X %p %016llX %p%s\n",
" %04X %p %016llX %p%s\n", i, i, le64_to_cpu(u0->a),
le64_to_cpu(u0->a),
le64_to_cpu(u0->b), le64_to_cpu(u0->b),
(u64)dma_unmap_addr(buffer_info, dma), (u64)dma_unmap_addr(buffer_info, dma),
dma_unmap_len(buffer_info, len), dma_unmap_len(buffer_info, len),
...@@ -519,10 +514,8 @@ static void igb_dump(struct igb_adapter *adapter) ...@@ -519,10 +514,8 @@ static void igb_dump(struct igb_adapter *adapter)
pr_info("------------------------------------\n"); pr_info("------------------------------------\n");
pr_info("RX QUEUE INDEX = %d\n", rx_ring->queue_index); pr_info("RX QUEUE INDEX = %d\n", rx_ring->queue_index);
pr_info("------------------------------------\n"); pr_info("------------------------------------\n");
pr_info("R [desc] [ PktBuf A0] [ HeadBuf DD] " pr_info("R [desc] [ PktBuf A0] [ HeadBuf DD] [bi->dma ] [bi->skb] <-- Adv Rx Read format\n");
"[bi->dma ] [bi->skb] <-- Adv Rx Read format\n"); pr_info("RWB[desc] [PcsmIpSHl PtRs] [vl er S cks ln] ---------------- [bi->skb] <-- Adv Rx Write-Back format\n");
pr_info("RWB[desc] [PcsmIpSHl PtRs] [vl er S cks ln] -----"
"----------- [bi->skb] <-- Adv Rx Write-Back format\n");
for (i = 0; i < rx_ring->count; i++) { for (i = 0; i < rx_ring->count; i++) {
const char *next_desc; const char *next_desc;
...@@ -4215,8 +4208,8 @@ static void igb_watchdog_task(struct work_struct *work) ...@@ -4215,8 +4208,8 @@ static void igb_watchdog_task(struct work_struct *work)
ctrl = rd32(E1000_CTRL); ctrl = rd32(E1000_CTRL);
/* Links status message must follow this format */ /* Links status message must follow this format */
printk(KERN_INFO "igb: %s NIC Link is Up %d Mbps %s " netdev_info(netdev,
"Duplex, Flow Control: %s\n", "igb: %s NIC Link is Up %d Mbps %s Duplex, Flow Control: %s\n",
netdev->name, netdev->name,
adapter->link_speed, adapter->link_speed,
adapter->link_duplex == FULL_DUPLEX ? adapter->link_duplex == FULL_DUPLEX ?
...@@ -4243,9 +4236,7 @@ static void igb_watchdog_task(struct work_struct *work) ...@@ -4243,9 +4236,7 @@ static void igb_watchdog_task(struct work_struct *work)
/* check for thermal sensor event */ /* check for thermal sensor event */
if (igb_thermal_sensor_event(hw, if (igb_thermal_sensor_event(hw,
E1000_THSTAT_LINK_THROTTLE)) { E1000_THSTAT_LINK_THROTTLE)) {
netdev_info(netdev, "The network adapter link " netdev_info(netdev, "The network adapter link speed was downshifted because it overheated\n");
"speed was downshifted because it "
"overheated\n");
} }
/* adjust timeout factor according to speed/duplex */ /* adjust timeout factor according to speed/duplex */
...@@ -4277,12 +4268,11 @@ static void igb_watchdog_task(struct work_struct *work) ...@@ -4277,12 +4268,11 @@ static void igb_watchdog_task(struct work_struct *work)
/* check for thermal sensor event */ /* check for thermal sensor event */
if (igb_thermal_sensor_event(hw, if (igb_thermal_sensor_event(hw,
E1000_THSTAT_PWR_DOWN)) { E1000_THSTAT_PWR_DOWN)) {
netdev_err(netdev, "The network adapter was " netdev_err(netdev, "The network adapter was stopped because it overheated\n");
"stopped because it overheated\n");
} }
/* Links status message must follow this format */ /* Links status message must follow this format */
printk(KERN_INFO "igb: %s NIC Link is Down\n", netdev_info(netdev, "igb: %s NIC Link is Down\n",
netdev->name); netdev->name);
netif_carrier_off(netdev); netif_carrier_off(netdev);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment
GitLab Nexedi Edition | About GitLab | About Nexedi | 沪ICP备2021021310号-2 | 沪ICP备2021021310号-7