Re: [edk2-devel] [PATCH] IntelFsp2Pkg: Adopt FSP 2.3 specification.

2021-10-03 Thread Chiu, Chasel
Patch pushed: https://github.com/tianocore/edk2/commit/4cc1458dbe004b1d70534caa55f475f6d19fe14a Thanks, Chasel > -Original Message- > From: Chiu, Chasel > Sent: Friday, October 1, 2021 4:29 PM > To: devel@edk2.groups.io > Cc: Chiu, Chasel ; Ma, Maurice > ; Desimone, Nathaniel L > ; Z

Re: [edk2-devel] [PATCH] IntelFsp2Pkg: Adopt FSP 2.3 specification.

2021-10-03 Thread Chiu, Chasel
Thanks Nate! I will remove it. > -Original Message- > From: Desimone, Nathaniel L > Sent: Saturday, October 2, 2021 6:15 AM > To: Chiu, Chasel ; devel@edk2.groups.io > Cc: Ma, Maurice ; Zeng, Star > Subject: RE: [PATCH] IntelFsp2Pkg: Adopt FSP 2.3 specification. > > Hi Chasel, > > My

Re: [edk2-devel] [PATCH] IntelFsp2Pkg: Adopt FSP 2.3 specification.

2021-10-01 Thread Nate DeSimone
Hi Chasel, My only feedback is let's not add the EDK1 style GUID definition to FspNonVolatileStorageHob2.h. It is redundant with the EDK2 style definition. Please see inline. With that change... Reviewed-by: Nate DeSimone > -Original Message- > From: Chiu, Chasel > Sent: Friday, Oct

[edk2-devel] [PATCH] IntelFsp2Pkg: Adopt FSP 2.3 specification.

2021-10-01 Thread Chiu, Chasel
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3674 Add ExtendedImageRevision in FSP_INFO_HEADER structure, also add FSP_NON_VOLATILE_STORAGE_HOB2 header. Cc: Maurice Ma Cc: Nate DeSimone Cc: Star Zeng Signed-off-by: Chasel Chiu --- IntelFsp2Pkg/Include/Guid/FspHeaderFile.h |