Commit 00d594c3 authored by Kalesh AP's avatar Kalesh AP Committed by David S. Miller

be2net: fix failure case in setting flow control

When the FW cmd to set flow control fails, the adapter state must simply
reflect the old values.
Signed-off-by: default avatarKalesh AP <kalesh.purayil@emulex.com>
Signed-off-by: default avatarSathya Perla <sathya.perla@emulex.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 0700d816
...@@ -705,15 +705,17 @@ be_set_pauseparam(struct net_device *netdev, struct ethtool_pauseparam *ecmd) ...@@ -705,15 +705,17 @@ be_set_pauseparam(struct net_device *netdev, struct ethtool_pauseparam *ecmd)
if (ecmd->autoneg != adapter->phy.fc_autoneg) if (ecmd->autoneg != adapter->phy.fc_autoneg)
return -EINVAL; return -EINVAL;
adapter->tx_fc = ecmd->tx_pause;
adapter->rx_fc = ecmd->rx_pause;
status = be_cmd_set_flow_control(adapter, status = be_cmd_set_flow_control(adapter, ecmd->tx_pause,
adapter->tx_fc, adapter->rx_fc); ecmd->rx_pause);
if (status) if (status) {
dev_warn(&adapter->pdev->dev, "Pause param set failed\n"); dev_warn(&adapter->pdev->dev, "Pause param set failed\n");
return be_cmd_status(status); return be_cmd_status(status);
}
adapter->tx_fc = ecmd->tx_pause;
adapter->rx_fc = ecmd->rx_pause;
return 0;
} }
static int be_set_phys_id(struct net_device *netdev, static int be_set_phys_id(struct net_device *netdev,
......
...@@ -3667,7 +3667,6 @@ int be_update_queues(struct be_adapter *adapter) ...@@ -3667,7 +3667,6 @@ int be_update_queues(struct be_adapter *adapter)
static int be_setup(struct be_adapter *adapter) static int be_setup(struct be_adapter *adapter)
{ {
struct device *dev = &adapter->pdev->dev; struct device *dev = &adapter->pdev->dev;
u32 tx_fc, rx_fc;
int status; int status;
be_setup_init(adapter); be_setup_init(adapter);
...@@ -3717,11 +3716,14 @@ static int be_setup(struct be_adapter *adapter) ...@@ -3717,11 +3716,14 @@ static int be_setup(struct be_adapter *adapter)
be_cmd_get_acpi_wol_cap(adapter); be_cmd_get_acpi_wol_cap(adapter);
be_cmd_get_flow_control(adapter, &tx_fc, &rx_fc); status = be_cmd_set_flow_control(adapter, adapter->tx_fc,
if (rx_fc != adapter->rx_fc || tx_fc != adapter->tx_fc)
be_cmd_set_flow_control(adapter, adapter->tx_fc,
adapter->rx_fc); adapter->rx_fc);
if (status)
be_cmd_get_flow_control(adapter, &adapter->tx_fc,
&adapter->rx_fc);
dev_info(&adapter->pdev->dev, "HW Flow control - TX:%d RX:%d\n",
adapter->tx_fc, adapter->rx_fc);
if (be_physfn(adapter)) if (be_physfn(adapter))
be_cmd_set_logical_link_config(adapter, be_cmd_set_logical_link_config(adapter,
......
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