From patchwork Wed Jul 27 06:24:49 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Ivan Hu X-Patchwork-Id: 653104 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from huckleberry.canonical.com (huckleberry.canonical.com [91.189.94.19]) by ozlabs.org (Postfix) with ESMTP id 3rzlN036M6z9t22; Wed, 27 Jul 2016 16:25:04 +1000 (AEST) Received: from localhost ([127.0.0.1] helo=huckleberry.canonical.com) by huckleberry.canonical.com with esmtp (Exim 4.76) (envelope-from ) id 1bSIH6-00024m-Ht; Wed, 27 Jul 2016 06:25:00 +0000 Received: from youngberry.canonical.com ([91.189.89.112]) by huckleberry.canonical.com with esmtp (Exim 4.76) (envelope-from ) id 1bSIH1-00024g-B2 for fwts-devel@lists.ubuntu.com; Wed, 27 Jul 2016 06:24:55 +0000 Received: from [175.181.156.151] (helo=canonical.com) by youngberry.canonical.com with esmtpsa (TLS1.0:DHE_RSA_AES_128_CBC_SHA1:16) (Exim 4.76) (envelope-from ) id 1bSIH0-0004Ul-FQ; Wed, 27 Jul 2016 06:24:55 +0000 From: Ivan Hu To: fwts-devel@lists.ubuntu.com Subject: [PATCH] acpi/madt: fix build fail for i386 Date: Wed, 27 Jul 2016 14:24:49 +0800 Message-Id: <1469600689-24714-1-git-send-email-ivan.hu@canonical.com> X-Mailer: git-send-email 1.9.1 MIME-Version: 1.0 X-BeenThere: fwts-devel@lists.ubuntu.com X-Mailman-Version: 2.1.14 Precedence: list List-Id: Firmware Test Suite Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: fwts-devel-bounces@lists.ubuntu.com Sender: fwts-devel-bounces@lists.ubuntu.com In file included from ../src/lib/include/fwts_binpaths.h:27:0, from ../src/lib/include/fwts.h:54, from acpi/madt/madt.c:16: acpi/madt/madt.c: In function ‘madt_find_processor_uid’: acpi/madt/madt.c:304:20: error: format ‘%lu’ expects argument of type ‘long unsigned int’, but argument 8 has type ‘uint64_t {aka long long unsigned int}’ [-Werror=format=] fwts_passed(fw, "MADT %s has matching processor " ^ ../src/lib/include/fwts_framework.h:231:84: note: in definition of macro ‘fwts_passed’ fwts_framework_log(fw, LOG_PASSED, NULL, LOG_LEVEL_NONE, &fw->minor_tests.passed, fmt, ## args) ^ acpi/madt/madt.c:312:20: error: format ‘%lu’ expects argument of type ‘long unsigned int’, but argument 8 has type ‘uint64_t {aka long long unsigned int}’ [-Werror=format=] table_label, "%s has no matching processor UID %lu", ^ ../src/lib/include/fwts_framework.h:233:76: note: in definition of macro ‘fwts_failed’ fwts_framework_log(fw, LOG_FAILED, label, level, &fw->minor_tests.failed, fmt, ## args) ^ cc1: all warnings being treated as errors Signed-off-by: Ivan Hu Acked-by: Alex Hung Acked-by: Colin Ian King --- src/acpi/madt/madt.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/acpi/madt/madt.c b/src/acpi/madt/madt.c index 3032e7b..257d7d9 100644 --- a/src/acpi/madt/madt.c +++ b/src/acpi/madt/madt.c @@ -302,14 +302,14 @@ static ACPI_OBJECT_TYPE madt_find_processor_uid(fwts_framework *fw, listint = fwts_list_data(struct acpi_integer *, item); if (uid == listint->value) { fwts_passed(fw, "MADT %s has matching processor " - "UID %lu.", table_name, uid); + "UID %" PRIu64 ".", table_name, uid); return listint->type; } } sprintf(table_label, "MADT%sUidMismatch", table_name); fwts_failed(fw, LOG_LEVEL_MEDIUM, - table_label, "%s has no matching processor UID %lu", + table_label, "%s has no matching processor UID %" PRIu64, table_name, uid); return ACPI_NUM_TYPES; }