RE: [PATCH] usb: chipidea: imx: unify over-current polarity handling

2018-11-30 Thread Matthew Starr
> -Original Message- > From: Uwe Kleine-König [mailto:u.kleine-koe...@pengutronix.de] > Sent: Friday, November 30, 2018 3:35 PM > > Hello Fabio, > > On Fri, Nov 30, 2018 at 07:16:39PM -0200, Fabio Estevam wrote: > > [Adding Matt] > > > > On Fri, Nov 30, 2018 at 6:53 PM Uwe Kleine-König >

[PATCH v2] usb: chipidea: imx: Allow OC polarity active low

2018-11-30 Thread Matthew Starr
The implementation for setting the over current polarity has always been the over-current-active-high property. The problem with this is there is no way to enable over current active low polarity since the default state of the register bit that controls the over current polarity is a 0 which

Re: [PATCH] usb: chipidea: imx: unify over-current polarity handling

2018-11-30 Thread Uwe Kleine-König
Hello Fabio, On Fri, Nov 30, 2018 at 07:16:39PM -0200, Fabio Estevam wrote: > [Adding Matt] > > On Fri, Nov 30, 2018 at 6:53 PM Uwe Kleine-König > wrote: > > > > The status quo is: > > - on i.MX25 the overcurrent polarity is always explicitly configured as > >active high which matches the

Re: [PATCH] usb: chipidea: imx: unify over-current polarity handling

2018-11-30 Thread Fabio Estevam
Hi Uwe, [Adding Matt] On Fri, Nov 30, 2018 at 6:53 PM Uwe Kleine-König wrote: > > The status quo is: > - on i.MX25 the overcurrent polarity is always explicitly configured as >active high which matches the reset default. > - on i.MX6 and i.MX7 the overcurrent polarity is active high after

[PATCH] usb: chipidea: imx: unify over-current polarity handling

2018-11-30 Thread Uwe Kleine-König
The status quo is: - on i.MX25 the overcurrent polarity is always explicitly configured as active high which matches the reset default. - on i.MX6 and i.MX7 the overcurrent polarity is active high after reset. usbmisc_imx6q_init() and usbmisc_imx7d_init() keep the current state

Re: Support Mac address pass through on Dell DS1000 dock

2018-11-30 Thread Frédéric Parrenin
Le 29/11/2018 à 15:10, mario.limoncie...@dell.com a écrit : I tried with your patch but the Mac address pass-through still does not work with the usbc adaptor, so it probably does not support it. So I will buy a Dell one, it should work. Thanks anyway. Frédéric OK, sounds good. For your

RE: [PATCH] usb: chipidea: imx: Allow OC polarity active low

2018-11-30 Thread Matthew Starr
> -Original Message- > From: PETER CHEN [mailto:peter.c...@nxp.com] > Sent: Thursday, November 29, 2018 9:30 PM > > > > > The implementation for setting the over current polarity has always been > > the over- > > current-active-high property. The problem with this is there is no way to