On Wed, Feb 03, 2016 at 11:58:33AM -0500, Alan Stern wrote:
> On Wed, 3 Feb 2016, Fabio Estevam wrote:
>
> > Hi Alan,
> >
> > On Wed, Feb 3, 2016 at 2:29 PM, Alan Stern <stern@xxxxxxxxxxxxxxxxxxx> wrote:
> >
> > > It seems likely that this script does exactly the same thing that you
> > > did before by hand. Why does one work but not the other?
> >
> > The script enables all of these entries:
> >
> > $ find /sys -name wakeup | grep usb
> > /sys/devices/soc0/soc/2000000.aips-bus/20c9000.usbphy/power/wakeup
> > /sys/devices/soc0/soc/2000000.aips-bus/20ca000.usbphy/power/wakeup
> > /sys/devices/soc0/soc/2100000.aips-bus/2184000.usb/power/wakeup
> > /sys/devices/soc0/soc/2100000.aips-bus/2184000.usb/ci_hdrc.0/power/wakeup
> > /sys/devices/soc0/soc/2100000.aips-bus/2184200.usb/power/wakeup
>
> Okay. As far as I know, those files should have been set to "enabled"
> by default.
>
No, the power consumption will be higher if we enable controller
and phy's wakeup setting default since several regulators needs
to be on to know dp/dm/vbus change.
Besides, there are no wakeup setting for gadget, but we may still
need wakeup for gadget, eg, vbus wakeup, and wakeup from bus
reset/resume, but the default wakeup setting should be off, the user
may be surprise why plug in USB cable wakes up system.
> > /sys/devices/soc0/soc/2100000.aips-bus/2184200.usb/ci_hdrc.1/usb1/1-1/power/wakeup
> > /sys/devices/soc0/soc/2100000.aips-bus/2184200.usb/ci_hdrc.1/usb1/power/wakeup
> > /sys/devices/soc0/soc/2100000.aips-bus/2184200.usb/ci_hdrc.1/power/wakeup
>
> And so should that one. So maybe the problem is that the default
> settings are wrong.
>
> Alan Stern
Yes, maybe for others, but I have a patch before to let the individual
controller driver to decide wakeup setting.
commit 3c9740a117d40a74412775b5d3fe2b88a7635a0e
Author: Peter Chen <peter.chen@xxxxxxxxxxxxx>
Date: Tue Nov 5 10:46:02 2013 +0800
usb: hcd: move controller wakeup setting initialization to
individual driver
--
Best Regards,
Peter Chen
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
