<br><br><div class="gmail_quote">On Tue, Nov 25, 2008 at 10:06 AM, ron minnich <span dir="ltr"><<a href="mailto:rminnich@gmail.com">rminnich@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="Ih2E3d">On Tue, Nov 25, 2008 at 9:04 AM, Myles Watson <<a href="mailto:mylesgw@gmail.com">mylesgw@gmail.com</a>> wrote:<br>
<br>
> I'm comfortable with the idea that this is the way we want to go, but it<br>
> doesn't work for me yet.  We could commit to make patching easier.  I guess<br>
> it's up to you how functional you want it before committing.<br>
<br>
</div>We'll commit when<br>
1. it works for you<br>
2. I can build a working dbe62 target (which I still can)<br>
<br>
You have the token for now.</blockquote><div><br>I'm not sure that's wise.  I'm breaking more than fixing right now.<br><br>Here's the latest snippet:<br>  DTC     mainboard/amd/serengeti/dts (dts->lbh)<br>
name_node: pci@0,0(usb11)<br>name_node: pci@0,1(usb12)<br>name_node: pci@0,2(usb2)<br>name_node: pci@1,0(nic)<br>name_node: pci@0,0(multilevel_root)<br>name_node: pci@1,0(lpc)<br>name_node: pci@1,1(ide)<br>name_node: pci@1,2(smbus)<br>
name_node: pci@1,3(acpi)<br>name_node: pci@1,5(ac97audio)<br>name_node: pci@1,6(ac97modem)<br>name_node: pci@1,7(dev7)<br>name_node: NULL(nic)<br>name_node: NULL(ide)<br>name_node: ioport@2e(NULL)<br>name_node: NULL(lpc)<br>
fixup_properties: before fixups mainboard tree nic()<br>NULL(nic)<br>fixup_properties: before fixups chipnodes tree multilevel_root(pci@0,0)<br>pci@0,0(multilevel_root)<br> pci@0,0(usb11)<br> pci@0,1(usb12)<br> pci@0,2(usb2)<br>
 pci@1,0(nic)<br>fixup_properties: checking nic() and multilevel_root(pci@0,0)<br>fixup_properties: checking nic() and lpc(pci@1,0)<br>fixup_properties: checking nic() and ide(pci@1,1)<br>fixup_properties: checking nic() and smbus(pci@1,2)<br>
fixup_properties: checking nic() and acpi(pci@1,3)<br>fixup_properties: checking nic() and ac97audio(pci@1,5)<br>fixup_properties: checking nic() and ac97modem(pci@1,6)<br>fixup_properties: checking nic() and dev7(pci@1,7)<br>
fixup_properties: checking nic() and nic()<br>fixup_properties: last_child NULL()<br>fixup_properties: matched and deleting nic((null))<br>del_node: matched and deleting nic((null))<br>del_node: expect seg fault nic((null))<br>
/bin/sh: line 1:  3615 Segmentation fault      /home/myles/buildrom/try/buildrom/buildrom-devel/work/coreboot-v3/svn/build/util/dtc/dtc -O lbh mainboard/amd/serengeti/dts >/tmp/statictree.h.$<br>make: *** [/home/myles/buildrom/try/buildrom/buildrom-devel/work/coreboot-v3/svn/build/statictree.h] Error 139<br>
<br>The "expect seg fault" line means that del_node got passed a node with no parent.  That means I don't understand enough what's going on here.  I was expecting the parent to be set to pci@0,0 (from the mainboard dts), but it hasn't been made yet (no name_node for it yet)<br>
<br>Help me understand.  Should we just take out the sibling links all together from del?  Have they not been added yet?<br><br>Thanks,<br>Myles<br><br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>
<font color="#888888"><br>
ron<br>
</font></blockquote></div><br>