<br><br><div class="gmail_quote">On Thu, Nov 13, 2008 at 11:45 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><div></div><div class="Wj3C7c">On Thu, Nov 13, 2008 at 10:19 AM, Myles Watson <<a href="mailto:mylesgw@gmail.com">mylesgw@gmail.com</a>> wrote:<br>
> It didn't work for me.<br>
><br>
>> -----Original Message-----<br>
>> From: <a href="mailto:coreboot-bounces@coreboot.org">coreboot-bounces@coreboot.org</a> [mailto:<a href="mailto:coreboot-bounces@coreboot.org">coreboot-bounces@coreboot.org</a>]<br>
>> On Behalf Of ron minnich<br>
>> Sent: Thursday, November 13, 2008 10:51 AM<br>
>> To: Coreboot<br>
>> Subject: [coreboot] patch: allow dts include files to have nodes<br>
>><br>
>> per IRC discussions.<br>
>><br>
>> ron<br>
><br>
> It didn't work for me.  I put the pnp devices in the SIO dts, and it says it<br>
> can't read the input tree (unfortunately it gives a line number I didn't<br>
> change in the error.) It works fine on the old tree with no nodes.<br>
></div></div></blockquote><div class="Wj3C7c">Sorry,<br>

<br>

I didn't see this response. <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;">it's not going to do much yet. This is to allow subnodes to be in the<br>
tree syntactically. There is zero processing occuring in the<br>
flatten_tree at this point.</blockquote><div><br>It doesn't parse my tree if I put in an extra node in a dts.<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;">

I just built a geode target with it. Are you building against your<br>
modified tree?</blockquote><div> </div><div>Yes, but it works fine until I add a node. <br></div><div> </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
This is a step-by-step. It should work on unmodified trees -- it did for me.</blockquote><div><br>It did for me too. <br></div><div> </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

I would suggest that you start with a clean tree and let's go through<br>
your ideas/changes one by one. I think you are in the 'diverged too<br>
far' state.<font color="#888888"><br>
</font></blockquote></div><br>I don't think the PNP patch I sent is really that big, and it is separate from everything else.  How would you suggest I break it up?<br><br>Thanks,<br>Myles<br><br>