[coreboot] [PATCH]The resource allocator does not like zero-sized fixed resources
Myles Watson
mylesgw at gmail.com
Wed Aug 19 19:59:20 CEST 2009
On Wed, Aug 19, 2009 at 11:32 AM, Patrick Georgi<patrick at georgi-clan.de> wrote:
> Am 19.08.2009 19:08, schrieb Myles Watson:
>>
>> The example you gave in src/devices/pci_device.c was for the old way of
>> specifying PCI ROM locations. Now that CBFS is going mainstream, that
>> should disappear. If it doesn't, I think it should have a size. Even 1
>> would work, since no Option ROM can be smaller than that.
>>
>
> I'd love to get rid of the rom_address thing once we drop the pre-cbfs way
> of doing things (which will take quite some time, still), but I'm not sure
> if there isn't some need for it left.
OK.
>> What other fixed resources have 0 size, especially on Kontron? I'd like
>> to
>> fix the root cause.
>>
>
> That's the only one on Kontron. I just wanted to have proper behaviour even
> with such weird entries.
Let's add something like this patch. I'm happy to have the wording
changed, but I think it's important that we don't silently fail to
avoid fixed resources.
Signed-off-by: Myles Watson <mylesgw at gmail.com>
Thanks,
Myles
-------------- next part --------------
A non-text attachment was scrubbed...
Name: zero_fixed.diff
Type: text/x-patch
Size: 1224 bytes
Desc: not available
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20090819/b0b72f99/attachment.diff>
More information about the coreboot
mailing list