Also, please ensure your patch can pass tiano CI. I cannot find the PR to CI for those features. Would you please point to me?
Thank you Yao Jiewen > -----Original Message----- > From: Vang, Judah <judah.v...@intel.com> > Sent: Tuesday, November 8, 2022 2:45 AM > To: Yao, Jiewen <jiewen....@intel.com>; devel@edk2.groups.io > Subject: RE: [edk2-devel] [PATCH v2 0/2] CryptoPkg bug fixes > > Sure, I can do that. I will resubmit as separate patches. > > -----Original Message----- > From: Yao, Jiewen <jiewen....@intel.com> > Sent: Monday, November 7, 2022 10:42 AM > To: devel@edk2.groups.io; Vang, Judah <judah.v...@intel.com> > Subject: RE: [edk2-devel] [PATCH v2 0/2] CryptoPkg bug fixes > > Hey > Would you please split this patch set to two different one? They are two > different HSDs. > > Please aware that we are in software freeze phase now. > > I suggest we include 3991 in this release, because it is an important bug fix. > > I suggest we defer 3992 to next release, because it is feature enhancement. > > Comment is welcome! > > Thank you > Yao, Jiewen > > > > -----Original Message----- > > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Judah > > Vang > > Sent: Tuesday, November 8, 2022 2:37 AM > > To: devel@edk2.groups.io > > Subject: [edk2-devel] [PATCH v2 0/2] CryptoPkg bug fixes > > > > https://bugzilla.tianocore.org/show_bug.cgi?id=3991 > > https://bugzilla.tianocore.org/show_bug.cgi?id=3992 > > > > There is a #define to deprecate Sha1 functions but not all the Sha1 > > function are wrapped around this #define causing a build error. The > > fix is to wrap all Sha1 functions with the #define. > > > > Need crypto AES to be supported for PEI phase and need crypto KDF to > > be supported for SMM phase. Update Readme to show AES and HKDF > > defaults. > > > > Judah Vang (2): > > CryptoPkg: Sha1 functions causing build errors > > CryptoPkg: Need to enable crypto functions > > > > CryptoPkg/Library/BaseCryptLib/PeiCryptLib.inf | 2 +- > > CryptoPkg/Library/BaseCryptLib/SmmCryptLib.inf | 2 +- > > CryptoPkg/Library/BaseHashApiLib/BaseHashApiLib.c | 14 ++++++++++- > > CryptoPkg/Readme.md | 26 +++++++++++--------- > > 4 files changed, 29 insertions(+), 15 deletions(-) > > > > -- > > 2.35.1.windows.2 > > > > > > > > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#96046): https://edk2.groups.io/g/devel/message/96046 Mute This Topic: https://groups.io/mt/94873522/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
