Reviewed-by: Chen Gang C <gang.c.c...@intel.com> BR Gang
-----Original Message----- From: Guo, Gua <gua....@intel.com> Sent: Thursday, November 9, 2023 6:07 PM To: Hsu, JasonX <jasonx....@intel.com>; devel@edk2.groups.io Cc: Desimone, Nathaniel L <nathaniel.l.desim...@intel.com>; Zeng, Star <star.z...@intel.com>; Chiu, Chasel <chasel.c...@intel.com>; Chen, Gang C <gang.c.c...@intel.com>; Duggapu, Chinni B <chinni.b.dugg...@intel.com>; Kuo, Ted <ted....@intel.com>; S, Ashraf Ali <ashraf.al...@intel.com>; Mohapatra, Susovan <susovan.mohapa...@intel.com>; Lu, James <james...@intel.com> Subject: RE: [PATCH] IntelFsp2WrapperPkg: Add variable initialization Reviewed-by: Gua Guo <gua....@intel.com> -----Original Message----- From: Hsu, JasonX <jasonx....@intel.com> Sent: Thursday, November 9, 2023 6:01 PM To: devel@edk2.groups.io Cc: Hsu, JasonX <jasonx....@intel.com>; Desimone, Nathaniel L <nathaniel.l.desim...@intel.com>; Zeng, Star <star.z...@intel.com>; Chiu, Chasel <chasel.c...@intel.com>; Chen, Gang C <gang.c.c...@intel.com>; Duggapu, Chinni B <chinni.b.dugg...@intel.com>; Kuo, Ted <ted....@intel.com>; S, Ashraf Ali <ashraf.al...@intel.com>; Mohapatra, Susovan <susovan.mohapa...@intel.com>; Lu, James <james...@intel.com>; Guo, Gua <gua....@intel.com> Subject: [PATCH] IntelFsp2WrapperPkg: Add variable initialization From: "JasonX.Hsu" <jasonx....@intel.com> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4595 Fix build error when remove "-Wno-sometimes-uninitialized" option, Add variable "FspMultiPhaseApiOffset" initialization. Cc: Nate DeSimone <nathaniel.l.desim...@intel.com> Cc: Star Zeng <star.z...@intel.com> Cc: Chasel Chiu <chasel.c...@intel.com> Cc: Chen Gang C <gang.c.c...@intel.com> Cc: Duggapu Chinni B <chinni.b.dugg...@intel.com> Cc: Ted Kuo <ted....@intel.com> Cc: Ashraf Ali S <ashraf.al...@intel.com> Cc: Susovan Mohapatra <susovan.mohapa...@intel.com> Cc: James Lu <james...@intel.com> Cc: Gua Guo <gua....@intel.com> Signed-off-by: Jason Hsu <jasonx....@intel.com> --- .../PeiFspWrapperMultiPhaseProcessLib.c | 1 + 1 file changed, 1 insertion(+) diff --git a/IntelFsp2WrapperPkg/Library/FspWrapperMultiPhaseProcessLib/PeiFspWrapperMultiPhaseProcessLib.c b/IntelFsp2WrapperPkg/Library/FspWrapperMultiPhaseProcessLib/PeiFspWrapperMultiPhaseProcessLib.c index 834dca07a9..224c24881d 100644 --- a/IntelFsp2WrapperPkg/Library/FspWrapperMultiPhaseProcessLib/PeiFspWrapperMultiPhaseProcessLib.c +++ b/IntelFsp2WrapperPkg/Library/FspWrapperMultiPhaseProcessLib/PeiFspW +++ rapperMultiPhaseProcessLib.c @@ -80,6 +80,7 @@ CallFspMultiPhaseEntry ( BOOLEAN IsVariableServiceRequest; FSP_MULTI_PHASE_PARAMS *FspMultiPhaseParamsPtr; + FspMultiPhaseApiOffset = 0; FspMultiPhaseParamsPtr = (FSP_MULTI_PHASE_PARAMS *)FspMultiPhaseParams; IsVariableServiceRequest = FALSE; if ((FspMultiPhaseParamsPtr->MultiPhaseAction == EnumMultiPhaseGetVariableRequestInfo) ||-- 2.39.1.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#110964): https://edk2.groups.io/g/devel/message/110964 Mute This Topic: https://groups.io/mt/102482693/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-