From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bin Liu Subject: Re: [PATCH v4 4/4] ARM: dts: da850: Add the usb otg device nodeg Date: Tue, 15 Nov 2016 15:19:51 -0600 Message-ID: <20161115211951.GH11011@uda0271908> References: <1478188752-22447-1-git-send-email-abailon@baylibre.com> <1478188752-22447-5-git-send-email-abailon@baylibre.com> <0c94c5e0-9d2c-fff5-1213-2fb2adb20ab0@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Return-path: Content-Disposition: inline In-Reply-To: <0c94c5e0-9d2c-fff5-1213-2fb2adb20ab0-l0cyMroinI0@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Sekhar Nori Cc: Alexandre Bailon , khilman-rdvid1DuHRBWk0Htik3J/w@public.gmane.org, robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-usb-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org List-Id: devicetree@vger.kernel.org On Tue, Nov 15, 2016 at 04:16:02PM +0530, Sekhar Nori wrote: > 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 > > --- > > 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). Dropped this patch due to this comment. Regards, -Bin. > > > + > > +&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-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html