[SeaBIOS] [PATCH] Seabios: allow mapping of multiple PCI option ROMs to one

Kevin O'Connor kevin at koconnor.net
Fri May 24 05:01:00 CEST 2013


On Thu, May 23, 2013 at 05:25:42PM -0500, Dave Frodin wrote:
> Kevin,
> I've attached a patch that I hope is more along the lines
> of where you wanted me to go with this change. In my testing 
> of the patch I'm currently just stuffing the vendev-map.bin file
> into my coreboot image.

Thanks.

[...]
> +    for (i=0; i < filesize/(sizeof (u32)); i+=2) {
> +        if ( filedata[i] == ((pci->vendor << 16) | pci->device)) {
> +            dprintf(1, "Mapping PCI device 0x%8x to 0x%8x\n",
> +                ((pci->vendor << 16) | pci->device), filedata[i+1]);
> +            pci->vendor = filedata[i+1] >> 16;
> +            pci->device = filedata[i+1] & 0xFFFF;
> +            return;

I don't understand why struct pci_device is being modified.  Maybe you
could further explain the use case.  After modifying struct
pci_device, where does the video rom actually come from?

-Kevin



More information about the SeaBIOS mailing list