From patchwork Thu Oct 25 05:42:05 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Alex Hung X-Patchwork-Id: 194041 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from chlorine.canonical.com (chlorine.canonical.com [91.189.94.204]) by ozlabs.org (Postfix) with ESMTP id 1CE4E2C009A for ; Thu, 25 Oct 2012 16:43:35 +1100 (EST) Received: from localhost ([127.0.0.1] helo=chlorine.canonical.com) by chlorine.canonical.com with esmtp (Exim 4.71) (envelope-from ) id 1TRGE8-0002iy-5o; Thu, 25 Oct 2012 05:43:32 +0000 Received: from youngberry.canonical.com ([91.189.89.112]) by chlorine.canonical.com with esmtp (Exim 4.71) (envelope-from ) id 1TRGCo-0002di-4p for fwts-devel@lists.ubuntu.com; Thu, 25 Oct 2012 05:42:10 +0000 Received: from [175.41.48.77] (helo=canonical.com) by youngberry.canonical.com with esmtpsa (TLS1.0:DHE_RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1TRGCn-000355-ET; Thu, 25 Oct 2012 05:42:10 +0000 From: Alex Hung To: fwts-devel@lists.ubuntu.com Subject: [PATCH] acpi: mcfg: Set MCFGMMIONotReserved error to low Date: Thu, 25 Oct 2012 13:42:05 +0800 Message-Id: <1351143725-2781-1-git-send-email-alex.hung@canonical.com> X-Mailer: git-send-email 1.7.9.5 X-BeenThere: fwts-devel@lists.ubuntu.com X-Mailman-Version: 2.1.13 Precedence: list List-Id: Firmware Test Suite Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Sender: fwts-devel-bounces@lists.ubuntu.com Errors-To: fwts-devel-bounces@lists.ubuntu.com In PCI Firmware Specification, it claims "If the operating system does not natively comprehend reserving the MMCFG region, the MMCFG region must be reserved by firmware." However, Linux is an operating system that comprehend MMCFG, and therefore it is not a problem for not declaring it. Signed-off-by: Alex Hung Acked-by: Keng-Yu Lin Acked-by: Colin Ian King --- src/acpi/mcfg/mcfg.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/acpi/mcfg/mcfg.c b/src/acpi/mcfg/mcfg.c index 9dfd9fd..878c0f1 100644 --- a/src/acpi/mcfg/mcfg.c +++ b/src/acpi/mcfg/mcfg.c @@ -200,7 +200,7 @@ static int mcfg_test1(fwts_framework *fw) if ((memory_map_list != NULL) && (!fwts_memory_map_is_reserved(memory_map_list, config->base_address))) { - fwts_failed(fw, LOG_LEVEL_MEDIUM, "MCFGMMIONotReserved", + fwts_failed(fw, LOG_LEVEL_LOW, "MCFGMMIONotReserved", "MCFG mmio config space at 0x%" PRIx64 " is not reserved in the memory map table", config->base_address);