Hi Andrey, Am Mittwoch, den 22.05.2019, 00:12 -0700 schrieb Andrey Smirnov: > Add node for UCS1002 USB charger chip connected to front panel USB and > replace "regulator-fixed" previously used to control VBUS.
I've had a similar version of this patch, but also added GPIO hogs for the UCS1002 configuration pins, so the device is put into the expected state even before driver load. Maybe something worth to consider? Regards, Lucas > Signed-off-by: Andrey Smirnov <[email protected]> > > Cc: Shawn Guo <[email protected]> > > Cc: Chris Healy <[email protected]> > > Cc: Fabio Estevam <[email protected]> > > Cc: Lucas Stach <[email protected]> > Cc: [email protected] > Cc: [email protected] > --- > arch/arm/boot/dts/imx6qdl-zii-rdu2.dtsi | 35 ++++++++++++------------- > 1 file changed, 17 insertions(+), 18 deletions(-) > > diff --git a/arch/arm/boot/dts/imx6qdl-zii-rdu2.dtsi > b/arch/arm/boot/dts/imx6qdl-zii-rdu2.dtsi > index 93be00a60c88..977d923e35df 100644 > --- a/arch/arm/boot/dts/imx6qdl-zii-rdu2.dtsi > +++ b/arch/arm/boot/dts/imx6qdl-zii-rdu2.dtsi > @@ -60,18 +60,6 @@ > > regulator-always-on; > > }; > > > - reg_5p0v_user_usb: regulator-5p0v-user-usb { > > - compatible = "regulator-fixed"; > > - pinctrl-names = "default"; > > - pinctrl-0 = <&pinctrl_reg_user_usb>; > > - vin-supply = <®_5p0v_main>; > > - regulator-name = "5V_USER_USB"; > > - regulator-min-microvolt = <5000000>; > > - regulator-max-microvolt = <5000000>; > > - gpio = <&gpio3 22 GPIO_ACTIVE_LOW>; > > - startup-delay-us = <1000>; > > - }; > - > > reg_3p3v_pmic: regulator-3p3v-pmic { > > compatible = "regulator-fixed"; > > vin-supply = <®_12p0v>; > @@ -590,6 +578,16 @@ > > status = "disabled"; > > }; > > > > + reg_5p0v_user_usb: charger@32 { > > + compatible = "microchip,ucs1002"; > > + pinctrl-names = "default"; > > + pinctrl-0 = <&pinctrl_ucs1002_pins>; > > + reg = <0x32>; > > + interrupts-extended = <&gpio5 2 IRQ_TYPE_EDGE_BOTH>, > > + <&gpio3 21 IRQ_TYPE_EDGE_BOTH>; > > + interrupt-names = "a_det", "alert"; > > + }; > + > > > hpa1: amp@60 { > > compatible = "ti,tpa6130a2"; > > pinctrl-names = "default"; > @@ -982,12 +980,6 @@ > > >; > > }; > > > - pinctrl_reg_user_usb: usbotggrp { > > - fsl,pins = < > > > - MX6QDL_PAD_EIM_D22__GPIO3_IO22 0x40000038 > > - >; > > - }; > - > > pinctrl_rmii_phy_irq: phygrp { > > fsl,pins = < > > > MX6QDL_PAD_EIM_D30__GPIO3_IO30 0x40010000 > @@ -1047,6 +1039,13 @@ > > >; > > }; > > > + pinctrl_ucs1002_pins: ucs1002grp { > > + fsl,pins = < > > > + MX6QDL_PAD_EIM_A25__GPIO5_IO02 0x1b0b0 > > > + MX6QDL_PAD_EIM_D21__GPIO3_IO21 0x1b0b0 > > + >; > > + }; > + > > pinctrl_usdhc2: usdhc2grp { > > fsl,pins = < > > > MX6QDL_PAD_SD2_CMD__SD2_CMD 0x10059
