Re: [PATCH 2/4] ARM: dts: omap5-uevm: Add USB Host support

[Nishanth Menon <nm@ti.com>]

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 <rogerq@ti.com>
> > [...]
> >> 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
> > <snip>
> >> +	/* 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