Commit 21af70fb authored by Catherine Sullivan's avatar Catherine Sullivan Committed by Jeff Kirsher

i40e/i40evf: Remove unused variable an_enable and function update_link_info

An_enable was never used only set so lets remove it. The function
update_link_info only did two things, call get_link_info and set
an_enabled. Therefore we should also remove update_link_info and
change all references to it to get_link_info.

Change-ID: Ie3022680fa7a94bfd495a4f5fc76a73701d85569
Signed-off-by: default avatarCatherine Sullivan <catherine.sullivan@intel.com>
Tested-by: default avatarJim Young <james.m.young@intel.com>
Signed-off-by: default avatarJeff Kirsher <jeffrey.t.kirsher@intel.com>
parent 9df42d1a
...@@ -1297,14 +1297,14 @@ enum i40e_status_code i40e_set_fc(struct i40e_hw *hw, u8 *aq_failures, ...@@ -1297,14 +1297,14 @@ enum i40e_status_code i40e_set_fc(struct i40e_hw *hw, u8 *aq_failures,
*aq_failures |= I40E_SET_FC_AQ_FAIL_SET; *aq_failures |= I40E_SET_FC_AQ_FAIL_SET;
} }
/* Update the link info */ /* Update the link info */
status = i40e_update_link_info(hw, true); status = i40e_aq_get_link_info(hw, true, NULL, NULL);
if (status) { if (status) {
/* Wait a little bit (on 40G cards it sometimes takes a really /* Wait a little bit (on 40G cards it sometimes takes a really
* long time for link to come back from the atomic reset) * long time for link to come back from the atomic reset)
* and try once more * and try once more
*/ */
msleep(1000); msleep(1000);
status = i40e_update_link_info(hw, true); status = i40e_aq_get_link_info(hw, true, NULL, NULL);
} }
if (status) if (status)
*aq_failures |= I40E_SET_FC_AQ_FAIL_UPDATE; *aq_failures |= I40E_SET_FC_AQ_FAIL_UPDATE;
...@@ -1451,35 +1451,6 @@ i40e_status i40e_aq_get_link_info(struct i40e_hw *hw, ...@@ -1451,35 +1451,6 @@ i40e_status i40e_aq_get_link_info(struct i40e_hw *hw,
return status; return status;
} }
/**
* i40e_update_link_info
* @hw: pointer to the hw struct
* @enable_lse: enable/disable LinkStatusEvent reporting
*
* Returns the link status of the adapter
**/
i40e_status i40e_update_link_info(struct i40e_hw *hw, bool enable_lse)
{
struct i40e_aq_get_phy_abilities_resp abilities;
i40e_status status;
status = i40e_aq_get_link_info(hw, enable_lse, NULL, NULL);
if (status)
return status;
status = i40e_aq_get_phy_capabilities(hw, false, false,
&abilities, NULL);
if (status)
return status;
if (abilities.abilities & I40E_AQ_PHY_AN_ENABLED)
hw->phy.link_info.an_enabled = true;
else
hw->phy.link_info.an_enabled = false;
return status;
}
/** /**
* i40e_aq_set_phy_int_mask * i40e_aq_set_phy_int_mask
* @hw: pointer to the hw struct * @hw: pointer to the hw struct
......
...@@ -620,7 +620,7 @@ static int i40e_set_settings(struct net_device *netdev, ...@@ -620,7 +620,7 @@ static int i40e_set_settings(struct net_device *netdev,
return -EAGAIN; return -EAGAIN;
} }
status = i40e_update_link_info(hw, true); status = i40e_aq_get_link_info(hw, true, NULL, NULL);
if (status) if (status)
netdev_info(netdev, "Updating link info failed with error %d\n", netdev_info(netdev, "Updating link info failed with error %d\n",
status); status);
...@@ -766,7 +766,7 @@ static int i40e_set_pauseparam(struct net_device *netdev, ...@@ -766,7 +766,7 @@ static int i40e_set_pauseparam(struct net_device *netdev,
err = -EAGAIN; err = -EAGAIN;
} }
if (aq_failures & I40E_SET_FC_AQ_FAIL_UPDATE) { if (aq_failures & I40E_SET_FC_AQ_FAIL_UPDATE) {
netdev_info(netdev, "Set fc failed on the update_link_info call with error %d and status %d\n", netdev_info(netdev, "Set fc failed on the get_link_info call with error %d and status %d\n",
status, hw->aq.asq_last_status); status, hw->aq.asq_last_status);
err = -EAGAIN; err = -EAGAIN;
} }
......
...@@ -8885,7 +8885,7 @@ static int i40e_setup_pf_switch(struct i40e_pf *pf, bool reinit) ...@@ -8885,7 +8885,7 @@ static int i40e_setup_pf_switch(struct i40e_pf *pf, bool reinit)
i40e_config_rss(pf); i40e_config_rss(pf);
/* fill in link information and enable LSE reporting */ /* fill in link information and enable LSE reporting */
i40e_update_link_info(&pf->hw, true); i40e_aq_get_link_info(&pf->hw, true, NULL, NULL);
i40e_link_event(pf); i40e_link_event(pf);
/* Initialize user-specific link properties */ /* Initialize user-specific link properties */
...@@ -8893,7 +8893,7 @@ static int i40e_setup_pf_switch(struct i40e_pf *pf, bool reinit) ...@@ -8893,7 +8893,7 @@ static int i40e_setup_pf_switch(struct i40e_pf *pf, bool reinit)
I40E_AQ_AN_COMPLETED) ? true : false); I40E_AQ_AN_COMPLETED) ? true : false);
/* fill in link information and enable LSE reporting */ /* fill in link information and enable LSE reporting */
i40e_update_link_info(&pf->hw, true); i40e_aq_get_link_info(&pf->hw, true, NULL, NULL);
i40e_link_event(pf); i40e_link_event(pf);
/* Initialize user-specific link properties */ /* Initialize user-specific link properties */
......
...@@ -97,7 +97,6 @@ i40e_status i40e_aq_set_link_restart_an(struct i40e_hw *hw, ...@@ -97,7 +97,6 @@ i40e_status i40e_aq_set_link_restart_an(struct i40e_hw *hw,
i40e_status i40e_aq_get_link_info(struct i40e_hw *hw, i40e_status i40e_aq_get_link_info(struct i40e_hw *hw,
bool enable_lse, struct i40e_link_status *link, bool enable_lse, struct i40e_link_status *link,
struct i40e_asq_cmd_details *cmd_details); struct i40e_asq_cmd_details *cmd_details);
i40e_status i40e_update_link_info(struct i40e_hw *hw, bool enable_lse);
i40e_status i40e_aq_set_local_advt_reg(struct i40e_hw *hw, i40e_status i40e_aq_set_local_advt_reg(struct i40e_hw *hw,
u64 advt_reg, u64 advt_reg,
struct i40e_asq_cmd_details *cmd_details); struct i40e_asq_cmd_details *cmd_details);
......
...@@ -175,7 +175,6 @@ struct i40e_link_status { ...@@ -175,7 +175,6 @@ struct i40e_link_status {
u8 an_info; u8 an_info;
u8 ext_info; u8 ext_info;
u8 loopback; u8 loopback;
bool an_enabled;
/* is Link Status Event notification to SW enabled */ /* is Link Status Event notification to SW enabled */
bool lse_enable; bool lse_enable;
u16 max_frame_size; u16 max_frame_size;
......
...@@ -175,7 +175,6 @@ struct i40e_link_status { ...@@ -175,7 +175,6 @@ struct i40e_link_status {
u8 an_info; u8 an_info;
u8 ext_info; u8 ext_info;
u8 loopback; u8 loopback;
bool an_enabled;
/* is Link Status Event notification to SW enabled */ /* is Link Status Event notification to SW enabled */
bool lse_enable; bool lse_enable;
u16 max_frame_size; u16 max_frame_size;
......
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