[net-next v3 5/9] igc: Use netdev log helpers in igc_ptp.c

[Jeff Kirsher]

From: Andre Guedes <andre.gue...@intel.com>

In igc_ptp.c we print log messages using dev_* helpers, generating
inconsistent output with the rest of the driver. Since this is a network
device driver, we should preferably use netdev_* helpers because they
append the interface name to the message, helping making sense out of
the logs.

This patch converts all dev_* calls to netdev_*. It also takes this
opportunity to remove the '\n' character at the end of messages since it
is automatically added by netdev_* log helpers.

Signed-off-by: Andre Guedes <andre.gue...@intel.com>
Tested-by: Aaron Brown <aaron.f.br...@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirs...@intel.com>
---
 drivers/net/ethernet/intel/igc/igc_ptp.c | 12 +++++-------
 1 file changed, 5 insertions(+), 7 deletions(-)

diff --git a/drivers/net/ethernet/intel/igc/igc_ptp.c 
b/drivers/net/ethernet/intel/igc/igc_ptp.c
index f99c514ad0f4..1bf016398b9d 100644
--- a/drivers/net/ethernet/intel/igc/igc_ptp.c
+++ b/drivers/net/ethernet/intel/igc/igc_ptp.c
@@ -466,7 +466,7 @@ void igc_ptp_tx_hang(struct igc_adapter *adapter)
                 * interrupt
                 */
                rd32(IGC_TXSTMPH);
-               dev_warn(&adapter->pdev->dev, "clearing Tx timestamp hang\n");
+               netdev_warn(adapter->netdev, "Clearing Tx timestamp hang\n");
        }
 }
 
@@ -529,7 +529,7 @@ static void igc_ptp_tx_work(struct work_struct *work)
                 * interrupt
                 */
                rd32(IGC_TXSTMPH);
-               dev_warn(&adapter->pdev->dev, "clearing Tx timestamp hang\n");
+               netdev_warn(adapter->netdev, "Clearing Tx timestamp hang\n");
                return;
        }
 
@@ -626,10 +626,9 @@ void igc_ptp_init(struct igc_adapter *adapter)
                                                &adapter->pdev->dev);
        if (IS_ERR(adapter->ptp_clock)) {
                adapter->ptp_clock = NULL;
-               dev_err(&adapter->pdev->dev, "ptp_clock_register failed\n");
+               netdev_err(netdev, "ptp_clock_register failed\n");
        } else if (adapter->ptp_clock) {
-               dev_info(&adapter->pdev->dev, "added PHC on %s\n",
-                        adapter->netdev->name);
+               netdev_info(netdev, "PHC added\n");
                adapter->ptp_flags |= IGC_PTP_ENABLED;
        }
 }
@@ -666,8 +665,7 @@ void igc_ptp_stop(struct igc_adapter *adapter)
 
        if (adapter->ptp_clock) {
                ptp_clock_unregister(adapter->ptp_clock);
-               dev_info(&adapter->pdev->dev, "removed PHC on %s\n",
-                        adapter->netdev->name);
+               netdev_info(adapter->netdev, "PHC removed\n");
                adapter->ptp_flags &= ~IGC_PTP_ENABLED;
        }
 }
-- 
2.26.2