From mboxrd@z Thu Jan  1 00:00:00 1970
Subject: [PATCH] UefiPayloadPkg: Define default values for the DynamicEX PCDs
To: devel@edk2.groups.io
From: "Jiading Zhang" <jdzhang@kunluntech.com.cn>
X-Originating-Location: AU (78.31.211.224)
X-Originating-Platform: Windows Chrome 107
User-Agent: GROUPS.IO Web Poster
MIME-Version: 1.0
Date: Thu, 10 Nov 2022 16:56:29 -0800
Message-ID: <7A82.1668128189032493181.vFXn@groups.io>
Content-Type: multipart/alternative; boundary="N5eDxVj54uiHmPUK2pNo"

--N5eDxVj54uiHmPUK2pNo
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable

The following PCDs have no value in UefiPayloadPkg.dsc=EF=BC=8Cand they can=
 not pass the Ecc tool check, so assign the default values the same as they=
 are in *.dec file.
gEfiMdeModulePkgTokenSpaceGuid.PcdAriSupport
gEfiMdeModulePkgTokenSpaceGuid.PcdMrIovSupport
gEfiMdeModulePkgTokenSpaceGuid.PcdSrIovSupport
gEfiMdeModulePkgTokenSpaceGuid.PcdSrIovSystemPageSize
gUefiCpuPkgTokenSpaceGuid.PcdCpuApInitTimeOutInMicroSeconds
gUefiCpuPkgTokenSpaceGuid.PcdCpuApLoopMode
gUefiCpuPkgTokenSpaceGuid.PcdCpuMicrocodePatchAddress
gUefiCpuPkgTokenSpaceGuid.PcdCpuMicrocodePatchRegionSize

Signed-off-by: jdzhang <jdzhang@kunluntech.com.cn>
---
UefiPayloadPkg/UefiPayloadPkg.dsc | 16 ++++++++--------
1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/UefiPayloadPkg/UefiPayloadPkg.dsc b/UefiPayloadPkg/UefiPayload=
Pkg.dsc
index 1150be6acd..ce7d3a7f89 100644
--- a/UefiPayloadPkg/UefiPayloadPkg.dsc
+++ b/UefiPayloadPkg/UefiPayloadPkg.dsc
@@ -530,14 +530,14 @@
gEfiMdePkgTokenSpaceGuid.PcdUartDefaultParity|$(UART_DEFAULT_PARITY)
gEfiMdePkgTokenSpaceGuid.PcdUartDefaultStopBits|$(UART_DEFAULT_STOP_BITS)
gEfiMdePkgTokenSpaceGuid.PcdDefaultTerminalType|$(DEFAULT_TERMINAL_TYPE)
-=C2=A0 gEfiMdeModulePkgTokenSpaceGuid.PcdAriSupport
-=C2=A0 gEfiMdeModulePkgTokenSpaceGuid.PcdMrIovSupport
-=C2=A0 gEfiMdeModulePkgTokenSpaceGuid.PcdSrIovSupport
-=C2=A0 gEfiMdeModulePkgTokenSpaceGuid.PcdSrIovSystemPageSize
-=C2=A0 gUefiCpuPkgTokenSpaceGuid.PcdCpuApInitTimeOutInMicroSeconds
-=C2=A0 gUefiCpuPkgTokenSpaceGuid.PcdCpuApLoopMode
-=C2=A0 gUefiCpuPkgTokenSpaceGuid.PcdCpuMicrocodePatchAddress
-=C2=A0 gUefiCpuPkgTokenSpaceGuid.PcdCpuMicrocodePatchRegionSize
+=C2=A0 gEfiMdeModulePkgTokenSpaceGuid.PcdAriSupport|TRUE
+=C2=A0 gEfiMdeModulePkgTokenSpaceGuid.PcdMrIovSupport|FALSE
+=C2=A0 gEfiMdeModulePkgTokenSpaceGuid.PcdSrIovSupport|TRUE
+=C2=A0 gEfiMdeModulePkgTokenSpaceGuid.PcdSrIovSystemPageSize|0x1
+=C2=A0 gUefiCpuPkgTokenSpaceGuid.PcdCpuApInitTimeOutInMicroSeconds|50000
+=C2=A0 gUefiCpuPkgTokenSpaceGuid.PcdCpuApLoopMode|1
+=C2=A0 gUefiCpuPkgTokenSpaceGuid.PcdCpuMicrocodePatchAddress|0x0
+=C2=A0 gUefiCpuPkgTokenSpaceGuid.PcdCpuMicrocodePatchRegionSize|0x0
!if ($(TARGET) =3D=3D DEBUG || $(USE_CBMEM_FOR_CONSOLE) =3D=3D TRUE)
gEfiMdeModulePkgTokenSpaceGuid.PcdStatusCodeUseSerial|TRUE
!else
--
2.38.1.windows.1

--N5eDxVj54uiHmPUK2pNo
Content-Type: text/html; charset="utf-8"
Content-Transfer-Encoding: quoted-printable

<div>The following PCDs have no value in UefiPayloadPkg.dsc=EF=BC=8Cand the=
y can not pass the Ecc tool check, so assign the default values the same as=
 they are in *.dec file.</div>
