Re: [edk2-devel] [PATCH 1/1] MdePkg/UefiDevicePathLib: Add support for PEIMs

["Michael D Kinney" <michael.d.kinney@intel.com>]

Mateusz,

I recommend just adding PEIM to the existing INF and not changing the name 
of the C/INF file.  Please update the file headers for both the existing
INF and C file to describe the module types this library is compatible with.

This allows platform developers to understand the compatibility even though
the INF filename does not express all the compatibility.

This minimizes the impact to and existing DSC files that may be using the 
current INF filename.

Mike

> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Albecki, Mateusz
> Sent: Tuesday, January 11, 2022 4:14 AM
> To: devel@edk2.groups.io
> Cc: Albecki, Mateusz <mateusz.albecki@intel.com>; Kinney, Michael D <michael.d.kinney@intel.com>; Gao, Liming
> <gaoliming@byosoft.com.cn>; Liu, Zhiguang <zhiguang.liu@intel.com>
> Subject: [edk2-devel] [PATCH 1/1] MdePkg/UefiDevicePathLib: Add support for PEIMs
> 
> DevicePathLib utilities are useful in PEI to locate the devices which need
> an opal unlock on S3 resume. This commit reuses the implementation done
> to support Standalone MM for PEI.
> 
> Signed-off-by: Mateusz Albecki <mateusz.albecki@intel.com>
> 
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Liming Gao <gaoliming@byosoft.com.cn>
> Cc: Zhiguang Liu <zhiguang.liu@intel.com>
> ---
>  ...iesStandaloneMm.c => DevicePathUtilitiesPeiStandaloneMm.c} | 0
>  ...bStandaloneMm.inf => UefiDevicePathLibPeiStandaloneMm.inf} | 4 ++--
>  MdePkg/MdePkg.dsc                                             | 2 +-
>  3 files changed, 3 insertions(+), 3 deletions(-)
>  rename MdePkg/Library/UefiDevicePathLib/{DevicePathUtilitiesStandaloneMm.c => DevicePathUtilitiesPeiStandaloneMm.c} (100%)
>  rename MdePkg/Library/UefiDevicePathLib/{UefiDevicePathLibStandaloneMm.inf => UefiDevicePathLibPeiStandaloneMm.inf} (93%)
> 
> diff --git a/MdePkg/Library/UefiDevicePathLib/DevicePathUtilitiesStandaloneMm.c
> b/MdePkg/Library/UefiDevicePathLib/DevicePathUtilitiesPeiStandaloneMm.c
> similarity index 100%
> rename from MdePkg/Library/UefiDevicePathLib/DevicePathUtilitiesStandaloneMm.c
> rename to MdePkg/Library/UefiDevicePathLib/DevicePathUtilitiesPeiStandaloneMm.c
> diff --git a/MdePkg/Library/UefiDevicePathLib/UefiDevicePathLibStandaloneMm.inf
> b/MdePkg/Library/UefiDevicePathLib/UefiDevicePathLibPeiStandaloneMm.inf
> similarity index 93%
> rename from MdePkg/Library/UefiDevicePathLib/UefiDevicePathLibStandaloneMm.inf
> rename to MdePkg/Library/UefiDevicePathLib/UefiDevicePathLibPeiStandaloneMm.inf
> index 23fedf38b7..2f39a29aa2 100644
> --- a/MdePkg/Library/UefiDevicePathLib/UefiDevicePathLibStandaloneMm.inf
> +++ b/MdePkg/Library/UefiDevicePathLib/UefiDevicePathLibPeiStandaloneMm.inf
> @@ -19,7 +19,7 @@
>    MODULE_TYPE                    = MM_STANDALONE
>    PI_SPECIFICATION_VERSION       = 0x00010032
>    VERSION_STRING                 = 1.0
> -  LIBRARY_CLASS                  = DevicePathLib | MM_STANDALONE MM_CORE_STANDALONE
> +  LIBRARY_CLASS                  = DevicePathLib | MM_STANDALONE MM_CORE_STANDALONE PEIM
> 
> 
>  #
> @@ -28,7 +28,7 @@
> 
>  [Sources]
>    DevicePathUtilities.c
> -  DevicePathUtilitiesStandaloneMm.c
> +  DevicePathUtilitiesPeiStandaloneMm.c
>    DevicePathToText.c
>    DevicePathFromText.c
>    UefiDevicePathLib.c
> diff --git a/MdePkg/MdePkg.dsc b/MdePkg/MdePkg.dsc
> index a94959169b..3345404cc1 100644
> --- a/MdePkg/MdePkg.dsc
> +++ b/MdePkg/MdePkg.dsc
> @@ -111,7 +111,7 @@
>    MdePkg/Library/UefiDebugLibDebugPortProtocol/UefiDebugLibDebugPortProtocol.inf
>    MdePkg/Library/UefiDebugLibStdErr/UefiDebugLibStdErr.inf
>    MdePkg/Library/UefiDevicePathLib/UefiDevicePathLib.inf
> -  MdePkg/Library/UefiDevicePathLib/UefiDevicePathLibStandaloneMm.inf
> +  MdePkg/Library/UefiDevicePathLib/UefiDevicePathLibPeiStandaloneMm.inf
>    MdePkg/Library/UefiDevicePathLib/UefiDevicePathLibOptionalDevicePathProtocol.inf
>    MdePkg/Library/UefiDevicePathLibDevicePathProtocol/UefiDevicePathLibDevicePathProtocol.inf
>    MdePkg/Library/UefiDriverEntryPoint/UefiDriverEntryPoint.inf
> --
> 2.28.0.windows.1
> 
> ---------------------------------------------------------------------
> Intel Technology Poland sp. z o.o.
> ul. Slowackiego 173 | 80-298 Gdansk | Sad Rejonowy Gdansk Polnoc | VII Wydzial Gospodarczy Krajowego Rejestru Sadowego -
> KRS 101882 | NIP 957-07-52-316 | Kapital zakladowy 200.000 PLN.
> Ta wiadomosc wraz z zalacznikami jest przeznaczona dla okreslonego adresata i moze zawierac informacje poufne. W razie
> przypadkowego otrzymania tej wiadomosci, prosimy o powiadomienie nadawcy oraz trwale jej usuniecie; jakiekolwiek
> przegladanie lub rozpowszechnianie jest zabronione.
> This e-mail and any attachments may contain confidential material for the sole use of the intended recipient(s). If you are
> not the intended recipient, please contact the sender and delete all copies; any review or distribution by others is
> strictly prohibited.
> 
> 
> 
> 
>