[coreboot] [PATCH 5/6] Changes required to the device allocator

Stefan Reinauer stepan at coresystems.de
Wed Oct 29 04:16:04 CET 2008

Myles Watson wrote:
>> Changes required to the device allocator:
>> - leave a hole for mmapped PCIe config space if
>>   is set.
>> - Mask moving bits to 32bit when resources are not supposed above 4G.
> Linux
>>   does not like this, even though the resource is disabled.
>> Signed-off-by: Stefan Reinauer <stepan at coresystems.de>
> Looks good.  would it be better to set the resource to 0 instead of to
> moving bits?  Then there would not be a need for the special case.  It also
> makes it easier to read the config space.
> Acked-by: Myles Watson <mylesgw at gmail.com>

Thanks! r3699.

According to Ron and Eric, setting the resource to 0 will not work on
some devices. I know factory bios has done it on at least the bridges
I've seen so I am not sure what the right behavior would be. Setting
them to 0 would seem the right thing.


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/20081028/aa7ca11b/attachment.sig>

More information about the coreboot mailing list