From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=40.107.8.41; helo=eur04-vi1-obe.outbound.protection.outlook.com; envelope-from=sami.mujawar@arm.com; receiver=edk2-devel@lists.01.org Received: from EUR04-VI1-obe.outbound.protection.outlook.com (mail-eopbgr80041.outbound.protection.outlook.com [40.107.8.41]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id A48D6211DB422 for ; Tue, 19 Mar 2019 06:48:01 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=armh.onmicrosoft.com; s=selector1-arm-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=6kWYCKWOvKuolQugSWERbdSJYdRX16m9QbLv/GNIhNM=; b=racrWKGx+CDx0QyzaOQ/aK9wlwvuDNXCkoCTLv0e4yGMxg4etRZfFftcBsmSzCxcz+QAaonx1I/gEhEUtow8jgcheMnkuuakhhtFquWlbrnfJOSG7tRn+JMllj87DxJUfr4CKY0KQHaHkJ/OSqcbqz1AqKyxiNMGIlT0Y1LWemY= Received: from VI1PR0801CA0080.eurprd08.prod.outlook.com (10.173.67.152) by AM4PR0801MB1475.eurprd08.prod.outlook.com (10.168.5.137) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1709.14; Tue, 19 Mar 2019 13:47:59 +0000 Received: from VE1EUR03FT042.eop-EUR03.prod.protection.outlook.com (2a01:111:f400:7e09::205) by VI1PR0801CA0080.outlook.office365.com (2603:10a6:800:7d::24) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384) id 15.20.1709.13 via Frontend Transport; Tue, 19 Mar 2019 13:47:59 +0000 Authentication-Results: spf=pass (sender IP is 40.67.248.234) smtp.mailfrom=arm.com; lists.01.org; dkim=none (message not signed) header.d=none;lists.01.org; dmarc=bestguesspass action=none header.from=arm.com; Received-SPF: Pass (protection.outlook.com: domain of arm.com designates 40.67.248.234 as permitted sender) receiver=protection.outlook.com; client-ip=40.67.248.234; helo=nebula.arm.com; Received: from nebula.arm.com (40.67.248.234) by VE1EUR03FT042.mail.protection.outlook.com (10.152.19.62) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256) id 15.20.1730.9 via Frontend Transport; Tue, 19 Mar 2019 13:47:58 +0000 Received: from AZ-NEU-EX03.Arm.com (10.251.24.31) by AZ-NEU-EX03.Arm.com (10.251.24.31) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.1415.2; Tue, 19 Mar 2019 13:47:36 +0000 Received: from E107187.Arm.com (10.1.195.55) by mail.arm.com (10.251.24.31) with Microsoft SMTP Server id 15.1.1415.2 via Frontend Transport; Tue, 19 Mar 2019 13:47:36 +0000 From: Sami Mujawar To: CC: , , , , , Date: Tue, 19 Mar 2019 13:47:25 +0000 Message-ID: <20190319134726.95524-6-sami.mujawar@arm.com> X-Mailer: git-send-email 2.11.0.windows.3 In-Reply-To: <20190319134726.95524-1-sami.mujawar@arm.com> References: <20190319134726.95524-1-sami.mujawar@arm.com> MIME-Version: 1.0 X-EOPAttributedMessage: 0 X-MS-Office365-Filtering-HT: Tenant X-Forefront-Antispam-Report: CIP:40.67.248.234; IPV:NLI; CTRY:US; EFV:NLI; SFV:NSPM; SFS:(10009020)(346002)(39860400002)(396003)(376002)(136003)(2980300002)(189003)(199004)(106002)(81156014)(76176011)(476003)(4326008)(51416003)(54906003)(53936002)(68736007)(8676002)(356004)(1076003)(6666004)(486006)(2616005)(446003)(7696005)(316002)(8936002)(11346002)(16586007)(336012)(44832011)(126002)(72206003)(426003)(81166006)(305945005)(14444005)(50226002)(6916009)(97736004)(36756003)(186003)(478600001)(50466002)(5660300002)(53416004)(2351001)(48376002)(77096007)(26005)(2906002)(47776003)(106466001)(86362001); DIR:OUT; SFP:1101; SCL:1; SRVR:AM4PR0801MB1475; H:nebula.arm.com; FPR:; SPF:Pass; LANG:en; PTR:InfoDomainNonexistent; MX:1; A:1; X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: b92abacd-05c0-4c36-8ef9-08d6ac717f78 X-Microsoft-Antispam: BCL:0; PCL:0; RULEID:(2390118)(7020095)(4652040)(8989299)(4534185)(4627221)(201703031133081)(201702281549075)(8990200)(5600127)(711020)(4605104)(4608103)(4709054)(2017052603328)(7153060); SRVR:AM4PR0801MB1475; X-MS-TrafficTypeDiagnostic: AM4PR0801MB1475: NoDisclaimer: True X-Microsoft-Antispam-PRVS: X-Forefront-PRVS: 0981815F2F X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam-Message-Info: rBhADYxPmovMVZYM9OZTQbPtm0KKIbx+G7NdiKwoeu3YHUWUzeksV40UbGn36v7BZdstu0TQTv1CFcWRFTHAuwJZI4f5RFo8WQRW1VhOHd9dT3BZtHpwdAB0XOccvHG1oODsnP5eQRl+pcg+6QYmK4FPyF531wyx06hv9aEfdeO/vLlxhTQsYfRKjIbCu8v6shXo6naSeeN6xda1fnlI6BqjWhgUVUZpxER721NosSf7d0VADc+vPV8cyegQ/Mjon7xb70yv3Xm/iWiqk4Kt2vjFUbVm7HDWybfc0yU4m+njmG9oYdhrYn+tA9aM9AzNWA6OQIH5piQrk5RJqqkPJgVOsKydfqcu07MV49uvaJ7K2yNMHDhsPCG0Rrm9zIADEHrG/mdruHfSw+vDR94u2urBewOij1vyG1UIorBcITk= X-OriginatorOrg: arm.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Mar 2019 13:47:58.7992 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: b92abacd-05c0-4c36-8ef9-08d6ac717f78 X-MS-Exchange-CrossTenant-Id: f34e5979-57d9-4aaa-ad4d-b122a662184d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=f34e5979-57d9-4aaa-ad4d-b122a662184d; Ip=[40.67.248.234]; Helo=[nebula.arm.com] X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM4PR0801MB1475 Subject: [PATCH v2 5/6] DynamicTablesPkg: Remove GIC Distributor Id field X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 19 Mar 2019 13:48:02 -0000 X-List-Received-Date: Tue, 19 Mar 2019 13:48:02 -0000 X-List-Received-Date: Tue, 19 Mar 2019 13:48:02 -0000 X-List-Received-Date: Tue, 19 Mar 2019 13:48:02 -0000 X-List-Received-Date: Tue, 19 Mar 2019 13:48:02 -0000 X-List-Received-Date: Tue, 19 Mar 2019 13:48:02 -0000 X-List-Received-Date: Tue, 19 Mar 2019 13:48:02 -0000 X-List-Received-Date: Tue, 19 Mar 2019 13:48:02 -0000 X-List-Received-Date: Tue, 19 Mar 2019 13:48:02 -0000 X-List-Received-Date: Tue, 19 Mar 2019 13:48:02 -0000 X-List-Received-Date: Tue, 19 Mar 2019 13:48:02 -0000 X-List-Received-Date: Tue, 19 Mar 2019 13:48:02 -0000 X-List-Received-Date: Tue, 19 Mar 2019 13:48:02 -0000 X-List-Received-Date: Tue, 19 Mar 2019 13:48:02 -0000 X-List-Received-Date: Tue, 19 Mar 2019 13:48:02 -0000 X-List-Received-Date: Tue, 19 Mar 2019 13:48:02 -0000 X-List-Received-Date: Tue, 19 Mar 2019 13:48:02 -0000 X-List-Received-Date: Tue, 19 Mar 2019 13:48:02 -0000 X-List-Received-Date: Tue, 19 Mar 2019 13:48:02 -0000 X-List-Received-Date: Tue, 19 Mar 2019 13:48:02 -0000 X-List-Received-Date: Tue, 19 Mar 2019 13:48:02 -0000 X-List-Received-Date: Tue, 19 Mar 2019 13:48:02 -0000 X-List-Received-Date: Tue, 19 Mar 2019 13:48:02 -0000 X-List-Received-Date: Tue, 19 Mar 2019 13:48:02 -0000 X-List-Received-Date: Tue, 19 Mar 2019 13:48:02 -0000 Content-Type: text/plain This patch was originally merged in edk2 master at d3a15f435f9716aa2f2ea5e9b35fcda04f267ab4. However, this was later reverted at 82c4426a176c271d24915b403b267a431bdc52f5 as it was merged during the Soft Feature Freeze for edk2-stable201903. Resubmitting this patch as the edk2 merge window is now open. According to ACPI 6.2 Specification - Errata A, 'One, and only one, GIC distributor structure must be present in the MADT for an ARM based system'. Therefore, the GIC Distributor ID field in the ACPI MADT GICD substructure can be set to zero and there is no need for the Configuration Manager to provide this information. Update the CM_ARM_GICD_INFO structure to remove the GicId field. Similarly update the MADT Generator to set the GicId field in the GICD substructure to zero. Cc: Laszlo Ersek Cc: Alexei Fedorov Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Sami Mujawar Reviewed-by: Alexei Fedorov --- DynamicTablesPkg/Include/ArmNameSpaceObjects.h | 5 +---- DynamicTablesPkg/Library/Acpi/Arm/AcpiMadtLibArm/MadtGenerator.c | 4 +++- 2 files changed, 4 insertions(+), 5 deletions(-) diff --git a/DynamicTablesPkg/Include/ArmNameSpaceObjects.h b/DynamicTablesPkg/Include/ArmNameSpaceObjects.h index 2adaa788083e4e15d049f5f6b6312f77f3c02488..73139eb3273e89205ef9e6bee67b1b842f7f516b 100644 --- a/DynamicTablesPkg/Include/ArmNameSpaceObjects.h +++ b/DynamicTablesPkg/Include/ArmNameSpaceObjects.h @@ -1,6 +1,6 @@ /** @file - Copyright (c) 2017 - 2018, ARM Limited. All rights reserved. + Copyright (c) 2017 - 2019, ARM Limited. All rights reserved. This program and the accompanying materials are licensed and made available under the terms and conditions of the BSD License @@ -166,9 +166,6 @@ typedef struct CmArmGicCInfo { GIC Distributor information for the Platform. */ typedef struct CmArmGicDInfo { - /// The GIC Distributor ID. - UINT32 GicId; - /// The Physical Base address for the GIC Distributor. UINT64 PhysicalBaseAddress; diff --git a/DynamicTablesPkg/Library/Acpi/Arm/AcpiMadtLibArm/MadtGenerator.c b/DynamicTablesPkg/Library/Acpi/Arm/AcpiMadtLibArm/MadtGenerator.c index 1ee0b9e006bfa00ea5bbf6aa64209dac244c90a0..6b89c3f65d36ff35fcfc81cdffdeae8e4306c976 100644 --- a/DynamicTablesPkg/Library/Acpi/Arm/AcpiMadtLibArm/MadtGenerator.c +++ b/DynamicTablesPkg/Library/Acpi/Arm/AcpiMadtLibArm/MadtGenerator.c @@ -192,7 +192,9 @@ AddGICD ( // UINT16 Reserved Gicd->Reserved1 = EFI_ACPI_RESERVED_WORD; // UINT32 Identifier - Gicd->GicId = GicDInfo->GicId; + // One, and only one, GIC distributor structure must be present + // in the MADT for an ARM based system + Gicd->GicId = 0; // UINT64 PhysicalBaseAddress Gicd->PhysicalBaseAddress = GicDInfo->PhysicalBaseAddress; // UINT32 VectorBase -- 'Guid(CE165669-3EF3-493F-B85D-6190EE5B9759)'