<div>&nbsp; gEfiMdeModulePkgTokenSpaceGuid.PcdAriSupport</div>
<div>&nbsp; gEfiMdeModulePkgTokenSpaceGuid.PcdMrIovSupport</div>
<div>&nbsp; gEfiMdeModulePkgTokenSpaceGuid.PcdSrIovSupport</div>
<div>&nbsp; gEfiMdeModulePkgTokenSpaceGuid.PcdSrIovSystemPageSize</div>
<div>&nbsp; gUefiCpuPkgTokenSpaceGuid.PcdCpuApInitTimeOutInMicroSeconds</di=
v>
<div>&nbsp; gUefiCpuPkgTokenSpaceGuid.PcdCpuApLoopMode</div>
<div>&nbsp; gUefiCpuPkgTokenSpaceGuid.PcdCpuMicrocodePatchAddress</div>
<div>&nbsp; gUefiCpuPkgTokenSpaceGuid.PcdCpuMicrocodePatchRegionSize</div>
<div>&nbsp;</div>
<div>Signed-off-by: jdzhang &lt;jdzhang@kunluntech.com.cn&gt;</div>
<div>---</div>
<div>&nbsp;UefiPayloadPkg/UefiPayloadPkg.dsc | 16 ++++++++--------</div>
<div>&nbsp;1 file changed, 8 insertions(+), 8 deletions(-)</div>
<div>&nbsp;</div>
<div>diff --git a/UefiPayloadPkg/UefiPayloadPkg.dsc b/UefiPayloadPkg/UefiPa=
yloadPkg.dsc</div>
<div>index 1150be6acd..ce7d3a7f89 100644</div>
<div>--- a/UefiPayloadPkg/UefiPayloadPkg.dsc</div>
<div>+++ b/UefiPayloadPkg/UefiPayloadPkg.dsc</div>
<div>@@ -530,14 +530,14 @@</div>
<div>&nbsp; &nbsp;gEfiMdePkgTokenSpaceGuid.PcdUartDefaultParity|$(UART_DEFA=
ULT_PARITY)</div>
<div>&nbsp; &nbsp;gEfiMdePkgTokenSpaceGuid.PcdUartDefaultStopBits|$(UART_DE=
FAULT_STOP_BITS)</div>
<div>&nbsp; &nbsp;gEfiMdePkgTokenSpaceGuid.PcdDefaultTerminalType|$(DEFAULT=
_TERMINAL_TYPE)</div>
<div>-&nbsp; gEfiMdeModulePkgTokenSpaceGuid.PcdAriSupport</div>
<div>-&nbsp; gEfiMdeModulePkgTokenSpaceGuid.PcdMrIovSupport</div>
<div>-&nbsp; gEfiMdeModulePkgTokenSpaceGuid.PcdSrIovSupport</div>
<div>-&nbsp; gEfiMdeModulePkgTokenSpaceGuid.PcdSrIovSystemPageSize</div>
<div>-&nbsp; gUefiCpuPkgTokenSpaceGuid.PcdCpuApInitTimeOutInMicroSeconds</d=
iv>
<div>-&nbsp; gUefiCpuPkgTokenSpaceGuid.PcdCpuApLoopMode</div>
<div>-&nbsp; gUefiCpuPkgTokenSpaceGuid.PcdCpuMicrocodePatchAddress</div>
<div>-&nbsp; gUefiCpuPkgTokenSpaceGuid.PcdCpuMicrocodePatchRegionSize</div>
<div>+&nbsp; gEfiMdeModulePkgTokenSpaceGuid.PcdAriSupport|TRUE</div>
<div>+&nbsp; gEfiMdeModulePkgTokenSpaceGuid.PcdMrIovSupport|FALSE</div>
<div>+&nbsp; gEfiMdeModulePkgTokenSpaceGuid.PcdSrIovSupport|TRUE</div>
<div>+&nbsp; gEfiMdeModulePkgTokenSpaceGuid.PcdSrIovSystemPageSize|0x1</div=
>
<div>+&nbsp; gUefiCpuPkgTokenSpaceGuid.PcdCpuApInitTimeOutInMicroSeconds|50=
000</div>
<div>+&nbsp; gUefiCpuPkgTokenSpaceGuid.PcdCpuApLoopMode|1</div>
<div>+&nbsp; gUefiCpuPkgTokenSpaceGuid.PcdCpuMicrocodePatchAddress|0x0</div=
>
<div>+&nbsp; gUefiCpuPkgTokenSpaceGuid.PcdCpuMicrocodePatchRegionSize|0x0</=
div>
<div>&nbsp;!if ($(TARGET) =3D=3D DEBUG || $(USE_CBMEM_FOR_CONSOLE) =3D=3D T=
RUE)</div>
<div>&nbsp; &nbsp;gEfiMdeModulePkgTokenSpaceGuid.PcdStatusCodeUseSerial|TRU=
E</div>
<div>&nbsp;!else</div>
<div>--&nbsp;</div>
<div>2.38.1.windows.1</div>
<div>&nbsp;</div>

--N5eDxVj54uiHmPUK2pNo--