From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <sami.mujawar@arm.com>
Received-SPF: Pass (sender SPF authorized) identity=mailfrom;
 client-ip=217.140.96.140; helo=cam-smtp0.cambridge.arm.com;
 envelope-from=sami.mujawar@arm.com; receiver=edk2-devel@lists.01.org 
Received: from cam-smtp0.cambridge.arm.com (fw-tnat.cambridge.arm.com
 [217.140.96.140]) (using TLSv1 with cipher AES256-SHA (256/256 bits))
 (No client certificate requested)
 by ml01.01.org (Postfix) with ESMTPS id EFF822111FE4E
 for <edk2-devel@lists.01.org>; Wed, 20 Jun 2018 09:43:03 -0700 (PDT)
Received: from E107187.Emea.Arm.com (E107187.Emea.Arm.com [10.1.206.129])
 by cam-smtp0.cambridge.arm.com (8.13.8/8.13.8) with ESMTP id w5KGgxHL028741;
 Wed, 20 Jun 2018 17:42:59 +0100
From: Sami Mujawar <sami.mujawar@arm.com>
To: edk2-devel@lists.01.org
Cc: Arvind Chauhan <Arvind.Chauhan@arm.com>,
 Daniil Egranov <Daniil.Egranov@arm.com>,
 Thomas Panakamattam Abraham <thomas.abraham@arm.com>,
 leif.lindholm@linaro.org, evan.lloyd@arm.com, Matteo.Carlini@arm.com,
 Stephanie.Hughes-Fitt@arm.com, nd@arm.com
Date: Wed, 20 Jun 2018 17:42:54 +0100
Message-Id: <20180620164254.5076-1-sami.mujawar@arm.com>
X-Mailer: git-send-email 2.11.0.windows.3
Subject: [PATCH edk2-platforms v1][platforms/devel-dynamictables] Platform/ARM: Fix OEM Table ID length in DSDT
X-BeenThere: edk2-devel@lists.01.org
X-Mailman-Version: 2.1.26
Precedence: list
List-Id: EDK II Development  <edk2-devel.lists.01.org>
List-Unsubscribe: <https://lists.01.org/mailman/options/edk2-devel>,
 <mailto:edk2-devel-request@lists.01.org?subject=unsubscribe>
List-Archive: <http://lists.01.org/pipermail/edk2-devel/>
List-Post: <mailto:edk2-devel@lists.01.org>
List-Help: <mailto:edk2-devel-request@lists.01.org?subject=help>
List-Subscribe: <https://lists.01.org/mailman/listinfo/edk2-devel>,
 <mailto:edk2-devel-request@lists.01.org?subject=subscribe>
X-List-Received-Date: Wed, 20 Jun 2018 16:43:04 -0000

The OEM Table ID length cannot be more than 8 characters.
Moreover, the ACPICA iASL compiler patch "Add error messages
for ACPI tables with bad OEM and OEM Table IDs" dated
13 April 2018 [20180413]
(https://github.com/acpica/acpica/commit/ca828bd15604699c14a02e98f4d0d9c4d6885e44)
introduced checks for the OEM Table ID length. If the
OEM Table ID length is more than 8 characters in the
DSDT/SSDT table's DefinitionBlock(), it flags them as
an error.

This patch adjusts the length of the OEM Table ID to
8 characters in the DSDT and SSDT tables.

Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Sami Mujawar <sami.mujawar@arm.com>
---

The changes can be seen at https://github.com/samimujawar/edk2-platforms/tree/289_fix_dsdt_definitionblock_v1

Notes:
    v1:
    - Fixed DSDT build error flagged by latest ACPICA iasl compiler [SAMI]

 Platform/ARM/VExpressPkg/ConfigurationManager/PlatformASLTablesLib/Dsdt.asl | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Platform/ARM/VExpressPkg/ConfigurationManager/PlatformASLTablesLib/Dsdt.asl b/Platform/ARM/VExpressPkg/ConfigurationManager/PlatformASLTablesLib/Dsdt.asl
index 15ce7c020045d42f08cf92ad76644b07f0a1e25e..fae6a4be423b4b7da5e9677139f2ab3deac5520e 100644
--- a/Platform/ARM/VExpressPkg/ConfigurationManager/PlatformASLTablesLib/Dsdt.asl
+++ b/Platform/ARM/VExpressPkg/ConfigurationManager/PlatformASLTablesLib/Dsdt.asl
@@ -12,7 +12,7 @@
 
 **/
 
-DefinitionBlock("DsdtTable.aml", "DSDT", 1, "ARMLTD", "ARM-VEXPRESS", 1) {
+DefinitionBlock("DsdtTable.aml", "DSDT", 1, "ARMLTD", "ARM-VEXP", 1) {
   Scope(_SB) {
     //
     // Processor
-- 
'Guid(CE165669-3EF3-493F-B85D-6190EE5B9759)'