On Thursday 03 November 2016 09:29 PM, Alexandre Bailon wrote: > This adds the device tree node for the usb otg > controller present in the da850 family of SoC's. > This also enables the otg usb controller for the lcdk board. > > Signed-off-by: Alexandre Bailon <abailon@xxxxxxxxxxxx> > --- > arch/arm/boot/dts/da850-lcdk.dts | 8 ++++++++ > arch/arm/boot/dts/da850.dtsi | 15 +++++++++++++++ > 2 files changed, 23 insertions(+) > > diff --git a/arch/arm/boot/dts/da850-lcdk.dts b/arch/arm/boot/dts/da850-lcdk.dts > index 7b8ab21..9f5040c 100644 > --- a/arch/arm/boot/dts/da850-lcdk.dts > +++ b/arch/arm/boot/dts/da850-lcdk.dts > @@ -158,6 +158,14 @@ > rx-num-evt = <32>; > }; > > +&usb_phy { > + status = "okay"; > + }; As mentioned by David already, this node needs to be removed. Please rebase this on top of latest linux-davinci/master when ready for merging (driver changes accepted). > + > +&usb0 { > + status = "okay"; > +}; > + > &aemif { > pinctrl-names = "default"; > pinctrl-0 = <&nand_pins>; > diff --git a/arch/arm/boot/dts/da850.dtsi b/arch/arm/boot/dts/da850.dtsi > index f79e1b9..322a31a 100644 > --- a/arch/arm/boot/dts/da850.dtsi > +++ b/arch/arm/boot/dts/da850.dtsi > @@ -372,6 +372,21 @@ > >; > status = "disabled"; > }; > + usb_phy: usb-phy { > + compatible = "ti,da830-usb-phy"; > + #phy-cells = <1>; > + status = "disabled"; > + }; > + usb0: usb@200000 { > + compatible = "ti,da830-musb"; > + reg = <0x200000 0x10000>; > + interrupts = <58>; > + interrupt-names = "mc"; > + dr_mode = "otg"; > + phys = <&usb_phy 0>; > + phy-names = "usb-phy"; > + status = "disabled"; > + }; Can you separate out the soc specific changes from board changes? Please place the usb0 node above the mdio node. I am trying to get to a rough ordering based on reg property. Thanks, Sekhar -- 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