Subject: Re: [PATCH 2/8] Add uli1575 pci-bridge sector to MPC8641HPCN dts file.

From: Segher Boessenkool <[email-masked]>
Date: Mon, 4 Jun 2007 21:27:19 +0200

>>>> "reg" included in "ranges"?  Something is wrong here.
>>>
>>> I think it's correct for soc nodes.
>>
>> It is not.  "ranges" is the address space translation
>> between the parent and child busses; "reg" is the stuff
>> that's on the bridge device itself.
>>
>>> At least, it appears that all of
>>> the dts files with soc nodes do similar things (including this one 
>>> even
>>> without this patch).
>>
>> Yes, but that doesn't make it right.
>
> But removing it does break things.

What breaks?  This sounds like a Linux bug, in
your port perhaps, or in some OF parsing code.

> I'm going to leave
> it as-is for now, and we can wrestle over "fixing" it
> in a follow up patch.

Fine with me -- please figure out what exactly it
is that is breaking though.

Thanks,

Segher