Re: [PATCH 0/1] TWL: USB: disable VUSB regulators

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Dave,

Sorry for the late response, I was out of office yesterday.

On Wed, 2009-01-21 at 12:38 -0800, David Brownell wrote:
> On Wednesday 21 January 2009, Kalle Jokiniemi wrote:
> > The TWL5030/4030 internal VUSB regulators are eating current
> > like small piggies.
> 
> Feral pigs are a big problem in many places ...
> no surprise that feral code is too!
> 
> 
> > The following patch makes an aptempt at 
> > making the twl4030-usb consumer driver dynamically disable
> > and enable the regulators it needs.
> 
> Thanks, I'll have a look.  It's a bit different
> from some old hacks I didn't finish debugging.
> 
> Shouldn't the power down sequence mirror the
> power up sequence?  Up 1.5, 1.8, 3.1 ... then
> down 3.1, 1.8, 1.5?  Or does that matter?

Good question. The TRM tells quite little on these VUSB regulators,
other than they supply the USB sub-chip in TWL. For VUSB3V1 there is a
bit more information, telling that it supplies USB PHY, CEA Carkit
modules and MCPC (seems to be some special usb UART specification). 

I would think they all can be shut down in whatever order, if there is
nothing connected to the usb connector.

> 
> Also, I'm thinking that the DEV_GRP and TYPE
> register updates belong in regulator code, not
> transceiver code.  That's something to take care
> of with a separate patch though.

I agree.

- Kalle 

> 
> - Dave
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux