Commit a0beb3c1 authored by Sasha Neftin's avatar Sasha Neftin Committed by Jeff Kirsher
Browse files

igc: Refactor the igc_power_down_link() 



Currently the implementation of igc_power_down_link()
method was just calling igc_power_down_phy_copper_base()
method.
We can just call igc_power_down_phy_copper_base()
method directly.

Signed-off-by: default avatarSasha Neftin <sasha.neftin@intel.com>
Tested-by: default avatarAaron Brown <aaron.f.brown@intel.com>
Signed-off-by: default avatarJeff Kirsher <jeffrey.t.kirsher@intel.com>
parent 725fa16d

drivers/net/ethernet/intel/igc/igc_main.c

+3 −13
Original line number Diff line number Diff line
@@ -61,16 +61,6 @@ enum latency_range { 
	latency_invalid = 255

};



/**

 * igc_power_down_link - Power down the phy/serdes link

 * @adapter: address of board private structure

 */

static void igc_power_down_link(struct igc_adapter *adapter)

{

	if (adapter->hw.phy.media_type == igc_media_type_copper)

		igc_power_down_phy_copper_base(&adapter->hw);

}



void igc_reset(struct igc_adapter *adapter)

{

	struct net_device *dev = adapter->netdev;
@@ -106,7 +96,7 @@ void igc_reset(struct igc_adapter *adapter) 
	igc_set_eee_i225(hw, true, true, true);



	if (!netif_running(adapter->netdev))

		igc_power_down_link(adapter);

		igc_power_down_phy_copper_base(&adapter->hw);



	/* Re-enable PTP, where applicable. */

	igc_ptp_reset(adapter);
@@ -4615,7 +4605,7 @@ err_set_queues: 
	igc_free_irq(adapter);

err_req_irq:

	igc_release_hw_control(adapter);

	igc_power_down_link(adapter);

	igc_power_down_phy_copper_base(&adapter->hw);

	igc_free_all_rx_resources(adapter);

err_setup_rx:

	igc_free_all_tx_resources(adapter);
@@ -5313,7 +5303,7 @@ static int __igc_shutdown(struct pci_dev *pdev, bool *enable_wake, 


	wake = wufc || adapter->en_mng_pt;

	if (!wake)

		igc_power_down_link(adapter);

		igc_power_down_phy_copper_base(&adapter->hw);

	else

		igc_power_up_link(adapter);

CRA Git | Maintained and supported by SUSTech CRA and CCSE