Re: [PATCH v4 04/10] usb: ehci-orion: add S2RAM support
flat view
From: Alan Stern <stern@rowland.harvard.edu>
Date: 2019-01-28 15:27:53
Also in:
linux-devicetree, linux-usb
On Mon, 28 Jan 2019, Miquel Raynal wrote:
Hi Alan, Alan Stern [off-list ref] wrote on Fri, 25 Jan 2019 21:56:27 -0500 (EST):quoted
On Fri, 25 Jan 2019, Miquel Raynal wrote:quoted
Add suspend/resume callbacks to reset the host controller properly during S2RAM operation. Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com> --- drivers/usb/host/ehci-orion.c | 18 ++++++++++++++++++ 1 file changed, 18 insertions(+)diff --git a/drivers/usb/host/ehci-orion.c b/drivers/usb/host/ehci-orion.c index 3109f082949e..0dab099b5d15 100644 --- a/drivers/usb/host/ehci-orion.c +++ b/drivers/usb/host/ehci-orion.c@@ -182,6 +182,23 @@ static int ehci_orion_drv_reset(struct usb_hcd *hcd) return ret; } +static int __maybe_unused ehci_orion_drv_suspend(struct device *dev) +{ + struct usb_hcd *hcd = dev_get_drvdata(dev); + + return ehci_suspend(hcd, device_may_wakeup(dev));Okay, good.quoted
+} + +static int __maybe_unused ehci_orion_drv_resume(struct device *dev) +{ + struct usb_hcd *hcd = dev_get_drvdata(dev); + + return ehci_resume(hcd, device_may_wakeup(dev));Not so good. The second argument here is force_reset; presumably you want it always to be false. (Note that the v3 version of this patch did not call device_may_wakeup() in ehci_orion_drv_resume.) Yes, the API is not symmetrical. So sue me...Oh right, I completely overlooked that one. Indeed the second parameter should be "false", as in the v3. Do you mind if I send a v5 only for this patch? If the rest looks good to you of course.
To avoid confusing Greg, it would be best to send v5 of all the patches that he should apply. The rest of the EHCI changes look okay, and when you fix this you can add: Acked-by: Alan Stern <stern@rowland.harvard.edu> Alan Stern _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel