From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from linux.microsoft.com (linux.microsoft.com [13.77.154.182]) by mx.groups.io with SMTP id smtpd.web09.11764.1617820439110696421 for ; Wed, 07 Apr 2021 11:33:59 -0700 Authentication-Results: mx.groups.io; dkim=fail reason="body hash did not verify" header.i=@linux.microsoft.com header.s=default header.b=AZkD51Iq; spf=pass (domain: linux.microsoft.com, ip: 13.77.154.182, mailfrom: mikuback@linux.microsoft.com) Received: from localhost.localdomain (unknown [167.220.2.74]) by linux.microsoft.com (Postfix) with ESMTPSA id 6C1C320B5680; Wed, 7 Apr 2021 11:33:58 -0700 (PDT) DKIM-Filter: OpenDKIM Filter v2.11.0 linux.microsoft.com 6C1C320B5680 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.microsoft.com; s=default; t=1617820438; bh=HNNVV3wOZJLvSJSQSdYBDYpkzD7ZjHFEaaydyOFOyd8=; h=From:To:Cc:Subject:Date:From; b=AZkD51Iq/5dTf0zOcMMvZnScfgSmyOrycOIoQZTr3KnMhQveuBWKi+JmNf887k83N PUIHHPRim1UZasbTfv5Z5EYc40shzj+vFQgKSiPQsVyJrfzlIjcKlP7F3qmOSzu6zU lCIcGJow+YdE5/6kQN1q48WmqRUlzp1WuVzJUmPA= From: "Michael Kubacki" To: devel@edk2.groups.io Cc: Chasel Chiu , Nate DeSimone , Liming Gao , Eric Dong Subject: [edk2-platforms][PATCH v1 0/2] MinPlatformPkg: Improve NULL ACPI table lib maintainability Date: Wed, 7 Apr 2021 11:33:22 -0700 Message-Id: <20210407183324.1659-1-mikuback@linux.microsoft.com> X-Mailer: git-send-email 2.28.0.windows.1 MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable From: Michael Kubacki This series makes some simple changes to the organization of BoardAcpiEnableLibNull and BoardAcpiTableLibNull to better align the library instances with patterns typically used so they are easier to integrate into platforms. The following issues are resolved: 1. Sharing of 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 instances have unnecessary dependencies. 4. The BASE_NAME does not indicate the library instance is the NULL instance. 5. The C source file names do not match the INF file name making finding the C source by search more cumbersome than needed. Cc: Chasel Chiu Cc: Nate DeSimone Cc: Liming Gao Cc: Eric Dong Signed-off-by: Michael Kubacki Michael Kubacki (2): MinPlatformPkg/BoardAcpiEnableLibNull: Improve maintainability MinPlatformPkg/BoardAcpiTableLibNull: Improve maintainability Platform/Intel/MinPlatformPkg/Acpi/Library/{BoardAcpiLibNull/BoardAcpiEn= ableLib.c =3D> BoardAcpiEnableLibNull/BoardAcpiEnableLibNull.c} | 4 +--- Platform/Intel/MinPlatformPkg/Acpi/Library/{BoardAcpiLibNull/BoardAcpiTa= bleLib.c =3D> BoardAcpiTableLibNull/BoardAcpiTableLibNull.c} | 4 +--- Platform/Intel/MinPlatformPkg/Acpi/Library/{BoardAcpiLibNull =3D> BoardA= cpiEnableLibNull}/BoardAcpiEnableLibNull.inf | 12 ++++= -------- Platform/Intel/MinPlatformPkg/Acpi/Library/{BoardAcpiLibNull =3D> BoardA= cpiTableLibNull}/BoardAcpiTableLibNull.inf | 12 ++++= -------- Platform/Intel/MinPlatformPkg/MinPlatformPkg.dsc = | 8 ++++--= -- 5 files changed, 14 insertions(+), 26 deletions(-) rename Platform/Intel/MinPlatformPkg/Acpi/Library/{BoardAcpiLibNull/Boar= dAcpiEnableLib.c =3D> BoardAcpiEnableLibNull/BoardAcpiEnableLibNull.c} (7= 3%) rename Platform/Intel/MinPlatformPkg/Acpi/Library/{BoardAcpiLibNull/Boar= dAcpiTableLib.c =3D> BoardAcpiTableLibNull/BoardAcpiTableLibNull.c} (73%) rename Platform/Intel/MinPlatformPkg/Acpi/Library/{BoardAcpiLibNull =3D>= BoardAcpiEnableLibNull}/BoardAcpiEnableLibNull.inf (67%) rename Platform/Intel/MinPlatformPkg/Acpi/Library/{BoardAcpiLibNull =3D>= BoardAcpiTableLibNull}/BoardAcpiTableLibNull.inf (67%) --=20 2.28.0.windows.1