From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from NAM10-DM6-obe.outbound.protection.outlook.com (NAM10-DM6-obe.outbound.protection.outlook.com [40.107.93.91]) by mx.groups.io with SMTP id smtpd.web10.2371.1648522667746888484 for ; Mon, 28 Mar 2022 19:57:48 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@os.amperecomputing.com header.s=selector2 header.b=lv8jYb4H; spf=pass (domain: os.amperecomputing.com, ip: 40.107.93.91, mailfrom: nhi@os.amperecomputing.com) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=fGTzLhu+yuz9J7OLvWulvpmYu6pBBHce5PULX+5ztnsIbidy9g+yuHwC4McWQENQsJy3GoaF7Scdx8HGFbKkSaCt5TvH7sloK0THDRpMtTE5SHn6t5Wc6BA427p9GsoyLO+6EqV6T5nzqwA+g95MEvl/VzU4uCi5At+qlI6Cy+UpWfcWrRTtCm4YKskbFZS5bvTv/la5mXpJVAJAFjXKHcmfuGjruu1jT/m/IaJ+nrPXEeyQ40xtJzgCcpZkMI6rKh+QDNYoDC11OANaUMOdUXg+WuShQyz3UNVquI+Uad/UjQKM1dtL7wjgvryHK5TUHpuhlGobLNAno7jIAK0GYQ== 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=lM1fqPtfbIVTa9d9CP8DB6hNe0tpRq7dDhcBD1Itge4=; b=McIa3R6WjS1pNBiV3qZKca7HgAUlIR1KDtA1S39eIOUKItaRtPrmixFRGiXAjbyyCB2DhL1Mo53wNif/yK6NkCBS93TLJvKPZrJBGhc/7V75mecF39ZvKLA9PtVh4sZUAXIEqhloNxKq3d/Mb2CRz79vj+WWif1xr93/7qIKurg+/3B+aPacplAP2gGcZBtBVBwRRnkPMOhthqEpSz7P5bHy6zpEZw/gtTOHX5Y1w28k46vetKSk54qr1KdsKk4Hcxpvi6H0JnkoO8cgHBnTxh3TuD0Et6M63/FGK6qlw9M41pzHb9z6iiDFgbh/jK0sckYwggUA7WvX/UqMMO8zdA== 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=lM1fqPtfbIVTa9d9CP8DB6hNe0tpRq7dDhcBD1Itge4=; b=lv8jYb4H8Cze6j86/t9Y0EhTfFgX0/9YKDMzm0nDL7PUzwqumiUSq3LwI6E/kMWOGYgaztsp4oGrwBhEVFksA+6xo5zKiRunbGF5aKKNq8B8YLpjAhFFENJnfyTZ+KmlDVP9u76Nm2VuHFWtkqDdz+d6IK2Y8wuiQLnW4DuXsB8= 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 BYAPR01MB5560.prod.exchangelabs.com (2603:10b6:a03:11f::11) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5102.22; Tue, 29 Mar 2022 02:57:43 +0000 Received: from PH0PR01MB7287.prod.exchangelabs.com ([fe80::94c:517c:70a8:8d98]) by PH0PR01MB7287.prod.exchangelabs.com ([fe80::94c:517c:70a8:8d98%8]) with mapi id 15.20.5102.023; Tue, 29 Mar 2022 02:57:43 +0000 Message-ID: <14b1e88c-dbfe-90b3-401f-8c2c4100c522@os.amperecomputing.com> Date: Tue, 29 Mar 2022 09:57:31 +0700 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:91.0) Gecko/20100101 Thunderbird/91.7.0 Subject: Re: [edk2-devel] [PATCH v2 14/17] Silicon/Ampere: Update ArmPlatformLib to work with changed ARM_CORE_INFO To: Leif Lindholm , devel@edk2.groups.io Cc: Rebecca Cran , Ard Biesheuvel , Samer El-Haj-Mahmoud , Leif Lindholm , Sami Mujawar , Wenyi Xie , Peng Xie , Ling Jia , Yiqi Shu , Vu Nguyen , Thang Nguyen , Chuong Tran , Pete Batard , Masami Hiramatsu , Graeme Gregory , Radoslaw Biernacki , Marcin Wojtas References: <20211216220800.9628-1-rebecca@nuviainc.com> <20211216220800.9628-15-rebecca@nuviainc.com> From: "Nhi Pham" In-Reply-To: X-ClientProxiedBy: CH2PR10CA0005.namprd10.prod.outlook.com (2603:10b6:610:4c::15) 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: 817c86f0-3ce7-4339-8fb1-08da112fe4fc X-MS-TrafficTypeDiagnostic: BYAPR01MB5560:EE_ X-LD-Processed: 3bc2b170-fd94-476d-b0ce-4229bdc904a7,ExtAddr X-Microsoft-Antispam-PRVS: X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: qTLNC22858VWCQgZ4hGE8gTjyqdSPSYFrPBmnIiFoT4tVmb7BNb9w8nd+rCw5vTWwDW6Un6cnm+PlQlP+6FtGRH8Y8fSjpoFk2kPYBernJzZBitSL6iY8pEMYxUrLFDQ6cYdUjtQtqKtit0k3PrytDeJh9M1qYZMPvfNM5ImJf/Vynn5iUCblxEj6AH0nZ2uWcLhgphnXPfUTc/ViShcyRNlm8GLNo7tGaQf+Fb2NM06+ncsgIW+MH0Far1LGn40kiIYQSuAIWDxS3US/7RjtjYJHvt4FRyMHKQ+m6/CL0v225K9YgIhp46+RPXX2L9PEgXsa32A9QPVwhrysf5SisgXm6U6qAQTZMuzglItMhbOVagU+G0DkIicdKXoi050OYMRIP41f8El0beTcRh2nNv63EQpW7N+FxbpNhyDuBtsJpdzpzjZfz26ohSbetpmsmG5tUxA26c+JiWHuzqNtu2jABB1Rer7IQFy28vbkBAS/vqOKity11p4RmXVSxbR3KLCV0ZnflZtCb1CNBKElhwnRAh+q6nbJMMI6Om+F4d/nPJz0VuS9wyK/+0MTUDCbLnvY2/1BLJWV3oNIRFXc75s8sTlIqj4ZAgR4gvchIXGQKAzSuHCxfDWob1tPDrNIDOckfQtIKy9AEUWqGitocnkC6XVEDSPUN1qIzQlm+Nit7HloWbUTVRTc+YJzoHtvu55dviLnXJEe+YUt/MPS0pbAL5NlnpCi/zeCEd/gXE= 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:(13230001)(4636009)(366004)(66946007)(54906003)(31696002)(8936002)(8676002)(66556008)(66476007)(26005)(4326008)(316002)(508600001)(186003)(7416002)(2616005)(6666004)(83380400001)(31686004)(2906002)(5660300002)(6506007)(53546011)(52116002)(15650500001)(86362001)(6486002)(38350700002)(6512007)(38100700002)(43740500002);DIR:OUT;SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?SGdCQTFFWG9lWGJ3d3ErLzRtVUh6RWFrbXUwbnN2U2xlR1JBMXN2enBEYk16?= =?utf-8?B?T0E5VUNQQk1FVUs0UFdOTXdIU2lFR3AvbCt4TnBiV0xZbSsxRVJNcGJOMXBr?= =?utf-8?B?WHRrRk9TMkttVzlSbHRYT3lhT1Y3TkNobVRXVWxMNkVFK2N5dWlwZ2pMdnZo?= =?utf-8?B?Q1ZERFJHR0JDVXRhSmhSRlZiUGtNTEJXRXNuZXhwb3JDcUVDS3RQYXZhOVRa?= =?utf-8?B?cHdrQ0REY3NxV2JxcVZKbkg2QVlWRG9MdnNIOVF2aFJhZldvdVVUUWJFRkV6?= =?utf-8?B?NzZEaXc1bDFLVjduSUJFSTIvNmI1L3VNMHJDc2lLcVVKdzVFVDhyQ1kvbG8z?= =?utf-8?B?NkZpQkorR3FaNUNlc1NZTVZSU2NGQ0VrK0VTVURhOUplZnFGaTNnM3d2SG9o?= =?utf-8?B?MjlCNVdKM1RpR3Fnd0NzU2lvMk91NmthbTlaYTR1Nnk2TW13dnpTMjFpZVM5?= =?utf-8?B?N1VZQW1CdmhrNm53cHFRY0svMTcydkRWWjdmemRpL3V5UFhLVjExbTRuYW1q?= =?utf-8?B?R1dhenQ4WWlYY0g0OXI1VVMxQWYvMHFFeHpyMlhSMnd0N2o1QmRPYzVKb2lL?= =?utf-8?B?RWFNREdkdE95RC9OWE16Vis4ODRxZHNsTkdPOENYb25nczVXY0N0T05XUGtH?= =?utf-8?B?SUxQQ2hHRmt4NlQreXE3em9DLzZCby9iWjNnc3loYUxwc0xMZWN3NUVYNlpM?= =?utf-8?B?U2NkbEdTOExiVUtNSmxPdE90UEt1Si9XUUNHZW9kanFRMXZkeUREaTVjampB?= =?utf-8?B?dk8yZXo2clVLWXE3Ykw2NlRFVUhnSjJwNG00NUw2aXFITDlBaHdyUURKV0pW?= =?utf-8?B?ckgwaWwvak0zcjVlbmFVMENzTnlOZ1dIZXVRTUR6cTA1T20xcC8yZ0JiV01r?= =?utf-8?B?ZndyTXd1d2lOSTU1MXYyM3pneG1EYWorWUlLL2w1RnJNZkpGRG1oeGJlcTdn?= =?utf-8?B?cEw0NEN6a2RHM2J4OHp3NlFoSDJvRWlaSGlQdTVoOGFPOTFua05lTU40UXNQ?= =?utf-8?B?THllK2Z2QWdsb0tac3QvNWxKSGIrRHNPaGs2OWVIRnY4NTVpWC9valFGT0pi?= =?utf-8?B?WDVDYnlYMGRVemNpcUEwSTd6Y2NQSFNRakR6WXd4RlU0d1Fja05FT2tMUzA3?= =?utf-8?B?V1RYS3hveEFoRnFsd3U5RFBVSWlhWGUyc0VsTURmeFhnVExBUUVNbDYrd09J?= =?utf-8?B?VEJiZTEwaDF4TFZ1MkhNTG5WMlpPaTNNQ0llQ3JPNU5PbXhMTkJQYzNTbHN5?= =?utf-8?B?RUc5eDZncHlwUkZKbktGZXRubHN2b2hJMmhOaS9kUU1LY05ERHdVRHFiNW1U?= =?utf-8?B?VTVkc0M3SGNYVGE0NjhuNmtoM1ZsK1I5RU5wSlhhNnJ3Y1VSQ3RLcE9YOHhW?= =?utf-8?B?TTdZT0YzZkpWRmtYOHlldmpvOUFta3hPWG5pa29vQ295TzhDUVV0djNWTTNF?= =?utf-8?B?T1FZdFM0MncvZGJQYTF4K0E4Nmo1UFdwQSszc0hxTG9peENNMTBNd05COTl5?= =?utf-8?B?aWZ2VUtGcjVIY2NzblQvYnZrYUxsMUlkWHZRZCtlRTVvUk1IVklFSmplUzZX?= =?utf-8?B?SzNpV0trQ2dmREVUUHNNZWN0ekpYbnJiRHhWMERTMDEvTUZ5MFZELytaQjQw?= =?utf-8?B?amI1WFFZeURxWmhzd0tmRGplcUpDRjZPWGN1OHFaMFhPUUhISU9SdFFkUGlR?= =?utf-8?B?QmJ0REE1dG5wRUpZMTViMjJwY2NCajRuVHdpWXp2WERGY0dQU25hMnN5WG5U?= =?utf-8?B?WEk0eDRNdDBhUEtpRXNSM3h6ekRiWDZ0dGtzVlZWak83d1pXUUVzK2k5ai9j?= =?utf-8?B?N2Q0UHF3b0RmbytYOFpuWUZwa09QQmI0V3JrdWdkQW5ka1VSUHQ0WGk5QUNG?= =?utf-8?B?c1RjUGc5M2VzSUFPSGhkYnZoVlRVYmZ4d2hiQjBDVFJsQ1ZBYVh0L0RUeSt6?= =?utf-8?B?eUdvVjZaVy9oaEs4aTNaQ3hCRjV2YzRpSmZvbEtCY3ZsaFpHc3Z6ektxUUti?= =?utf-8?B?OHZpeVpHY0JhSnltWXdVdGx2czhYTGhNVnRKNGhuT21IQlh1UkMwZkFxdkRm?= =?utf-8?B?WDFHaVBaNzFEMzJMQnJMQXhDUkx0cGh4SW1VVjJyWFYwRDkydGZIdm0yTUV0?= =?utf-8?B?cFhxTjIxV2NLaWZ4cmVIUnU5YnE0UTR3VUdZUW1TZHh6bldtQzJZb2tBRjQ0?= =?utf-8?B?d1JDRWVsSm9nODl3ODFveTg3L0ZBcjQ2Sk1mL2toOG5xVTdEUU9Qai94OHFy?= =?utf-8?B?cUpsaFRSSmFQYUNqNTlBSVNKalZheE56QjZCUWtTYTIyQ0NWN2M4SHN2NG05?= =?utf-8?B?cEdvTHBmUDZWQ052MGxPeWQ1VDdWR0FZZ3RyNVVXcEpiWWtpbGgxNE81ek9i?= =?utf-8?Q?F+0ppb2oR/FRoQvicw95Oysbkda+55GUExeiA?= X-OriginatorOrg: os.amperecomputing.com X-MS-Exchange-CrossTenant-Network-Message-Id: 817c86f0-3ce7-4339-8fb1-08da112fe4fc X-MS-Exchange-CrossTenant-AuthSource: PH0PR01MB7287.prod.exchangelabs.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 29 Mar 2022 02:57:43.1894 (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: HNgM2LgjeDRIHwJEsv0YQVQpMinNt3DD1lAD0slCvoPsglBlDb3vMv4xQ/T1x+/KBXRMM9+Ceg5i1eA5POQt4ee0dXDMxWEuxh2TRS5aIjQ= X-MS-Exchange-Transport-CrossTenantHeadersStamped: BYAPR01MB5560 Content-Language: en-US Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit On 27/03/2022 05:00, Leif Lindholm wrote: > On Sat, Mar 26, 2022 at 16:12:07 +0700, Nhi Pham via groups.io wrote: >>> Would you be happy for me to fold that into >>> "AmpereAltraPkg, JadePkg: Add ACPI support", or would you be able to >>> submit a v6 of that patch only? >>> >>> Best Regards, >>> >>> Leif >> Thanks much for the patch. The MPIDR decoding matches with Rebecca's update >> for the ArmPlatformLib earlier and the ARM_CORE_INFO is just consumed in the >> AcpiSrat.c. So, that is good for now. Please help fold that into the ACPI >> patch when merging the rest of the Mt. Jade support patchset. > Done. I just pushed up to and including patch 29 as > 41628dcf3332..e18e208e7105. Thanks, Leif. I'm happy to hear that. > > (I will take a look at patch 30 on Monday, but I couldn't find that I > had signed off on it anywhere.) It seems like you have not reviewed the v5 of patch 30 yet. Best regards, Nhi > > Apologies for the delay in getting this merged. > >> In the future, I will follow up with a patch that makes the representation >> of MPIDR in the ARM_CORE_INFO the same as in MADT table. > That sounds ideal, thanks. > > Best Regards, > > Leif