[coreboot] v3: dts and arrays
Stefan Reinauer
stepan at coresystems.de
Thu Feb 14 23:23:29 CET 2008
* Marc Jones <Marc.Jones at amd.com> [080214 23:16]:
>>> +++ southbridge/amd/cs5536/dts (Arbeitskopie)
>>> @@ -57,4 +57,7 @@
>>> com2_enable = "0";
>>> com2_address = "0x2f8";
>>> com2_irq = "3";
>>> +
>>> + /* Disable unwanted virtualized PCI devices */
>>> + unwanted_vpci[0] = "0";
>>> };
>>>
>>>
>> I am blissfully ignorant on the topic, but the first thing I wondered:
>>
>> Should a virtual device not be disabled in the same way as a physical one?
>>
>> pci at 0,0 {
>> disabled;
>> }
>>
>> ?
>>
>> Stefan
>>
> This is an interesting idea but the devices are CS5536 specific and I don't
> think we want to make dts CS5536 aware.
Yes, that's what I think, too.
We should use the standard pci mechanism rather than extending the dts with
unwanted_vpci[]
> I think it better use the device ID rather than the device location to
> do the disable.
Yes. This would be interesting for other platforms, too.
--
coresystems GmbH • Brahmsstr. 16 • D-79104 Freiburg i. Br.
Tel.: +49 761 7668825 • Fax: +49 761 7664613
Email: info at coresystems.de • http://www.coresystems.de/
Registergericht: Amtsgericht Freiburg • HRB 7656
Geschäftsführer: Stefan Reinauer • Ust-IdNr.: DE245674866
More information about the coreboot
mailing list