[coreboot] domain vs device statictree order

Marc Jones marcj303 at gmail.com
Wed Jan 7 00:13:06 CET 2009


On Tue, Jan 6, 2009 at 3:37 PM, Myles Watson <mylesgw at gmail.com> wrote:
> Marc,
>
> Hopefully this makes everything right again.  I still think some of
> the geode functions should be moved, but that's really a separate
> issue.
>
>> You have to specify to make things breadth first, which seems like the
>> correct way.
>
> I guess it's not really breadth first.  It's just parents before siblings.
>
> I think this should be done for all the phases unless there's some
> compelling reason not to.
>
> Signed-off-by: Myles Watson <mylesgw at gmail.com>

I agree it is parents before siblings but I thought it looked more
like a problem with the statictree. I must not understand .next.

from http://pastebin.ca/1301081

dev_apic_0 has a .sibling = &dev_domain_0 and .next = &dev_domain_0_pci_1_0

Then later in the last device domain dev_domain_0_pci_f_2 has a .next
= &dev_domain_0.

I would expect .sibling and .next to point to the &dev_domain_0 and
then dev_domain_0 to have .next = &dev_domain_0_pci_1_0

dev_cpus looks like I would expect .sibling = &dev_apic_0 and .next =
&dev_apic_0.
dev_domain_0_pci_f_0 also has the same .sibling and .next.

Can you explain the last pci dev .next pointing back to the domain?

I am not setup to test this either. I was talking with Mart on NAND
device problems. Maybe he can test it tomorrow.

Marc




More information about the coreboot mailing list