Reviewed-by: Isaac Oram <isaac.w.oram@...>

Pushed as 60053f3077..edc718521e

Pushed as ea2a520c83..60053f3077

Pushed as 436a8612ae..ea2a520c83

Reviewed-by: Marvin Häuser <mhaeuser@...>

Hi, I would like to propose to add Alert standard format support which is based on ASF2.0 specification(DSP0136). REF:https://www.dmtf.org/sites/default/files/standards/documents/DSP0136.pdf Could

SerDesProtocolMap is a UINT64, but BIT0 is an unadorned int. Shifting BIT0 left by more than 31 bits is undefined behaviour. This would cause SerDesProtocolMap to be filled with incorrect values when

From: Shengfengx Xue <shengfengx.xue@...> REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3D4014 if Configured Memory Speed is 65,535 MT/s or greater, and the actual speed is stored in the

--- ShellPkg/Library/UefiShellLevel2CommandsLib/Cp.c | 1 + 1 file changed, 1 insertion(+) diff --git a/ShellPkg/Library/UefiShellLevel2CommandsLib/Cp.c

Hi Kun, I have just replied to your patch series. I think patches 4 & 5 may need some adjustment. Can you look into those, please? Regards, Sami Mujawar On 08/08/2022, 16:34, "Kun Qin"

Hi Kun, Please find my response inline marked [SAMI]. Regards, Sami Mujawar [SAMI] When running Kvmtool guest firmware I break from here with EFI_NOT_FOUND. The problem is

Hi Kun, I have just tried testing your patch with Kvmtool guest firmware and think this patch may need some modifications. Also, the patch 4/6 may need some adjustment, which I will

Hi DynamicTablePkg maintainers, Just a gentle ping on this patch series. If there is no further feedback, can we merge them into the mainline? Otherwise, any feedback is appreciated. Regards, Kun

Hi Kun, Thank you for this patch. This change looks good to me. Reviewed-by: Sami Mujawar <sami.mujawar@...> Regards, Sami Mujawar

Hi Kun, Thank you for this patch. These changes look good to me. Reviewed-by: Sami Mujawar <sami.mujawar@...> Regards, Sami Mujawar

Hi Kun, Thank you for this patch. Please find my response inline marked [SAMI]. Regards, Sami Mujawar [SAMI] If I understand correctly, the mAcpiVerifyTables[x].Presence filed cannot be

Hi Jianyong, Thank you for this patch. This change looks good to me. Reviewed-by: Sami Mujawar <sami.mujawar@...> Regards, Sami Mujawar

Hi Jianyong, Thank you for this patch. Please find my response inline marked [SAMI]. Regards, Sami Mujawar On 29/07/2022 08:21 am, Jianyong Wu wrote:

Hi, all We enter into Soft Feature Freeze phase on 2022-08-08. In this phase, the feature under review will not be allowed to be pushed. The feature passed review can still be merged. The

Hi Michael, Please find my response inline marked [SAMI]. Regards, Sami Mujawar [SAMI] I have not been following the convention to add Cc lines to the patches. However, I will start from the next