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: [email protected] <[email protected]> On Behalf Of Albecki, 
> Mateusz
> Sent: Tuesday, January 11, 2022 4:14 AM
> To: [email protected]
> Cc: Albecki, Mateusz <[email protected]>; Kinney, Michael D 
> <[email protected]>; Gao, Liming
> <[email protected]>; Liu, Zhiguang <[email protected]>
> 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 <[email protected]>
> 
> Cc: Michael D Kinney <[email protected]>
> Cc: Liming Gao <[email protected]>
> Cc: Zhiguang Liu <[email protected]>
> ---
>  ...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.
> 
> 
> 
> 
> 



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#89200): https://edk2.groups.io/g/devel/message/89200
Mute This Topic: https://groups.io/mt/88346995/21656
Group Owner: [email protected]
Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to