From nobody Sat Feb 7 11:05:11 2026 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) client-ip=66.175.222.108; envelope-from=bounce+27952+73881+1787277+3901457@groups.io; helo=mail02.groups.io; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce+27952+73881+1787277+3901457@groups.io; dmarc=fail(p=none dis=none) header.from=linux.microsoft.com ARC-Seal: i=1; a=rsa-sha256; t=1617917097; cv=none; d=zohomail.com; s=zohoarc; b=kVBos3vrowLGEwmSSyzSAvCO/3XdrsDMsV0+1fOi0gj5UjSEtgBf7OFHRQtwXpQvkrgba3p0lw3zzONcPaxiarRVNsNaoKcqbQ+3vFIWLC8ytX+ZDpUzvMWPNFXS6pWDQkaN7YAW9H332zTt2tMsbu7Cdzihn2+aKI2qxHd8sJs= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1617917097; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Id:List-Help:List-Unsubscribe:MIME-Version:Message-ID:Reply-To:References:Sender:Subject:To; bh=UCR575myzbTmykhyF7CkpRntdX6J3IGmups1bUyF7q8=; b=U0Ptsd5cBolUu2K8k8jtPEfmO0S6/E3J5aK1mCZN8BcUD/SJqR7J1tqcxzlUcZKg1WkcBRaJOsY2dJkV7Q6gxSOEIEkleLRUP9JgMh1O8rYnLN53yGH6DPuN0JaYl2XXIgG9lbqFHyfAXX4F35rqcN3M+85imgzl6KPEJguW418= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce+27952+73881+1787277+3901457@groups.io; dmarc=fail header.from= (p=none dis=none) header.from= Received: from mail02.groups.io (mail02.groups.io [66.175.222.108]) by mx.zohomail.com with SMTPS id 1617917097564135.648835958885; Thu, 8 Apr 2021 14:24:57 -0700 (PDT) Return-Path: X-Received: by 127.0.0.2 with SMTP id aEzTYY1788612xp2NSgistF3; Thu, 08 Apr 2021 14:24:56 -0700 X-Received: from linux.microsoft.com (linux.microsoft.com [13.77.154.182]) by mx.groups.io with SMTP id smtpd.web12.294.1617917095398635006 for ; Thu, 08 Apr 2021 14:24:55 -0700 X-Received: from localhost.localdomain (unknown [167.220.2.74]) by linux.microsoft.com (Postfix) with ESMTPSA id 0F58D20B5681; Thu, 8 Apr 2021 14:24:55 -0700 (PDT) DKIM-Filter: OpenDKIM Filter v2.11.0 linux.microsoft.com 0F58D20B5681 From: "Michael Kubacki" To: devel@edk2.groups.io Cc: Chasel Chiu , Nate DeSimone , Liming Gao , Eric Dong Subject: [edk2-devel] [edk2-platforms][PATCH v2 1/2] MinPlatformPkg/BoardAcpiEnableLibNull: Improve maintainability Date: Thu, 8 Apr 2021 14:24:32 -0700 Message-Id: <20210408212433.632-2-mikuback@linux.microsoft.com> In-Reply-To: <20210408212433.632-1-mikuback@linux.microsoft.com> References: <20210408212433.632-1-mikuback@linux.microsoft.com> MIME-Version: 1.0 Precedence: Bulk List-Unsubscribe: List-Subscribe: List-Help: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Reply-To: devel@edk2.groups.io,mikuback@linux.microsoft.com X-Gm-Message-State: eyss87t9rs11sZUKJmO0JXfMx1787277AA= Content-Transfer-Encoding: quoted-printable DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=groups.io; q=dns/txt; s=20140610; t=1617917096; bh=nbUeIT1K/peBFja8UzKpmH8rg8DQ1Xm3TQQBnG4HD5c=; h=Cc:Date:From:Reply-To:Subject:To; b=NGmgthhV4hhJ/JzIFLtBCyhHHETz9kLihCyrbEpg4V1nXO4GkPgWvWGJSGfjE0uuWMa pSaJtQP0RDwuZ++mWfX61y/JeHP7yJZ9LhaVsfBzvbCj9XnUemyJBOokf8LtP4pUQLv6h kxTzPM3GmRvSs5HahZFfu3xMe0ob5ZbIbhM= X-ZohoMail-DKIM: pass (identity @groups.io) Content-Type: text/plain; charset="utf-8" From: Michael Kubacki The NULL instance of BoardAcpiEnableLib in MinPlatformPkg currently has a few organization issues that make it more difficult to find and use than a typical NULL library instance. 1. It shares a directory with another unrelated library instance. 2. The directory name "BoardAcpiLibNull" is not directly related to either library instance name in the directory. 3. The library instance has unnecessary dependencies. 4. The BASE_NAME does not indicate the library instance is the NULL instance. 5. The C source file name does not match the INF file name making finding the C source by search more cumbersome than needed. This change resolves the above issues to improve maintainability. Cc: Chasel Chiu Cc: Nate DeSimone Cc: Liming Gao Cc: Eric Dong Signed-off-by: Michael Kubacki --- Platform/Intel/MinPlatformPkg/Acpi/Library/{BoardAcpiLibNull/BoardAcpiEnab= leLib.c =3D> BoardAcpiEnableLibNull/BoardAcpiEnableLibNull.c} | 3 --- Platform/Intel/MinPlatformPkg/Acpi/Library/{BoardAcpiLibNull =3D> BoardAcp= iEnableLibNull}/BoardAcpiEnableLibNull.inf | 12 ++++----= ---- Platform/Intel/MinPlatformPkg/MinPlatformPkg.dsc = | 4 ++-- 3 files changed, 6 insertions(+), 13 deletions(-) diff --git a/Platform/Intel/MinPlatformPkg/Acpi/Library/BoardAcpiLibNull/Bo= ardAcpiEnableLib.c b/Platform/Intel/MinPlatformPkg/Acpi/Library/BoardAcpiEn= ableLibNull/BoardAcpiEnableLibNull.c similarity index 73% rename from Platform/Intel/MinPlatformPkg/Acpi/Library/BoardAcpiLibNull/Boa= rdAcpiEnableLib.c rename to Platform/Intel/MinPlatformPkg/Acpi/Library/BoardAcpiEnableLibNull= /BoardAcpiEnableLibNull.c index 334cb21714ac..66d52c143c4a 100644 --- a/Platform/Intel/MinPlatformPkg/Acpi/Library/BoardAcpiLibNull/BoardAcpi= EnableLib.c +++ b/Platform/Intel/MinPlatformPkg/Acpi/Library/BoardAcpiEnableLibNull/Boa= rdAcpiEnableLibNull.c @@ -5,10 +5,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent =20 **/ =20 -#include #include -#include -#include =20 EFI_STATUS EFIAPI diff --git a/Platform/Intel/MinPlatformPkg/Acpi/Library/BoardAcpiLibNull/Bo= ardAcpiEnableLibNull.inf b/Platform/Intel/MinPlatformPkg/Acpi/Library/Board= AcpiEnableLibNull/BoardAcpiEnableLibNull.inf similarity index 67% rename from Platform/Intel/MinPlatformPkg/Acpi/Library/BoardAcpiLibNull/Boa= rdAcpiEnableLibNull.inf rename to Platform/Intel/MinPlatformPkg/Acpi/Library/BoardAcpiEnableLibNull= /BoardAcpiEnableLibNull.inf index 54985ee50d5f..90bed3760e2a 100644 --- a/Platform/Intel/MinPlatformPkg/Acpi/Library/BoardAcpiLibNull/BoardAcpi= EnableLibNull.inf +++ b/Platform/Intel/MinPlatformPkg/Acpi/Library/BoardAcpiEnableLibNull/Boa= rdAcpiEnableLibNull.inf @@ -1,7 +1,8 @@ ## @file -# Component information file for Board Acpi Library +# Component information file for NULL instance of the Board ACPI Enable li= brary # # Copyright (c) 2017, Intel Corporation. All rights reserved.
+# Copyright (c) Microsoft Corporation.
# # SPDX-License-Identifier: BSD-2-Clause-Patent # @@ -9,20 +10,15 @@ =20 [Defines] INF_VERSION =3D 0x00010005 - BASE_NAME =3D BoardAcpiEnableLib + BASE_NAME =3D BoardAcpiEnableLibNull FILE_GUID =3D EAF8AA6D-549A-40CC-8FB2-3A1FFEC41601 MODULE_TYPE =3D BASE VERSION_STRING =3D 1.0 LIBRARY_CLASS =3D BoardAcpiEnableLib =20 -[LibraryClasses] - BaseLib - PcdLib - DebugLib - [Packages] MinPlatformPkg/MinPlatformPkg.dec MdePkg/MdePkg.dec =20 [Sources] - BoardAcpiEnableLib.c + BoardAcpiEnableLibNull.c diff --git a/Platform/Intel/MinPlatformPkg/MinPlatformPkg.dsc b/Platform/In= tel/MinPlatformPkg/MinPlatformPkg.dsc index 998ee7909568..da27aa1c4227 100644 --- a/Platform/Intel/MinPlatformPkg/MinPlatformPkg.dsc +++ b/Platform/Intel/MinPlatformPkg/MinPlatformPkg.dsc @@ -79,7 +79,7 @@ [LibraryClasses.common] =20 BoardInitLib|MinPlatformPkg/PlatformInit/Library/BoardInitLibNull/BoardI= nitLibNull.inf BoardAcpiTableLib|MinPlatformPkg/Acpi/Library/BoardAcpiLibNull/BoardAcpi= TableLibNull.inf - BoardAcpiEnableLib|MinPlatformPkg/Acpi/Library/BoardAcpiLibNull/BoardAcp= iEnableLibNull.inf + BoardAcpiEnableLib|MinPlatformPkg/Acpi/Library/BoardAcpiEnableLibNull/Bo= ardAcpiEnableLibNull.inf SiliconPolicyInitLib|MinPlatformPkg/PlatformInit/Library/SiliconPolicyIn= itLibNull/SiliconPolicyInitLibNull.inf SiliconPolicyUpdateLib|MinPlatformPkg/PlatformInit/Library/SiliconPolicy= UpdateLibNull/SiliconPolicyUpdateLibNull.inf =20 @@ -150,7 +150,7 @@ [Components] MinPlatformPkg/Acpi/AcpiSmm/AcpiSmm.inf MinPlatformPkg/Acpi/AcpiSmm/AcpiStandaloneMm.inf MinPlatformPkg/Acpi/Library/DxeAslUpdateLib/DxeAslUpdateLib.inf - MinPlatformPkg/Acpi/Library/BoardAcpiLibNull/BoardAcpiEnableLibNull.inf + MinPlatformPkg/Acpi/Library/BoardAcpiEnableLibNull/BoardAcpiEnableLibNul= l.inf MinPlatformPkg/Acpi/Library/BoardAcpiLibNull/BoardAcpiTableLibNull.inf MinPlatformPkg/Acpi/Library/MultiBoardAcpiSupportLib/DxeMultiBoardAcpiSu= pportLib.inf MinPlatformPkg/Acpi/Library/MultiBoardAcpiSupportLib/SmmMultiBoardAcpiSu= pportLib.inf --=20 2.28.0.windows.1 -=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 (#73881): https://edk2.groups.io/g/devel/message/73881 Mute This Topic: https://groups.io/mt/81953910/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-