On Wed, Jan 20, 2016 at 02:09:19PM +0100, Lothar Waßmann wrote: > Signed-off-by: Lothar Waßmann <LW@xxxxxxxxxxxxxxxxxxx> > --- > arch/arm/boot/dts/imx53-tx53.dtsi | 40 ++++++++++++++++++++++++++++++--------- > 1 file changed, 31 insertions(+), 9 deletions(-) > > diff --git a/arch/arm/boot/dts/imx53-tx53.dtsi b/arch/arm/boot/dts/imx53-tx53.dtsi > index d3e50b2..8f08389 100644 > --- a/arch/arm/boot/dts/imx53-tx53.dtsi > +++ b/arch/arm/boot/dts/imx53-tx53.dtsi > @@ -87,9 +87,20 @@ > regulator-max-microvolt = <3300000>; > }; > > - reg_can_xcvr: regulator@2 { > + reg_3v3_etn: regulator@2 { Why cannot this be added as regulator@5 to avoid changing every node? > compatible = "regulator-fixed"; > reg = <2>; > + regulator-name = "ETN 3V3"; > + regulator-min-microvolt = <3300000>; > + regulator-max-microvolt = <3300000>; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_etn_phy>; > + gpio = <&gpio3 20 GPIO_ACTIVE_HIGH>; > + }; > + > + reg_can_xcvr: regulator@3 { > + compatible = "regulator-fixed"; > + reg = <3>; > regulator-name = "CAN XCVR"; > regulator-min-microvolt = <3300000>; > regulator-max-microvolt = <3300000>; > @@ -98,9 +109,9 @@ > gpio = <&gpio4 21 GPIO_ACTIVE_HIGH>; > }; > > - reg_usbh1_vbus: regulator@3 { > + reg_usbh1_vbus: regulator@4 { > compatible = "regulator-fixed"; > - reg = <3>; > + reg = <4>; > regulator-name = "usbh1_vbus"; > regulator-min-microvolt = <5000000>; > regulator-max-microvolt = <5000000>; > @@ -110,9 +121,9 @@ > enable-active-high; > }; > > - reg_usbotg_vbus: regulator@4 { > + reg_usbotg_vbus: regulator@5 { > compatible = "regulator-fixed"; > - reg = <4>; > + reg = <5>; > regulator-name = "usbotg_vbus"; > regulator-min-microvolt = <5000000>; > regulator-max-microvolt = <5000000>; > @@ -204,13 +215,20 @@ > phy-mode = "rmii"; > phy-reset-gpios = <&gpio7 6 GPIO_ACTIVE_HIGH>; > phy-handle = <&phy0>; > + phy-supply = <®_3v3_etn>; > mac-address = [000000000000]; /* placeholder; will be overwritten by bootloader */ > status = "okay"; > > - phy0: ethernet-phy@0 { > - interrupt-parent = <&gpio2>; > - interrupts = <4>; > - device_type = "ethernet-phy"; > + mdio { > + #address-cells = <1>; > + #size-cells = <0>; > + > + phy0: ethernet-phy@0 { > + compatible = "ethernet-phy-id0007.c0f1", "ethernet-phy-ieee802.3-c22"; > + reg = <0>; > + interrupt-parent = <&gpio2>; > + interrupts = <4>; > + }; This should be a separate change. Shawn > }; > }; > > @@ -309,6 +327,10 @@ > fsl,pins = <MX53_PAD_DISP0_DAT0__GPIO4_21 0xe0>; /* Flexcan XCVR enable */ > }; > > + pinctrl_etn_phy: etn-phygrp { > + fsl,pins = <MX53_PAD_EIM_D20__GPIO3_20 0xe0>; /* ETN PHY Power enable */ > + }; > + > pinctrl_ds1339: ds1339grp { > fsl,pins = <MX53_PAD_DI0_PIN4__GPIO4_20 0xe0>; > }; > -- > 2.1.4 > > -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html