From mboxrd@z Thu Jan 1 00:00:00 1970 From: nm@ti.com (Nishanth Menon) Date: Thu, 6 Jun 2013 13:46:17 -0500 Subject: [PATCH 2/4] ARM: dts: omap5-uevm: Add USB Host support In-Reply-To: <51B0CC15.6010809@ti.com> References: <1370414770-1485-1-git-send-email-r.sricharan@ti.com> <1370414770-1485-3-git-send-email-r.sricharan@ti.com> <20130605135739.GA11193@kahuna> <51B0CC15.6010809@ti.com> Message-ID: <20130606184617.GA25579@kahuna> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 23:21-20130606, Sricharan R wrote: > Hi, > On Wednesday 05 June 2013 07:27 PM, Nishanth Menon wrote: > > On 12:16-20130605, Sricharan R wrote: > >> From: Roger Quadros > > [...] > >> diff --git a/arch/arm/boot/dts/omap5-uevm.dts b/arch/arm/boot/dts/omap5-uevm.dts > >> index 843a001..cf862df 100644 > >> --- a/arch/arm/boot/dts/omap5-uevm.dts > >> +++ b/arch/arm/boot/dts/omap5-uevm.dts > > > >> + /* hsusb2_phy is clocked by FREF_CLK1 i.e. auxclk1 */ > >> + clock_alias { > >> + clock-name = "auxclk1_ck"; > >> + clock-alias = "main_clk"; > > is'nt this better to use with clock node? > Roger was suggesting to remove this for now. Posted > V2 following that. > Are you suggesting to go with your clock-binding patch ? you cannot. Mike has already rejected that approach and I have dropped that series and wont be following up. See discussion https://patchwork.kernel.org/patch/2541331/ instead the new approach being discussed https://patchwork.kernel.org/patch/2657351/ and related discussions is the way to go. That said, it practically blocks introduction of all code that depend on dts nodes :(, no matter how I dont like being blocked and my many series delayed, I believe Mike, Tero, Rajendra are doing the right architecture and we should wait for it's upstream introduction. That is just my 2 cents. -- Regards, Nishanth Menon