iwlwifi: check return value of pci_enable_device
pci_enable_device is tagged with __must_check therefore don't ignore the return value in pci_resume handlers Signed-off-by: Tomas Winkler <tomas.winkler@intel.com> Signed-off-by: Reinette Chatre <reinette.chatre@intel.com> Signed-off-by: John W. Linville <linville@tuxdriver.com>
This commit is contained in:
parent
dfb39e8295
commit
450154e4f4
2 changed files with 8 additions and 2 deletions
|
@ -3767,9 +3767,12 @@ static int iwl_pci_suspend(struct pci_dev *pdev, pm_message_t state)
|
||||||
static int iwl_pci_resume(struct pci_dev *pdev)
|
static int iwl_pci_resume(struct pci_dev *pdev)
|
||||||
{
|
{
|
||||||
struct iwl_priv *priv = pci_get_drvdata(pdev);
|
struct iwl_priv *priv = pci_get_drvdata(pdev);
|
||||||
|
int ret;
|
||||||
|
|
||||||
pci_set_power_state(pdev, PCI_D0);
|
pci_set_power_state(pdev, PCI_D0);
|
||||||
pci_enable_device(pdev);
|
ret = pci_enable_device(pdev);
|
||||||
|
if (ret)
|
||||||
|
return ret;
|
||||||
pci_restore_state(pdev);
|
pci_restore_state(pdev);
|
||||||
iwl_enable_interrupts(priv);
|
iwl_enable_interrupts(priv);
|
||||||
|
|
||||||
|
|
|
@ -5824,9 +5824,12 @@ static int iwl3945_pci_suspend(struct pci_dev *pdev, pm_message_t state)
|
||||||
static int iwl3945_pci_resume(struct pci_dev *pdev)
|
static int iwl3945_pci_resume(struct pci_dev *pdev)
|
||||||
{
|
{
|
||||||
struct iwl_priv *priv = pci_get_drvdata(pdev);
|
struct iwl_priv *priv = pci_get_drvdata(pdev);
|
||||||
|
int ret;
|
||||||
|
|
||||||
pci_set_power_state(pdev, PCI_D0);
|
pci_set_power_state(pdev, PCI_D0);
|
||||||
pci_enable_device(pdev);
|
ret = pci_enable_device(pdev);
|
||||||
|
if (ret)
|
||||||
|
return ret;
|
||||||
pci_restore_state(pdev);
|
pci_restore_state(pdev);
|
||||||
|
|
||||||
if (priv->is_open)
|
if (priv->is_open)
|
||||||
|
|
Loading…
Add table
Reference in a new issue