[edk2-devel] [edk2-platforms: PATCH v2 5/9] KabylakeOpenBoardPkg/KabylakeRvp3: Use same variable name for FspNvsHob.

2021-10-07 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 To simplify the implementation the variable Name/GUID has been changed to "FspNvsBuffer" and gFspNvsBufferVariableGuid regardless it stores the data from FSP_NON_VOLATILE_STORAGE_HOB2 or FSP_NON_VOLATILE_STORAGE_HOB. Cc: Nate DeSimone

[edk2-devel] [edk2-platforms: PATCH v2 6/9] TigerlakeOpenBoardPkg: Use same variable name for FspNvsHob.

2021-10-07 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 To simplify the implementation the variable Name/GUID has been changed to "FspNvsBuffer" and gFspNvsBufferVariableGuid regardless it stores the data from FSP_NON_VOLATILE_STORAGE_HOB2 or FSP_NON_VOLATILE_STORAGE_HOB. Cc: Sai Chaganty Cc:

[edk2-devel] [edk2-platforms: PATCH v2 8/9] WhitleyOpenBoardPkg: Support FSP 2.3 FSP_NON_VOLATILE_STORAGE_HOB2.

2021-10-07 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 Implementation should search FSP_NON_VOLATILE_STORAGE_HOB2 firstly and only search FSP_NON_VOLATILE_STORAGE_HOB when former one is not found. Also added PeiGetLargeVariable () to support the scenarios where the variable data size is bigger

[edk2-devel] [edk2-platforms: PATCH v2 9/9] WhitleySiliconPkg: Use same variable name for FspNvsHob.

2021-10-07 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 To simplify the implementation the variable Name/GUID has been changed to "FspNvsBuffer" and gFspNvsBufferVariableGuid regardless it stores the data from FSP_NON_VOLATILE_STORAGE_HOB2 or FSP_NON_VOLATILE_STORAGE_HOB. Cc: Isaac Oram Cc:

[edk2-devel] [edk2-platforms: PATCH v2 7/9] WhiskeylakeOpenBoardPkg: Use same variable name for FspNvsHob.

2021-10-07 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 To simplify the implementation the variable Name/GUID has been changed to "FspNvsBuffer" and gFspNvsBufferVariableGuid regardless it stores the data from FSP_NON_VOLATILE_STORAGE_HOB2 or FSP_NON_VOLATILE_STORAGE_HOB. Cc: Nate DeSimone

[edk2-devel] [edk2-platforms: PATCH v4 9/9] WhitleySiliconPkg: Use same variable name for FspNvsHob.

2021-10-14 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 To simplify the implementation the variable Name/GUID has been changed to "FspNvsBuffer" and gFspNvsBufferVariableGuid regardless it stores the data from FSP_NON_VOLATILE_STORAGE_HOB2 or FSP_NON_VOLATILE_STORAGE_HOB. Cc: Isaac Oram Cc:

[edk2-devel] [edk2-platforms: PATCH v4 7/9] WhiskeylakeOpenBoardPkg: Use same variable name for FspNvsHob.

2021-10-14 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 To simplify the implementation the variable Name/GUID has been changed to "FspNvsBuffer" and gFspNvsBufferVariableGuid regardless it stores the data from FSP_NON_VOLATILE_STORAGE_HOB2 or FSP_NON_VOLATILE_STORAGE_HOB. Cc: Nate DeSimone

[edk2-devel] [edk2-platforms: PATCH v4 8/9] WhitleyOpenBoardPkg: Support FSP 2.3 FSP_NON_VOLATILE_STORAGE_HOB2.

2021-10-14 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 Implementation should search FSP_NON_VOLATILE_STORAGE_HOB2 firstly and only search FSP_NON_VOLATILE_STORAGE_HOB when former one is not found. Also added PeiGetLargeVariable () to support the scenarios where the variable data size is bigger

[edk2-devel] [edk2-platforms: PATCH v4 5/9] KabylakeOpenBoardPkg/KabylakeRvp3: Use same variable name for FspNvsHob.

2021-10-14 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 To simplify the implementation the variable Name/GUID has been changed to "FspNvsBuffer" and gFspNvsBufferVariableGuid regardless it stores the data from FSP_NON_VOLATILE_STORAGE_HOB2 or FSP_NON_VOLATILE_STORAGE_HOB. Cc: Nate DeSimone

[edk2-devel] [edk2-platforms: PATCH v4 0/9] MinPlatformPkg: Support FSP 2.3 FSP_NON_VOLATILE_STORAGE_HOB2.

2021-10-14 Thread Chiu, Chasel
Esakkithevar Cc: Liming Gao Cc: Eric Dong Signed-off-by: Chasel Chiu Chasel Chiu (9): MinPlatformPkg: Support FSP 2.3 FSP_NON_VOLATILE_STORAGE_HOB2. CometlakeOpenBoardPkg: Use same variable name for FspNvsHob. KabylakeOpenBoardPkg/AspireVn7Dash572G:Use same variable name for FspNvsHob

[edk2-devel] [edk2-platforms: PATCH v4 1/9] MinPlatformPkg: Support FSP 2.3 FSP_NON_VOLATILE_STORAGE_HOB2.

2021-10-14 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 Implementation should search FSP_NON_VOLATILE_STORAGE_HOB2 firstly and only search FSP_NON_VOLATILE_STORAGE_HOB when former one is not found. Also added PeiGetLargeVariable () to support the scenarios where the variable data size is bigger

[edk2-devel] [edk2-platforms: PATCH v4 2/9] CometlakeOpenBoardPkg: Use same variable name for FspNvsHob.

2021-10-14 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 To simplify the implementation the variable Name/GUID has been changed to "FspNvsBuffer" and gFspNvsBufferVariableGuid regardless it stores the data from FSP_NON_VOLATILE_STORAGE_HOB2 or FSP_NON_VOLATILE_STORAGE_HOB. Cc: Nate DeSimone Cc:

[edk2-devel] [edk2-platforms: PATCH v4 3/9] KabylakeOpenBoardPkg/AspireVn7Dash572G:Use same variable name for FspNvsHob

2021-10-14 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 To simplify the implementation the variable Name/GUID has been changed to "FspNvsBuffer" and gFspNvsBufferVariableGuid regardless it stores the data from FSP_NON_VOLATILE_STORAGE_HOB2 or FSP_NON_VOLATILE_STORAGE_HOB. Cc: Nate DeSimone Cc:

