[edk2-devel] [Patch 1/1] ShellPkg/Library: Fix 32-bit truncation of pointer values

2023-03-11 Thread Michael D Kinney
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4366 Update C and UNI files that are incorrectly using %x or %08x instead of %p for pointer values. On 64-bit systems, this is truncating pointer values above 4GB. In reviewing ShellPkg for this issue some unused UNI strings with incorrect form

[edk2-devel] [PATCH edk2-platforms 2/2] IpmiFeaturePkg: fix IpmiSendCommand response size in GetDeviceId function

2023-03-11 Thread Mike Maslenkin
It is required to reinitialize DataSize variable before subsequent IpmiSendCommand calls. It is especially required after processing IpmiSendCommand with IPMI_GET_BMC_EXECUTION_CONTEXT because it returns only 2 bytes as a response. So to the next call of IpmiSendCommand with IPMI_APP_GET_DEVICE_ID

[edk2-devel] [PATCH edk2-platforms 1/2] IpmiFeaturePkg: add missing update of BMC status after GetSelfTest

2023-03-11 Thread Mike Maslenkin
Signed-off-by: Mike Maslenkin Cc: Arunk Cc: Isaac Oram Cc: Nate DeSimone Cc: Liming Gao --- .../IpmiFeaturePkg/GenericIpmi/Dxe/IpmiInit.c| 1 + 1 file changed, 1 insertion(+) diff --git a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/GenericIpmi/Dxe/IpmiInit.c b/Fea

[edk2-devel] [PATCH edk2-platforms 0/2] IpmiFeaturePkg: small fixes

2023-03-11 Thread Mike Maslenkin
During review of a previous patchset with IpmiFeaturePkg changes, Arunk asked to add additional update of BmcStatus [*]. Also I found another issue at GetDeviceId() function that could be solved by trivial fix. [*] https://edk2.groups.io/g/devel/message/100945 Signed-off-by: Mike Maslenkin Cc: A

[edk2-devel] [PATCH 1/3] Silicon/RISC-V: Fix build failures caused by duplicated definition

2023-03-11 Thread Chai, Evan
Clean up redundant function code becasue they were merged to MdePkg. Cc: Daniel Schaefer Signed-off-by: Evan Chai Reviewed-by: Andrei Warkentin Reviewed-by: Sunil V L --- .../Library/ResetSystemLib/ResetSystemLib.c | 3 +- .../Library/ResetSystemLib/ResetSystemLib.inf | 3 +- .../Include/

[edk2-devel] [PATCH 3/3] Platform/SiFive: Fixed build failure casued by wrong alignment

2023-03-11 Thread Chai, Evan
The lib and driver should be aligned to Mdepkg. Cc: Daniel Schaefer Signed-off-by: Evan Chai Reviewed-by: Andrei Warkentin Reviewed-by: Sunil V L --- .../U5SeriesPkg/FreedomU500VC707Board/U500.dsc | 15 --- .../U5SeriesPkg/FreedomU500VC707Board/U500.fdf | 2 +- .../FreedomU540H

[edk2-devel] [PATCH 2/3] Platform/SiFive: align the library and driver to MdePkg

2023-03-11 Thread Chai, Evan
There are almost same implementation in MdePkg, the part in platform package can be deprecated. Cc: Daniel Schaefer Signed-off-by: Evan Chai Reviewed-by: Andrei Warkentin Reviewed-by: Sunil V L --- .../RiscVTimerLib/BaseRiscVTimerLib.inf | 35 -- .../Library/RiscVTimerLib/RiscVTimerLib

[edk2-devel] [PATCH 1/3] Silicon/RISC-V: Fix build failures caused by duplicated definition

2023-03-11 Thread Chai, Evan
Clean up redundant function code becasue they were merged to MdePkg. Cc: Daniel Schaefer Signed-off-by: Evan Chai Reviewed-by: Andrei Warkentin Reviewed-by: Sunil V L --- .../Library/ResetSystemLib/ResetSystemLib.c | 3 +- .../Library/ResetSystemLib/ResetSystemLib.inf | 3 +- .../Include/