Hello Laszlo, I have filed a new TianoCore Feature request BZ for this. https://bugzilla.tianocore.org/show_bug.cgi?id=3467 I will refer this bug in future commit messages for this patch-set.

Thanks for the clarification. I will work on v-next with flexible array as Data field. Regards, Kun

Looks good to me. Thanks for working on this, Greg. Reviewed-by: Sunny Wang <sunny.wang@...>

Internally reviewed this patch before sending the edk2 mailing list and Greg already addressed all my comments, so It looks good to me. This patch is to replace my previous commit

Internally reviewed this patch before sending the edk2 mailing list and Greg already addressed all my comments, so It looks good to me. Reviewed-by: Sunny Wang <sunny.wang@...> Add Ray and

Ok. I understand your patch just merges the existing PCDs from individual XXSIliconPkgs to IntelSiliconPkg. Are there any modules that use these PCDs in XXSiliconPkgs?

Can you explain why VERBOSE is chosen instead of INFO? Thanks, Ray

Reviewed-by: Maurice Ma <maurice.ma@...> -Maurice

Add Ray Ni as UefiPayloadPkg Maintainer. Update Maurice Ma and Benjamin You as reviewers to continue support UefiPayloadPkg patch review. Cc: Benjamin You <benjamin.you@...> Cc: Maurice Ma

Reviewed-by: Guo Dong <guo.dong@...>

Reviewed-by: Guo Dong <guo.dong@...>

Patch pushed: 580b11201ed001f9533c6782ec87d430b1736040 Thanks, Chasel

Unfortunately, those cannot be used in the silicon packages. This is not new to this series, see existing

I see. Thanks for your quick response. I will work on it. Nickle

This is a separate commit. You can send it together with previous patch. Thanks Liming

Hi Liming, No problem. It looks like I have to add OnigurumaUefiPort.c into "IgnoreFiles" object in MdeModulePkg.ci.yaml. And should I send this patch alone? Or I have to send it as a part of patch

If no other comment, I will create PR to merge this patch. Thanks Liming 发件人: devel@edk2.groups.io <devel@edk2.groups.io> 代表 gaoliming 发送时间: 2021年6月24日 8:54 收件人:

If it breaks in the future, then that would be due to a new compiler that or changes to the configuration of an existing compiler that break compatibility with UEFI ABI. The compiler issue must be

Laszlo: Yes. I agree to add OnigurumaUefiPort.c into ECC exception in MdeModulePkg.ci.yaml. Nickle: Can you provide the patch to update MdeModulePkg.ci.yaml? Thanks Liming

Hi Mike, Thanks for the note. I will add this check for sanity check in v-next, assuming this will not fail for currently supported compilers. Just curious, what do we normally do if this type of