[edk2-devel] [edk2-platforms: PATCH v4 4/9] KabylakeOpenBoardPkg/GalagoPro3: Use same variable name for FspNvsHob.

2021-10-14 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 To simplify the implementation the variable Name/GUID has been changed to "FspNvsBuffer" and gFspNvsBufferVariableGuid regardless it stores the data from FSP_NON_VOLATILE_STORAGE_HOB2 or FSP_NON_VOLATILE_STORAGE_HOB. Cc: Nate DeSimone Cc:

[edk2-devel] [edk2-platforms: PATCH v4 6/9] TigerlakeOpenBoardPkg: Use same variable name for FspNvsHob.

2021-10-14 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 To simplify the implementation the variable Name/GUID has been changed to "FspNvsBuffer" and gFspNvsBufferVariableGuid regardless it stores the data from FSP_NON_VOLATILE_STORAGE_HOB2 or FSP_NON_VOLATILE_STORAGE_HOB. Cc: Sai Chaganty Cc:

Re: [edk2-devel] [edk2-platforms: PATCH v3 0/9] MinPlatformPkg: Support FSP 2.3 FSP_NON_VOLATILE_STORAGE_HOB2.

2021-10-14 Thread Chiu, Chasel
Hello, Thanks for all your valuable feedbacks, I have sent V4 to incorporate all of them, please help to review. Thanks, Chasel > -Original Message- > From: Chiu, Chasel > Sent: Friday, October 8, 2021 2:43 PM > To: devel@edk2.groups.io > Cc: Chiu, Chasel ; Oram, Isaac

[edk2-devel] [edk2-platforms: PATCH v5 0/9] MinPlatformPkg: Support FSP 2.3 FSP_NON_VOLATILE_STORAGE_HOB2.

2021-10-14 Thread Chiu, Chasel
: Rangasai V Chaganty Cc: Deepika Kethi Reddy Cc: Kathappan Esakkithevar Cc: Liming Gao Cc: Eric Dong Signed-off-by: Chasel Chiu Chasel Chiu (9): MinPlatformPkg: Support FSP 2.3 FSP_NON_VOLATILE_STORAGE_HOB2. CometlakeOpenBoardPkg: Use same variable name for FspNvsHob. KabylakeOpenBoardPkg

[edk2-devel] [edk2-platforms: PATCH v5 1/9] MinPlatformPkg: Support FSP 2.3 FSP_NON_VOLATILE_STORAGE_HOB2.

2021-10-14 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 Implementation should search FSP_NON_VOLATILE_STORAGE_HOB2 firstly and only search FSP_NON_VOLATILE_STORAGE_HOB when former one is not found. Also added PeiGetLargeVariable () to support the scenarios where the variable data size is bigger

[edk2-devel] [edk2-platforms: PATCH v5 2/9] CometlakeOpenBoardPkg: Use same variable name for FspNvsHob.

2021-10-14 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 To simplify the implementation the variable Name/GUID has been changed to "FspNvsBuffer" and gFspNvsBufferVariableGuid regardless it stores the data from FSP_NON_VOLATILE_STORAGE_HOB2 or FSP_NON_VOLATILE_STORAGE_HOB. Cc: Nate DeSimone Cc:

[edk2-devel] [edk2-platforms: PATCH v5 3/9] KabylakeOpenBoardPkg/AspireVn7Dash572G:Use same variable name for FspNvsHob

2021-10-14 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 To simplify the implementation the variable Name/GUID has been changed to "FspNvsBuffer" and gFspNvsBufferVariableGuid regardless it stores the data from FSP_NON_VOLATILE_STORAGE_HOB2 or FSP_NON_VOLATILE_STORAGE_HOB. Cc: Nate DeSimone Cc:

[edk2-devel] [edk2-platforms: PATCH v5 4/9] KabylakeOpenBoardPkg/GalagoPro3: Use same variable name for FspNvsHob.

2021-10-14 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 To simplify the implementation the variable Name/GUID has been changed to "FspNvsBuffer" and gFspNvsBufferVariableGuid regardless it stores the data from FSP_NON_VOLATILE_STORAGE_HOB2 or FSP_NON_VOLATILE_STORAGE_HOB. Cc: Nate DeSimone Cc:

[edk2-devel] [edk2-platforms: PATCH v5 5/9] KabylakeOpenBoardPkg/KabylakeRvp3: Use same variable name for FspNvsHob.

2021-10-14 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 To simplify the implementation the variable Name/GUID has been changed to "FspNvsBuffer" and gFspNvsBufferVariableGuid regardless it stores the data from FSP_NON_VOLATILE_STORAGE_HOB2 or FSP_NON_VOLATILE_STORAGE_HOB. Cc: Nate DeSimone

[edk2-devel] [edk2-platforms: PATCH v5 6/9] TigerlakeOpenBoardPkg: Use same variable name for FspNvsHob.

2021-10-14 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 To simplify the implementation the variable Name/GUID has been changed to "FspNvsBuffer" and gFspNvsBufferVariableGuid regardless it stores the data from FSP_NON_VOLATILE_STORAGE_HOB2 or FSP_NON_VOLATILE_STORAGE_HOB. Cc: Sai Chaganty Cc:

[edk2-devel] [edk2-platforms: PATCH v5 7/9] WhiskeylakeOpenBoardPkg: Use same variable name for FspNvsHob.

2021-10-14 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 To simplify the implementation the variable Name/GUID has been changed to "FspNvsBuffer" and gFspNvsBufferVariableGuid regardless it stores the data from FSP_NON_VOLATILE_STORAGE_HOB2 or FSP_NON_VOLATILE_STORAGE_HOB. Cc: Nate DeSimone

[edk2-devel] [edk2-platforms: PATCH v5 8/9] WhitleyOpenBoardPkg: Support FSP 2.3 FSP_NON_VOLATILE_STORAGE_HOB2.

2021-10-14 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 Implementation should search FSP_NON_VOLATILE_STORAGE_HOB2 firstly and only search FSP_NON_VOLATILE_STORAGE_HOB when former one is not found. Also added PeiGetLargeVariable () to support the scenarios where the variable data size is bigger

[edk2-devel] [edk2-platforms: PATCH v5 9/9] WhitleySiliconPkg: Use same variable name for FspNvsHob.

2021-10-14 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 To simplify the implementation the variable Name/GUID has been changed to "FspNvsBuffer" and gFspNvsBufferVariableGuid regardless it stores the data from FSP_NON_VOLATILE_STORAGE_HOB2 or FSP_NON_VOLATILE_STORAGE_HOB. Cc: Isaac Oram Cc:

