Commit 47bca7de authored by Sasha Neftin's avatar Sasha Neftin Committed by Tony Nguyen

igc: Remove phy->type checking

i225 devices have only one phy->type: copper. There is no point checking
phy->type during the igc_has_link method from the watchdog that
invoked every 2 seconds.
This patch comes to clean up these pointless checkings.
Signed-off-by: default avatarSasha Neftin <sasha.neftin@intel.com>
Tested-by: default avatarDvora Fuxbrumer <dvorax.fuxbrumer@linux.intel.com>
Signed-off-by: default avatarTony Nguyen <anthony.l.nguyen@intel.com>
parent 7c496de5
...@@ -5219,17 +5219,10 @@ bool igc_has_link(struct igc_adapter *adapter) ...@@ -5219,17 +5219,10 @@ bool igc_has_link(struct igc_adapter *adapter)
* false until the igc_check_for_link establishes link * false until the igc_check_for_link establishes link
* for copper adapters ONLY * for copper adapters ONLY
*/ */
switch (hw->phy.media_type) { if (!hw->mac.get_link_status)
case igc_media_type_copper: return true;
if (!hw->mac.get_link_status) hw->mac.ops.check_for_link(hw);
return true; link_active = !hw->mac.get_link_status;
hw->mac.ops.check_for_link(hw);
link_active = !hw->mac.get_link_status;
break;
default:
case igc_media_type_unknown:
break;
}
if (hw->mac.type == igc_i225) { if (hw->mac.type == igc_i225) {
if (!netif_carrier_ok(adapter->netdev)) { if (!netif_carrier_ok(adapter->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