[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
>>
> CONFIG_PCIE_CONFIGSPACE_HOLE
>
>> 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.
Stefan
--
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...
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