[coreboot] [PATCH 5/6] Changes required to the device allocator
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.
>> 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>
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.de • http://www.coresystems.de/
Registergericht: Amtsgericht Freiburg • HRB 7656
Geschäftsführer: Stefan Reinauer • Ust-IdNr.: DE245674866
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 249 bytes
Desc: OpenPGP digital signature
More information about the coreboot