<br><br><div class="gmail_quote">On Thu, Nov 5, 2009 at 12:12 PM, Stefan Reinauer <span dir="ltr"><<a href="mailto:stepan@coresystems.de">stepan@coresystems.de</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="im">Myles Watson wrote:<br>
><br>
><br>
> On Wed, Oct 28, 2009 at 2:18 PM, Myles Watson <<a href="mailto:mylesgw@gmail.com">mylesgw@gmail.com</a><br>
</div><div class="im">> <mailto:<a href="mailto:mylesgw@gmail.com">mylesgw@gmail.com</a>>> wrote:<br>
><br>
><br>
><br>
>     On Wed, Oct 28, 2009 at 1:58 PM, Uwe Hermann <<a href="mailto:uwe@hermann-uwe.de">uwe@hermann-uwe.de</a><br>
</div><div class="im">>     <mailto:<a href="mailto:uwe@hermann-uwe.de">uwe@hermann-uwe.de</a>>> wrote:<br>
><br>
>         See patch.<br>
><br>
>     I think it would be nice to figure out what we do with the<br>
>     "onboard" device at the same time.<br>
><br>
> Ping.  I think this patch is an improvement, but I didn't want the<br>
> opportunity to remove more dead code pass by.  Any comments?<br>
</div>Acked-by: Stefan Reinauer <<a href="mailto:stepan@coresystems.de">stepan@coresystems.de</a>> for removing all the<br>
->onboard stuff.<br>
<div class="im"><br>
><br>
>      from src/drivers/pci/onboard.c:<br>
><br>
>     static void onboard_enable(device_t dev)<br>
>     {<br>
>         struct drivers_pci_onboard_config *conf;<br>
>             conf = dev->chip_info;<br>
>         dev->rom_address = conf->rom_address;<br>
>     }<br>
><br>
>     Does it make sense to change ROM handling so that only "onboard"<br>
>     devices can have their ROMs run?  That's the way it used to be, right?<br>
><br>
</div>No. Plugin cards (graphics cards) need to have their ROM run, too.<br></blockquote><div>I meant should "onboard" devices be the only ones allowed to have ROMs in CBFS, but I can see that that wasn't right either.  It just seems like when we made the change to CBFS we allowed a lot more devices to have ROMs in CBFS, and I was wondering if that was intentional, or a side-effect.<br>
<br>Thanks,<br>Myles<br></div></div><br>