diff mbox series

usb: dwc3: core: don't enable wakeup for runtime PM

Message ID 1614767857-12723-1-git-send-email-jun.li@nxp.com
State Superseded
Headers show
Series usb: dwc3: core: don't enable wakeup for runtime PM | expand

Commit Message

Jun Li March 3, 2021, 10:37 a.m. UTC
Don't enable dwc3 core to wakeup system by default if runtime suspended,
this will make the power domain of dwc3 core or its parent device can't be
off while system sleep even system wakeup by usb is not required.

Signed-off-by: Li Jun <jun.li@nxp.com>
---
 drivers/usb/dwc3/core.c | 4 ----
 1 file changed, 4 deletions(-)

Comments

Greg Kroah-Hartman March 23, 2021, 11:52 a.m. UTC | #1
On Wed, Mar 03, 2021 at 06:37:37PM +0800, Li Jun wrote:
> Don't enable dwc3 core to wakeup system by default if runtime suspended,

> this will make the power domain of dwc3 core or its parent device can't be

> off while system sleep even system wakeup by usb is not required.

> 

> Signed-off-by: Li Jun <jun.li@nxp.com>

> ---


What commit does this fix?  Does it need to go to stable kernels?  If
so, how far back?

Please add that information to the signed-off-by area when resending.

thanks,

greg k-h
Jun Li March 24, 2021, 5:54 a.m. UTC | #2
> -----Original Message-----

> From: Greg KH <gregkh@linuxfoundation.org>

> Sent: Tuesday, March 23, 2021 7:52 PM

> To: Jun Li <jun.li@nxp.com>

> Cc: balbi@kernel.org; linux-usb@vger.kernel.org

> Subject: Re: [PATCH] usb: dwc3: core: don't enable wakeup for runtime PM

> 

> On Wed, Mar 03, 2021 at 06:37:37PM +0800, Li Jun wrote:

> > Don't enable dwc3 core to wakeup system by default if runtime

> > suspended, this will make the power domain of dwc3 core or its parent

> > device can't be off while system sleep even system wakeup by usb is not

> required.

> >

> > Signed-off-by: Li Jun <jun.li@nxp.com>

> > ---

> 

> What commit does this fix?  Does it need to go to stable kernels?  If so,

> how far back?

> 

> Please add that information to the signed-off-by area when resending.


Just sent out v2 with fix tag and cc to stable kernel.

Thanks
Li Jun
> 

> thanks,

> 

> greg k-h
diff mbox series

Patch

diff --git a/drivers/usb/dwc3/core.c b/drivers/usb/dwc3/core.c
index 6969196..a8466f6 100644
--- a/drivers/usb/dwc3/core.c
+++ b/drivers/usb/dwc3/core.c
@@ -1788,8 +1788,6 @@  static int dwc3_runtime_suspend(struct device *dev)
 	if (ret)
 		return ret;
 
-	device_init_wakeup(dev, true);
-
 	return 0;
 }
 
@@ -1798,8 +1796,6 @@  static int dwc3_runtime_resume(struct device *dev)
 	struct dwc3     *dwc = dev_get_drvdata(dev);
 	int		ret;
 
-	device_init_wakeup(dev, false);
-
 	ret = dwc3_resume_common(dwc, PMSG_AUTO_RESUME);
 	if (ret)
 		return ret;