> Should the following change be made to make sure RegularExpressionDxe is
> loaded before RedfishPlatformConfigDxe?

It is ok that RegularExpressionDxe is loaded after RedfishPlatformConfigDxe 
because EfiCreateProtocolNotifyEvent() is used to get Regular expression 
protocol. And I don't have objection to add the dependency as you suggested to 
eliminate the false alarm.

Regards,
Nickle

> -----Original Message-----
> From: Rebecca Cran <rebe...@bsdio.com>
> Sent: Tuesday, July 9, 2024 4:25 AM
> To: Abner Chang <abner.ch...@amd.com>; Nickle Wang <nick...@nvidia.com>;
> Igor Kulchytskyy <ig...@ami.com>
> Cc: devel@edk2.groups.io
> Subject: RedfishPlatformConfigDxe needs Depex on
> gEfiRegularExpressionProtocolGuid?
> 
> External email: Use caution opening links or attachments
> 
> 
> I noticed that RedfishPlatformConfigDxe prints an error message that it can't 
> find
> EFI_REGULAR_EXPRESSION_PROTOCOL.
> 
> Should the following change be made to make sure RegularExpressionDxe is
> loaded before RedfishPlatformConfigDxe?
> 
> diff --git
> a/RedfishPkg/RedfishPlatformConfigDxe/RedfishPlatformConfigDxe.inf
> b/RedfishPkg/RedfishPlatformConfigDxe/RedfishPlatformConfigDxe.inf
> index 2db47c2cf..7490d89be 100644
> --- a/RedfishPkg/RedfishPlatformConfigDxe/RedfishPlatformConfigDxe.inf
> +++ b/RedfishPkg/RedfishPlatformConfigDxe/RedfishPlatformConfigDxe.inf
> @@ -60,4 +60,4 @@
>    gEfiRedfishPkgTokenSpaceGuid.PcdRedfishPlatformConfigDebugProperty
> 
>   [Depex]
> -  TRUE
> +  gEfiRegularExpressionProtocolGuid^M
> 
> --
> Rebecca Cran


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#119825): https://edk2.groups.io/g/devel/message/119825
Mute This Topic: https://groups.io/mt/107110826/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to