Re: [edk2-devel] [edk2-platforms: PATCH v5 0/9] MinPlatformPkg: Support FSP 2.3 FSP_NON_VOLATILE_STORAGE_HOB2.

2021-10-14 Thread Chiu, Chasel
age- > From: Chiu, Chasel > Sent: Thursday, October 14, 2021 5:16 PM > To: devel@edk2.groups.io > Cc: Chiu, Chasel ; Oram, Isaac W > ; Desimone, Nathaniel L > ; Luo, Heng ; Jeremy > Soller ; Benjamin Doron > ; Chaganty, Rangasai V > ; Kethi Reddy, Deepika > ; Esakk

Re: [edk2-devel] [PATCH] IntelFsp2Pkg: Add support for config editor to handle multiple UPD

2021-10-17 Thread Chiu, Chasel
l L > ; Zeng, Star ; Chiu, > Chasel > Subject: [PATCH] IntelFsp2Pkg: Add support for config editor to handle > multiple > UPD > > BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=3692 > > In several use cases in bootloader, there are multiple instances of UPD with >

Re: [edk2-devel] [PATCH v7] IntelFsp2WrapperPkg : FSPM/S UPD data address based on Build Type

2021-09-26 Thread Chiu, Chasel
Reviewed-by: Chasel Chiu > -Original Message- > From: S, Ashraf Ali > Sent: Friday, September 24, 2021 7:43 PM > To: devel@edk2.groups.io > Cc: S, Ashraf Ali ; Chiu, Chasel > ; > Desimone, Nathaniel L ; Zeng, Star > ; Kuo, Ted ; Duggapu, Chinni B > ; Chaga

Re: [edk2-devel] [edk2-platforms] [PATCH V1] KabylakeOpenBoardPkg: Fix Build

2021-09-28 Thread Chiu, Chasel
Reviewed-by: Chasel Chiu > -Original Message- > From: Desimone, Nathaniel L > Sent: Wednesday, September 29, 2021 9:03 AM > To: devel@edk2.groups.io > Cc: Chiu, Chasel > Subject: [edk2-platforms] [PATCH V1] KabylakeOpenBoardPkg: Fix Build > > Commit

Re: [edk2-devel] [edk2-platforms] [PATCH V1] KabylakeOpenBoardPkg/AspireVn7Dash572G: Fix Visual Studio Build

2021-09-28 Thread Chiu, Chasel
Reviewed-by: Chasel Chiu > -Original Message- > From: Desimone, Nathaniel L > Sent: Wednesday, September 29, 2021 9:03 AM > To: devel@edk2.groups.io > Cc: Chiu, Chasel ; Benjamin Doron > > Subject: [edk2-platforms] [PATCH V1] > KabylakeOpenBoardPkg/Aspir

Re: [edk2-devel] [PATCH v1] IntelFsp2WrapperPkg: Make PcdFspModeSelection dynamic and set it accordingly

2021-09-22 Thread Chiu, Chasel
Patch pushed: https://github.com/tianocore/edk2/commit/f334c5a41dc4702b1c767c6614025aa2f802f9a4 Thanks, Chasel > -Original Message- > From: Zhang, Xiaoqiang > Sent: Thursday, September 16, 2021 11:45 AM > To: devel@edk2.groups.io > Cc: Zhang, Xiaoqiang ; Chiu, Chas

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 specificati

