Kconfig allows you to run all PCI ROMs, VGA only, or non-VGA only.
authorMyles Watson <mylesgw@gmail.com>
Wed, 7 Oct 2009 18:41:08 +0000 (18:41 +0000)
committerMyles Watson <mylesgw@gmail.com>
Wed, 7 Oct 2009 18:41:08 +0000 (18:41 +0000)
Update the code to support that too.

Remove an unused variable.

Signed-off-by: Myles Watson <mylesgw@gmail.com>
Acked-by: Peter Stuge <peter@stuge.se>
git-svn-id: svn://svn.coreboot.org/coreboot/trunk@4738 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1

src/devices/pci_device.c

index e45b1ecf67ba5bb47767c7c7dd922280efc99637..5c73db5ed0de495786b19cbad5eb1a4eaef5de50 100644 (file)
@@ -669,14 +669,13 @@ void pci_dev_init(struct device *dev)
        void run_bios(struct device *dev, unsigned long addr);
        struct rom_header *rom, *ram;
 
-#if CONFIG_PCI_ROM_RUN != 1
-       /* We want to execute VGA option ROMs when CONFIG_VGA_ROM_RUN
-        * is set but CONFIG_PCI_ROM_RUN is not. In this case we skip
-        * all other option ROM types.
-        */
-       if ((dev->class >> 8) != PCI_CLASS_DISPLAY_VGA)
+       if (CONFIG_PCI_ROM_RUN != 1 && /* Only execute VGA ROMs. */
+           ((dev->class >> 8) != PCI_CLASS_DISPLAY_VGA))
+               return;
+
+       if (CONFIG_VGA_ROM_RUN != 1 && /* Only execute non-VGA ROMs. */
+           ((dev->class >> 8) == PCI_CLASS_DISPLAY_VGA))
                return;
-#endif
 
        rom = pci_rom_probe(dev);
        if (rom == NULL)
@@ -1224,7 +1223,6 @@ void pci_assign_irqs(unsigned bus, unsigned slot,
        device_t pdev;
        u8 line;
        u8 irq;
-       u8 readback;
 
        /* Each slot may contain up to eight functions */
        for (funct = 0; funct < 8; funct++) {
@@ -1248,8 +1246,8 @@ void pci_assign_irqs(unsigned bus, unsigned slot,
                        pIntAtoD[line - 1]);
 
 #ifdef PARANOID_IRQ_ASSIGNMENTS
-               readback = pci_read_config8(pdev, PCI_INTERRUPT_LINE);
-               printk_debug("  Readback = %d\n", readback);
+               irq = pci_read_config8(pdev, PCI_INTERRUPT_LINE);
+               printk_debug("  Readback = %d\n", irq);
 #endif
 
                // Change to level triggered