From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail05.groups.io (mail05.groups.io [45.79.224.7]) by spool.mail.gandi.net (Postfix) with ESMTPS id 44825D81111 for ; Wed, 24 Apr 2024 09:50:53 +0000 (UTC) DKIM-Signature: a=rsa-sha256; bh=4kAa2+BW2dsG8dkNG4/HOFYV7jqyoeU2Qv60DdFYuZI=; c=relaxed/simple; d=groups.io; h=Received-SPF:From:To:CC:Subject:Thread-Topic:Thread-Index:Date:Message-ID:References:In-Reply-To:Accept-Language:msip_labels:Authentication-Results-Original:nodisclaimer:MIME-Version:Original-Authentication-Results:Precedence:List-Subscribe:List-Help:Sender:List-Id:Mailing-List:Delivered-To:Resent-Date:Resent-From:Reply-To:List-Unsubscribe-Post:List-Unsubscribe:Content-Language:Content-Type:Content-Transfer-Encoding; s=20240206; t=1713952251; v=1; b=4D6KlW4MnyiB4b9T7oRH/5qw3unDScIWfvWRvGz8DdADG9zjDR0pywPxUNQQ09Rl1t0x6mYW x5SATZskozCZEGrUe4KX9kfX5IgnDHNPtTcATDFcMwDd7N6z1CaJ0jXW2qKo+2fhZhVq8HRIes8 HsC7ue/hX+vN9IM+Y517KTOerwcYb6hIQxKAnZ0fQENbZz+kW+Qy7H1EVcgaNI5/IF6SGM+SOpC Sx35vxVnD8f0a3yGNd/XzGtvU1CuKzaDKO5bdnbWJ22xZLlofjFmJ32IudccxSffo2SNA4zh+tZ PYrsw1bjj/ssaFhKG79MB8oQaQqwuRogmCOdmqn4iBaMA== X-Received: by 127.0.0.2 with SMTP id VZ8fYY7687511xGeRUqDDYjK; Wed, 24 Apr 2024 02:50:51 -0700 X-Received: from EUR05-DB8-obe.outbound.protection.outlook.com (EUR05-DB8-obe.outbound.protection.outlook.com [40.107.20.69]) by mx.groups.io with SMTP id smtpd.web10.12583.1713952250554023784 for ; Wed, 24 Apr 2024 02:50:51 -0700 X-Received: from AS4P189CA0017.EURP189.PROD.OUTLOOK.COM (2603:10a6:20b:5db::7) by DB4PR08MB9863.eurprd08.prod.outlook.com (2603:10a6:10:3f0::16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7472.44; Wed, 24 Apr 2024 09:50:43 +0000 X-Received: from AM3PEPF0000A797.eurprd04.prod.outlook.com (2603:10a6:20b:5db:cafe::db) by AS4P189CA0017.outlook.office365.com (2603:10a6:20b:5db::7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7519.22 via Frontend Transport; Wed, 24 Apr 2024 09:50:43 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 63.35.35.123) smtp.mailfrom=arm.com; dkim=pass (signature was verified) header.d=arm.com;dmarc=pass action=none header.from=arm.com; Received-SPF: Pass (protection.outlook.com: domain of arm.com designates 63.35.35.123 as permitted sender) receiver=protection.outlook.com; client-ip=63.35.35.123; helo=64aa7808-outbound-1.mta.getcheckrecipient.com; pr=C X-Received: from 64aa7808-outbound-1.mta.getcheckrecipient.com (63.35.35.123) by AM3PEPF0000A797.mail.protection.outlook.com (10.167.16.102) with Microsoft SMTP Server (version=TLS1_3, cipher=TLS_AES_256_GCM_SHA384) id 15.20.7519.19 via Frontend Transport; Wed, 24 Apr 2024 09:50:43 +0000 X-Received: ("Tessian outbound 8c03561b2da6:v313"); Wed, 24 Apr 2024 09:50:42 +0000 X-CheckRecipientChecked: true X-CR-MTA-CID: ae6e60dcb93b30de X-CR-MTA-TID: 64aa7808 X-Received: from abed7de03f1a.1 by 64aa7808-outbound-1.mta.getcheckrecipient.com id FE276BD7-5BD4-48BC-B779-4EBCE617B3AD.1; Wed, 24 Apr 2024 09:50:36 +0000 X-Received: from EUR05-AM6-obe.outbound.protection.outlook.com by 64aa7808-outbound-1.mta.getcheckrecipient.com with ESMTPS id abed7de03f1a.1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384); Wed, 24 Apr 2024 09:50:36 +0000 X-Received: from AM0PR08MB4289.eurprd08.prod.outlook.com (2603:10a6:208:148::12) by PAWPR08MB11088.eurprd08.prod.outlook.com (2603:10a6:102:468::14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7472.44; Wed, 24 Apr 2024 09:50:31 +0000 X-Received: from AM0PR08MB4289.eurprd08.prod.outlook.com ([fe80::4a5d:48:bf49:a524]) by AM0PR08MB4289.eurprd08.prod.outlook.com ([fe80::4a5d:48:bf49:a524%7]) with mapi id 15.20.7472.044; Wed, 24 Apr 2024 09:50:31 +0000 From: "levi.yun" To: "devel@edk2.groups.io" , Sahil Kaushal CC: Ard Biesheuvel , =?koi8-r?Q?Leif_Lindholm_=9A?= , Sami Mujawar Subject: Re: [edk2-devel] [PATCH RESEND edk2-platforms][PATCH V2 07/14] Platform/ARM/NorFlashDxe: Switch from NorFlash.c to NorFlashDeviceLib Thread-Topic: [edk2-devel] [PATCH RESEND edk2-platforms][PATCH V2 07/14] Platform/ARM/NorFlashDxe: Switch from NorFlash.c to NorFlashDeviceLib Thread-Index: AQHalY+vvZJqxJjHeECbEGFMNXKn47F3Lo9n Date: Wed, 24 Apr 2024 09:50:31 +0000 Message-ID: References: <20240423055638.1271531-1-Sahil.Kaushal@arm.com> <20240423055638.1271531-8-Sahil.Kaushal@arm.com> In-Reply-To: <20240423055638.1271531-8-Sahil.Kaushal@arm.com> Accept-Language: en-GB, en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: msip_labels: Authentication-Results-Original: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com; x-ms-traffictypediagnostic: AM0PR08MB4289:EE_|PAWPR08MB11088:EE_|AM3PEPF0000A797:EE_|DB4PR08MB9863:EE_ X-MS-Office365-Filtering-Correlation-Id: 3ec4c2ea-ae9a-48f8-5904-08dc644401e1 x-checkrecipientrouted: true nodisclaimer: true X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam-Untrusted: BCL:0; X-Microsoft-Antispam-Message-Info-Original: =?koi8-r?Q?XSDug89nXeSoLV/cdfzPI8K86QdAiha7DPpJOBy9cx2ypsqVKT7xZpJZ8RTjBT?= =?koi8-r?Q?kULd8/42B2kowDy5Cb7Mh83iGO5rmjkJIVq5MNzfP6cgZK5mzQFTnAdu6Cp0oJ?= =?koi8-r?Q?8QfWm8ENskJjvWuN81OABhgRCSIUlaqRynYvGAs+7w4T0HUQMczhWERpH8EgSt?= =?koi8-r?Q?UyX7VnUxTwJO8KZSEqTVrJnrF8pTh9HSBWP9PS/7Ke/0QOBUgq8aAtYokUg6m3?= =?koi8-r?Q?20LDaiABmm9f+iHIMQujp81flCt5GqofWOn9BD5ZoBP1DOkwBeJEeepaQl+hNw?= =?koi8-r?Q?ng+gJHVx/o/3/jiJSTwLLaIthrEx9P438p0WB+J+W41Lmyfx8DWjC56kC49O1/?= =?koi8-r?Q?XA3CTjSDJMhXlec+iF1tZRRg9vBdIAHwWW0sJ7XWlwcMe9Vrr752uY8Oda6Av/?= =?koi8-r?Q?CfOkEnQGw/2U+9T6Vcul06nmljXK8uK9/0SpRn+sRc/glsvpIXwQSyQ5Kc+uOO?= =?koi8-r?Q?HHLYmWTiIl3TVNWJjJjlXcY0EzgUxeiCllnOk/gSiTuXdda6WvUHR9Rez/f96s?= =?koi8-r?Q?o9VUcoB+XXjAqyS0U2398nP8bk8FecMHm1xVNr6R9aFXfMCR5RTV8CMs3hzRmC?= =?koi8-r?Q?odHDJadqLEAk3tRmm0h5p2GTXYLIx7M9f3k/7V2kaUCwanffcHYrLQbt2RnmY6?= =?koi8-r?Q?aORkikMumruSsbM9UQ/IaFgXsHK/8E0cUlYVGwE9hIfuSIYUsw5voQ5QoB/e0q?= =?koi8-r?Q?ko0o8hgXZ4o6wLFI1tDZDt5bG6ZfyBRyJdBUhgMkThXbx4ga4hDUlHyV20FoPb?= =?koi8-r?Q?QOnYuTPtQGi/RwCXBn1120Q/15tYx8WE3PHSfirkUpUiR+qi77iQZdQhjIJPzd?= =?koi8-r?Q?VlPy9/hwTw19NY2b/Sl9mORx60Kf/gWBFIHiXh4+OtM5X3xagGLthM+6KPPWpx?= =?koi8-r?Q?wq8fioJrZachp3ZFHvG4YJ8mLe8dJxOVuc+1zKlJiaMUtGor71kQVt2AQeKpwR?= =?koi8-r?Q?5a/pYmoiUIW5jLohLlN0bdvmYVYYwqg1oX2SIH2mmsp41fgV+1rONWAAZ06zXj?= =?koi8-r?Q?ffrKp1X0R4+1u/SXk2291SvFXFzKkSn39TURoMs4MGVq9unKHtTN2xmM1/7+tJ?= =?koi8-r?Q?ua3hTEfrlZfhAtwpFq6uuoNYKZY+KS14QS1kCu1DzEJMo5lvm+QtnnQitIPvqf?= =?koi8-r?Q?hqXUdenJHe/yTHxIXunUZKXwGah6d2qriZw3ojXbpI308EazWxcsiYSrLjjLhQ?= =?koi8-r?Q?8qRCXSGEL8TKaRZvOLbTXTmeF3gn3LLRSIqWtDkoINOz3hXwLIcktwfvzRbrQn?= =?koi8-r?Q?G60bFME4O6enxPsF8aupZJG6cqI6Zkw0dJ2Y50Dq6Qu8dWzwe/OI1rWxOBKh+E?= =?koi8-r?Q?0Q=3D=3D?= X-Forefront-Antispam-Report-Untrusted: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:AM0PR08MB4289.eurprd08.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230031)(366007)(1800799015)(376005)(38070700009);DIR:OUT;SFP:1101; MIME-Version: 1.0 X-MS-Exchange-Transport-CrossTenantHeadersStamped: PAWPR08MB11088 Original-Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com; X-EOPAttributedMessage: 0 X-MS-Exchange-Transport-CrossTenantHeadersStripped: AM3PEPF0000A797.eurprd04.prod.outlook.com X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id-Prvs: 08c33131-bee8-45db-24e6-08dc6443fabe X-Microsoft-Antispam-Message-Info: =?koi8-r?Q?V0R69TxXGYCt0ndZdS2HPmV6jANP91otdcVoZuaBimulGsbbEOqD8T6FBK+97i?= =?koi8-r?Q?ZqaQz5Wt0PREHuozxw5Fhqjl4yhRLMndGzn6gPzs6MaEL8C8/zj+gZ4Y0dlpeS?= =?koi8-r?Q?LCZ6lmP3EIYFV77iyPjpuGktv2ROHe2YPOMp6kQuzERJq5Fyvc06gVsyA21Ym+?= =?koi8-r?Q?Kw3AlKrHaNdFeuPND7Oo0wLliZ5Z1kH7D7VIrf6jVOIhmRwJryyCJBQFAHPPwW?= =?koi8-r?Q?O2TyN/H3uavPJlpwia7G4SZ+uyeXZHv0Y4FrQ2227nS9TIaTXpmyVpaFqII6Hk?= =?koi8-r?Q?o6WbJyCtfrK96L0Pwkfdlr2YXohwB/icgdJPVNKiCXv+n3Pr/uf6Mv+iGFeblt?= =?koi8-r?Q?ZwHXCUHWIRU3fp1W71vG+oW6l0CX2v3eR7tW9b0ADUsLuUHEX0wlkp0q4CLkaH?= =?koi8-r?Q?iWQ2qlfVVautcoc98pzK4A93NpVdZCLWMOlJ+g1o+0UAfGNwavYa/6gUklIt/2?= =?koi8-r?Q?9lBqnTUUJJvsRa0N107UrTGsgnjo12ztjflg6rpD4449zZw7PVHP//EaK0XaRl?= =?koi8-r?Q?2wrOC2REUfmiFND1gETKjzPCRTxdbnw0+pXCjCGL08CeH09XHqVWe0pNtbYWnt?= =?koi8-r?Q?qxqVe/TPlSx7GbqvWvGABh3Kj78udnj26IgWrSDtII7smdWuw2SrvuB7SareLS?= =?koi8-r?Q?tVGUGCYGFRwDUCqdqc+wk+0VlFT/0FahSw7oplBnYZG54LtuxbCSdp3sw0t7e6?= =?koi8-r?Q?OR7+m+KHPh3Z/no2PTZTQxmlXj3erzvwWPM1fS5U77/b79HVH6ijxSLFPVxfs0?= =?koi8-r?Q?hdB/qcfw9yiMewG7HV5yBflx9ssL0p7CQY24/1looc+i2mEdJly9RokZAZxQJ5?= =?koi8-r?Q?Gn42MUrZp9pQAfGYhVzEID35my60ewnfsmmc60/HjPVSo9AG/8Ge5T6zemvk85?= =?koi8-r?Q?vXKUunUor2+ICzUMyz+rOpI8JpQ9fwZXqxgGD0WqIIlisxXmScLwaq9abnYEMv?= =?koi8-r?Q?V4K8OJZ9C9uTxEru8NynW3SY1/Ch7e3G/vaDcJX5G9/Gcdv0/DcKhkSJaqsNmy?= =?koi8-r?Q?fePlo02doo9+Z4MnzO1I2Bt4HLce+XszJBDxmTmgz/QJVJCcrE1GudwGEp1zWX?= =?koi8-r?Q?UpH7Dg8qF/fc4DR0ebh9NbUEmj3a04DZqe1nQRTo+he9ogQFLycPJw7oR2Zjpz?= =?koi8-r?Q?bQUnKMyGIjdlyxtTl9ruV/fIDzlyFzQPI7mt4Lu/VmXTCTE3V2MEpILUAA/cve?= =?koi8-r?Q?gbFWoVc6GY1azS5DpWoy4548fpZf2x8YQNIbZ7ZdtvtCHSk2jn0FB05hv7WOZP?= =?koi8-r?Q?o0cZ+hJ5LU8GC+5z1MvSe1d13F3IMKyGh8L6w1h4uN33dxElbBlX96xWlAD1+i?= =?koi8-r?Q?qTL2DLZA5sSYt5IqW6/+h7PfA9LZpHsERHWcfDVtoxII?= X-OriginatorOrg: arm.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 24 Apr 2024 09:50:43.1242 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 3ec4c2ea-ae9a-48f8-5904-08dc644401e1 X-MS-Exchange-CrossTenant-Id: f34e5979-57d9-4aaa-ad4d-b122a662184d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=f34e5979-57d9-4aaa-ad4d-b122a662184d;Ip=[63.35.35.123];Helo=[64aa7808-outbound-1.mta.getcheckrecipient.com] X-MS-Exchange-CrossTenant-AuthSource: AM3PEPF0000A797.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB4PR08MB9863 Precedence: Bulk List-Subscribe: List-Help: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Resent-Date: Wed, 24 Apr 2024 02:50:51 -0700 Resent-From: yeoreum.yun@arm.com Reply-To: devel@edk2.groups.io,yeoreum.yun@arm.com List-Unsubscribe-Post: List-Unsubscribe=One-Click List-Unsubscribe: X-Gm-Message-State: 30lkvWsEIFoEuW0HQLYXFyz6x7686176AA= Content-Language: en-GB Content-Type: text/plain; charset="koi8-r" Content-Transfer-Encoding: quoted-printable X-GND-Status: LEGIT Authentication-Results: spool.mail.gandi.net; dkim=pass header.d=groups.io header.s=20240206 header.b=4D6KlW4M; spf=pass (spool.mail.gandi.net: domain of bounce@groups.io designates 45.79.224.7 as permitted sender) smtp.mailfrom=bounce@groups.io; dmarc=fail reason="SPF not aligned (relaxed), DKIM not aligned (relaxed)" header.from=arm.com (policy=none) Reviewed-by: levi.yun ________________________________________ From: devel@edk2.groups.io on behalf of Sahil Kausha= l via groups.io Sent: 23 April 2024 06:56 To: devel@edk2.groups.io Cc: Ard Biesheuvel; Leif Lindholm ; Sami Mujawar; Sahil Kaushal Subject: [edk2-devel] [PATCH RESEND edk2-platforms][PATCH V2 07/14] Platfor= m/ARM/NorFlashDxe: Switch from NorFlash.c to NorFlashDeviceLib From: sahil NorFlashDeviceLib can be used to provide implementations of different NOR Flash to NorFlashDxe, i.e. NorFlashDxe links with NorFlashDeviceLib and the platforms can specify their respective NorFlashDeviceLib instances. This patch adds the following major changes: 1. Adds changes in NorFlashDxe to look for NorFlashDeviceLib instead of NorFlash.c for flash specific implementation 2. Remove NorFlash.c and NorFlash.h as they are no longer required. 3. Add changes to platform description files to provide P30NorFlashDeviceLib as NorFlashDeviceLib Signed-off-by: sahil --- Platform/ARM/SgiPkg/SgiPlatform.dsc.inc | 1 + Platform/ARM/SgiPkg/SgiPlatformMm.dsc.inc | 1 + Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc | 1 + Platform/ARM/JunoPkg/ArmJuno.dsc | 1 + Platform/ARM/VExpressPkg/PlatformStandaloneMm.dsc | 1 + Platform/ARM/Drivers/NorFlashDxe/NorFlashDxe.inf | 5 +- Platform/ARM/Drivers/NorFlashDxe/NorFlashStandaloneMm.inf | 6 +- Platform/ARM/Drivers/NorFlashDxe/NorFlash.h | 108 --- Platform/ARM/Drivers/NorFlashDxe/NorFlash.c | 953 ----------= ---------- Platform/ARM/Drivers/NorFlashDxe/NorFlashBlockIoDxe.c | 1 - Platform/ARM/Drivers/NorFlashDxe/NorFlashDxe.c | 1 - Platform/ARM/Drivers/NorFlashDxe/NorFlashFvb.c | 1 - Platform/ARM/Drivers/NorFlashDxe/NorFlashStandaloneMm.c | 1 - 13 files changed, 7 insertions(+), 1074 deletions(-) diff --git a/Platform/ARM/SgiPkg/SgiPlatform.dsc.inc b/Platform/ARM/SgiPkg/= SgiPlatform.dsc.inc index 1bf489ffeb39..3dcf422eab4b 100644 --- a/Platform/ARM/SgiPkg/SgiPlatform.dsc.inc +++ b/Platform/ARM/SgiPkg/SgiPlatform.dsc.inc @@ -34,6 +34,7 @@ BasePathLib|ShellPkg/Library/UefiShellLib/UefiShellLib.inf LcdHwLib|ArmPlatformPkg/Library/HdLcd/HdLcd.inf LcdPlatformLib|Platform/ARM/SgiPkg/Library/HdLcdArmSgiLib/HdLcdArmSgiLib= .inf + NorFlashDeviceLib|Platform/ARM/Library/P30NorFlashDeviceLib/P30NorFlashD= eviceLib.inf NorFlashPlatformLib|Platform/ARM/SgiPkg/Library/NorFlashLib/NorFlashLib.= inf HobLib|MdePkg/Library/DxeHobLib/DxeHobLib.inf ResetSystemLib|ArmPkg/Library/ArmSmcPsciResetSystemLib/ArmSmcPsciResetSy= stemLib.inf diff --git a/Platform/ARM/SgiPkg/SgiPlatformMm.dsc.inc b/Platform/ARM/SgiPk= g/SgiPlatformMm.dsc.inc index 0dd9ebbfc16c..ab0e2a957a1b 100644 --- a/Platform/ARM/SgiPkg/SgiPlatformMm.dsc.inc +++ b/Platform/ARM/SgiPkg/SgiPlatformMm.dsc.inc @@ -63,6 +63,7 @@ AuthVariableLib|SecurityPkg/Library/AuthVariableLib/AuthVariableLib.inf BaseCryptLib|CryptoPkg/Library/BaseCryptLib/SmmCryptLib.inf IntrinsicLib|CryptoPkg/Library/IntrinsicLib/IntrinsicLib.inf + NorFlashDeviceLib|Platform/ARM/Library/P30NorFlashDeviceLib/P30NorFlashD= eviceLib.inf NorFlashPlatformLib|Platform/ARM/SgiPkg/Library/NorFlashLib/StandaloneMm= NorFlashLib.inf OpensslLib|CryptoPkg/Library/OpensslLib/OpensslLib.inf RngLib|MdePkg/Library/BaseRngLibTimerLib/BaseRngLibTimerLib.inf diff --git a/Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc b/Platform/ARM/VE= xpressPkg/ArmVExpress.dsc.inc index 9f51e05af561..70ff049d3248 100644 --- a/Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc +++ b/Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc @@ -93,6 +93,7 @@ # Versatile Express Specific Libraries PlatformPeiLib|ArmPlatformPkg/PlatformPei/PlatformPeiLib.inf ArmPlatformSysConfigLib|Platform/ARM/VExpressPkg/Library/ArmVExpressSysC= onfigLib/ArmVExpressSysConfigLib.inf + NorFlashDeviceLib|Platform/ARM/Library/P30NorFlashDeviceLib/P30NorFlashD= eviceLib.inf NorFlashPlatformLib|Platform/ARM/VExpressPkg/Library/NorFlashArmVExpress= Lib/NorFlashArmVExpressLib.inf ResetSystemLib|ArmPkg/Library/ArmSmcPsciResetSystemLib/ArmSmcPsciResetSy= stemLib.inf diff --git a/Platform/ARM/JunoPkg/ArmJuno.dsc b/Platform/ARM/JunoPkg/ArmJun= o.dsc index 7fe796a53433..81d2cbe4359f 100644 --- a/Platform/ARM/JunoPkg/ArmJuno.dsc +++ b/Platform/ARM/JunoPkg/ArmJuno.dsc @@ -40,6 +40,7 @@ ArmPlatformLib|Platform/ARM/JunoPkg/Library/ArmJunoLib/ArmJunoLib.inf ArmSmcLib|ArmPkg/Library/ArmSmcLib/ArmSmcLib.inf + NorFlashDeviceLib|Platform/ARM/Library/P30NorFlashDeviceLib/P30NorFlashD= eviceLib.inf NorFlashPlatformLib|Platform/ARM/JunoPkg/Library/NorFlashJunoLib/NorFlas= hJunoLib.inf CapsuleLib|MdeModulePkg/Library/DxeCapsuleLibNull/DxeCapsuleLibNull.inf diff --git a/Platform/ARM/VExpressPkg/PlatformStandaloneMm.dsc b/Platform/A= RM/VExpressPkg/PlatformStandaloneMm.dsc index 0e77b76cae69..a5805da49c92 100644 --- a/Platform/ARM/VExpressPkg/PlatformStandaloneMm.dsc +++ b/Platform/ARM/VExpressPkg/PlatformStandaloneMm.dsc @@ -100,6 +100,7 @@ # STMM for Variable runtime service. !if $(ENABLE_UEFI_SECURE_VARIABLE) =3D=3D TRUE + NorFlashDeviceLib|Platform/ARM/Library/P30NorFlashDeviceLib/P30NorFlashD= eviceLib.inf NorFlashPlatformLib|Platform/ARM/VExpressPkg/Library/NorFlashArmVExpress= Lib/NorFlashStMmLib.inf VarCheckLib|MdeModulePkg/Library/VarCheckLib/VarCheckLib.inf VariableFlashInfoLib|MdeModulePkg/Library/BaseVariableFlashInfoLib/BaseV= ariableFlashInfoLib.inf diff --git a/Platform/ARM/Drivers/NorFlashDxe/NorFlashDxe.inf b/Platform/AR= M/Drivers/NorFlashDxe/NorFlashDxe.inf index 18e99bac8888..de160025b632 100644 --- a/Platform/ARM/Drivers/NorFlashDxe/NorFlashDxe.inf +++ b/Platform/ARM/Drivers/NorFlashDxe/NorFlashDxe.inf @@ -17,8 +17,6 @@ ENTRY_POINT =3D NorFlashInitialise [Sources.common] - NorFlash.c - NorFlash.h NorFlashBlockIoDxe.c NorFlashDxe.c NorFlashCommon.h @@ -36,6 +34,7 @@ DxeServicesTableLib HobLib IoLib + NorFlashDeviceLib NorFlashPlatformLib UefiBootServicesTableLib UefiDriverEntryPoint @@ -66,7 +65,5 @@ gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageFtwSpareBase gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageFtwSpareSize - gPlatformArmTokenSpaceGuid.PcdNorFlashCheckBlockLocked - [Depex] gEfiCpuArchProtocolGuid diff --git a/Platform/ARM/Drivers/NorFlashDxe/NorFlashStandaloneMm.inf b/Pl= atform/ARM/Drivers/NorFlashDxe/NorFlashStandaloneMm.inf index 69c40ccf9c27..d9e7de07165c 100644 --- a/Platform/ARM/Drivers/NorFlashDxe/NorFlashStandaloneMm.inf +++ b/Platform/ARM/Drivers/NorFlashDxe/NorFlashStandaloneMm.inf @@ -19,8 +19,6 @@ ENTRY_POINT =3D NorFlashInitialise [Sources.common] - NorFlash.c - NorFlash.h NorFlashFvb.c NorFlashCommon.h NorFlashStandaloneMm.c @@ -38,6 +36,7 @@ IoLib MemoryAllocationLib MmServicesTableLib + NorFlashDeviceLib NorFlashPlatformLib StandaloneMmDriverEntryPoint @@ -60,8 +59,5 @@ gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageFtwSpareBase gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageFtwSpareSize -[FeaturePcd] - gPlatformArmTokenSpaceGuid.PcdNorFlashCheckBlockLocked - [Depex] TRUE diff --git a/Platform/ARM/Drivers/NorFlashDxe/NorFlash.h b/Platform/ARM/Dri= vers/NorFlashDxe/NorFlash.h deleted file mode 100644 index 6cb1f64b9875..000000000000 --- a/Platform/ARM/Drivers/NorFlashDxe/NorFlash.h +++ /dev/null @@ -1,108 +0,0 @@ -/** @file NorFlash.h - - Copyright (c) 2011 - 2014, ARM Ltd. All rights reserved.
- - SPDX-License-Identifier: BSD-2-Clause-Patent - -**/ - -#ifndef __NOR_FLASH_H__ -#define __NOR_FLASH_H__ - -#include -#include - -#include - -#include -#include -#include - -#include -#include -#include - -#define NOR_FLASH_ERASE_RETRY 10 - -// Device access macros -// These are necessary because we use 2 x 16bit parts to make up 32bit dat= a - -#define HIGH_16_BITS 0xFFFF0000 -#define LOW_16_BITS 0x0000FFFF -#define LOW_8_BITS 0x000000FF - -#define FOLD_32BIT_INTO_16BIT(value) ( ( value >> 16 ) | ( value & LOW_16= _BITS ) ) - -#define GET_LOW_BYTE(value) ( value & LOW_8_BITS ) -#define GET_HIGH_BYTE(value) ( GET_LOW_BYTE( value >> 16 ) ) - -// Each command must be sent simultaneously to both chips, -// i.e. at the lower 16 bits AND at the higher 16 bits -#define CREATE_NOR_ADDRESS(BaseAddr, OffsetAddr) ((BaseAddr) + ((Off= setAddr) << 2)) -#define CREATE_DUAL_CMD(Cmd) ( ( Cmd << 16) | ( = Cmd & LOW_16_BITS) ) -#define SEND_NOR_COMMAND(BaseAddr, Offset, Cmd) MmioWrite32 (CREATE= _NOR_ADDRESS(BaseAddr,Offset), CREATE_DUAL_CMD(Cmd)) - -// Status Register Bits -#define P30_SR_BIT_WRITE (BIT7 << 16 | BIT7) -#define P30_SR_BIT_ERASE_SUSPEND (BIT6 << 16 | BIT6) -#define P30_SR_BIT_ERASE (BIT5 << 16 | BIT5) -#define P30_SR_BIT_PROGRAM (BIT4 << 16 | BIT4) -#define P30_SR_BIT_VPP (BIT3 << 16 | BIT3) -#define P30_SR_BIT_PROGRAM_SUSPEND (BIT2 << 16 | BIT2) -#define P30_SR_BIT_BLOCK_LOCKED (BIT1 << 16 | BIT1) -#define P30_SR_BIT_BEFP (BIT0 << 16 | BIT0) - -// Device Commands for Intel StrataFlash(R) Embedded Memory (P30) Family - -// On chip buffer size for buffered programming operations -// There are 2 chips, each chip can buffer up to 32 (16-bit)words, and eac= h word is 2 bytes. -// Therefore the total size of the buffer is 2 x 32 x 2 =3D 128 bytes -#define P30_MAX_BUFFER_SIZE_IN_BYTES ((UINTN)128) -#define P30_MAX_BUFFER_SIZE_IN_WORDS (P30_MAX_BUFFER_SIZE_IN_BYTES/((UINT= N)4)) -#define MAX_BUFFERED_PROG_ITERATIONS 10000000 -#define BOUNDARY_OF_32_WORDS 0x7F - -// CFI Addresses -#define P30_CFI_ADDR_QUERY_UNIQUE_QRY 0x10 -#define P30_CFI_ADDR_VENDOR_ID 0x13 - -// CFI Data -#define CFI_QRY 0x00595251 - -// READ Commands -#define P30_CMD_READ_DEVICE_ID 0x0090 -#define P30_CMD_READ_STATUS_REGISTER 0x0070 -#define P30_CMD_CLEAR_STATUS_REGISTER 0x0050 -#define P30_CMD_READ_ARRAY 0x00FF -#define P30_CMD_READ_CFI_QUERY 0x0098 - -// WRITE Commands -#define P30_CMD_WORD_PROGRAM_SETUP 0x0040 -#define P30_CMD_ALTERNATE_WORD_PROGRAM_SETUP 0x0010 -#define P30_CMD_BUFFERED_PROGRAM_SETUP 0x00E8 -#define P30_CMD_BUFFERED_PROGRAM_CONFIRM 0x00D0 -#define P30_CMD_BEFP_SETUP 0x0080 -#define P30_CMD_BEFP_CONFIRM 0x00D0 - -// ERASE Commands -#define P30_CMD_BLOCK_ERASE_SETUP 0x0020 -#define P30_CMD_BLOCK_ERASE_CONFIRM 0x00D0 - -// SUSPEND Commands -#define P30_CMD_PROGRAM_OR_ERASE_SUSPEND 0x00B0 -#define P30_CMD_SUSPEND_RESUME 0x00D0 - -// BLOCK LOCKING / UNLOCKING Commands -#define P30_CMD_LOCK_BLOCK_SETUP 0x0060 -#define P30_CMD_LOCK_BLOCK 0x0001 -#define P30_CMD_UNLOCK_BLOCK 0x00D0 -#define P30_CMD_LOCK_DOWN_BLOCK 0x002F - -// PROTECTION Commands -#define P30_CMD_PROGRAM_PROTECTION_REGISTER_SETUP 0x00C0 - -// CONFIGURATION Commands -#define P30_CMD_READ_CONFIGURATION_REGISTER_SETUP 0x0060 -#define P30_CMD_READ_CONFIGURATION_REGISTER 0x0003 - -#endif /* __NOR_FLASH_H__ */ diff --git a/Platform/ARM/Drivers/NorFlashDxe/NorFlash.c b/Platform/ARM/Dri= vers/NorFlashDxe/NorFlash.c deleted file mode 100644 index 15000a692b02..000000000000 --- a/Platform/ARM/Drivers/NorFlashDxe/NorFlash.c +++ /dev/null @@ -1,953 +0,0 @@ -/** @file NorFlash.c - - Copyright (c) 2011 - 2020, Arm Limited. All rights reserved.
- Copyright (c) 2020, Linaro, Ltd. All rights reserved.
- - SPDX-License-Identifier: BSD-2-Clause-Patent - -**/ - -#include - -#include "NorFlash.h" - -// -// Global variable declarations -// -extern NOR_FLASH_INSTANCE **mNorFlashInstances; -extern UINT32 mNorFlashDeviceCount; - -UINT32 -NorFlashReadStatusRegister ( - IN NOR_FLASH_INSTANCE *Instance, - IN UINTN SR_Address - ) -{ - // Prepare to read the status register - SEND_NOR_COMMAND (Instance->DeviceBaseAddress, 0, P30_CMD_READ_STATUS_RE= GISTER); - return MmioRead32 (Instance->DeviceBaseAddress); -} - -STATIC -BOOLEAN -NorFlashBlockIsLocked ( - IN NOR_FLASH_INSTANCE *Instance, - IN UINTN BlockAddress - ) -{ - UINT32 LockStatus; - - // Send command for reading device id - SEND_NOR_COMMAND (BlockAddress, 2, P30_CMD_READ_DEVICE_ID); - - // Read block lock status - LockStatus =3D MmioRead32 (CREATE_NOR_ADDRESS (BlockAddress, 2)); - - // Decode block lock status - LockStatus =3D FOLD_32BIT_INTO_16BIT (LockStatus); - - if ((LockStatus & 0x2) !=3D 0) { - DEBUG ((DEBUG_ERROR, "NorFlashBlockIsLocked: WARNING: Block LOCKED DOW= N\n")); - } - - return ((LockStatus & 0x1) !=3D 0); -} - -STATIC -EFI_STATUS -NorFlashUnlockSingleBlock ( - IN NOR_FLASH_INSTANCE *Instance, - IN UINTN BlockAddress - ) -{ - UINT32 LockStatus; - - // Raise the Task Priority Level to TPL_NOTIFY to serialise all its oper= ations - // and to protect shared data structures. - - if (FeaturePcdGet (PcdNorFlashCheckBlockLocked) =3D=3D TRUE) { - do { - // Request a lock setup - SEND_NOR_COMMAND (BlockAddress, 0, P30_CMD_LOCK_BLOCK_SETUP); - - // Request an unlock - SEND_NOR_COMMAND (BlockAddress, 0, P30_CMD_UNLOCK_BLOCK); - - // Send command for reading device id - SEND_NOR_COMMAND (BlockAddress, 2, P30_CMD_READ_DEVICE_ID); - - // Read block lock status - LockStatus =3D MmioRead32 (CREATE_NOR_ADDRESS (BlockAddress, 2)); - - // Decode block lock status - LockStatus =3D FOLD_32BIT_INTO_16BIT (LockStatus); - } while ((LockStatus & 0x1) =3D=3D 1); - } else { - // Request a lock setup - SEND_NOR_COMMAND (BlockAddress, 0, P30_CMD_LOCK_BLOCK_SETUP); - - // Request an unlock - SEND_NOR_COMMAND (BlockAddress, 0, P30_CMD_UNLOCK_BLOCK); - - // Wait until the status register gives us the all clear - do { - LockStatus =3D NorFlashReadStatusRegister (Instance, BlockAddress); - } while ((LockStatus & P30_SR_BIT_WRITE) !=3D P30_SR_BIT_WRITE); - } - - // Put device back into Read Array mode - SEND_NOR_COMMAND (BlockAddress, 0, P30_CMD_READ_ARRAY); - - DEBUG ((DEBUG_BLKIO, "UnlockSingleBlock: BlockAddress=3D0x%08x\n", Block= Address)); - - return EFI_SUCCESS; -} - -EFI_STATUS -NorFlashUnlockSingleBlockIfNecessary ( - IN NOR_FLASH_INSTANCE *Instance, - IN UINTN BlockAddress - ) -{ - EFI_STATUS Status; - - Status =3D EFI_SUCCESS; - - if (NorFlashBlockIsLocked (Instance, BlockAddress)) { - Status =3D NorFlashUnlockSingleBlock (Instance, BlockAddress); - } - - return Status; -} - -/** - * The following function presumes that the block has already been unlocke= d. - **/ -EFI_STATUS -NorFlashEraseSingleBlock ( - IN NOR_FLASH_INSTANCE *Instance, - IN UINTN BlockAddress - ) -{ - EFI_STATUS Status; - UINT32 StatusRegister; - - Status =3D EFI_SUCCESS; - - // Request a block erase and then confirm it - SEND_NOR_COMMAND (BlockAddress, 0, P30_CMD_BLOCK_ERASE_SETUP); - SEND_NOR_COMMAND (BlockAddress, 0, P30_CMD_BLOCK_ERASE_CONFIRM); - - // Wait until the status register gives us the all clear - do { - StatusRegister =3D NorFlashReadStatusRegister (Instance, BlockAddress)= ; - } while ((StatusRegister & P30_SR_BIT_WRITE) !=3D P30_SR_BIT_WRITE); - - if (StatusRegister & P30_SR_BIT_VPP) { - DEBUG ((DEBUG_ERROR, "EraseSingleBlock(BlockAddress=3D0x%08x: VPP Rang= e Error\n", BlockAddress)); - Status =3D EFI_DEVICE_ERROR; - } - - if ((StatusRegister & (P30_SR_BIT_ERASE | P30_SR_BIT_PROGRAM)) =3D=3D (P= 30_SR_BIT_ERASE | P30_SR_BIT_PROGRAM)) { - DEBUG ((DEBUG_ERROR, "EraseSingleBlock(BlockAddress=3D0x%08x: Command = Sequence Error\n", BlockAddress)); - Status =3D EFI_DEVICE_ERROR; - } - - if (StatusRegister & P30_SR_BIT_ERASE) { - DEBUG ((DEBUG_ERROR, "EraseSingleBlock(BlockAddress=3D0x%08x: Block Er= ase Error StatusRegister:0x%X\n", BlockAddress, StatusRegister)); - Status =3D EFI_DEVICE_ERROR; - } - - if (StatusRegister & P30_SR_BIT_BLOCK_LOCKED) { - // The debug level message has been reduced because a device lock migh= t happen. In this case we just retry it ... - DEBUG ((DEBUG_INFO, "EraseSingleBlock(BlockAddress=3D0x%08x: Block Loc= ked Error\n", BlockAddress)); - Status =3D EFI_WRITE_PROTECTED; - } - - if (EFI_ERROR (Status)) { - // Clear the Status Register - SEND_NOR_COMMAND (Instance->DeviceBaseAddress, 0, P30_CMD_CLEAR_STATUS= _REGISTER); - } - - // Put device back into Read Array mode - SEND_NOR_COMMAND (Instance->DeviceBaseAddress, 0, P30_CMD_READ_ARRAY); - - return Status; -} - -EFI_STATUS -NorFlashWriteSingleWord ( - IN NOR_FLASH_INSTANCE *Instance, - IN UINTN WordAddress, - IN UINT32 WriteData - ) -{ - EFI_STATUS Status; - UINT32 StatusRegister; - - Status =3D EFI_SUCCESS; - - // Request a write single word command - SEND_NOR_COMMAND (WordAddress, 0, P30_CMD_WORD_PROGRAM_SETUP); - - // Store the word into NOR Flash; - MmioWrite32 (WordAddress, WriteData); - - // Wait for the write to complete and then check for any errors; i.e. ch= eck the Status Register - do { - // Prepare to read the status register - StatusRegister =3D NorFlashReadStatusRegister (Instance, WordAddress); - // The chip is busy while the WRITE bit is not asserted - } while ((StatusRegister & P30_SR_BIT_WRITE) !=3D P30_SR_BIT_WRITE); - - // Perform a full status check: - // Mask the relevant bits of Status Register. - // Everything should be zero, if not, we have a problem - - if (StatusRegister & P30_SR_BIT_VPP) { - DEBUG ((DEBUG_ERROR, "NorFlashWriteSingleWord(WordAddress:0x%X): VPP R= ange Error\n", WordAddress)); - Status =3D EFI_DEVICE_ERROR; - } - - if (StatusRegister & P30_SR_BIT_PROGRAM) { - DEBUG ((DEBUG_ERROR, "NorFlashWriteSingleWord(WordAddress:0x%X): Progr= am Error\n", WordAddress)); - Status =3D EFI_DEVICE_ERROR; - } - - if (StatusRegister & P30_SR_BIT_BLOCK_LOCKED) { - DEBUG ((DEBUG_ERROR, "NorFlashWriteSingleWord(WordAddress:0x%X): Devic= e Protect Error\n", WordAddress)); - Status =3D EFI_DEVICE_ERROR; - } - - if (!EFI_ERROR (Status)) { - // Clear the Status Register - SEND_NOR_COMMAND (Instance->DeviceBaseAddress, 0, P30_CMD_CLEAR_STATUS= _REGISTER); - } - - // Put device back into Read Array mode - SEND_NOR_COMMAND (Instance->DeviceBaseAddress, 0, P30_CMD_READ_ARRAY); - - return Status; -} - -/* - * Writes data to the NOR Flash using the Buffered Programming method. - * - * The maximum size of the on-chip buffer is 32-words, because of hardware= restrictions. - * Therefore this function will only handle buffers up to 32 words or 128 = bytes. - * To deal with larger buffers, call this function again. - * - * This function presumes that both the TargetAddress and the TargetAddres= s+BufferSize - * exist entirely within the NOR Flash. Therefore these conditions will no= t be checked here. - * - * In buffered programming, if the target address not at the beginning of = a 32-bit word boundary, - * then programming time is doubled and power consumption is increased. - * Therefore, it is a requirement to align buffer writes to 32-bit word bo= undaries. - * i.e. the last 4 bits of the target start address must be zero: 0x......= 00 - */ -EFI_STATUS -NorFlashWriteBuffer ( - IN NOR_FLASH_INSTANCE *Instance, - IN UINTN TargetAddress, - IN UINTN BufferSizeInBytes, - IN UINT32 *Buffer - ) -{ - EFI_STATUS Status; - UINTN BufferSizeInWords; - UINTN Count; - volatile UINT32 *Data; - UINTN WaitForBuffer; - BOOLEAN BufferAvailable; - UINT32 StatusRegister; - - WaitForBuffer =3D MAX_BUFFERED_PROG_ITERATIONS; - BufferAvailable =3D FALSE; - - // Check that the target address does not cross a 32-word boundary. - if ((TargetAddress & BOUNDARY_OF_32_WORDS) !=3D 0) { - return EFI_INVALID_PARAMETER; - } - - // Check there are some data to program - if (BufferSizeInBytes =3D=3D 0) { - return EFI_BUFFER_TOO_SMALL; - } - - // Check that the buffer size does not exceed the maximum hardware buffe= r size on chip. - if (BufferSizeInBytes > P30_MAX_BUFFER_SIZE_IN_BYTES) { - return EFI_BAD_BUFFER_SIZE; - } - - // Check that the buffer size is a multiple of 32-bit words - if ((BufferSizeInBytes % 4) !=3D 0) { - return EFI_BAD_BUFFER_SIZE; - } - - // Pre-programming conditions checked, now start the algorithm. - - // Prepare the data destination address - Data =3D (UINT32 *)TargetAddress; - - // Check the availability of the buffer - do { - // Issue the Buffered Program Setup command - SEND_NOR_COMMAND (TargetAddress, 0, P30_CMD_BUFFERED_PROGRAM_SETUP); - - // Read back the status register bit#7 from the same address - if (((*Data) & P30_SR_BIT_WRITE) =3D=3D P30_SR_BIT_WRITE) { - BufferAvailable =3D TRUE; - } - - // Update the loop counter - WaitForBuffer--; - } while ((WaitForBuffer > 0) && (BufferAvailable =3D=3D FALSE)); - - // The buffer was not available for writing - if (WaitForBuffer =3D=3D 0) { - Status =3D EFI_DEVICE_ERROR; - goto EXIT; - } - - // From now on we work in 32-bit words - BufferSizeInWords =3D BufferSizeInBytes / (UINTN)4; - - // Write the word count, which is (buffer_size_in_words - 1), - // because word count 0 means one word. - SEND_NOR_COMMAND (TargetAddress, 0, (BufferSizeInWords - 1)); - - // Write the data to the NOR Flash, advancing each address by 4 bytes - for (Count =3D 0; Count < BufferSizeInWords; Count++, Data++, Buffer++) = { - MmioWrite32 ((UINTN)Data, *Buffer); - } - - // Issue the Buffered Program Confirm command, to start the programming = operation - SEND_NOR_COMMAND (Instance->DeviceBaseAddress, 0, P30_CMD_BUFFERED_PROGR= AM_CONFIRM); - - // Wait for the write to complete and then check for any errors; i.e. ch= eck the Status Register - do { - StatusRegister =3D NorFlashReadStatusRegister (Instance, TargetAddress= ); - // The chip is busy while the WRITE bit is not asserted - } while ((StatusRegister & P30_SR_BIT_WRITE) !=3D P30_SR_BIT_WRITE); - - // Perform a full status check: - // Mask the relevant bits of Status Register. - // Everything should be zero, if not, we have a problem - - Status =3D EFI_SUCCESS; - - if (StatusRegister & P30_SR_BIT_VPP) { - DEBUG ((DEBUG_ERROR, "NorFlashWriteBuffer(TargetAddress:0x%X): VPP Ran= ge Error\n", TargetAddress)); - Status =3D EFI_DEVICE_ERROR; - } - - if (StatusRegister & P30_SR_BIT_PROGRAM) { - DEBUG ((DEBUG_ERROR, "NorFlashWriteBuffer(TargetAddress:0x%X): Program= Error\n", TargetAddress)); - Status =3D EFI_DEVICE_ERROR; - } - - if (StatusRegister & P30_SR_BIT_BLOCK_LOCKED) { - DEBUG ((DEBUG_ERROR, "NorFlashWriteBuffer(TargetAddress:0x%X): Device = Protect Error\n", TargetAddress)); - Status =3D EFI_DEVICE_ERROR; - } - - if (!EFI_ERROR (Status)) { - // Clear the Status Register - SEND_NOR_COMMAND (Instance->DeviceBaseAddress, 0, P30_CMD_CLEAR_STATUS= _REGISTER); - } - -EXIT: - // Put device back into Read Array mode - SEND_NOR_COMMAND (Instance->DeviceBaseAddress, 0, P30_CMD_READ_ARRAY); - - return Status; -} - -EFI_STATUS -NorFlashWriteBlocks ( - IN NOR_FLASH_INSTANCE *Instance, - IN EFI_LBA Lba, - IN UINTN BufferSizeInBytes, - IN VOID *Buffer - ) -{ - UINT32 *pWriteBuffer; - EFI_STATUS Status; - EFI_LBA CurrentBlock; - UINT32 BlockSizeInWords; - UINT32 NumBlocks; - UINT32 BlockCount; - - Status =3D EFI_SUCCESS; - - // The buffer must be valid - if (Buffer =3D=3D NULL) { - return EFI_INVALID_PARAMETER; - } - - if (Instance->Media.ReadOnly =3D=3D TRUE) { - return EFI_WRITE_PROTECTED; - } - - // We must have some bytes to read - DEBUG ((DEBUG_BLKIO, "NorFlashWriteBlocks: BufferSizeInBytes=3D0x%x\n", = BufferSizeInBytes)); - if (BufferSizeInBytes =3D=3D 0) { - return EFI_BAD_BUFFER_SIZE; - } - - // The size of the buffer must be a multiple of the block size - DEBUG ((DEBUG_BLKIO, "NorFlashWriteBlocks: BlockSize in bytes =3D0x%x\n"= , Instance->Media.BlockSize)); - if ((BufferSizeInBytes % Instance->Media.BlockSize) !=3D 0) { - return EFI_BAD_BUFFER_SIZE; - } - - // All blocks must be within the device - NumBlocks =3D ((UINT32)BufferSizeInBytes) / Instance->Media.BlockSize; - - DEBUG ((DEBUG_BLKIO, "NorFlashWriteBlocks: NumBlocks=3D%d, LastBlock=3D%= ld, Lba=3D%ld.\n", NumBlocks, Instance->Media.LastBlock, Lba)); - - if ((Lba + NumBlocks) > (Instance->Media.LastBlock + 1)) { - DEBUG ((DEBUG_ERROR, "NorFlashWriteBlocks: ERROR - Write will exceed l= ast block.\n")); - return EFI_INVALID_PARAMETER; - } - - BlockSizeInWords =3D Instance->Media.BlockSize / 4; - - // Because the target *Buffer is a pointer to VOID, we must put all the = data into a pointer - // to a proper data type, so use *ReadBuffer - pWriteBuffer =3D (UINT32 *)Buffer; - - CurrentBlock =3D Lba; - for (BlockCount =3D 0; BlockCount < NumBlocks; BlockCount++, CurrentBloc= k++, pWriteBuffer =3D pWriteBuffer + BlockSizeInWords) { - DEBUG ((DEBUG_BLKIO, "NorFlashWriteBlocks: Writing block #%d\n", (UINT= N)CurrentBlock)); - - Status =3D NorFlashWriteFullBlock (Instance, CurrentBlock, pWriteBuffe= r, BlockSizeInWords); - - if (EFI_ERROR (Status)) { - break; - } - } - - DEBUG ((DEBUG_BLKIO, "NorFlashWriteBlocks: Exit Status =3D \"%r\".\n", S= tatus)); - return Status; -} - -#define BOTH_ALIGNED(a, b, align) ((((UINTN)(a) | (UINTN)(b)) & ((align) = - 1)) =3D=3D 0) - -/** - Copy Length bytes from Source to Destination, using aligned accesses onl= y. - Note that this implementation uses memcpy() semantics rather then memmov= e() - semantics, i.e., SourceBuffer and DestinationBuffer should not overlap. - - @param DestinationBuffer The target of the copy request. - @param SourceBuffer The place to copy from. - @param Length The number of bytes to copy. - - @return Destination - -**/ -STATIC -VOID * -AlignedCopyMem ( - OUT VOID *DestinationBuffer, - IN CONST VOID *SourceBuffer, - IN UINTN Length - ) -{ - UINT8 *Destination8; - CONST UINT8 *Source8; - UINT32 *Destination32; - CONST UINT32 *Source32; - UINT64 *Destination64; - CONST UINT64 *Source64; - - if (BOTH_ALIGNED (DestinationBuffer, SourceBuffer, 8) && (Length >=3D 8)= ) { - Destination64 =3D DestinationBuffer; - Source64 =3D SourceBuffer; - while (Length >=3D 8) { - *Destination64++ =3D *Source64++; - Length -=3D 8; - } - - Destination8 =3D (UINT8 *)Destination64; - Source8 =3D (CONST UINT8 *)Source64; - } else if (BOTH_ALIGNED (DestinationBuffer, SourceBuffer, 4) && (Length = >=3D 4)) { - Destination32 =3D DestinationBuffer; - Source32 =3D SourceBuffer; - while (Length >=3D 4) { - *Destination32++ =3D *Source32++; - Length -=3D 4; - } - - Destination8 =3D (UINT8 *)Destination32; - Source8 =3D (CONST UINT8 *)Source32; - } else { - Destination8 =3D DestinationBuffer; - Source8 =3D SourceBuffer; - } - - while (Length-- !=3D 0) { - *Destination8++ =3D *Source8++; - } - - return DestinationBuffer; -} - -EFI_STATUS -NorFlashReadBlocks ( - IN NOR_FLASH_INSTANCE *Instance, - IN EFI_LBA Lba, - IN UINTN BufferSizeInBytes, - OUT VOID *Buffer - ) -{ - UINT32 NumBlocks; - UINTN StartAddress; - - DEBUG (( - DEBUG_BLKIO, - "NorFlashReadBlocks: BufferSize=3D0x%xB BlockSize=3D0x%xB LastBlock=3D= %ld, Lba=3D%ld.\n", - BufferSizeInBytes, - Instance->Media.BlockSize, - Instance->Media.LastBlock, - Lba - )); - - // The buffer must be valid - if (Buffer =3D=3D NULL) { - return EFI_INVALID_PARAMETER; - } - - // Return if we have not any byte to read - if (BufferSizeInBytes =3D=3D 0) { - return EFI_SUCCESS; - } - - // The size of the buffer must be a multiple of the block size - if ((BufferSizeInBytes % Instance->Media.BlockSize) !=3D 0) { - return EFI_BAD_BUFFER_SIZE; - } - - // All blocks must be within the device - NumBlocks =3D ((UINT32)BufferSizeInBytes) / Instance->Media.BlockSize; - - if ((Lba + NumBlocks) > (Instance->Media.LastBlock + 1)) { - DEBUG ((DEBUG_ERROR, "NorFlashReadBlocks: ERROR - Read will exceed las= t block\n")); - return EFI_INVALID_PARAMETER; - } - - // Get the address to start reading from - StartAddress =3D GET_NOR_BLOCK_ADDRESS ( - Instance->RegionBaseAddress, - Lba, - Instance->Media.BlockSize - ); - - // Put the device into Read Array mode - SEND_NOR_COMMAND (Instance->DeviceBaseAddress, 0, P30_CMD_READ_ARRAY); - - // Readout the data - AlignedCopyMem (Buffer, (VOID *)StartAddress, BufferSizeInBytes); - - return EFI_SUCCESS; -} - -EFI_STATUS -NorFlashRead ( - IN NOR_FLASH_INSTANCE *Instance, - IN EFI_LBA Lba, - IN UINTN Offset, - IN UINTN BufferSizeInBytes, - OUT VOID *Buffer - ) -{ - UINTN StartAddress; - - // The buffer must be valid - if (Buffer =3D=3D NULL) { - return EFI_INVALID_PARAMETER; - } - - // Return if we have not any byte to read - if (BufferSizeInBytes =3D=3D 0) { - return EFI_SUCCESS; - } - - if (((Lba * Instance->Media.BlockSize) + Offset + BufferSizeInBytes) > I= nstance->Size) { - DEBUG ((DEBUG_ERROR, "NorFlashRead: ERROR - Read will exceed device si= ze.\n")); - return EFI_INVALID_PARAMETER; - } - - // Get the address to start reading from - StartAddress =3D GET_NOR_BLOCK_ADDRESS ( - Instance->RegionBaseAddress, - Lba, - Instance->Media.BlockSize - ); - - // Put the device into Read Array mode - SEND_NOR_COMMAND (Instance->DeviceBaseAddress, 0, P30_CMD_READ_ARRAY); - - // Readout the data - AlignedCopyMem (Buffer, (VOID *)(StartAddress + Offset), BufferSizeInByt= es); - - return EFI_SUCCESS; -} - -/* - Write a full or portion of a block. It must not span block boundaries; t= hat is, - Offset + *NumBytes <=3D Instance->Media.BlockSize. -*/ -EFI_STATUS -NorFlashWriteSingleBlock ( - IN NOR_FLASH_INSTANCE *Instance, - IN EFI_LBA Lba, - IN UINTN Offset, - IN OUT UINTN *NumBytes, - IN UINT8 *Buffer - ) -{ - EFI_STATUS TempStatus; - UINT32 Tmp; - UINT32 TmpBuf; - UINT32 WordToWrite; - UINT32 Mask; - BOOLEAN DoErase; - UINTN BytesToWrite; - UINTN CurOffset; - UINTN WordAddr; - UINTN BlockSize; - UINTN BlockAddress; - UINTN PrevBlockAddress; - - PrevBlockAddress =3D 0; - - DEBUG ((DEBUG_BLKIO, "NorFlashWriteSingleBlock(Parameters: Lba=3D%ld, Of= fset=3D0x%x, *NumBytes=3D0x%x, Buffer @ 0x%08x)\n", Lba, Offset, *NumBytes,= Buffer)); - - // Detect WriteDisabled state - if (Instance->Media.ReadOnly =3D=3D TRUE) { - DEBUG ((DEBUG_ERROR, "NorFlashWriteSingleBlock: ERROR - Can not write:= Device is in WriteDisabled state.\n")); - // It is in WriteDisabled state, return an error right away - return EFI_ACCESS_DENIED; - } - - // Cache the block size to avoid de-referencing pointers all the time - BlockSize =3D Instance->Media.BlockSize; - - // The write must not span block boundaries. - // We need to check each variable individually because adding two large = values together overflows. - if ((Offset >=3D BlockSize) || - (*NumBytes > BlockSize) || - ((Offset + *NumBytes) > BlockSize)) - { - DEBUG ((DEBUG_ERROR, "NorFlashWriteSingleBlock: ERROR - EFI_BAD_BUFFER= _SIZE: (Offset=3D0x%x + NumBytes=3D0x%x) > BlockSize=3D0x%x\n", Offset, *Nu= mBytes, BlockSize)); - return EFI_BAD_BUFFER_SIZE; - } - - // We must have some bytes to write - if (*NumBytes =3D=3D 0) { - DEBUG ((DEBUG_ERROR, "NorFlashWriteSingleBlock: ERROR - EFI_BAD_BUFFER= _SIZE: (Offset=3D0x%x + NumBytes=3D0x%x) > BlockSize=3D0x%x\n", Offset, *Nu= mBytes, BlockSize)); - return EFI_BAD_BUFFER_SIZE; - } - - // Pick 128bytes as a good start for word operations as opposed to erasi= ng the - // block and writing the data regardless if an erase is really needed. - // It looks like most individual NV variable writes are smaller than 128= bytes. - if (*NumBytes <=3D 128) { - // Check to see if we need to erase before programming the data into N= OR. - // If the destination bits are only changing from 1s to 0s we can just= write. - // After a block is erased all bits in the block is set to 1. - // If any byte requires us to erase we just give up and rewrite all of= it. - DoErase =3D FALSE; - BytesToWrite =3D *NumBytes; - CurOffset =3D Offset; - - while (BytesToWrite > 0) { - // Read full word from NOR, splice as required. A word is the smalle= st - // unit we can write. - TempStatus =3D NorFlashRead (Instance, Lba, CurOffset & ~(0x3), size= of (Tmp), &Tmp); - if (EFI_ERROR (TempStatus)) { - return EFI_DEVICE_ERROR; - } - - // Physical address of word in NOR to write. - WordAddr =3D (CurOffset & ~(0x3)) + GET_NOR_BLOCK_ADDRESS ( - Instance->RegionBaseAddress, - Lba, - BlockSize - ); - // The word of data that is to be written. - TmpBuf =3D *((UINT32 *)(Buffer + (*NumBytes - BytesToWrite))); - - // First do word aligned chunks. - if ((CurOffset & 0x3) =3D=3D 0) { - if (BytesToWrite >=3D 4) { - // Is the destination still in 'erased' state? - if (~Tmp !=3D 0) { - // Check to see if we are only changing bits to zero. - if ((Tmp ^ TmpBuf) & TmpBuf) { - DoErase =3D TRUE; - break; - } - } - - // Write this word to NOR - WordToWrite =3D TmpBuf; - CurOffset +=3D sizeof (TmpBuf); - BytesToWrite -=3D sizeof (TmpBuf); - } else { - // BytesToWrite < 4. Do small writes and left-overs - Mask =3D ~((~0) << (BytesToWrite * 8)); - // Mask out the bytes we want. - TmpBuf &=3D Mask; - // Is the destination still in 'erased' state? - if ((Tmp & Mask) !=3D Mask) { - // Check to see if we are only changing bits to zero. - if ((Tmp ^ TmpBuf) & TmpBuf) { - DoErase =3D TRUE; - break; - } - } - - // Merge old and new data. Write merged word to NOR - WordToWrite =3D (Tmp & ~Mask) | TmpBuf; - CurOffset +=3D BytesToWrite; - BytesToWrite =3D 0; - } - } else { - // Do multiple words, but starting unaligned. - if (BytesToWrite > (4 - (CurOffset & 0x3))) { - Mask =3D ((~0) << ((CurOffset & 0x3) * 8)); - // Mask out the bytes we want. - TmpBuf &=3D Mask; - // Is the destination still in 'erased' state? - if ((Tmp & Mask) !=3D Mask) { - // Check to see if we are only changing bits to zero. - if ((Tmp ^ TmpBuf) & TmpBuf) { - DoErase =3D TRUE; - break; - } - } - - // Merge old and new data. Write merged word to NOR - WordToWrite =3D (Tmp & ~Mask) | TmpBuf; - BytesToWrite -=3D (4 - (CurOffset & 0x3)); - CurOffset +=3D (4 - (CurOffset & 0x3)); - } else { - // Unaligned and fits in one word. - Mask =3D (~((~0) << (BytesToWrite * 8))) << ((CurOffset & 0x3) *= 8); - // Mask out the bytes we want. - TmpBuf =3D (TmpBuf << ((CurOffset & 0x3) * 8)) & Mask; - // Is the destination still in 'erased' state? - if ((Tmp & Mask) !=3D Mask) { - // Check to see if we are only changing bits to zero. - if ((Tmp ^ TmpBuf) & TmpBuf) { - DoErase =3D TRUE; - break; - } - } - - // Merge old and new data. Write merged word to NOR - WordToWrite =3D (Tmp & ~Mask) | TmpBuf; - CurOffset +=3D BytesToWrite; - BytesToWrite =3D 0; - } - } - - // - // Write the word to NOR. - // - - BlockAddress =3D GET_NOR_BLOCK_ADDRESS (Instance->RegionBaseAddress,= Lba, BlockSize); - if (BlockAddress !=3D PrevBlockAddress) { - TempStatus =3D NorFlashUnlockSingleBlockIfNecessary (Instance, Blo= ckAddress); - if (EFI_ERROR (TempStatus)) { - return EFI_DEVICE_ERROR; - } - - PrevBlockAddress =3D BlockAddress; - } - - TempStatus =3D NorFlashWriteSingleWord (Instance, WordAddr, WordToWr= ite); - if (EFI_ERROR (TempStatus)) { - return EFI_DEVICE_ERROR; - } - } - - // Exit if we got here and could write all the data. Otherwise do the - // Erase-Write cycle. - if (!DoErase) { - return EFI_SUCCESS; - } - } - - // Check we did get some memory. Buffer is BlockSize. - if (Instance->ShadowBuffer =3D=3D NULL) { - DEBUG ((DEBUG_ERROR, "FvbWrite: ERROR - Buffer not ready\n")); - return EFI_DEVICE_ERROR; - } - - // Read NOR Flash data into shadow buffer - TempStatus =3D NorFlashReadBlocks (Instance, Lba, BlockSize, Instance->S= hadowBuffer); - if (EFI_ERROR (TempStatus)) { - // Return one of the pre-approved error statuses - return EFI_DEVICE_ERROR; - } - - // Put the data at the appropriate location inside the buffer area - CopyMem ((VOID *)((UINTN)Instance->ShadowBuffer + Offset), Buffer, *NumB= ytes); - - // Write the modified buffer back to the NorFlash - TempStatus =3D NorFlashWriteBlocks (Instance, Lba, BlockSize, Instance->= ShadowBuffer); - if (EFI_ERROR (TempStatus)) { - // Return one of the pre-approved error statuses - return EFI_DEVICE_ERROR; - } - - return EFI_SUCCESS; -} - -EFI_STATUS -NorFlashReset ( - IN NOR_FLASH_INSTANCE *Instance - ) -{ - // As there is no specific RESET to perform, ensure that the devices is = in the default Read Array mode - SEND_NOR_COMMAND (Instance->DeviceBaseAddress, 0, P30_CMD_READ_ARRAY); - return EFI_SUCCESS; -} - -/** - * This function unlock and erase an entire NOR Flash block. -**/ -EFI_STATUS -NorFlashUnlockAndEraseSingleBlock ( - IN NOR_FLASH_INSTANCE *Instance, - IN UINTN BlockAddress - ) -{ - EFI_STATUS Status; - UINTN Index; - EFI_TPL OriginalTPL; - - NorFlashLock (&OriginalTPL); - - Index =3D 0; - // The block erase might fail a first time (SW bug ?). Retry it ... - do { - // Unlock the block if we have to - Status =3D NorFlashUnlockSingleBlockIfNecessary (Instance, BlockAddres= s); - if (EFI_ERROR (Status)) { - break; - } - - Status =3D NorFlashEraseSingleBlock (Instance, BlockAddress); - Index++; - } while ((Index < NOR_FLASH_ERASE_RETRY) && (Status =3D=3D EFI_WRITE_PRO= TECTED)); - - if (Index =3D=3D NOR_FLASH_ERASE_RETRY) { - DEBUG ((DEBUG_ERROR, "EraseSingleBlock(BlockAddress=3D0x%08x: Block Lo= cked Error (try to erase %d times)\n", BlockAddress, Index)); - } - - NorFlashUnlock (OriginalTPL); - - return Status; -} - -EFI_STATUS -NorFlashWriteFullBlock ( - IN NOR_FLASH_INSTANCE *Instance, - IN EFI_LBA Lba, - IN UINT32 *DataBuffer, - IN UINT32 BlockSizeInWords - ) -{ - EFI_STATUS Status; - UINTN WordAddress; - UINT32 WordIndex; - UINTN BufferIndex; - UINTN BlockAddress; - UINTN BuffersInBlock; - UINTN RemainingWords; - EFI_TPL OriginalTPL; - UINTN Cnt; - - Status =3D EFI_SUCCESS; - - // Get the physical address of the block - BlockAddress =3D GET_NOR_BLOCK_ADDRESS (Instance->RegionBaseAddress, Lba= , BlockSizeInWords * 4); - - // Start writing from the first address at the start of the block - WordAddress =3D BlockAddress; - - NorFlashLock (&OriginalTPL); - - Status =3D NorFlashUnlockAndEraseSingleBlock (Instance, BlockAddress); - if (EFI_ERROR (Status)) { - DEBUG ((DEBUG_ERROR, "WriteSingleBlock: ERROR - Failed to Unlock and E= rase the single block at 0x%X\n", BlockAddress)); - goto EXIT; - } - - // To speed up the programming operation, NOR Flash is programmed using = the Buffered Programming method. - - // Check that the address starts at a 32-word boundary, i.e. last 7 bits= must be zero - if ((WordAddress & BOUNDARY_OF_32_WORDS) =3D=3D 0x00) { - // First, break the entire block into buffer-sized chunks. - BuffersInBlock =3D (UINTN)(BlockSizeInWords * 4) / P30_MAX_BUFFER_SIZE= _IN_BYTES; - - // Then feed each buffer chunk to the NOR Flash - // If a buffer does not contain any data, don't write it. - for (BufferIndex =3D 0; - BufferIndex < BuffersInBlock; - BufferIndex++, WordAddress +=3D P30_MAX_BUFFER_SIZE_IN_BYTES, Dat= aBuffer +=3D P30_MAX_BUFFER_SIZE_IN_WORDS - ) - { - // Check the buffer to see if it contains any data (not set all 1s). - for (Cnt =3D 0; Cnt < P30_MAX_BUFFER_SIZE_IN_WORDS; Cnt++) { - if (~DataBuffer[Cnt] !=3D 0 ) { - // Some data found, write the buffer. - Status =3D NorFlashWriteBuffer ( - Instance, - WordAddress, - P30_MAX_BUFFER_SIZE_IN_BYTES, - DataBuffer - ); - if (EFI_ERROR (Status)) { - goto EXIT; - } - - break; - } - } - } - - // Finally, finish off any remaining words that are less than the maxi= mum size of the buffer - RemainingWords =3D BlockSizeInWords % P30_MAX_BUFFER_SIZE_IN_WORDS; - - if (RemainingWords !=3D 0) { - Status =3D NorFlashWriteBuffer (Instance, WordAddress, (RemainingWor= ds * 4), DataBuffer); - if (EFI_ERROR (Status)) { - goto EXIT; - } - } - } else { - // For now, use the single word programming algorithm - // It is unlikely that the NOR Flash will exist in an address which fa= lls within a 32 word boundary range, - // i.e. which ends in the range 0x......01 - 0x......7F. - for (WordIndex =3D 0; WordIndex < BlockSizeInWords; WordIndex++, DataB= uffer++, WordAddress =3D WordAddress + 4) { - Status =3D NorFlashWriteSingleWord (Instance, WordAddress, *DataBuff= er); - if (EFI_ERROR (Status)) { - goto EXIT; - } - } - } - -EXIT: - NorFlashUnlock (OriginalTPL); - - if (EFI_ERROR (Status)) { - DEBUG ((DEBUG_ERROR, "NOR FLASH Programming [WriteSingleBlock] failed = at address 0x%08x. Exit Status =3D \"%r\".\n", WordAddress, Status)); - } - - return Status; -} diff --git a/Platform/ARM/Drivers/NorFlashDxe/NorFlashBlockIoDxe.c b/Platfo= rm/ARM/Drivers/NorFlashDxe/NorFlashBlockIoDxe.c index a6c73312c596..4e7e88610200 100644 --- a/Platform/ARM/Drivers/NorFlashDxe/NorFlashBlockIoDxe.c +++ b/Platform/ARM/Drivers/NorFlashDxe/NorFlashBlockIoDxe.c @@ -9,7 +9,6 @@ #include #include -#include "NorFlash.h" #include "NorFlashCommon.h" // diff --git a/Platform/ARM/Drivers/NorFlashDxe/NorFlashDxe.c b/Platform/ARM/= Drivers/NorFlashDxe/NorFlashDxe.c index 4bad6e9b2a6b..1c12572ab663 100644 --- a/Platform/ARM/Drivers/NorFlashDxe/NorFlashDxe.c +++ b/Platform/ARM/Drivers/NorFlashDxe/NorFlashDxe.c @@ -14,7 +14,6 @@ #include #include -#include "NorFlash.h" #include "NorFlashCommon.h" STATIC EFI_EVENT mNorFlashVirtualAddrChangeEvent; diff --git a/Platform/ARM/Drivers/NorFlashDxe/NorFlashFvb.c b/Platform/ARM/= Drivers/NorFlashDxe/NorFlashFvb.c index f9f2856e0c81..e48c258a3008 100644 --- a/Platform/ARM/Drivers/NorFlashDxe/NorFlashFvb.c +++ b/Platform/ARM/Drivers/NorFlashDxe/NorFlashFvb.c @@ -18,7 +18,6 @@ #include #include -#include "NorFlash.h" #include "NorFlashCommon.h" extern UINTN mFlashNvStorageVariableBase; diff --git a/Platform/ARM/Drivers/NorFlashDxe/NorFlashStandaloneMm.c b/Plat= form/ARM/Drivers/NorFlashDxe/NorFlashStandaloneMm.c index 5bff524e5e18..85198b98e8a8 100644 --- a/Platform/ARM/Drivers/NorFlashDxe/NorFlashStandaloneMm.c +++ b/Platform/ARM/Drivers/NorFlashDxe/NorFlashStandaloneMm.c @@ -11,7 +11,6 @@ #include #include -#include "NorFlash.h" #include "NorFlashCommon.h" // -- 2.25.1 -=3D-=3D-=3D-=3D-=3D-=3D Groups.io Links: You receive all messages sent to this group. View/Reply Online (#118157): https://edk2.groups.io/g/devel/message/118157 Mute This Topic: https://groups.io/mt/105690942/7717249 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [yeoreum.yun@arm.com] -=3D-=3D-=3D-=3D-=3D-=3D IMPORTANT NOTICE: The contents of this email and any attachments are confid= ential and may also be privileged. If you are not the intended recipient, p= lease notify the sender immediately and do not disclose the contents to any= other person, use it for any purpose, or store or copy the information in = any medium. Thank you. -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#118191): https://edk2.groups.io/g/devel/message/118191 Mute This Topic: https://groups.io/mt/105690942/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-