[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

Re: [edk2-devel] [PATCH v8] IntelFsp2WrapperPkg : FSPM/S UPD data address based on Build Type

2021-12-19 Thread Chiu, Chasel
Patch pushed: https://github.com/tianocore/edk2/commit/de9e5b7dc721d4ca319c0455cf83577347e0abef Thanks, Chasel > -Original Message- > From: S, Ashraf Ali > Sent: Thursday, December 16, 2021 4:10 PM > To: devel@edk2.groups.io > Cc: S, Ashraf Ali ; Chiu, Chasel

[edk2-devel] [PATCH] IntelFsp2Pkg/FspSecCore: ExtendedImageRevision was not printed.

2021-12-28 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3791 ExtendedImageRevision should be printed when Header revision >= 6. Cc: Maurice Ma Cc: Nate DeSimone Cc: Star Zeng Signed-off-by: Chasel Chiu --- IntelFsp2Pkg/FspSecCore/SecFsp.c | 10 +++--- 1 file changed, 7 insertions(+), 3

Re: [edk2-devel] [PATCH v8] IntelFsp2WrapperPkg : FSPM/S UPD data address based on Build Type

2021-12-20 Thread Chiu, Chasel
Thanks Ray! I have sent a code review for removing EFIAPI, please help to review. Thanks, Chasel > -Original Message- > From: Ni, Ray > Sent: Monday, December 20, 2021 1:15 PM > To: S, Ashraf Ali ; devel@edk2.groups.io > Cc: Chiu, Chasel ; Desimone, Nathaniel L >

[edk2-devel] [PATCH] IntelFsp2WrapperPkg : Remove EFIAPI from local functions.

2021-12-20 Thread Chiu, Chasel
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3642 Local functions do not need EFIAPI. Cc: Nate DeSimone Cc: Star Zeng Cc: Ray Ni Cc: Ashraf Ali S Signed-off-by: Chasel Chiu --- IntelFsp2WrapperPkg/FspmWrapperPeim/FspmWrapperPeim.c | 1 -

Re: [edk2-devel] [PATCH] IntelFsp2Pkg/FspSecCore: ExtendedImageRevision was not printed.

2021-12-28 Thread Chiu, Chasel
Patch pushed: https://github.com/tianocore/edk2/commit/7935be0fbd8f47266e5972f4cba1a1e58505061a Thanks, Chasel > -Original Message- > From: devel@edk2.groups.io On Behalf Of Chiu, Chasel > Sent: Tuesday, December 28, 2021 6:58 PM > To: devel@edk2.groups.io > Cc: Ch

Re: [edk2-devel] [PATCH] IntelFsp2WrapperPkg : Remove EFIAPI from local functions.

2021-12-22 Thread Chiu, Chasel
Patch pushed: https://github.com/tianocore/edk2/commit/9ec2cc1f3138ef9314314df29fc6ce0aa5e120b9 Thanks, Chasel > -Original Message- > From: devel@edk2.groups.io On Behalf Of Chiu, Chasel > Sent: Tuesday, December 21, 2021 8:40 AM > To: devel@edk2.groups.io > C

Re: [edk2-devel] [PATCH] IntelFsp2Pkg/SplitFspBin.py: adopt FSP 2.3 specification.

2021-10-27 Thread Chiu, Chasel
Patch pushed: https://github.com/tianocore/edk2/commit/9a95d11023ac2f2ee49a2958cf80658270442c42 Thanks, Chasel > -Original Message- > From: devel@edk2.groups.io On Behalf Of Chiu, Chasel > Sent: Tuesday, October 26, 2021 4:06 PM > To: devel@edk2.groups.io > Cc: Ch

Re: [edk2-devel] [PATCH] IntelFsp2Pkg SplitFspBin.py: Correct file name in file header

2021-10-27 Thread Chiu, Chasel
Thanks Star. Reviewed-by: Chasel Chiu > -Original Message- > From: Zeng, Star > Sent: Wednesday, October 27, 2021 8:00 PM > To: devel@edk2.groups.io > Cc: Zeng, Star ; Ma, Maurice ; > Desimone, Nathaniel L ; Chiu, Chasel > > Subject: [PATCH] IntelFsp2Pkg Spli

Re: [edk2-devel] [PATCH] IntelFsp2Pkg SplitFspBin.py: Correct file name in file header

2021-10-31 Thread Chiu, Chasel
imone, Nathaniel L ; Chiu, Chasel > > Subject: [PATCH] IntelFsp2Pkg SplitFspBin.py: Correct file name in file header > > Cc: Maurice Ma > Cc: Nate DeSimone > Cc: Chasel Chiu > Signed-off-by: Star Zeng > --- > IntelFsp2Pkg/Tools/SplitFspBin.py | 2 +- > 1 file change

[edk2-devel] [PATCH] IntelFsp2Pkg/SplitFspBin.py: adopt FSP 2.3 specification.

2021-10-26 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3705 FSP 2.3 has updated FSP_INFO_HEADER to support ExtendedImageRevision and SplitFspBin.py needs to support it. Also updated script to display integer value basing on length. Cc: Maurice Ma Cc: Nate DeSimone Cc: Star Zeng Signed-off-by:

Re: [edk2-devel] [PATCH v8] IntelFsp2WrapperPkg : FSPM/S UPD data address based on Build Type

2021-12-16 Thread Chiu, Chasel
M > To: S, Ashraf Ali ; devel@edk2.groups.io > Cc: Chiu, Chasel ; Desimone, Nathaniel L > ; Zeng, Star ; Duggapu, > Chinni B ; Chaganty, Rangasai V > ; Solanki, Digant H > ; V, Sangeetha ; Ni, Ray > > Subject: RE: [PATCH v8] IntelFsp2WrapperPkg : FSPM/S UPD data address base

Re: [edk2-devel] [PATCH v8] IntelFsp2WrapperPkg : FSPM/S UPD data address based on Build Type

2021-12-16 Thread Chiu, Chasel
Thanks Ashraf! Reviewed-by: Chasel Chiu > -Original Message- > From: S, Ashraf Ali > Sent: Thursday, December 16, 2021 4:10 PM > To: devel@edk2.groups.io > Cc: S, Ashraf Ali ; Chiu, Chasel > ; > Desimone, Nathaniel L ; Zeng, Star > ; Kuo, Ted ; Duggap

Re: [edk2-devel] Uncrustify Conversion Detailed Plan and Extended Hard Freeze Update #4

2021-12-06 Thread Chiu, Chasel
For below 2 packages, Reviewed-by: Chasel Chiu Thanks, Chasel * IntelFsp2Pkg * IntelFsp2WrapperPkg > -Original Message- > From: devel@edk2.groups.io On Behalf Of Michael D > Kinney > Sent: Monday, December 6, 2021 9:18 AM > To: Michael Kubacki ; devel@edk2.groups.io; >

Re: [edk2-devel] Uncrustify Conversion Detailed Plan and Extended Hard Freeze Update #4

2021-11-30 Thread Chiu, Chasel
For below patches, Reviewed-by: Chasel Chiu Thanks, Chasel IntelFsp2Pkg: Apply uncrusitify changes IntelFsp2WrapperPkg: Apply uncrusitify changes > -Original Message- > From: devel@edk2.groups.io On Behalf Of Michael D > Kinney > Sent: Wednesday, December 1, 2021 6:34 AM > To:

Re: [edk2-devel] [edk2-platforms] [PATCH V1] WhitleySiliconPkg: Improve comments for silicon policy structures

2021-07-20 Thread Chiu, Chasel
Hi Nate, Just one small feedbacks inline, please check them. With that resolved: Reviewed-by: Chasel Chiu Thanks, Chasel > -Original Message- > From: Desimone, Nathaniel L > Sent: Wednesday, July 21, 2021 4:22 AM > To: devel@edk2.groups.io > Cc: Oram, Isaac W

Re: [edk2-devel] [edk2-platforms][PATCH V1] KabylakeOpenBoardPkg: Implement the PEI Reset Services

2021-07-20 Thread Chiu, Chasel
Reviewed-by: Chasel Chiu > -Original Message- > From: Desimone, Nathaniel L > Sent: Tuesday, July 13, 2021 5:27 PM > To: devel@edk2.groups.io > Cc: Chiu, Chasel ; Jeremy Soller > ; Michael Kubacki ; > Benjamin Doron > Subject: [edk2-platforms][PATCH

Re: [edk2-devel] [edk2-platforms: PATCH] Features/Intel/IpmiFeaturePkg: Use MdePkg macros instead of redefining.

2021-07-21 Thread Chiu, Chasel
Patch pushed: d549e39ca1a9da14d86ff841358f990a0ace71f5 Thanks, Chasel > -Original Message- > From: devel@edk2.groups.io On Behalf Of Chiu, > Chasel > Sent: Thursday, July 15, 2021 10:38 PM > To: devel@edk2.groups.io > Cc: Chiu, Chasel ; Desimone, Nathaniel L > ;

Re: [edk2-devel] [PATCH v2] MinPlatformPkg/PciHostBridgeLibSimple: Fix Mem.Limit assignment

2021-07-22 Thread Chiu, Chasel
Thanks for fixing this issue! Reviewed-by: Chasel Chiu > -Original Message- > From: Benjamin Doron > Sent: Friday, July 23, 2021 10:27 AM > To: devel@edk2.groups.io > Cc: Chiu, Chasel ; Desimone, Nathaniel L > ; Liming Gao > ; Dong, Eric > Subject: [

Re: [edk2-devel] [edk2-platforms: PATCH v3] MinPlatformPkg/SaveMemoryConfig: Variable may not be locked.

2022-02-13 Thread Chiu, Chasel
Thanks Isaac! I will correct them when pushing the patch. > -Original Message- > From: Oram, Isaac W > Sent: Saturday, February 12, 2022 8:48 AM > To: devel@edk2.groups.io; Chiu, Chasel > Cc: Desimone, Nathaniel L ; Gao, Liming > ; Dong, Eric > Subject: R

Re: [edk2-devel][PATCH v2] IntelFsp2Pkg: BaseFspSwitchStackLib Support for X64

2022-03-14 Thread Chiu, Chasel
Thanks Ted for updating patch! Looks good to me. Reviewed-by: Chasel Chiu > -Original Message- > From: Kuo, Ted > Sent: Monday, March 14, 2022 10:48 AM > To: devel@edk2.groups.io > Cc: Chiu, Chasel ; Desimone, Nathaniel L > ; Zeng, Star ; S, Ashraf > Ali &

Re: [edk2-devel][PATCH v2] IntelFsp2Pkg: BaseFspDebugLibSerialPort Support for X64

2022-03-14 Thread Chiu, Chasel
Thanks Ted for updating patch! Reviewed-by: Chasel Chiu > -Original Message- > From: Kuo, Ted > Sent: Monday, March 14, 2022 10:33 AM > To: devel@edk2.groups.io > Cc: Chiu, Chasel ; Desimone, Nathaniel L > ; Zeng, Star ; S, Ashraf > Ali > Subject: [edk2-devel

Re: [edk2-devel] [edk2-platforms: PATCH] MinPlatformPkg/SaveMemoryConfig: Fix GCC build failure.

2022-02-17 Thread Chiu, Chasel
Fix has pushed: https://github.com/tianocore/edk2-platforms/commit/c398756604f54515da7af75a3f60a0380429a78c Thanks, Chasel > -Original Message- > From: devel@edk2.groups.io On Behalf Of Chiu, Chasel > Sent: Friday, February 18, 2022 10:52 AM > To: devel@edk2.groups.i

Re: [edk2-devel] [edk2-platforms: PATCH v4] MinPlatformPkg/SaveMemoryConfig: Variable may not be locked.

2022-02-17 Thread Chiu, Chasel
Fix pushed: https://github.com/tianocore/edk2-platforms/commit/cbc8e420ac4505e9c51aa0d4f049026691024ca5 Thanks, Chasel > -Original Message- > From: devel@edk2.groups.io On Behalf Of Chiu, Chasel > Sent: Friday, February 18, 2022 7:56 AM > To: Desimone, Nathaniel L ;

[edk2-devel] [edk2-platforms: PATCH] MinPlatformPkg/SaveMemoryConfig: Fix GCC build failure.

2022-02-17 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3829 Commit cbc8e420ac4505e9c51aa0d4f049026691024ca5 caused GCC build failure which should be fixed. Cc: Nate DeSimone Cc: Liming Gao Cc: Eric Dong Signed-off-by: Chasel Chiu ---

Re: [edk2-devel] [edk2-platforms: PATCH v4] MinPlatformPkg/SaveMemoryConfig: Variable may not be locked.

2022-02-17 Thread Chiu, Chasel
Thanks Nate, Isaac for good catch! I will correct them before pushing. > -Original Message- > From: Desimone, Nathaniel L > Sent: Friday, February 18, 2022 7:22 AM > To: devel@edk2.groups.io; Chiu, Chasel > Cc: Gao, Liming ; Dong, Eric > ; Oram, Isaac W > Sub

[edk2-devel] [edk2-platforms: PATCH v4] MinPlatformPkg/SaveMemoryConfig: Variable may not be locked.

2022-02-16 Thread Chiu, Chasel
From: "Chiu, Chasel" REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3829 Fixed the bug that existing variable will not be locked when it is identical with hob data by creating LockLargeVariable function, also switched to VariablePolicyProtocol for locking variables. Faili

Re: [edk2-devel] [PATCH] IntelFsp2Pkg: BaseFspSwitchStackLib Support for X64 Build

2022-02-21 Thread Chiu, Chasel
nt: Monday, February 14, 2022 4:38 PM > To: devel@edk2.groups.io; S, Ashraf Ali > Cc: Chiu, Chasel ; Desimone, Nathaniel L > ; Zeng, Star ; Kuo, Ted > ; Duggapu, Chinni B ; > Chaganty, Rangasai V ; Solanki, Digant H > ; V, Sangeetha > Subject: RE: [edk2-devel] [PATCH] IntelFsp2Pkg: Ba

Re: [edk2-devel] [PATCH] IntelFsp2Pkg: BaseFspDebugLibSerialPort Support for X64 Build

2022-02-21 Thread Chiu, Chasel
Thanks Ray! @S, Ashraf Ali I think we need to return 64bit pointer in 64bit build, please help to verify and confirm. Thanks, Chasel > -Original Message- > From: Ni, Ray > Sent: Monday, February 14, 2022 4:27 PM > To: devel@edk2.groups.io; S, Ashraf Ali > C

Re: [edk2-devel] [PATCH v1 3/7] IntelFsp2Pkg: Invoke GetPhysicalAddressBits() and remove the duplicated code

2022-03-02 Thread Chiu, Chasel
Thanks Yu! Reviewed-by: Chasel Chiu > -Original Message- > From: Pu, Yu > Sent: Wednesday, March 2, 2022 5:19 PM > To: devel@edk2.groups.io > Cc: Pu, Yu ; Chiu, Chasel > Subject: [PATCH v1 3/7] IntelFsp2Pkg: Invoke GetPhysicalAddressBits() and > remove the dup

Re: [edk2-devel] [edk2-platforms][PATCH v1 1/1] MinPlatformPkg/TestPointCheckLib: Fix mis-parsed HSTI structures

2022-03-23 Thread Chiu, Chasel
Reviewed-by: Chasel Chiu > -Original Message- > From: mikub...@linux.microsoft.com > Sent: Wednesday, March 23, 2022 9:11 AM > To: devel@edk2.groups.io > Cc: Bret Barkelew ; Chiu, Chasel > ; Desimone, Nathaniel L > ; Gao, Liming > ; Dong, Eric > Subject:

Re: [edk2-devel] [edk2-platforms][PATCH v1 1/1] MinPlatformPkg: Add missing libraries to [Components] in DSC

2022-03-23 Thread Chiu, Chasel
Reviewed-by: Chasel Chiu > -Original Message- > From: mikub...@linux.microsoft.com > Sent: Wednesday, March 23, 2022 9:28 AM > To: devel@edk2.groups.io > Cc: Chiu, Chasel ; Desimone, Nathaniel L > ; Gao, Liming > ; Dong, Eric > Subject: [edk2-platforms][PATC

Re: [edk2-devel][PATCH] IntelFsp2Pkg: BaseFspCommonLib Support for X64

2022-03-22 Thread Chiu, Chasel
Thanks Ted! Reviewed-by: Chasel Chiu > -Original Message- > From: Kuo, Ted > Sent: Monday, March 14, 2022 12:23 PM > To: devel@edk2.groups.io > Cc: Chiu, Chasel ; Desimone, Nathaniel L > ; Zeng, Star ; S, > Ashraf Ali > Subject: [edk2-devel][PATCH] IntelF

Re: [edk2-devel][PATCH] IntelFsp2Pkg: BaseFspCommonLib Support for X64

2022-03-22 Thread Chiu, Chasel
Patch pushed: https://github.com/tianocore/edk2/commit/ec0b54849b23efa25caf0055b0eef8bf9b4dec98 Thanks, Chasel > -Original Message- > From: Kuo, Ted > Sent: Monday, March 14, 2022 12:23 PM > To: devel@edk2.groups.io > Cc: Chiu, Chasel ; Desimone, Nathaniel L >

[edk2-devel] [edk2-platforms: PATCH] MinPlatformPkg/SaveMemoryConfig: Variable may not be locked.

2022-02-08 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3829 Fixed the bug that existing variable will not be locked when it is identical with hob data, also switch to VariablePolicyProtocol for locking variables. This patch also updated SaveMemoryConfig driver to be unloaded after execution because

[edk2-devel] [edk2-platforms: PATCH v3] MinPlatformPkg/SaveMemoryConfig: Variable may not be locked.

2022-02-11 Thread Chiu, Chasel
From: "Chiu, Chasel" REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3829 Fixed the bug that existing variable will not be locked when it is identical with hob data by creating LockLargeVariable function, also switched to VariablePolicyProtocol for locking variables. Faili

Re: [edk2-devel] [PATCH] IntelFsp2WrapperPkg: Make GetFspm(s)UpdDataAddress reusable

2022-02-06 Thread Chiu, Chasel
Thanks for sending this patch, Ted! Our direction is to always use PcdFspmUpdDataAddress64 and PcdFspsUpdDataAddress64 from platform side no matter building with IA32 or X64. This will help retirement of old PcdFspmUpdDataAddress and PcdFspsUpdDataAddress in the future. Please help to check

Re: [edk2-devel] [PATCH v1] MinPlatformPkg: Install memory relating PPIs

2022-02-07 Thread Chiu, Chasel
Thanks Xiaoqiang! Reviewed-by: Chasel Chiu > -Original Message- > From: Zhang, Xiaoqiang > Sent: Monday, February 7, 2022 4:11 PM > To: devel@edk2.groups.io > Cc: Zhang, Xiaoqiang ; Chiu, Chasel > ; Desimone, Nathaniel L > ; Oram, Isaac W > Subject: [

[edk2-devel] [edk2-platforms: PATCH v2] MinPlatformPkg/SaveMemoryConfig: Variable may not be locked.

2022-02-09 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3829 Fixed the bug that existing variable will not be locked when it is identical with hob data by creating LockLargeVariable function, also switched to VariablePolicyProtocol for locking variables. This patch also modified SaveMemoryConfig

Re: [edk2-devel][PATCH v4 0/8] Support PEI 64bit in IntelFsp2Pkg and IntelFsp2WrapperPkg

2022-04-12 Thread Chiu, Chasel
Thanks Ted for updating patch! For this patch series, Reviewed-by: Chasel Chiu > -Original Message- > From: devel@edk2.groups.io On Behalf Of Kuo, > Ted > Sent: Wednesday, April 13, 2022 10:43 AM > To: devel@edk2.groups.io > Subject: [edk2-devel][PATCH v4 0/8] Support PEI 64bit in

Re: [edk2-devel][PATCH v4 1/8] IntelFsp2Pkg: X64 compatible changes to support PEI in 64bit

2022-04-12 Thread Chiu, Chasel
Hi Ted, Please see my comments below inline. Thanks, Chasel > -Original Message- > From: Kuo, Ted > Sent: Wednesday, April 13, 2022 10:43 AM > To: devel@edk2.groups.io > Cc: Chiu, Chasel ; Desimone, Nathaniel L > ; Zeng, Star ; S, > Ashraf Ali > Subject: [

Re: [edk2-devel] [PATCH v1 1/6] IntelFsp2Pkg: Add CpuLib to module INFs that depend on UefiCpuLib

2022-04-05 Thread Chiu, Chasel
Thanks Yu! Reviewed-by: Chasel Chiu > -Original Message- > From: Pu, Yu > Sent: Friday, April 1, 2022 1:22 PM > To: devel@edk2.groups.io > Cc: Pu, Yu ; Chiu, Chasel ; > Desimone, Nathaniel L ; Zeng, Star > > Subject: [PATCH v1 1/6] IntelFsp2Pkg: A

Re: [edk2-devel] [PATCH v1 2/6] IntelFsp2WrapperPkg: Add CpuLib to module INFs that depend on UefiCpuLib.

2022-04-05 Thread Chiu, Chasel
Thanks Yu! Reviewed-by: Chasel Chiu > -Original Message- > From: Pu, Yu > Sent: Friday, April 1, 2022 1:22 PM > To: devel@edk2.groups.io > Cc: Pu, Yu ; Chiu, Chasel ; > Desimone, Nathaniel L ; Zeng, Star > > Subject: [PATCH v1 2/6] IntelFsp2WrapperPkg: A

Re: [edk2-devel][PATCH v2 8/8] IntelFsp2WrapperPkg: SecFspWrapperPlatformSecLibSample support for X64

2022-04-06 Thread Chiu, Chasel
HI Ted, Please see my comments below inline. Thanks, Chasel > -Original Message- > From: Kuo, Ted > Sent: Monday, April 4, 2022 2:23 PM > To: devel@edk2.groups.io > Cc: Chiu, Chasel ; Desimone, Nathaniel L > ; Zeng, Star ; S, > Ashraf Ali > Subject: [

Re: [edk2-devel][PATCH v2 2/8] IntelFsp2Pkg: Add FSPx_ARCH2_UPD support for X64

2022-04-06 Thread Chiu, Chasel
Hi Ted, Please see my comments inline below. Thanks, Chasel > -Original Message- > From: Kuo, Ted > Sent: Monday, April 4, 2022 2:23 PM > To: devel@edk2.groups.io > Cc: Chiu, Chasel ; Desimone, Nathaniel L > ; Zeng, Star ; S, > Ashraf Ali > Subject: [

Re: [edk2-devel][PATCH v2 1/8] IntelFsp2Pkg: X64 compatible changes to support PEI in 64bit

2022-04-06 Thread Chiu, Chasel
Hi Ted, Please see my comments inline below. Thanks, Chasel > -Original Message- > From: Kuo, Ted > Sent: Monday, April 4, 2022 2:23 PM > To: devel@edk2.groups.io > Cc: Chiu, Chasel ; Desimone, Nathaniel L > ; Zeng, Star ; S, > Ashraf Ali > Subject: [

Re: [edk2-devel][PATCH v2 2/8] IntelFsp2Pkg: Add FSPx_ARCH2_UPD support for X64

2022-04-06 Thread Chiu, Chasel
One more comments. Thanks, Chasel > -Original Message- > From: Chiu, Chasel > Sent: Wednesday, April 6, 2022 10:40 PM > To: Kuo, Ted ; devel@edk2.groups.io > Cc: Desimone, Nathaniel L ; Zeng, Star > ; S, Ashraf Ali > Subject: RE: [edk2-devel][PATCH v2 2

Re: [edk2-devel] [PATCH v1 02/15] IntelFsp2WrapperPkg: Add CpuLib to module INFs that depend on UefiCpuLib.

2022-04-11 Thread Chiu, Chasel
Reviewed-by: Chasel Chiu Thanks, Chasel > -Original Message- > From: Pu, Yu > Sent: Monday, April 11, 2022 5:46 PM > To: devel@edk2.groups.io > Cc: Pu, Yu ; Chiu, Chasel ; > Desimone, Nathaniel L ; Zeng, Star > > Subject: [PATCH v1 02/15] IntelFsp2WrapperP

Re: [edk2-devel] [PATCH v1 01/15] IntelFsp2Pkg: Add CpuLib to module INFs that depend on UefiCpuLib

2022-04-11 Thread Chiu, Chasel
Reviewed-by: Chasel Chiu Thanks, Chasel > -Original Message- > From: Pu, Yu > Sent: Monday, April 11, 2022 5:46 PM > To: devel@edk2.groups.io > Cc: Pu, Yu ; Chiu, Chasel ; > Desimone, Nathaniel L ; Zeng, Star > > Subject: [PATCH v1 01/15] IntelFsp2Pkg: A

Re: [edk2-devel] [PATCH v1 09/15] IntelFsp2Pkg: Remove UefiCpuLib from module INFs.

2022-04-11 Thread Chiu, Chasel
Reviewed-by: Chasel Chiu Thanks, Chasel > -Original Message- > From: Pu, Yu > Sent: Monday, April 11, 2022 5:46 PM > To: devel@edk2.groups.io > Cc: Pu, Yu ; Chiu, Chasel ; > Desimone, Nathaniel L ; Zeng, Star > > Subject: [PATCH v1 09/15] IntelFsp2Pkg: Remov

Re: [edk2-devel] [PATCH v2 02/14] IntelFsp2Pkg: Add CpuLib to module INFs that depend on UefiCpuLib

2022-04-12 Thread Chiu, Chasel
Reviewed-by: Chasel Chiu Thanks, Chasel > -Original Message- > From: Pu, Yu > Sent: Tuesday, April 12, 2022 10:52 PM > To: devel@edk2.groups.io > Cc: Pu, Yu ; Chiu, Chasel ; > Desimone, Nathaniel L ; Zeng, Star > > Subject: [PATCH v2 02/14] IntelFsp2Pkg: A

Re: [edk2-devel] [PATCH v2 03/14] IntelFsp2WrapperPkg: Add CpuLib to module INFs that depend on UefiCpuLib.

2022-04-12 Thread Chiu, Chasel
Reviewed-by: Chasel Chiu Thanks, Chasel > -Original Message- > From: Pu, Yu > Sent: Tuesday, April 12, 2022 10:52 PM > To: devel@edk2.groups.io > Cc: Pu, Yu ; Chiu, Chasel ; > Desimone, Nathaniel L ; Zeng, Star > > Subject: [PATCH v2 03/14] IntelFsp2WrapperP

Re: [edk2-devel] [PATCH v2 08/14] IntelFsp2Pkg: Remove UefiCpuLib from module INFs.

2022-04-12 Thread Chiu, Chasel
Reviewed-by: Chasel Chiu Thanks, Chasel > -Original Message- > From: Pu, Yu > Sent: Tuesday, April 12, 2022 10:52 PM > To: devel@edk2.groups.io > Cc: Pu, Yu ; Chiu, Chasel ; > Desimone, Nathaniel L ; Zeng, Star > > Subject: [PATCH v2 08/14] IntelFsp2Pkg

Re: [edk2-devel][PATCH v5 0/8] Support PEI 64bit in IntelFsp2Pkg and IntelFsp2WrapperPkg

2022-04-15 Thread Chiu, Chasel
Thanks for fixing issues Ted! For this patch series: Reviewed-by: Chasel Chiu > -Original Message- > From: devel@edk2.groups.io On Behalf Of Kuo, > Ted > Sent: Friday, April 15, 2022 4:38 PM > To: devel@edk2.groups.io > Subject: [edk2-devel][PATCH v5 0/8] Support PEI 64bit in

Re: [edk2-devel] [PATCH v1] IntelFsp2Pkg: Add get and set FspSmmInit upd data pointer functions

2023-08-24 Thread Chiu, Chasel
Reviewed-by: Chasel Chiu Thanks, Chasel > -Original Message- > From: Zhang, Hongbin1 > Sent: Thursday, August 24, 2023 8:15 PM > To: devel@edk2.groups.io > Cc: Zhang, Hongbin1 ; Chiu, Chasel > ; Desimone, Nathaniel L > ; Duggapu, Chinni B > ; Ng, Ray Han Lim ;

Re: [edk2-devel][PATCH v2] IntelFsp2Pkg: Support child FV in SplitFspBin.py

2023-08-28 Thread Chiu, Chasel
Merged: https://github.com/tianocore/edk2/commit/2c7fd32676272e22ed44fdfc8fa7e47f5c7a93b8 Thanks, Chasel > -Original Message- > From: Kuo, Ted > Sent: Thursday, July 20, 2023 3:20 AM > To: devel@edk2.groups.io > Cc: Chiu, Chasel ; Desimone, Nathaniel L > ; Zeng

Re: [edk2-devel] [PATCH v1] IntelFsp2Pkg: Add get and set FspSmmInit upd data pointer functions

2023-08-28 Thread Chiu, Chasel
merged: https://github.com/tianocore/edk2/commit/b1e558f6369fc85ae053e0c16ffcd600880fe78d Thanks, Chasel > -Original Message- > From: Zhang, Hongbin1 > Sent: Thursday, August 24, 2023 8:15 PM > To: devel@edk2.groups.io > Cc: Zhang, Hongbin1 ; Chiu, Chasel > ; Des

Re: [edk2-devel] [edk2-platforms][PATCH v1 0/3] MinPlatformPkg: Add FADT duty offset and width PCDs

2022-04-24 Thread Chiu, Chasel
Thanks Michael! For this patch series: Reviewed-by: Chasel Chiu > -Original Message- > From: mikub...@linux.microsoft.com > Sent: Tuesday, April 12, 2022 4:58 AM > To: devel@edk2.groups.io > Cc: Chiu, Chasel ; Desimone, Nathaniel L > ; Gao, Liming > ; Dong, E

Re: [edk2-devel] [PATCH] IntelFsp2Pkg\Tools\ConfigEditor: Added new USF config workstream support Config Edit utility addition/changes.

2023-11-08 Thread Chiu, Chasel
-patch -v2 -1) Thanks, Chasel > -Original Message- > From: Soundara Pandian, Arun SuraX > Sent: Tuesday, November 7, 2023 8:53 PM > To: devel@edk2.groups.io > Cc: Soundara Pandian, Arun SuraX ; > Chiu, Chasel ; Duggapu, Chinni B > ; Desimone, Nathaniel L > ;

Re: [edk2-devel] [edk2-platforms PATCH 1/2] WhitleyOpenBoardPkg: remove references

2023-11-09 Thread Chiu, Chasel
nty, Rangasai V ; Desimone, Nathaniel > L ; Chiu, Chasel > Subject: [edk2-devel] [edk2-platforms PATCH 1/2] WhitleyOpenBoardPkg: > remove references > > For removing "MdePkg/Include/IndustryStandard/LegacyBiosMpTable.h" from > edk2, first remove the edk2-platforms referen

[edk2-devel] [PATCH V1 1/1] MinPlatformPkg: Add PCDs to update FADT entries from board package

2022-05-16 Thread Chiu, Chasel
From: Ankit Sinha Adds new PCDs to allow entries in FADT to be customized during platform integration. Board packages will can update these PCDs during boot. Cc: Chasel Chiu Cc: Nate DeSimone Cc: Liming Gao Cc: Eric Dong Signed-off-by: Ankit Sinha ---

Re: [edk2-devel] [PATCH v4 1/2] IntelFsp2Pkg: FSP_TEMP_RAM_INIT call must follow X64 Calling Convention

2022-05-16 Thread Chiu, Chasel
Thanks Chinni! Reviewed-by: Chasel Chiu > -Original Message- > From: Duggapu, Chinni B > Sent: Tuesday, May 17, 2022 1:07 PM > To: devel@edk2.groups.io > Cc: Chiu, Chasel ; Desimone, Nathaniel L > ; Zeng, Star ; S, > Ashraf Ali > Subject: [PA

Re: [edk2-devel] [PATCH v4 2/2] IntelFsp2WrapperPkg: FSP_TEMP_RAM_INIT call must follow X64 Calling Convention

2022-05-16 Thread Chiu, Chasel
Thanks Chinni! Reviewed-by: Chasel Chiu > -Original Message- > From: Duggapu, Chinni B > Sent: Tuesday, May 17, 2022 1:07 PM > To: devel@edk2.groups.io > Cc: Chiu, Chasel ; Desimone, Nathaniel L > ; Zeng, Star ; S, > Ashraf Ali > Subject: [PATCH v4

Re: [edk2-devel] [PATCH v3] IntelFsp2Pkg: FSP_TEMP_RAM_INIT call must follow X64 Calling Convention

2022-05-16 Thread Chiu, Chasel
Thanks for clarification! In this case, please use "mov rsp, rcx" to support 64bit addressing. Thanks, Chasel > -Original Message- > From: Duggapu, Chinni B > Sent: Tuesday, May 17, 2022 12:33 AM > To: Chiu, Chasel ; devel@edk2.groups.io > Cc: Desimone, Natha

Re: [edk2-devel] [PATCH v5 1/2] IntelFsp2Pkg: FSP_TEMP_RAM_INIT call must follow X64 Calling Convention

2022-05-17 Thread Chiu, Chasel
Thanks for fixing a small coding issue! Reviewed-by: Chasel Chiu > -Original Message- > From: Duggapu, Chinni B > Sent: Tuesday, May 17, 2022 4:44 PM > To: devel@edk2.groups.io > Cc: Chiu, Chasel ; Desimone, Nathaniel L > ; Zeng, Star ; S, > Ashraf Ali >

Re: [edk2-devel] [PATCH v5 2/2] IntelFsp2WrapperPkg: FSP_TEMP_RAM_INIT call must follow X64 Calling Convention

2022-05-17 Thread Chiu, Chasel
Thanks Chinni! Reviewed-by: Chasel Chiu > -Original Message- > From: Duggapu, Chinni B > Sent: Tuesday, May 17, 2022 4:44 PM > To: devel@edk2.groups.io > Cc: Chiu, Chasel ; Desimone, Nathaniel L > ; Zeng, Star ; S, > Ashraf Ali > Subject: [PATCH v5

Re: [edk2-devel] [PATCH v3] IntelFsp2Pkg: FSP_TEMP_RAM_INIT call must follow X64 Calling Convention

2022-05-16 Thread Chiu, Chasel
: [1/2] IntelFsp2Pkg patch [2/2] IntelFsp2WrapperPkg patch Thanks, Chasel > -Original Message- > From: Duggapu, Chinni B > Sent: Monday, May 16, 2022 6:54 PM > To: devel@edk2.groups.io > Cc: Chiu, Chasel ; Desimone, Nathaniel L > ; Zeng, Star ; S, >

<    2   3   4   5   6   7   8   9   10   >