ARM: dts: imx6*-hummingboard2: rework regulators
authorLucas Stach <dev@lynxeye.de>
Mon, 11 Dec 2017 16:58:34 +0000 (16:58 +0000)
committerShawn Guo <shawnguo@kernel.org>
Tue, 26 Dec 2017 08:15:44 +0000 (16:15 +0800)
Name the regulators according to the voltage rails they are feeding in
the schematic. The USB 2-4 regulators are fixed regulators and are always
on, as they are not a VBUS supply for a single USB host port on the i.MX6,
but supply VBUS to ports behind a USB hub and there is currently no way
to model this in mainline.

Signed-off-by: Lucas Stach <dev@lynxeye.de>
Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk>
Reviewed-by: Fabio Estevam <fabio.estevam@nxp.com>
Signed-off-by: Shawn Guo <shawnguo@kernel.org>
arch/arm/boot/dts/imx6qdl-hummingboard2.dtsi

index 6fbfa97..52ed580 100644 (file)
@@ -54,7 +54,7 @@
 
        reg_3p3v: regulator-3p3v {
                compatible = "regulator-fixed";
-               regulator-name = "3P3V";
+               regulator-name = "VCC_3V2";
                regulator-min-microvolt = <3300000>;
                regulator-max-microvolt = <3300000>;
                regulator-always-on;
@@ -62,7 +62,7 @@
 
        reg_1p8v: regulator-1p8v {
                compatible = "regulator-fixed";
-               regulator-name = "1P8V";
+               regulator-name = "VCC_1V8";
                regulator-min-microvolt = <1800000>;
                regulator-max-microvolt = <1800000>;
                regulator-always-on;
                startup-delay-us = <1000>;
        };
 
-       reg_usbh1_vbus: regulator-usb-h1-vbus {
+       reg_usbh2_vbus: regulator-usb-h1-vbus {
                compatible = "regulator-fixed";
                enable-active-high;
                gpio = <&gpio1 0 GPIO_ACTIVE_HIGH>;
                pinctrl-names = "default";
                pinctrl-0 = <&pinctrl_hummingboard2_usbh1_vbus>;
-               regulator-name = "usb_h1_vbus";
+               regulator-name = "V_USB2";
                regulator-min-microvolt = <5000000>;
                regulator-max-microvolt = <5000000>;
+               regulator-always-on;
        };
 
        reg_usbotg_vbus: regulator-usb-otg-vbus {
                gpio = <&gpio3 22 GPIO_ACTIVE_HIGH>;
                pinctrl-names = "default";
                pinctrl-0 = <&pinctrl_hummingboard2_usbotg_vbus>;
-               regulator-name = "usb_otg_vbus";
+               regulator-name = "V_USB1";
                regulator-min-microvolt = <5000000>;
                regulator-max-microvolt = <5000000>;
        };
 
-       reg_usbh2_vbus: regulator-usb-h2-vbus {
-               compatible = "regulator-gpio";
+       reg_usbh3_vbus: regulator-usb-h2-vbus {
+               compatible = "regulator-fixed";
                enable-active-high;
                enable-gpio = <&gpio2 13 GPIO_ACTIVE_HIGH>;
                pinctrl-names = "default";
                pinctrl-0 = <&pinctrl_hummingboard2_usbh2_vbus>;
-               regulator-name = "usb_h2_vbus";
+               regulator-name = "V_USB3";
                regulator-min-microvolt = <5000000>;
                regulator-max-microvolt = <5000000>;
-               regulator-boot-on;
+               regulator-always-on;
        };
 
-       reg_usbh3_vbus: regulator-usb-h3-vbus {
-               compatible = "regulator-gpio";
+       reg_usbh4_vbus: regulator-usb-h3-vbus {
+               compatible = "regulator-fixed";
                enable-active-high;
                enable-gpio = <&gpio7 10 GPIO_ACTIVE_HIGH>;
                pinctrl-names = "default";
                pinctrl-0 = <&pinctrl_hummingboard2_usbh3_vbus>;
-               regulator-name = "usb_h3_vbus";
+               regulator-name = "V_USB4";
                regulator-min-microvolt = <5000000>;
                regulator-max-microvolt = <5000000>;
-               regulator-boot-on;
+               regulator-always-on;
        };
 
        sound-sgtl5000 {
 
 &usbh1 {
        disable-over-current;
-       vbus-supply = <&reg_usbh1_vbus>;
        status = "okay";
 };