Hi Uwe,

[Adding Matt]

On Fri, Nov 30, 2018 at 6:53 PM Uwe Kleine-König
[off-list ref] 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
   reset. usbmisc_imx6q_init() and usbmisc_imx7d_init() keep the current
   state (probably as setup by the bootloader or still the reset
   default) unless the polarity is explicitly configured as active high
   which then is configured. (So if the pin is active low and the
   bootloader didn't set this up the configuration is wrong.)

To improve the situation always configure the reset default value unless
the device tree configures the polarity (and then use this one).

Note that as the reset default is active high on all platforms there is
no need to check for the presence of "over-current-active-high". In the
absence of "over-current-active-low" it doesn't matter if active high is
configured because that's the default or because it is configured
explicitly.

Signed-off-by: Uwe Kleine-König <redacted>

It seems that Matt is also interested in adding the
'over-current-active-low' property:
https://patchwork.kernel.org/patch/10701311/

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel