diff options
author | Feng Tang <feng.tang@intel.com> | 2010-05-05 05:08:49 -0400 |
---|---|---|
committer | Jesse Barnes <jbarnes@virtuousgeek.org> | 2010-05-18 18:03:27 -0400 |
commit | a02ce953a14d6a8aab721b129b3c8ff4981a76e6 (patch) | |
tree | 6131f29db636ad779c27201b96056d8be64fc0e7 | |
parent | 8f6bce3c4f48bd79b57d6ac9f337f5aabee43ea7 (diff) |
x86/PCI: make ACPI MCFG reserved error messages ACPI specific
Both ACPI and SFI firmwares will have MCFG space, but the error message
isn't valid on SFI, so don't print the message in that case.
Signed-off-by: Feng Tang <feng.tang@intel.com>
Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
-rw-r--r-- | arch/x86/pci/mmconfig-shared.c | 17 |
1 files changed, 9 insertions, 8 deletions
diff --git a/arch/x86/pci/mmconfig-shared.c b/arch/x86/pci/mmconfig-shared.c index 39b9ebe8f88..a918553ebc7 100644 --- a/arch/x86/pci/mmconfig-shared.c +++ b/arch/x86/pci/mmconfig-shared.c | |||
@@ -483,16 +483,17 @@ static void __init pci_mmcfg_reject_broken(int early) | |||
483 | list_for_each_entry(cfg, &pci_mmcfg_list, list) { | 483 | list_for_each_entry(cfg, &pci_mmcfg_list, list) { |
484 | int valid = 0; | 484 | int valid = 0; |
485 | 485 | ||
486 | if (!early && !acpi_disabled) | 486 | if (!early && !acpi_disabled) { |
487 | valid = is_mmconf_reserved(is_acpi_reserved, cfg, 0); | 487 | valid = is_mmconf_reserved(is_acpi_reserved, cfg, 0); |
488 | 488 | ||
489 | if (valid) | 489 | if (valid) |
490 | continue; | 490 | continue; |
491 | 491 | else | |
492 | if (!early) | 492 | printk(KERN_ERR FW_BUG PREFIX |
493 | printk(KERN_ERR FW_BUG PREFIX | 493 | "MMCONFIG at %pR not reserved in " |
494 | "MMCONFIG at %pR not reserved in " | 494 | "ACPI motherboard resources\n", |
495 | "ACPI motherboard resources\n", &cfg->res); | 495 | &cfg->res); |
496 | } | ||
496 | 497 | ||
497 | /* Don't try to do this check unless configuration | 498 | /* Don't try to do this check unless configuration |
498 | type 1 is available. how about type 2 ?*/ | 499 | type 1 is available. how about type 2 ?*/ |