Hi @Rebecca Cran<mailto:rebe...@bsdio.com>,
I create pull request here to downgrade the message level to DEBUG_INFO: https://github.com/tianocore/edk2/pull/5897 Regards, Nickle > -----Original Message----- > From: Rebecca Cran <rebe...@bsdio.com> > Sent: Wednesday, July 10, 2024 2:48 AM > To: Nickle Wang <nick...@nvidia.com>; Abner Chang <abner.ch...@amd.com>; > Igor Kulchytskyy <ig...@ami.com> > Cc: devel@edk2.groups.io > Subject: Re: RedfishPlatformConfigDxe needs Depex on > gEfiRegularExpressionProtocolGuid? > > External email: Use caution opening links or attachments > > > I'm ok keeping the current behavior as long as the DEBUG_ERROR message is > removed or downgraded to DEBUG_INFO. As it is, it makes it look as though the > driver has failed when error messages are highlighted during the boot. > > -- > Rebecca Cran > > On 7/8/24 21:53, Nickle Wang wrote: > >> 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<mailto:rebe...@bsdio.com>> > >> Sent: Tuesday, July 9, 2024 4:25 AM > >> To: Abner Chang <abner.ch...@amd.com<mailto:abner.ch...@amd.com>>; Nickle > >> Wang > >> <nick...@nvidia.com<mailto:nick...@nvidia.com>>; Igor Kulchytskyy > >> <ig...@ami.com<mailto:ig...@ami.com>> > >> Cc: devel@edk2.groups.io<mailto: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.in > >> +++ f > >> @@ -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 (#119861): https://edk2.groups.io/g/devel/message/119861 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] -=-=-=-=-=-=-=-=-=-=-=-
