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

[Sricharan R <r.sricharan@ti.com>]

On Friday 07 June 2013 12:16 AM, Nishanth Menon wrote:
> 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. Postedhen 
>> 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.
>
 Thanks for the details. So dropping the alias for now, should be fine.
 
Regards,
 Sricharan