<br><br><div class="gmail_quote">On Tue, Nov 11, 2008 at 8:24 PM, Myles Watson <span dir="ltr"><<a href="mailto:mylesgw@gmail.com">mylesgw@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"><br>
<br>
> -----Original Message-----<br>
> From: ron minnich [mailto:<a href="mailto:rminnich@gmail.com">rminnich@gmail.com</a>]<br>
> Sent: Tuesday, November 11, 2008 4:13 PM<br>
> To: Marc Jones<br>
> Cc: Myles Watson; Coreboot<br>
> Subject: Re: [coreboot] Resource allocation<br>
><br>
</div><div><div></div><div class="Wj3C7c">> On Tue, Nov 11, 2008 at 2:58 PM, Marc Jones <<a href="mailto:marcj303@yahoo.com">marcj303@yahoo.com</a>> wrote:<br>
><br>
> ><br>
> > This would be a good reason to have a generic enable for each device<br>
> like in v2. Then the enable function can handle anything. I am not sure<br>
> how or why this changed in v3.<br>
> ><br>
><br>
> I thought you and I just finished getting this right. It's called<br>
> phase3_setup_chip_dev. Is this not right? It does what the v2 thing<br>
> did IIRC.<br>
<br>
</div></div>It goes through and allocates new devices and resources for each PNP device.<br>
It doesn't read any of the dts information.  That's the missing part.<br>
<br>
I think I could add more fields to the struct that gets passed to the chip<br>
enable and populate them from the dts.  I just wasn't sure how you were<br>
thinking it would work.<br>
</blockquote><div><br>The other way to do it would be to add the PNP devices back into the dts. I personally think that's the way to go.  If we put PNP devices around the configuration it won't be so huge, and the old PNP functions will work better again.<br>
<br>Was it deliberate that the devices disappeared?<br><br>Thanks,<br><br>Myles<br></div></div><br>