From ac826c8fd7b616f8b65a268e7474d8c0cf84f93f Mon Sep 17 00:00:00 2001 From: Nico Huber Date: Fri, 31 Mar 2017 12:09:58 +0200 Subject: [PATCH] inteltool: Don't use PCI_FILL_SIZES This is supposed to fill the `size[]` array with the actual sizes of a device' MMIO ranges, but apparently isn't implemented for every access method in libpci (we let the library choose one). It tells us by clearing `PCI_FILL_SIZES` in the return value of `pci_fill_info()` (which we don't check). Since we don't ever use `size`, we can just make it clear and don't ask for it. Change-Id: I3fb9334472f1c7563a9e17910190f73affbe067a Signed-off-by: Nico Huber Reviewed-on: https://review.coreboot.org/19582 Tested-by: build bot (Jenkins) Reviewed-by: Stefan Reinauer --- util/inteltool/inteltool.c | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/util/inteltool/inteltool.c b/util/inteltool/inteltool.c index a4814f8577..cefce7e27e 100644 --- a/util/inteltool/inteltool.c +++ b/util/inteltool/inteltool.c @@ -425,7 +425,7 @@ int main(int argc, char *argv[]) exit(1); } - pci_fill_info(sb, PCI_FILL_IDENT|PCI_FILL_BASES|PCI_FILL_SIZES|PCI_FILL_CLASS); + pci_fill_info(sb, PCI_FILL_IDENT | PCI_FILL_BASES | PCI_FILL_CLASS); if (sb->vendor_id != PCI_VENDOR_ID_INTEL) { printf("Not an Intel(R) southbridge.\n"); @@ -438,7 +438,7 @@ int main(int argc, char *argv[]) exit(1); } - pci_fill_info(nb, PCI_FILL_IDENT|PCI_FILL_BASES|PCI_FILL_SIZES|PCI_FILL_CLASS); + pci_fill_info(nb, PCI_FILL_IDENT | PCI_FILL_BASES | PCI_FILL_CLASS); if (nb->vendor_id != PCI_VENDOR_ID_INTEL) { printf("Not an Intel(R) northbridge.\n"); @@ -448,7 +448,8 @@ int main(int argc, char *argv[]) gfx = pci_get_dev(pacc, 0, 0, 0x02, 0); if (gfx) { - pci_fill_info(gfx, PCI_FILL_IDENT|PCI_FILL_BASES|PCI_FILL_SIZES|PCI_FILL_CLASS); + pci_fill_info(gfx, PCI_FILL_IDENT | PCI_FILL_BASES | + PCI_FILL_CLASS); if (gfx->vendor_id != PCI_VENDOR_ID_INTEL) gfx = 0; @@ -460,7 +461,8 @@ int main(int argc, char *argv[]) ahci = pci_get_dev(pacc, 0, 0, 0x1f, 2); if (ahci) { - pci_fill_info(ahci, PCI_FILL_IDENT|PCI_FILL_BASES|PCI_FILL_SIZES|PCI_FILL_CLASS); + pci_fill_info(ahci, PCI_FILL_IDENT | PCI_FILL_BASES | + PCI_FILL_CLASS); if (ahci->vendor_id != PCI_VENDOR_ID_INTEL) ahci = 0;