From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from NAM04-MW2-obe.outbound.protection.outlook.com (NAM04-MW2-obe.outbound.protection.outlook.com [40.107.101.128]) by mx.groups.io with SMTP id smtpd.web12.3585.1657769706529545320 for ; Wed, 13 Jul 2022 20:35:06 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@os.amperecomputing.com header.s=selector2 header.b=kZ+Kw4of; spf=pass (domain: os.amperecomputing.com, ip: 40.107.101.128, mailfrom: nhi@os.amperecomputing.com) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=FwAhp0WXfqX3Iwc2+xw4pZHimch0IXA9g7HHrbqzA5f9Og69pXU+PP1UG7Os8NymXsFKhNxtewuELt2Dd88QlcrYaHec9/hf/Slq2PUmYrIy5+pNNNhvsnXjIPht2I6hgauusE1JOTlZlkBt9VF8DzqRllEX0mYE7MD5+f7yiG/+/5db4/E1x/uR0Kif+/kSqiw82wo2D0evse8my9QuGVrw90QcPofGL7LPQaLvWwUVaXze18BtKRl1zsGiHilV1g3x4JFjJqVwRSB2z9QSCuiapSsojQo+6riXeUkE0I8kS9337kA+iTvXUQGQHs4aHHuFEwqRTCB4+PfxfzMX9A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=QmQzFe13BRIjcWCdUTM+A9TbL+xh5pHtBSBNm39ldDw=; b=TJ2vs6RfNlvZDPfC+mXiPx0fljVS8HNATqdwIgrg+BhJ1u7zfzk9Aw8HtHHvXFDsrhpMUpncyVh9HsHGunH3ymVQWCLutrJECrXwYGVc9nccm9CqrxflxmGd57sJvRnsnQcLH/NDKpzo3j7jfxzflkjpwvKM3ETXWI663Y6AOtLL/BOvF2LDivuCgymwh2PMJmY0wEvW/Gdh7MsW0x0pOO3PRcTDrmtCiAQrPqxXSilrWUjEdLMg12ktchM5cJmB8PgOxV9UNqwaLPC8FZuESh3rkelWtat3NiJRu8kUfa1zijjJEXdZIMCW9fHZ8UlwvzWFdNlKkRd21SQ5DM7V1g== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=os.amperecomputing.com; dmarc=pass action=none header.from=os.amperecomputing.com; dkim=pass header.d=os.amperecomputing.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=os.amperecomputing.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=QmQzFe13BRIjcWCdUTM+A9TbL+xh5pHtBSBNm39ldDw=; b=kZ+Kw4ofQpteAz74+9mVdiD/yROUQxfMDZiAY4NOt1ML540JFRNPhfY4AAoAXOoGoXrt19ASFCFnErP+zD92EtcPXjwcsuFWjU1Yd/dTYj5pGCPQrS9DJkSke7zOLUa1iqdtJ4inqwMxlHgnBZUOzkXvR5stDRHUaKEFJnYv0Kk= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=os.amperecomputing.com; Received: from PH0PR01MB7287.prod.exchangelabs.com (2603:10b6:510:10a::21) by SJ0PR01MB7527.prod.exchangelabs.com (2603:10b6:a03:3da::12) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5417.25; Thu, 14 Jul 2022 03:35:04 +0000 Received: from PH0PR01MB7287.prod.exchangelabs.com ([fe80::b476:d1a1:cacb:d7fe]) by PH0PR01MB7287.prod.exchangelabs.com ([fe80::b476:d1a1:cacb:d7fe%9]) with mapi id 15.20.5417.026; Thu, 14 Jul 2022 03:35:04 +0000 Message-ID: <143bd56e-98ee-6467-717b-c4a983be136a@os.amperecomputing.com> Date: Thu, 14 Jul 2022 10:34:53 +0700 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:91.0) Gecko/20100101 Thunderbird/91.11.0 Subject: Re: [PATCH edk2-platforms v3 09/12] Platform/Ampere: JadePkg: Update for IORT revision macro renaming To: Sami Mujawar , devel@edk2.groups.io Cc: vunguyen@os.amperecomputing.com, thang@os.amperecomputing.com, chuong@os.amperecomputing.com, quic_llindhol@quicinc.com, Matteo.Carlini@arm.com, Akanksha.Jain2@arm.com, Ben.Adderson@arm.com, nd@arm.com References: <20220712144047.3080-1-sami.mujawar@arm.com> <20220712144047.3080-10-sami.mujawar@arm.com> From: "Nhi Pham" In-Reply-To: <20220712144047.3080-10-sami.mujawar@arm.com> X-ClientProxiedBy: SG2PR02CA0056.apcprd02.prod.outlook.com (2603:1096:4:54::20) To PH0PR01MB7287.prod.exchangelabs.com (2603:10b6:510:10a::21) Return-Path: nhi@os.amperecomputing.com MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 57c3aba6-97fe-472f-080c-08da6549d742 X-MS-TrafficTypeDiagnostic: SJ0PR01MB7527:EE_ X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: FHuMm7uRBQsrrNzN6QprHgx5jqv36f+kJ2oF+ejp6e0DCIU4348FisddpDEiZOTM3eICmcNINC76zw7BRZlm+0w+eLcczoidWeR+KHsnjTrwVdvDrUHdZLM7RaSDJPjWt3/f5fR/gTe7DO7SfcEMZRg5pKuuFdB0lS+JjRbjWogee5K+D7CgcjSjYUlKBFDihgOsV0P1X2u8uZYla6Z5fZydSSvSjA9aQbDBmiDvHPoudHVyyAi7pkN2iXTWCMKT2CwRETT9hrv0HSNXjQnfOw8stX5zwN0s6lLjPeeWeOE5sNWfP6nmb874+M1zI8FiQurX9B/Td/tEdFD02I5hFeiAhS5RFpJLur8nu0hgdJMdqkUIYFsAkf1FML+BLJXP+saHq5nUr11oHgaiRweyLhkRnljZKMZjvmWtLH2lotI2OsOsgwQmkQPU0JtxapFJGnDJeZOPvKWGs3vs06GN6u0L4zzDZC1zTkjXENClUYakybJzPwGaC1UXwG+0FoYuu/wtgWvpmHrBXh1gBEltLtk41clDrVSgKzMjQ141bnSCRFF2hCmzgdW6KRarQ/6VrVsf/DLKaUBwqcV5oJeXWyBCcRN5af7bZ34MNncDV3ErzxtTz5yV0BKfI0r9MEvYGp1b7QrsnHecMNxEgnbSUSVFetpOL9HLLDC+iQA3ZbeFXijQrifxXobouDwy4iOCEDGddCL7a+sNkaXfY0ci6yzNeh89C4rjfPWySvFhn84bNlUABOKSn7WnzjzCXLTaePVxOiCDGeiZkWU+8Dyg8pcCH7VbI2wG/pu6fuQ4+4LDb3dPMxBI2mcrt+zrX9U3JDQxwDXmVYVXvRiwBv4Zy5oaR8bOAiNHm0OYjO4476s= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:PH0PR01MB7287.prod.exchangelabs.com;PTR:;CAT:NONE;SFS:(13230016)(4636009)(39850400004)(396003)(366004)(376002)(346002)(136003)(2906002)(8676002)(31686004)(6666004)(38100700002)(478600001)(38350700002)(31696002)(5660300002)(8936002)(316002)(83380400001)(15650500001)(41300700001)(186003)(4326008)(66556008)(52116002)(6506007)(66476007)(86362001)(66946007)(6512007)(2616005)(26005)(53546011)(6486002)(43740500002);DIR:OUT;SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?QzlpTjdrY1gybmk4cWhsWkpzQUc2WTRjWlVOUEx6NnVvTkV0TzhvU3JVVnFK?= =?utf-8?B?WDRCTlZ5VUQvYWNpQVN4SEYvNElxWEx4RDZubHFmWnRhN0RxNDdEcVFqdzBn?= =?utf-8?B?akNLc0Z3bmNqOU10L0V3TTFWN3VrWWdoaTRGYUwvZWVwcmFNNWk3UzVXS243?= =?utf-8?B?SDRLYVIyS09ldHdkdUhYMVdRRlg5YmpSNHR6MUQ2L0xjMjhPUW9CTTcwVU1j?= =?utf-8?B?aWo4eG5QbXRYQ1pjMXZVdEJqVWpSSU1KcmU1MmsxMEhibUllbTdtUzhXaGsw?= =?utf-8?B?YTNPeGlJcHh6eDZyTmZUdjVOTThsb1FvZnR2ZmxPSHZSUzVaa2cvZ0ZVdGJ2?= =?utf-8?B?V3Q4RnhzWGtSMWozVjNNVk1XZFNORS8vSmJJWkFacHplclU1MW0vUG1iNUww?= =?utf-8?B?YUlWdjBmSFl3ajlHUnd5cVZxellJWitocDZia1p5Tll3WDVsY1lXdVhJdlpP?= =?utf-8?B?MWdxbm82cDFqUllFQ2F3R1N2SEtuVm5PR2hoZFVzTXJjMWlUaVBJRFlBaXR1?= =?utf-8?B?ZzlZdzhQdm52QUw5V3BqRCtEaVNYc0tQTlJRVS9hSnN5d09ISDZsclZoUmNT?= =?utf-8?B?UEVQNlN2RmowTysyYzVXVlFxSGxyOW9SSEdKd1lpbTVzNnZ6ZTI5RXNLOG00?= =?utf-8?B?Ukx3NFlpMHJpeURQK25NeE5tMXdWTEhBdmdxSTNlYlZYVmE5MmprQjIzekc4?= =?utf-8?B?T1dEM3ZzQWEwbVh0OEFjSlprV0dkeGtCdWxhTWtNYzJHZk5ZeXdpMzVpSTJ5?= =?utf-8?B?YlpzRXBpNkoyb2dYY1hrbVdTSEJ4bm1ZY2RWM05KNG55V045WldhTEZIK2lW?= =?utf-8?B?b0RuQ2o0U0pLY2ZDam9oc1ZqNTBVTGNjZmFnYjQ2V094N2J4dnFJcjNqYlVz?= =?utf-8?B?eTQ3ZEJGcVNqY0ZZbk1KemtlRk9uTEI5ekhsZE00dGFneVJHWldsa09uejlX?= =?utf-8?B?cCs3c1RKZktEU2EvN1JpVXNJTCtmRGRCcE5SWUg5clNqam5rTklOaEtzTENX?= =?utf-8?B?Z0dTTUNPZWI5SzB4SnVEekFSQUxlSGl2Szh4VzIrRmszeDgyNHIzVzlOVnBi?= =?utf-8?B?ZTR3L3IydjRIbERWRGsvVUhabWNacW1CTVpkQlBXWUhxQ3Y5N0RFZnRKVXk2?= =?utf-8?B?ek10ekxZdUJrTFk3aUZ2YnJtTjZnZkJXdFo0V3FCUUYrZjQ2emdzV3RoR1BT?= =?utf-8?B?QTZWS0R5MU0zb243QXpxSXZ3Z0Z5UUIvdkRsY3F4eDZJejF6TnluTUZId09Y?= =?utf-8?B?ZVdpMkloMG5TWHRRbm80ai9NMWhUWWJvaDBRb2MxRzFWcUlIMUZDQWVGOTl6?= =?utf-8?B?L2RMSkxnd3ByTmk4RmVaQjM3ZEUzeUhTVStsdjdHa0Q0WW9ZTTRUckVYeFJE?= =?utf-8?B?STl1TGtOcWZ1b3BLR3ViWXhrRkZpRGJac2tiUFpqdHEzbWloNUhyNG92YVBa?= =?utf-8?B?RnVSZ1hDcUlrTFA4V0wyZTFNQ09leGd1aGlmQWN0cS9mOWt0UXpYb0tQS3cx?= =?utf-8?B?cXVLY3FTdUNhSlBaWkEwcDY4N3I3UWtkV1RwVkp1SzlJUWFTYjlIb2NyUG1i?= =?utf-8?B?RmlqNHVSU2oxMHdoSnhVMGtkOFVqcjhrZDM1b1R2TGFXbFlWeHFNMEVXWFdl?= =?utf-8?B?SXhFZm44NTVzQ05IWVFQSEF6SDZGNGkzUGVUSTcrWXI4L0tPbUh6YisyaTZZ?= =?utf-8?B?cUg0eGtlMnBkVWdwMTdMU2FINzdqUUJnUzlLUEtCUW9lb0ZOSHNQeDBtNlkw?= =?utf-8?B?VDJkam5MbS9nclpjWmhxVmtSNDZNeHU4d1pMT1V2bGh3UjQwcXovam9GeEdi?= =?utf-8?B?OEdsVEJFeXNrUlpTalB4ZFVQeDRFblIrZjJlWTR6MGEzZ2VtM1Q2anUxdDc3?= =?utf-8?B?NDFhdllqRnFaUzc5YVl2c2VoVmNMUFVWMzV2V0ZYbVBKYkpOVHlkNldSWUJX?= =?utf-8?B?WGtSWmc5ZjhpLzZ2MkFmWnc5a1ZURC91TDdNN3hGSUpxejFibURsY3U3YUR3?= =?utf-8?B?NnhEclNxNmdneXdvVnIvWHpXWE4yK3VWam9uNWMrMzhhT3VEUy9IY2x1Wkla?= =?utf-8?B?SlIzMFNiQ1VtbXI0emlZajhzU3JwK1BoMi9XcU4zQ283U2kvbStWaFFTUGhB?= =?utf-8?B?ZTQ3RDhjU2ZXT0NaSnJtK1plYmZQM0JIMWluQ3UwRzgyVTBtTTArV0tCVXRL?= =?utf-8?Q?72KiCYcE7Txn6BpePQoJTh0=3D?= X-OriginatorOrg: os.amperecomputing.com X-MS-Exchange-CrossTenant-Network-Message-Id: 57c3aba6-97fe-472f-080c-08da6549d742 X-MS-Exchange-CrossTenant-AuthSource: PH0PR01MB7287.prod.exchangelabs.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 14 Jul 2022 03:35:04.8073 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 3bc2b170-fd94-476d-b0ce-4229bdc904a7 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: +jaKNic50gLRRJf0k4UB2fF63alHkml1CL66rVZ5yjdSbayC80ZwH+dcW+kC1A/SrtfK9DDuzcuk81Twuski/gne3syg0lieIA8jD8JNF4c= X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ0PR01MB7527 Content-Language: en-US Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Reviewed-by: Nhi Pham Thanks, Nhi On 12/07/2022 21:40, Sami Mujawar wrote: > The IORT Specification E.d updates the IORT table revision to 5. To > reflect this change the IORT header file has been updated to rename > the EFI_ACPI_IO_REMAPPING_TABLE_REVISION macro which was at Rev 0 to > EFI_ACPI_IO_REMAPPING_TABLE_REVISION_00. > > Therefore, update the usage of EFI_ACPI_IO_REMAPPING_TABLE_REVISION > macro in the IORT table for SynQuacer platform to > EFI_ACPI_IO_REMAPPING_TABLE_REVISION_00. > > Signed-off-by: Sami Mujawar > --- > Platform/Ampere/JadePkg/Drivers/AcpiPlatformDxe/AcpiIort.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/Platform/Ampere/JadePkg/Drivers/AcpiPlatformDxe/AcpiIort.c b/Platform/Ampere/JadePkg/Drivers/AcpiPlatformDxe/AcpiIort.c > index 97be85c51f253422851ba139ee60bc8fc681aa87..958708810a4918b1683ecc5fb021b59a7191e207 100644 > --- a/Platform/Ampere/JadePkg/Drivers/AcpiPlatformDxe/AcpiIort.c > +++ b/Platform/Ampere/JadePkg/Drivers/AcpiPlatformDxe/AcpiIort.c > @@ -87,7 +87,7 @@ EFI_ACPI_6_0_IO_REMAPPING_TABLE mIortHeader = { > .Header = __ACPI_HEADER ( > EFI_ACPI_6_0_IO_REMAPPING_TABLE_SIGNATURE, > AC01_IO_REMAPPING_STRUCTURE, > - EFI_ACPI_IO_REMAPPING_TABLE_REVISION > + EFI_ACPI_IO_REMAPPING_TABLE_REVISION_00 > ), > .NumNodes = 0, // To be filled > .NodeOffset = sizeof (EFI_ACPI_6_0_IO_REMAPPING_TABLE),