RE: [PATCH v2 1/3] usb: chipidea: imx: support configuring for active low oc signal

2018-12-03 Thread Matthew Starr
> -Original Message- > From: Uwe Kleine-König [mailto:u.kleine-koe...@pengutronix.de] > Sent: Sunday, December 02, 2018 3:33 PM > > The status quo on i.MX6 is that if "over-current-active-high" is > specified in the device tree this is configured as expected. If > the property is missing

RE: [PATCH 1/1] usb: chipidea: imx: improve the over current handling

2018-12-03 Thread Matthew Starr
ity as high explicitly; > > > > otherwise, we set it as low explicitly. > > > > > > > > Cc: stable > > > > Cc: Peter Chen > > > > Cc: Uwe Kleine-König > > > > Cc: Matthew Starr > > > > Signed-off-by: Peter Chen &

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
-current-active-high property with the over-current-active-low property. Without this property the over current polarity will be active high by default like it always has been. With the property the over current polarity will be active low. Signed-off-by: Matthew Starr --- .../devicetree/bindings/usb

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

2018-11-30 Thread Matthew Starr
been missed up till now because the bit next to the OVER_CUR_POL bit is the PWR_POL which has the exact inverse logic with a value of 1 meaning high enable and 0 meaning low enable. Also I rarely have seen the USB interface go into an overcurrent situation so maybe no one ever caught thi

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

2018-11-27 Thread Matthew Starr
-current-active-high property with the over-current-active-low property. Without this property the over current polarity will be active high by default like it always has been. With the property the over current polarity will be active low. Signed-off-by: Matthew Starr --- Documentation/devicetree