<div class="gmail_quote">On Tue, Mar 24, 2009 at 12:32 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="im"><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 Peter Stuge<br>
> Sent: Tuesday, March 24, 2009 10:06 AM<br>
> To: <a href="mailto:coreboot@coreboot.org">coreboot@coreboot.org</a><br>
> Subject: Re: [coreboot] [PATCH] fix object/driver confusion ofall<br>
> northbridge code<br>
><br>
> Stefan Reinauer wrote:<br>
> > Every object file with a struct pci_driver ... __pci_driver needs to<br>
> > be marked as "driver" instead of "object" in order to get the init<br>
> > code actually executed.<br>
> ><br>
> > This patch fixes up all northbridges that did not do this before.<br>
> ><br>
> > Signed-off-by: Stefan Reinauer <<a href="mailto:stepan@coresystems.de">stepan@coresystems.de</a>><br>
><br>
> Acked-by: Peter Stuge <<a href="mailto:peter@stuge.se">peter@stuge.se</a>><br>
><br>
> How come we have not noticed this before? Do these NB drivers not do<br>
> anything critical?<br>
<br>
</div>It is hard to believe that the amdk8 northbridge code never got executed.<br>
Is this only true if it's not contained in a "chip", or if you don't specify<br>
its position in the device tree?</blockquote><div><br>I'm grasping at straws, but I think it had something to do with multiple drivers in northbridge.c, if it's marked as a driver only the first driver would get run, or it might have been something to do with the special pci domain and cpu bus drivers. Then again, I could be entirely wrong, it's been a while.<br>
<br>-Corey <br></div></div>