This is a replacement patch of Abner's with title ProcessorPkg/RiscVEdk2SbiLib: Error handling for locating firmware context
I realized that the error handling wasn't bad but unnecessary. Those funtions can never return an error, since the SBI call they use also doesn't. Tested by booting to UEFI Shell. No new error message logged or ASSERT triggered. Cc: Abner Chang <abner.ch...@hpe.com> Cc: Leif Lindholm <l...@nuviainc.com> Daniel Schaefer (1): Silicon/SiFive: Handle case of NULL FirmwareContext .../Include/Library/RiscVEdk2SbiLib.h | 12 ++--- .../PlatformPkg/Universal/Sec/SecMain.c | 11 +++-- .../Library/RiscVEdk2SbiLib/RiscVEdk2SbiLib.c | 46 +++++++------------ .../Library/PeiCoreInfoHobLib/CoreInfoHob.c | 13 ++++-- 4 files changed, 36 insertions(+), 46 deletions(-) -- 2.28.0 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#65762): https://edk2.groups.io/g/devel/message/65762 Mute This Topic: https://groups.io/mt/77213798/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
