ASoC: Intel: Skylake: remove pci device enabling calls on suspend
authorJayachandran B <jayachandran.b@intel.com>
Thu, 3 Nov 2016 11:37:22 +0000 (17:07 +0530)
committerMark Brown <broonie@kernel.org>
Wed, 9 Nov 2016 15:22:48 +0000 (15:22 +0000)
We were invoking pci_disable_device() while going to suspend-to-idle and
pci_enable_device() while coming back to active state.

Turns out that we do not need these calls as we only need system to be
wake capable when in suspend-to-idle state. The wake capability is
already done by enable_irq_wake() calls, so remove these unwanted calls
in driver.

Signed-off-by: Jayachandran B <jayachandran.b@intel.com>
Signed-off-by: Vinod Koul <vinod.koul@intel.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/intel/skylake/skl.c

index b9209af89915bc3b95b214a9c7d7df58aae6616e..ed59783e98461ba8fc9281c246aaf931de32a420 100644 (file)
@@ -290,7 +290,6 @@ static int skl_suspend(struct device *dev)
 
                enable_irq_wake(bus->irq);
                pci_save_state(pci);
-               pci_disable_device(pci);
        } else {
                ret = _skl_suspend(ebus);
                if (ret < 0)
@@ -333,7 +332,6 @@ static int skl_resume(struct device *dev)
         */
        if (skl->supend_active) {
                pci_restore_state(pci);
-               ret = pci_enable_device(pci);
                snd_hdac_ext_bus_link_power_up_all(ebus);
                disable_irq_wake(bus->irq);
                /*