[PATCH] Fix FEC node in 8540 ADS dts

[Andy Fleming]

On Aug 21, 2006, at 14:39, Hollis Blanchard wrote:

> On Mon, 2006-08-21 at 14:29 -0500, Andy Fleming wrote:
>>
>> diff --git a/arch/powerpc/boot/dts/mpc8540ads.dts
>> b/arch/powerpc/boot/dts/mpc8540ads.dts
>> index 93d2c2d..5f41c1f 100644
>> --- a/arch/powerpc/boot/dts/mpc8540ads.dts
>> +++ b/arch/powerpc/boot/dts/mpc8540ads.dts
>> @@ -83,11 +83,11 @@
>>                                 reg = <1>;
>>                                 device_type = "ethernet-phy";
>>                         };
>> -                       ethernet-phy at 2 {
>> -                               linux,phandle = <2452002>;
>> +                       ethernet-phy at 3 {
>> +                               linux,phandle = <2452003>;
>
> Again, doesn't DTC assign linux,phandle properties as needed (i.e.  
> when
> the node is referenced with & syntax)?


That's what I hear.  I think this would be a good place for using  
that, but I'm waiting for someone to determine whether some sort of  
labeling system is going to be added, first.  As it doesn't require  
changes to u-boot or linux, it's an easy fix for a later time.

I will, of course, accede to popular opinion.  But I'd rather change  
all the phandles in exactly the same way, and I feel that the  
notation gets clunky when seen iterated in a table (as shown in Jon's  
patch for that change).

Andy