ipq40xx: net: ethernet: edma: use generic PHY print
authorRobert Marko <robert.marko@sartura.hr>
Thu, 26 Nov 2020 12:54:34 +0000 (13:54 +0100)
committerPetr Štetiar <ynezz@true.cz>
Wed, 23 Dec 2020 15:36:08 +0000 (16:36 +0100)
Lets use the generic upstream phy_print_status() instead of doing
something similar by hand.

Before:
ess_edma c080000.edma: eth1: GMAC Link is up with phy_speed=1000

After:
ess_edma c080000.edma eth1: Link is Up - 1Gbps/Full - flow control rx/tx

Signed-off-by: Robert Marko <robert.marko@sartura.hr>
target/linux/ipq40xx/files/drivers/net/ethernet/qualcomm/essedma/edma.c

index ecbc946de9334b816b250dbbd733748db4e35137..80db1f0acbe47d65a8ce76a4886325a165dfb7d9 100644 (file)
@@ -1310,7 +1310,7 @@ void edma_adjust_link(struct net_device *netdev)
        status = edma_check_link(adapter);
 
        if (status == __EDMA_LINKUP && adapter->link_state == __EDMA_LINKDOWN) {
-               dev_info(&adapter->pdev->dev, "%s: GMAC Link is up with phy_speed=%d\n", netdev->name, phydev->speed);
+               phy_print_status(phydev);
                adapter->link_state = __EDMA_LINKUP;
                if (adapter->edma_cinfo->is_single_phy) {
                        ess_set_port_status_speed(adapter->edma_cinfo, phydev,
@@ -1320,7 +1320,7 @@ void edma_adjust_link(struct net_device *netdev)
                if (netif_running(netdev))
                        netif_tx_wake_all_queues(netdev);
        } else if (status == __EDMA_LINKDOWN && adapter->link_state == __EDMA_LINKUP) {
-               dev_info(&adapter->pdev->dev, "%s: GMAC Link is down\n", netdev->name);
+               phy_print_status(phydev);
                adapter->link_state = __EDMA_LINKDOWN;
                netif_carrier_off(netdev);
                netif_tx_stop_all_queues(netdev);