[coreboot] subsystem IDs

Myles Watson mylesgw at gmail.com
Wed Nov 19 04:06:43 CET 2008



> -----Original Message-----
> From: Uwe Hermann [mailto:uwe at hermann-uwe.de]
> Sent: Tuesday, November 18, 2008 3:47 PM
> To: Myles Watson
> Cc: Coreboot
> Subject: Re: [coreboot] subsystem IDs
> 
> On Tue, Nov 18, 2008 at 02:49:37PM -0700, Myles Watson wrote:
> > This patch makes subsystem ids work.  Here are the changes by file:
> 
> Nice!
> 
> Acked-by: Uwe Hermann <uwe at hermann-uwe.de>
> 
> So just to make sure -- we can now set per-board IDs _and_ override them
> for individual PCI devices in the dts?

It should work.  You should just have to add the subsystem values to the dts
in the same way that it was added for the mainboard.  You'll have to add
them in the mainboard's dts, though, or override the function that sets it.
If you add them in the device's dts they end up in the device_configuration
structure.  That's part of the reason why the superio code ended up the way
it did.
 
> > Index: mainboard/amd/serengeti/dts
> > ===================================================================
> > --- mainboard/amd/serengeti/dts	(revision 1043)
> > +++ mainboard/amd/serengeti/dts	(working copy)
> > @@ -22,6 +22,8 @@
> >  	device_operations="serengeti";
> >  	mainboard_vendor = "AMD";
> >  	mainboard_name = "Serengeti";
> > +	subsystem_vendor = "PCI_VENDOR_ID_AMD";
> 
> Nice, I didn't know that #defines could be used in dts.

Yeah.  I like the dts.  The strings from here go straight into statictree.c,
so anything that compiles there is fine.

Rev 1045. 

Thanks,
Myles






More information about the coreboot mailing list