diff options
author | Frans Pop <elendil@planet.nl> | 2009-10-02 13:04:12 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2009-10-02 13:04:12 -0400 |
commit | 19d5afd4f0d26201d8d8bec351ee0442775a5379 (patch) | |
tree | 8bf3b440d53c2e3a60c0a0ea69cb99bc3b729428 /drivers/net/ixgbe/ixgbe_main.c | |
parent | b607bd900051efc3308c4edc65dd98b34b230021 (diff) |
e1000e/igb/ixgbe: Don't report an error if devices don't support AER
The only error returned by pci_{en,dis}able_pcie_error_reporting() is
-EIO which simply means that Advanced Error Reporting is not supported.
There is no need to report that, so remove the error check from e1000e,
igb and ixgbe.
Signed-off-by: Frans Pop <elendil@planet.nl>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ixgbe/ixgbe_main.c')
-rw-r--r-- | drivers/net/ixgbe/ixgbe_main.c | 13 |
1 files changed, 2 insertions, 11 deletions
diff --git a/drivers/net/ixgbe/ixgbe_main.c b/drivers/net/ixgbe/ixgbe_main.c index 1cbc6a318b67..28fbb9d281f9 100644 --- a/drivers/net/ixgbe/ixgbe_main.c +++ b/drivers/net/ixgbe/ixgbe_main.c | |||
@@ -5507,12 +5507,7 @@ static int __devinit ixgbe_probe(struct pci_dev *pdev, | |||
5507 | goto err_pci_reg; | 5507 | goto err_pci_reg; |
5508 | } | 5508 | } |
5509 | 5509 | ||
5510 | err = pci_enable_pcie_error_reporting(pdev); | 5510 | pci_enable_pcie_error_reporting(pdev); |
5511 | if (err) { | ||
5512 | dev_err(&pdev->dev, "pci_enable_pcie_error_reporting failed " | ||
5513 | "0x%x\n", err); | ||
5514 | /* non-fatal, continue */ | ||
5515 | } | ||
5516 | 5511 | ||
5517 | pci_set_master(pdev); | 5512 | pci_set_master(pdev); |
5518 | pci_save_state(pdev); | 5513 | pci_save_state(pdev); |
@@ -5821,7 +5816,6 @@ static void __devexit ixgbe_remove(struct pci_dev *pdev) | |||
5821 | { | 5816 | { |
5822 | struct net_device *netdev = pci_get_drvdata(pdev); | 5817 | struct net_device *netdev = pci_get_drvdata(pdev); |
5823 | struct ixgbe_adapter *adapter = netdev_priv(netdev); | 5818 | struct ixgbe_adapter *adapter = netdev_priv(netdev); |
5824 | int err; | ||
5825 | 5819 | ||
5826 | set_bit(__IXGBE_DOWN, &adapter->state); | 5820 | set_bit(__IXGBE_DOWN, &adapter->state); |
5827 | /* clear the module not found bit to make sure the worker won't | 5821 | /* clear the module not found bit to make sure the worker won't |
@@ -5872,10 +5866,7 @@ static void __devexit ixgbe_remove(struct pci_dev *pdev) | |||
5872 | 5866 | ||
5873 | free_netdev(netdev); | 5867 | free_netdev(netdev); |
5874 | 5868 | ||
5875 | err = pci_disable_pcie_error_reporting(pdev); | 5869 | pci_disable_pcie_error_reporting(pdev); |
5876 | if (err) | ||
5877 | dev_err(&pdev->dev, | ||
5878 | "pci_disable_pcie_error_reporting failed 0x%x\n", err); | ||
5879 | 5870 | ||
5880 | pci_disable_device(pdev); | 5871 | pci_disable_device(pdev); |
5881 | } | 5872 | } |