Re: [PATCH 0/6] twl4030: Various fixes for charing-from-USB

[NeilBrown <neilb@suse.de>]

[-- Attachment #1: Type: text/plain, Size: 2929 bytes --]

On Thu, 10 May 2012 09:24:23 +0800 Peter Chen <peter.chen@freescale.com>
wrote:

> On Wed, Apr 25, 2012 at 05:33:10PM +1000, NeilBrown wrote:
> > Following are a collection of patches that I've need using for a while
> > to make sure the charge-from-usb on my GTA04 works.
> > Hopefully I've included the right people in the recipient list :-)
> > 
> > The issues are:
> >  - charge the backup battery as well as the main battery
> >  - charge from a charger which ties ID to ground via a resistor
> >  - charge while device is suspended, or when no gadget module is
> >    loaded (i.e. when the USB side thinks the phy should be powered
> >    down).
> According to USB Spec, when the USB bus goes to suspend, the device
> should not draw more than 2.5mA. So when the device is suspended,
> it should notify the battery driver do NOT charge any more.

Hi Peter,
 thanks for your input.

 Would you be able to help me understand exactly what it means for the USB
 bus to go to suspend?  Is this something that the host initiates or
 something that the gadget initiates?
 How does this relate to a simple charger?  It that considered to have a
 non-suspended bus, or is it considered as a separate case?

 Maybe we need to not suspend the USB OTG interface when the device as a
 whole enters suspend-to-RAM.  Maybe we need to register a dummy gadget to
 the bus active while in suspend?
 Does the gadget have to be responsive while-ever the bus is not suspended?
 Maybe we can arrange for any USB activity to trigger a resume?

 Entering suspend while still charging really should be possible - at least
 with a dump charger and hopefully from a host port as well.  I'll like to
 find the best way to achieve that, and any help you can provide would be
 greatly appreciated.

Thanks,
NeilBrown


> > 
> > Questions and comments more welcome.
> > 
> > Thanks,
> > NeilBrown
> > 
> > 
> > ---
> > 
> > NeilBrown (6):
> >       twl4030-usb: Don't report EVENT_ID when there is VBUS.
> >       twl4030-usb: Don't power down phy when it is in-use by charger.
> >       twl4030_charger: Allow charger to control the regulator that feeds it.
> >       twl4030_charger: allow charging whenever VBUS is present.
> >       twl4030_charger: add backup-battery charging.
> >       twl4030_charger: Fix some typos
> > 
> > 
> >  drivers/mfd/twl-core.c          |    9 ++--
> >  drivers/power/twl4030_charger.c |   86 +++++++++++++++++++++++++++++++++++----
> >  drivers/usb/otg/twl4030-usb.c   |   27 ++++++++----
> >  include/linux/i2c/twl.h         |    2 +
> >  4 files changed, 102 insertions(+), 22 deletions(-)
> > 
> > -- 
> > Signature
> > 
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-usb" in
> > the body of a message to majordomo@vger.kernel.org
> > More majordomo info at  http://vger.kernel.org/majordomo-info.html
> > 
> 


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 828 bytes --]