From patchwork Wed Dec 12 19:11:57 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Colin Ian King X-Patchwork-Id: 205626 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 629862C0084 for ; Thu, 13 Dec 2012 06:12:00 +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 1Tirip-0006ys-88; Wed, 12 Dec 2012 19:11:59 +0000 Received: from youngberry.canonical.com ([91.189.89.112]) by chlorine.canonical.com with esmtp (Exim 4.71) (envelope-from ) id 1Tirio-0006yn-0F for fwts-devel@lists.ubuntu.com; Wed, 12 Dec 2012 19:11:58 +0000 Received: from cpc3-craw6-2-0-cust180.croy.cable.virginmedia.com ([77.100.248.181] helo=localhost) by youngberry.canonical.com with esmtpsa (TLS1.0:DHE_RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1Tirin-0005Wm-RL for fwts-devel@lists.ubuntu.com; Wed, 12 Dec 2012 19:11:57 +0000 From: Colin King To: fwts-devel@lists.ubuntu.com Subject: [PATCH] apci: method: don't bother dumping out the _BST contents Date: Wed, 12 Dec 2012 19:11:57 +0000 Message-Id: <1355339517-1964-1-git-send-email-colin.king@canonical.com> X-Mailer: git-send-email 1.8.0 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 From: Colin Ian King Since we just want to sanity check the type of data being returned and we can't gather _BST information from the EC we shouldn't dump the package, it makes no sense and clutters up the output. Signed-off-by: Colin Ian King Acked-by: Keng-Yu Lin Acked-by: Ivan Hu --- src/acpi/method/method.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/acpi/method/method.c b/src/acpi/method/method.c index 2dcbee0..7831a92 100644 --- a/src/acpi/method/method.c +++ b/src/acpi/method/method.c @@ -2750,8 +2750,6 @@ static void method_test_BST_return( uint32_t i; int failed = 0; - fwts_method_dump_object(fw, obj); - if (obj->Package.Count != 4) { fwts_failed(fw, LOG_LEVEL_MEDIUM, "Method_BSTElementCount",