Star, I agree. I will integrate into a V6 patch series.
Thanks, Mike > -----Original Message----- > From: Zeng, Star > Sent: Wednesday, August 1, 2018 12:53 AM > To: Kinney, Michael D <michael.d.kin...@intel.com>; > edk2-devel@lists.01.org > Cc: Ni, Ruiyu <ruiyu...@intel.com>; Dong, Eric > <eric.d...@intel.com>; Zeng, Star <star.z...@intel.com> > Subject: RE: [Patch v5 21/21] MdeModulePkg/BdsDxe: Move > display of test key usage into BDS module > > A very minor comment. > > if (PcdGetBool (PcdTestKeyUsed)) > can be used directly instead of > if (PcdGetBool (PcdTestKeyUsed) == TRUE) > > With update, Reviewed-by: Star Zeng > <star.z...@intel.com>. > > > Thanks, > Star > -----Original Message----- > From: Kinney, Michael D > Sent: Wednesday, August 1, 2018 2:55 PM > To: edk2-devel@lists.01.org > Cc: Ni, Ruiyu <ruiyu...@intel.com>; Zeng, Star > <star.z...@intel.com>; Dong, Eric > <eric.d...@intel.com>; Kinney, Michael D > <michael.d.kin...@intel.com> > Subject: [Patch v5 21/21] MdeModulePkg/BdsDxe: Move > display of test key usage into BDS module > > Cc: Ruiyu Ni <ruiyu...@intel.com> > Cc: Star Zeng <star.z...@intel.com> > Cc: Eric Dong <eric.d...@intel.com> > Contributed-under: TianoCore Contribution Agreement 1.1 > Signed-off-by: Michael D Kinney > <michael.d.kin...@intel.com> > --- > MdeModulePkg/Universal/BdsDxe/BdsDxe.inf | 1 + > MdeModulePkg/Universal/BdsDxe/BdsEntry.c | 12 > ++++++++++++ > 2 files changed, 13 insertions(+) > > diff --git a/MdeModulePkg/Universal/BdsDxe/BdsDxe.inf > b/MdeModulePkg/Universal/BdsDxe/BdsDxe.inf > index 7e644aa995..7030d67907 100644 > --- a/MdeModulePkg/Universal/BdsDxe/BdsDxe.inf > +++ b/MdeModulePkg/Universal/BdsDxe/BdsDxe.inf > @@ -100,6 +100,7 @@ [Pcd] > gEfiMdeModulePkgTokenSpaceGuid.PcdFirmwareRevision > ## CONSUMES > > gEfiMdeModulePkgTokenSpaceGuid.PcdConInConnectOnDemand > ## CONSUMES > > gEfiMdeModulePkgTokenSpaceGuid.PcdErrorCodeSetVariable > ## SOMETIMES_CONSUMES > + gEfiMdeModulePkgTokenSpaceGuid.PcdTestKeyUsed > ## CONSUMES > > [Depex] > TRUE > diff --git a/MdeModulePkg/Universal/BdsDxe/BdsEntry.c > b/MdeModulePkg/Universal/BdsDxe/BdsEntry.c > index a25663ea43..b9ca31741e 100644 > --- a/MdeModulePkg/Universal/BdsDxe/BdsEntry.c > +++ b/MdeModulePkg/Universal/BdsDxe/BdsEntry.c > @@ -884,6 +884,18 @@ BdsEntry ( > > PERF_INMODULE_BEGIN("PlatformBootManagerAfterConsole"); > PlatformBootManagerAfterConsole (); > > PERF_INMODULE_END("PlatformBootManagerAfterConsole"); > + > + // > + // If any component set PcdTestKeyUsed to TRUE > because use of a test key > + // was detected, then display a warning message on > the debug log and the console > + // > + if (PcdGetBool (PcdTestKeyUsed) == TRUE) { > + DEBUG ((DEBUG_ERROR, > "**********************************\n")); > + DEBUG ((DEBUG_ERROR, "** WARNING: Test Key is > used. **\n")); > + DEBUG ((DEBUG_ERROR, > "**********************************\n")); > + Print (L"** WARNING: Test Key is used. **\n"); > + } > + > // > // Boot to Boot Manager Menu when > EFI_OS_INDICATIONS_BOOT_TO_FW_UI is set. Skip > HotkeyBoot > // > -- > 2.14.2.windows.3 _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel