|
Re: [PATCH v3 1/2] UefiPayloadPkg: Implement a new SerialPortLib instance
Please try to not use AllocatePool in this library because in early part
of DxeMain, the memory service may not be available.
You could use a C array with maximum maybe 16 elements.
Then the
|
Please try to not use AllocatePool in this library because in early part
of DxeMain, the memory service may not be available.
You could use a C array with maximum maybe 16 elements.
Then the
|
By
Ni, Ray
·
#91626
·
|
|
[PATCH v3 2/2] UefiPayloadPkg: Allow DxeMain.inf to consume the new SerialPortLib
Let DxeMain.inf consume new SerialPortLib to support multiple
channel debug message on early DXE.
Cc: Ray Ni <ray.ni@...>
Cc: Sean Rhodes <sean@...>
Cc: Gua Guo
|
Let DxeMain.inf consume new SerialPortLib to support multiple
channel debug message on early DXE.
Cc: Ray Ni <ray.ni@...>
Cc: Sean Rhodes <sean@...>
Cc: Gua Guo
|
By
Kavya
·
#91625
·
|
|
[PATCH v3 1/2] UefiPayloadPkg: Implement a new SerialPortLib instance
Add new Serial port library instance that consumes the HOB defined
in MdeModulePkg/Include/UniversalPayload/SerialPortInfo.h to support
multiple UART's.
Cc: Ray Ni <ray.ni@...>
Cc: Sean Rhodes
|
Add new Serial port library instance that consumes the HOB defined
in MdeModulePkg/Include/UniversalPayload/SerialPortInfo.h to support
multiple UART's.
Cc: Ray Ni <ray.ni@...>
Cc: Sean Rhodes
|
By
Kavya
·
#91624
·
|
|
Re: [PATCH v3 00/22] Add Raw algorithm support using Arm FW-TRNG interface
Hi Jiewen,
I am not sure I understood what you want to move to the ArmPkg.
Nonetheless, doing the following removes the dependency over the ArmPkg
(and it seems to be the normal way way of removing
|
Hi Jiewen,
I am not sure I understood what you want to move to the ArmPkg.
Nonetheless, doing the following removes the dependency over the ArmPkg
(and it seems to be the normal way way of removing
|
By
PierreGondois
·
#91623
·
|
|
Re: [Patch v2 2/3] Maintainers.txt: Add missing github ids
Yes, that's my GitHub ID.
Can add reviewed-by or acked-by. Whatever makes sense here.
Thanks,
Daniel
|
Yes, that's my GitHub ID.
Can add reviewed-by or acked-by. Whatever makes sense here.
Thanks,
Daniel
|
By
Daniel Schaefer
·
#91622
·
|
|
Re: [PATCH v3] IntelFsp2Pkg: Add FSPI_ARCH_UPD.
The spec looks wrong to me.
Thanks,
Star
|
The spec looks wrong to me.
Thanks,
Star
|
By
Zeng, Star
·
#91621
·
|
|
Re: [PATCH v3] IntelFsp2Pkg: Add FSPI_ARCH_UPD.
Hi Star, this is following current specification.
Thanks,
Chasel
|
Hi Star, this is following current specification.
Thanks,
Chasel
|
By
Chiu, Chasel
·
#91620
·
|
|
[Patch v2 3/3] Maintainers.txt: Remove reviewer Harry Han
Cc: Harry Han <harry.han@...>
Cc: Andrew Fish <afish@...>
Cc: Leif Lindholm <quic_llindhol@...>
Signed-off-by: Michael D Kinney <michael.d.kinney@...>
---
Maintainers.txt |
|
Cc: Harry Han <harry.han@...>
Cc: Andrew Fish <afish@...>
Cc: Leif Lindholm <quic_llindhol@...>
Signed-off-by: Michael D Kinney <michael.d.kinney@...>
---
Maintainers.txt |
|
By
Michael D Kinney
·
#91619
·
|
|
[Patch v2 2/3] Maintainers.txt: Add missing github ids
Cc: Leif Lindholm <quic_llindhol@...>
Cc: Sean Rhodes <sean@...>
Cc: Debkumar De <debkumar.de@...>
Cc: Erdem Aktas <erdemaktas@...>
Cc: Gary Lin
|
Cc: Leif Lindholm <quic_llindhol@...>
Cc: Sean Rhodes <sean@...>
Cc: Debkumar De <debkumar.de@...>
Cc: Erdem Aktas <erdemaktas@...>
Cc: Gary Lin
|
By
Michael D Kinney
·
#91618
·
|
|
[Patch v2 1/3] Maintainers.txt: Remove OvmfPkg/XenTimerDxe reference
XenTimerDxe has been removed from the OvmfPkg. Remove
file pattern for XenTimerDxe reviews from Maintainers.txt.
Cc: Julien Grall <julien@...>
Cc: Andrew Fish <afish@...>
Cc: Leif Lindholm
|
XenTimerDxe has been removed from the OvmfPkg. Remove
file pattern for XenTimerDxe reviews from Maintainers.txt.
Cc: Julien Grall <julien@...>
Cc: Andrew Fish <afish@...>
Cc: Leif Lindholm
|
By
Michael D Kinney
·
#91617
·
|
|
[Patch v2 0/3] Update Maintainers.txt
Cc: Julien Grall <julien@...>
Cc: Sean Rhodes <sean@...>
Cc: Erdem Aktas <erdemaktas@...>
Cc: Gary Lin <gary.lin@...>
Cc: Andrew Fish <afish@...>
Cc: Leif Lindholm
|
Cc: Julien Grall <julien@...>
Cc: Sean Rhodes <sean@...>
Cc: Erdem Aktas <erdemaktas@...>
Cc: Gary Lin <gary.lin@...>
Cc: Andrew Fish <afish@...>
Cc: Leif Lindholm
|
By
Michael D Kinney
·
#91616
·
|
|
Re: [PATCH v3] IntelFsp2Pkg: Add FSPI_ARCH_UPD.
Is the reserved bytes number correct for FSPI_ARCH_UPD alignment?
UINT16 BootloaderSmmFvContextDataLength;
UINT8 Reserved1[24];
Thanks,
Star
|
Is the reserved bytes number correct for FSPI_ARCH_UPD alignment?
UINT16 BootloaderSmmFvContextDataLength;
UINT8 Reserved1[24];
Thanks,
Star
|
By
Zeng, Star
·
#91615
·
|
|
回复: [PATCH v5] MdeModulePkg/Include: Long debug string is truncated to 104 char
Reviewed-by: Liming Gao <gaoliming@...>
<jian.j.wang@...>;
to
104
phase
Status
|
Reviewed-by: Liming Gao <gaoliming@...>
<jian.j.wang@...>;
to
104
phase
Status
|
By
gaoliming
·
#91614
·
|
|
[PATCH v3] IntelFsp2Pkg: Add FSPI_ARCH_UPD.
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3D3993
Adding the missing FSPI_ARCH_UPD, FSP_GLOBAL_DATA_VERSION bumpping up,
and some comments for clarification.
Also fixed a bug in
|
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3D3993
Adding the missing FSPI_ARCH_UPD, FSP_GLOBAL_DATA_VERSION bumpping up,
and some comments for clarification.
Also fixed a bug in
|
By
Chiu, Chasel
·
#91613
·
|
|
Re: [PATCH v1 4/6] DynamicTablesPkg: DynamicTableManagerDxe: Added check for installed tables
Thank you for your reviews, Sami! I will incorporate your suggestions in v2 patch and send for review after validation.
Regards,
Kun
|
Thank you for your reviews, Sami! I will incorporate your suggestions in v2 patch and send for review after validation.
Regards,
Kun
|
By
Kun Qin
·
#91612
·
|
|
Re: [PATCH v1 0/6] Enhance DynamicTablesPkg modules
Thank you for the review, Pierre! We will update the patches per your feedback and send v2 after validation.
Regards,
Kun
|
Thank you for the review, Pierre! We will update the patches per your feedback and send v2 after validation.
Regards,
Kun
|
By
Kun Qin
·
#91611
·
|
|
Re: [PATCH v5] MdeModulePkg/Include: Long debug string is truncated to 104 char
Reviewed-by: Dandan Bi <dandan.bi@...>
Thanks,
Dandan
|
Reviewed-by: Dandan Bi <dandan.bi@...>
Thanks,
Dandan
|
By
Dandan Bi
·
#91610
·
|
|
Re: [edk2-platforms][PATCH V2 1/1] Features/Intel/PlatformPayloadPkg: add platform payload FV
Any comments for this patch?
Thanks,
Guo
|
Any comments for this patch?
Thanks,
Guo
|
By
Guo Dong
·
#91609
·
|
|
Re: [PATCH] UefiPayloadPkg: Fix RelaAddress type always mismatch in if condition
Reviewed-by: Guo Dong <guo.dong@...>
|
Reviewed-by: Guo Dong <guo.dong@...>
|
By
Guo Dong
·
#91608
·
|
|
Re: [edk2-platforms][PATCH V1 0/5] Resolve GCC compilation warnings
Pushed as a2763c425e..f5306c2b07
|
Pushed as a2763c425e..f5306c2b07
|
By
Isaac Oram
·
#91607
·
|
