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