diff mbox

[v3] ASoC: Intel: Skylake: remove call to pci_dev_put

Message ID 1458144438-28912-1-git-send-email-vinod.koul@intel.com (mailing list archive)
State New, archived
Headers show

Commit Message

Vinod Koul March 16, 2016, 4:07 p.m. UTC 
The PCI bus takes pci_dev_get() and pci_dev_put() is also there.
So no need for drivers to invoke these. In SKL driver we were
calling pci_dev_put() only which is not right, so remove this

Signed-off-by: Vinod Koul <vinod.koul@intel.com>
---
 sound/soc/intel/skylake/skl.c | 1 -
 1 file changed, 1 deletion(-)

Comments

Mark Brown March 16, 2016, 4:07 p.m. UTC | #1 
On Wed, Mar 16, 2016 at 09:37:18PM +0530, Vinod Koul wrote:
> The PCI bus takes pci_dev_get() and pci_dev_put() is also there.
> So no need for drivers to invoke these. In SKL driver we were
> calling pci_dev_put() only which is not right, so remove this

This doesn't seem to apply against current code (I don't have the
relevant SHA1s for the dependency) - can you please check and resend?
Vinod Koul March 16, 2016, 4:16 p.m. UTC | #2 
On Wed, Mar 16, 2016 at 04:07:17PM +0000, Mark Brown wrote:
> On Wed, Mar 16, 2016 at 09:37:18PM +0530, Vinod Koul wrote:
> > The PCI bus takes pci_dev_get() and pci_dev_put() is also there.
> > So no need for drivers to invoke these. In SKL driver we were
> > calling pci_dev_put() only which is not right, so remove this
> 
> This doesn't seem to apply against current code (I don't have the
> relevant SHA1s for the dependency) - can you please check and resend?

Thats surprising as I amended the commit in last series and sent this one. I
will re-fetch your tree and do this again.

Thanks
Mark Brown March 16, 2016, 4:18 p.m. UTC | #3 
On Wed, Mar 16, 2016 at 09:46:05PM +0530, Vinod Koul wrote:
> On Wed, Mar 16, 2016 at 04:07:17PM +0000, Mark Brown wrote:

> > This doesn't seem to apply against current code (I don't have the
> > relevant SHA1s for the dependency) - can you please check and resend?

> Thats surprising as I amended the commit in last series and sent this one. I
> will re-fetch your tree and do this again.

Is it depending on some -next only stuff or something?
diff mbox

Patch

diff --git a/sound/soc/intel/skylake/skl.c b/sound/soc/intel/skylake/skl.c
index b67f3bd0fccd..c337d06793ec 100644
--- a/sound/soc/intel/skylake/skl.c
+++ b/sound/soc/intel/skylake/skl.c
@@ -732,7 +732,6 @@  static void skl_remove(struct pci_dev *pci)
 
 	if (pci_dev_run_wake(pci))
 		pm_runtime_get_noresume(&pci->dev);
-	pci_dev_put(pci);
 	skl_codecs_remove(ebus);
 	skl_platform_unregister(&pci->dev);
 	skl_free_dsp(skl);