Thread (17 messages) 17 messages, 5 authors, 2013-03-04
STALE4848d

[PATCH v10 6/8] usb: chipidea: imx: add internal vbus regulator control

From: Felipe Balbi <hidden>
Date: 2013-02-27 08:36:10

Hi Darius,

On Wed, Feb 27, 2013 at 04:13:14PM +0800, Peter Chen wrote:
On Wed, Feb 27, 2013 at 09:07:35AM +0100, Sascha Hauer wrote:
quoted
On Wed, Feb 27, 2013 at 09:55:10AM +0200, Felipe Balbi wrote:
quoted
Hi,

On Wed, Feb 27, 2013 at 02:33:17PM +0800, Peter Chen wrote:
quoted
- For host, the vbus should always be on.
- For otg, the vbus is off defaultly, the vbus needs to be
turned on/off when usb role switches.

Signed-off-by: Peter Chen <redacted>
one quick question, if chipidea already has an imx glue layer, can I
delete the broken imx_udc ? It doesn't even compile and it's including
headers which don't exist.
This driver handles i.MX1. This one has a gadget only core which is not
chipidea compatible. You should ping Darius Augulis [off-list ref]
if he is willing to put some love into the driver.
Sascha is correct.
do you plan to put any work on drivers/usb/gadget/imx_udc.c ? It doesn't
even compile since it includes headers which don't exist (and no,
simply removing the include doesn't help).

If nobody is going to work on that driver, I'm thinking about scheduling
it for removal. We can't keep such drivers that nobody cares (and just
break tree compilation) around.

-- 
balbi
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20130227/34c39604/attachment.sig>
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help