diff mbox series

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

Message ID 1616563105-28565-1-git-send-email-jun.li@nxp.com
State New
Headers show
Series [v2] usb: dwc3: core: don't enable wakeup for runtime PM | expand

Commit Message

Jun Li March 24, 2021, 5:18 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.

Fixes: fc8bb91bc83e ("usb: dwc3: implement runtime PM")
Cc: <Stable@vger.kernel.org>
Signed-off-by: Li Jun <jun.li@nxp.com>
---
Change for v2:
- Add fix tag and cc to stable kernel.

 drivers/usb/dwc3/core.c | 4 ----
 1 file changed, 4 deletions(-)

Comments

Greg KH March 26, 2021, 1:38 p.m. UTC | #1
On Wed, Mar 24, 2021 at 01:18:25PM +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.

> 

> Fixes: fc8bb91bc83e ("usb: dwc3: implement runtime PM")

> Cc: <Stable@vger.kernel.org>

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

> ---

> Change for v2:

> - Add fix tag and cc to stable kernel.

> 

>  drivers/usb/dwc3/core.c | 4 ----

>  1 file changed, 4 deletions(-)

> 

> diff --git a/drivers/usb/dwc3/core.c b/drivers/usb/dwc3/core.c

> index 94fdbe502ce9..0bba5c21de56 100644

> --- a/drivers/usb/dwc3/core.c

> +++ b/drivers/usb/dwc3/core.c

> @@ -1840,8 +1840,6 @@ static int dwc3_runtime_suspend(struct device *dev)

>  	if (ret)

>  		return ret;

>  

> -	device_init_wakeup(dev, true);

> -

>  	return 0;

>  }

>  

> @@ -1850,8 +1848,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;

> -- 

> 2.25.1

> 


Can I get some verification by other users of this driver that this
actually fixes something?  I find it odd it's not shown up yet to anyone
else...

thanks,

greg k-h
Jun Li March 27, 2021, 4:07 p.m. UTC | #2
> -----Original Message-----

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

> Sent: Friday, March 26, 2021 9:39 PM

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

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

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

> PM

> 

> On Wed, Mar 24, 2021 at 01:18:25PM +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.

> >

> > Fixes: fc8bb91bc83e ("usb: dwc3: implement runtime PM")

> > Cc: <Stable@vger.kernel.org>

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

> > ---

> > Change for v2:

> > - Add fix tag and cc to stable kernel.

> >

> >  drivers/usb/dwc3/core.c | 4 ----

> >  1 file changed, 4 deletions(-)

> >

> > diff --git a/drivers/usb/dwc3/core.c b/drivers/usb/dwc3/core.c index

> > 94fdbe502ce9..0bba5c21de56 100644

> > --- a/drivers/usb/dwc3/core.c

> > +++ b/drivers/usb/dwc3/core.c

> > @@ -1840,8 +1840,6 @@ static int dwc3_runtime_suspend(struct device *dev)

> >  	if (ret)

> >  		return ret;

> >

> > -	device_init_wakeup(dev, true);

> > -

> >  	return 0;

> >  }

> >

> > @@ -1850,8 +1848,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;

> > --

> > 2.25.1

> >

> 

> Can I get some verification by other users of this driver that this

> actually fixes something?  I find it odd it's not shown up yet to anyone

> else...


This is not a common case for dwc3 users, I guess only a few users
support runtime PM as it's disabled by default, even with runtime PM
supported and enabled in my case, system sleeping with dwc3 core wakeup
enabled does not break any function, it's just leave the power domain
(attached to dwc3 core's parent device) _on_ even I don't need any
usb remote wakeup, so it's trying to "fix" a power waste.

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 94fdbe502ce9..0bba5c21de56 100644
--- a/drivers/usb/dwc3/core.c
+++ b/drivers/usb/dwc3/core.c
@@ -1840,8 +1840,6 @@  static int dwc3_runtime_suspend(struct device *dev)
 	if (ret)
 		return ret;
 
-	device_init_wakeup(dev, true);
-
 	return 0;
 }
 
@@ -1850,8 +1848,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;