Yes, these pcds is used by PciBus driver,  my comment is not correct. My intention is these PCDs have no value in the UefiPaylodPkd.dsc,  we can use the default values in MdeModulePkg.dec or  override the values in  UefiPaylodPkd.dsc.<br/><br/>On Thu, Nov 10, 2022 at 09:58 AM, Ni, Ray wrote:<br/>
<blockquote><!-- [if !mso]><style>v\:* {behavior:url(#default#VML);}
o\:* {behavior:url(#default#VML);}
w\:* {behavior:url(#default#VML);}
.shape {behavior:url(#default#VML);}
</style><![endif]--><!-- [if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!-- [if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
<div class="WordSection1">
<p class="DefangedMsoNormal" style="margin:0px;">These PCDs are used by PciBus driver and MP init logic which are included in the UefiPayloadPkg.</p>
<p class="DefangedMsoNormal" style="margin:0px;"> </p>
<div style="border: none; border-left: solid blue 1.5pt; padding: 0in 0in 0in 4.0pt;">
<div>
<div style="border: none; border-top: solid #E1E1E1 1.0pt; padding: 3.0pt 0in 0in 0in;">
<p class="DefangedMsoNormal" style="margin:0px;"><strong>From:</strong> devel@edk2.groups.io &lt;devel@edk2.groups.io&gt; <strong> On Behalf Of </strong>Jiading Zhang<br/><strong>Sent:</strong> Thursday, November 10, 2022 9:53 AM<br/><strong>To:</strong> devel@edk2.groups.io<br/><strong>Subject:</strong> [edk2-devel] [PATCH] UefiPayloadPkg: remove the redundant PCD definition.</p>
</div>
</div>
<p class="DefangedMsoNormal" style="margin:0px;"> </p>
<div>
<p class="DefangedMsoNormal" style="margin:0px;"> </p>
</div>
<div>
<p class="DefangedMsoNormal" style="margin:0px;">The following PCDs have no value in the file of UefiPayloadPkg.dsc, and they are not used in UefiPayloadPkg, so remove them from [PcdsDynamicExDefault] section.</p>
</div>
<div>
<p class="DefangedMsoNormal" style="margin:0px;">  gEfiMdeModulePkgTokenSpaceGuid.PcdAriSupport</p>
</div>
<div>
<p class="DefangedMsoNormal" style="margin:0px;">  gEfiMdeModulePkgTokenSpaceGuid.PcdMrIovSupport</p>
</div>
<div>
<p class="DefangedMsoNormal" style="margin:0px;">  gEfiMdeModulePkgTokenSpaceGuid.PcdSrIovSupport</p>
</div>
<div>
<p class="DefangedMsoNormal" style="margin:0px;">  gEfiMdeModulePkgTokenSpaceGuid.PcdSrIovSystemPageSize</p>
</div>
<div>
<p class="DefangedMsoNormal" style="margin:0px;">  gUefiCpuPkgTokenSpaceGuid.PcdCpuApInitTimeOutInMicroSeconds</p>
</div>
<div>
<p class="DefangedMsoNormal" style="margin:0px;">  gUefiCpuPkgTokenSpaceGuid.PcdCpuApLoopMode</p>
</div>
<div>
<p class="DefangedMsoNormal" style="margin:0px;">  gUefiCpuPkgTokenSpaceGuid.PcdCpuMicrocodePatchAddress</p>
</div>
<div>
<p class="DefangedMsoNormal" style="margin:0px;">  gUefiCpuPkgTokenSpaceGuid.PcdCpuMicrocodePatchRegionSize</p>
</div>
<div>
<p class="DefangedMsoNormal" style="margin:0px;"> </p>
</div>
<div>
<p class="DefangedMsoNormal" style="margin:0px;">Signed-off-by: jdzhang &lt;<a href="mailto:jdzhang@kunluntech.com.cn">jdzhang@kunluntech.com.cn</a>&gt;</p>
</div>
<div>
<p class="DefangedMsoNormal" style="margin:0px;">---</p>
</div>
<div>
<p class="DefangedMsoNormal" style="margin:0px;"> UefiPayloadPkg/UefiPayloadPkg.dsc | 8 --------</p>
</div>
<div>
<p class="DefangedMsoNormal" style="margin:0px;"> 1 file changed, 8 deletions(-)</p>
</div>
<div>
<p class="DefangedMsoNormal" style="margin:0px;"> </p>
</div>
<div>
<p class="DefangedMsoNormal" style="margin:0px;">diff --git a/UefiPayloadPkg/UefiPayloadPkg.dsc b/UefiPayloadPkg/UefiPayloadPkg.dsc</p>
</div>
<div>
<p class="DefangedMsoNormal" style="margin:0px;">index 1150be6acd..43768909b3 100644</p>
</div>
<div>
<p class="DefangedMsoNormal" style="margin:0px;">--- a/UefiPayloadPkg/UefiPayloadPkg.dsc</p>
</div>
<div>
<p class="DefangedMsoNormal" style="margin:0px;">+++ b/UefiPayloadPkg/UefiPayloadPkg.dsc</p>
</div>
<div>
<p class="DefangedMsoNormal" style="margin:0px;">@@ -530,14 +530,6 @@</p>
</div>
<div>
<p class="DefangedMsoNormal" style="margin:0px;">   gEfiMdePkgTokenSpaceGuid.PcdUartDefaultParity|$(UART_DEFAULT_PARITY)</p>
</div>
<div>
<p class="DefangedMsoNormal" style="margin:0px;">   gEfiMdePkgTokenSpaceGuid.PcdUartDefaultStopBits|$(UART_DEFAULT_STOP_BITS)</p>
</div>
<div>
<p class="DefangedMsoNormal" style="margin:0px;">   gEfiMdePkgTokenSpaceGuid.PcdDefaultTerminalType|$(DEFAULT_TERMINAL_TYPE)</p>
</div>
<div>
<p class="DefangedMsoNormal" style="margin:0px;">-  gEfiMdeModulePkgTokenSpaceGuid.PcdAriSupport</p>
</div>
<div>
<p class="DefangedMsoNormal" style="margin:0px;">-  gEfiMdeModulePkgTokenSpaceGuid.PcdMrIovSupport</p>
</div>
<div>
<p class="DefangedMsoNormal" style="margin:0px;">-  gEfiMdeModulePkgTokenSpaceGuid.PcdSrIovSupport</p>
</div>
<div>
<p class="DefangedMsoNormal" style="margin:0px;">-  gEfiMdeModulePkgTokenSpaceGuid.PcdSrIovSystemPageSize</p>
</div>
<div>
<p class="DefangedMsoNormal" style="margin:0px;">-  gUefiCpuPkgTokenSpaceGuid.PcdCpuApInitTimeOutInMicroSeconds</p>
</div>
<div>
<p class="DefangedMsoNormal" style="margin:0px;">-  gUefiCpuPkgTokenSpaceGuid.PcdCpuApLoopMode</p>
</div>
<div>
<p class="DefangedMsoNormal" style="margin:0px;">-  gUefiCpuPkgTokenSpaceGuid.PcdCpuMicrocodePatchAddress</p>
</div>
<div>
<p class="DefangedMsoNormal" style="margin:0px;">-  gUefiCpuPkgTokenSpaceGuid.PcdCpuMicrocodePatchRegionSize</p>
</div>
<div>
<p class="DefangedMsoNormal" style="margin:0px;"> !if ($(TARGET) == DEBUG || $(USE_CBMEM_FOR_CONSOLE) == TRUE)</p>
</div>
<div>
<p class="DefangedMsoNormal" style="margin:0px;">   gEfiMdeModulePkgTokenSpaceGuid.PcdStatusCodeUseSerial|TRUE</p>
</div>
<div>
<p class="DefangedMsoNormal" style="margin:0px;"> !else</p>
</div>
<div>
<p class="DefangedMsoNormal" style="margin:0px;">-- </p>
</div>
<div>
<p class="DefangedMsoNormal" style="margin:0px;">2.38.1.windows.1</p>
</div>
<div>
<p class="DefangedMsoNormal" style="margin:0px;"> </p>
</div>
<div>
<p class="DefangedMsoNormal" style="margin:0px;"> </p>
</div>
</div>
</div>
</blockquote>