[PATCH] powerpc/83xx: Update ranges in gianfar node to match other dts

Kumar Gala galak at kernel.crashing.org
Wed Mar 25 04:42:57 EST 2009


On Mar 24, 2009, at 11:00 AM, Scott Wood wrote:

> On Tue, Mar 24, 2009 at 08:35:45AM -0500, Kumar Gala wrote:
>> The gianfar node @ 25000 was missing its ranges prop for the mdio bus
>> and provided an explicit ranges property on gianfar at 24000 to match
>> all other .dts
>>
>> Signed-off-by: Kumar Gala <galak at kernel.crashing.org>
>> ---
>> arch/powerpc/boot/dts/mpc8313erdb.dts |    3 ++-
>> 1 files changed, 2 insertions(+), 1 deletions(-)
>>
>> diff --git a/arch/powerpc/boot/dts/mpc8313erdb.dts b/arch/powerpc/ 
>> boot/dts/mpc8313erdb.dts
>> index 3ebf7ec..20a0fa8 100644
>> --- a/arch/powerpc/boot/dts/mpc8313erdb.dts
>> +++ b/arch/powerpc/boot/dts/mpc8313erdb.dts
>> @@ -180,7 +180,7 @@
>> 			#address-cells = <1>;
>> 			#size-cells = <1>;
>> 			sleep = <&pmc 0x20000000>;
>> -			ranges;
>> +			ranges = <0x0 0x24000 0x1000>;
>>
>> 			cell-index = <0>;
>> 			device_type = "network";
>
> You need to fix the reg property of the child node to match this  
> change
> in ranges.  Was this boot-tested (or is it a case of Linux's device  
> tree
> parsing being excessively tolerant of broken trees)?
>
I didn't boot test.. Anton pointed this issue out as well.

- k



More information about the Linuxppc-dev mailing list