Re: [PATCH] usb: host: ohci-platform: Implement ohci_platform_shutdown

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



* Alan Stern <stern@xxxxxxxxxxxxxxxxxxx> [190322 20:03]:
> On Fri, 22 Mar 2019, Tony Lindgren wrote:
> 
> > * Alan Stern <stern@xxxxxxxxxxxxxxxxxxx> [190322 18:37]:
> > > How about putting these runtime PM additions into
> > > usb_hcd_platform_shutdown instead, so they will apply to all platform
> > > controller drivers?
> > 
> > OK let's do that then.
> > 
> > > Also, are you certain you want the pm_runtime_put_sync at the end?  If 
> > > the system is shutting down anyway, why waste time doing an extra 
> > > runtime suspend?
> > 
> > Well mostly to keep the calls paired. But maybe there are
> > also kexec reboot cases where we'd want to have things
> > properly disabled for PM before kexec.
> 
> I'm not sure that makes sense.  You can't actually disable anything for 
> runtime PM from within a driver; all you can do is tell the runtime PM 
> core that _you're_ not using the device any more.  But if some other 
> part of the system is still using it, it will remain at full power.

Right, it's just a usecount and whatever happens after that is
ouf of the driver control. And in the shutdown case PM runtime
does not have much of a chance to do anything here :)

Not much point checking the pm_runtime_get_sync() for errors
either then.. We can just add a comment there.

If the patch below looks OK to you I'll post with an updated
description.

Regards,

Tony

8< -------------------
diff --git a/drivers/usb/core/hcd.c b/drivers/usb/core/hcd.c
--- a/drivers/usb/core/hcd.c
+++ b/drivers/usb/core/hcd.c
@@ -3017,6 +3017,9 @@ usb_hcd_platform_shutdown(struct platform_device *dev)
 {
 	struct usb_hcd *hcd = platform_get_drvdata(dev);
 
+	/* No need for pm_runtime_put(), we're shutting down */
+	pm_runtime_get_sync(&dev->dev);
+
 	if (hcd->driver->shutdown)
 		hcd->driver->shutdown(hcd);
 }



[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux