There is a typo ' failue' in the commit message. Other than that, looks good.
Reviewed-by: Maurice Ma <maurice...@intel.com> > -----Original Message----- > From: Dong, Guo <guo.d...@intel.com> > Sent: Thursday, October 14, 2021 22:08 > To: devel@edk2.groups.io > Cc: Dong, Guo <guo.d...@intel.com>; Ni, Ray <ray...@intel.com>; Ma, > Maurice <maurice...@intel.com>; You, Benjamin > <benjamin....@intel.com> > Subject: [`edk2-devel][PATCH V2] UefiPayloadPkg: Fix the build issue for > coreboot > > From: Guo Dong <guo.d...@intel.com> > > V2: Fix GCC build issue. > > There is typo in previous change caused coreboot build failue. > This patch fixed the build issue. > > Cc: Ray Ni <ray...@intel.com> > Cc: Maurice Ma <maurice...@intel.com> > Cc: Benjamin You <benjamin....@intel.com> > Signed-off-by: Guo Dong <guo.d...@intel.com> > --- > UefiPayloadPkg/Library/CbParseLib/CbParseLib.c | 4 ++-- > UefiPayloadPkg/UefiPayloadEntry/UefiPayloadEntry.c | 4 +++- > 2 files changed, 5 insertions(+), 3 deletions(-) > > diff --git a/UefiPayloadPkg/Library/CbParseLib/CbParseLib.c > b/UefiPayloadPkg/Library/CbParseLib/CbParseLib.c > index f81aa0f301..4be5d66ba5 100644 > --- a/UefiPayloadPkg/Library/CbParseLib/CbParseLib.c > +++ b/UefiPayloadPkg/Library/CbParseLib/CbParseLib.c > @@ -194,8 +194,8 @@ GetParameterBase ( > return NULL; > > } > > > > - PcdSet64S (PcdBootloaderParameter, (UINTN)CbTablePtr); > > - ASSERT_EFI_STATUS (Status); > > + Status = PcdSet64S (PcdBootloaderParameter, (UINTN)CbTablePtr); > > + ASSERT_EFI_ERROR (Status); > > > > return CbTablePtr; > > } > > diff --git a/UefiPayloadPkg/UefiPayloadEntry/UefiPayloadEntry.c > b/UefiPayloadPkg/UefiPayloadEntry/UefiPayloadEntry.c > index 5a1e578668..0e6dcf10f9 100644 > --- a/UefiPayloadPkg/UefiPayloadEntry/UefiPayloadEntry.c > +++ b/UefiPayloadPkg/UefiPayloadEntry/UefiPayloadEntry.c > @@ -406,7 +406,9 @@ _ModuleEntryPoint ( > SERIAL_PORT_INFO SerialPortInfo; > > UNIVERSAL_PAYLOAD_SERIAL_PORT_INFO *UniversalSerialPort; > > > > - PcdSet64S (PcdBootloaderParameter, BootloaderParameter); > > + Status = PcdSet64S (PcdBootloaderParameter, BootloaderParameter); > > + ASSERT_EFI_ERROR (Status); > > + > > // Initialize floating point operating environment to be compliant with > UEFI > spec. > > InitializeFloatingPointUnits (); > > > > -- > 2.32.0.windows.2 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#82135): https://edk2.groups.io/g/devel/message/82135 Mute This Topic: https://groups.io/mt/86332763/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-