[coreboot] patch: more path support

Stefan Reinauer stepan at coresystems.de
Sun Feb 10 17:27:19 CET 2008


Peter Stuge wrote:
> On Sat, Feb 09, 2008 at 09:35:49PM -0800, ron minnich wrote:
>   
>>> It should be simple enough to get the type if struct property had
>>> a struct node *node; member so that the owner->parent node could
>>> be reached from within the property foreach loop.
>>>       
>> it's there.
>>     
>
> Hmm. How?
>
>
>   
>> but that doesn't totally tell you what type of resource you have. I
>> don't think.
>>     
>
> Maybe I'm oversimplifying? I was thinking that:
> (from your patch)
>
>         cpus {
> +               cpupath="0";
>
>         apic {
>                 /config/("northbridge/amd/geodelx/apic");
> +               apicpath="0,0,0";
>
>                 superio {
>                         /config/("superio/winbond/w83627hf/dts");
> +                       superiopath = "0x2e";
>
>
> ..it would be possible to determine resource type from the name of
> the containing block. "cpus" => cpupath, "apic" => apicpath etc.
>   

Ron suggested to make this cpu at 0 { ... } and apic at 0,0,0 { ... }

I like that idea a lot.

I think that would be the way to go.. It means we can not call the apic 
node "my_weird_apic" anymore, which is an improvement, too.

Stefan

-- 
coresystems GmbH • Brahmsstr. 16 • D-79104 Freiburg i. Br.
      Tel.: +49 761 7668825 • Fax: +49 761 7664613
Email: info at coresystems.dehttp://www.coresystems.de/
Registergericht: Amtsgericht Freiburg • HRB 7656
Geschäftsführer: Stefan Reinauer • Ust-IdNr.: DE245674866


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 249 bytes
Desc: OpenPGP digital signature
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20080210/1c68051f/attachment.sig>


More information about the coreboot mailing list