[U-Boot] [PATCH v3 2/2] pci: Disable expansion ROM address decoding when signature check fails

[Bin Meng]

We should not leave the expansion ROM address window open when there
is not a valid ROM.

Suggested-by: Matt Porter <mporter at konsulko.com>
Signed-off-by: Bin Meng <bmeng.cn at gmail.com>

---
I did not update pci_rom_probe() to handle the header type1 ROM address,
as I think pci_rom_probe() is a static routine which is only used for
VGA ROM only so far, so we don't need consider type1 case.

Changes in v3:
- Rebase on u-boot-x86/master

Changes in v2:
- New patch to disable expansion ROM address decoding when signature check fails

 drivers/pci/pci_rom.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/drivers/pci/pci_rom.c b/drivers/pci/pci_rom.c
index aa06767..a33efae 100644
--- a/drivers/pci/pci_rom.c
+++ b/drivers/pci/pci_rom.c
@@ -104,6 +104,10 @@ static int pci_rom_probe(pci_dev_t dev, uint class,
 	if (le16_to_cpu(rom_header->signature) != PCI_ROM_HDR) {
 		printf("Incorrect expansion ROM header signature %04x\n",
 		       le16_to_cpu(rom_header->signature));
+#ifndef CONFIG_VGA_BIOS_ADDR
+		/* Disable expansion ROM address decoding */
+		pci_write_config_dword(dev, PCI_ROM_ADDRESS, rom_address);
+#endif
 		return -EINVAL;
 	}
 
-- 
1.8.2.1