Patch pushed: fe5da0927aad98f3c005088197fa30c1b8f9d3e8
> -Original Message-
> From: Lou, Yun
> Sent: Thursday, April 15, 2021 2:49 PM
> To: devel@edk2.groups.io
> Cc: Lou, Yun ; Chiu, Chasel ;
> Desimone, Nathaniel L ; Zeng, Star
> ; Ni, Ray
> Subject: [PATCH v2] IntelFsp2WrapperPkg:
Reviewed-by: Nate DeSimone
-Original Message-
From: Lou, Yun
Sent: Wednesday, April 14, 2021 11:49 PM
To: devel@edk2.groups.io
Cc: Lou, Yun ; Chiu, Chasel ;
Desimone, Nathaniel L ; Zeng, Star
; Ni, Ray
Subject: [PATCH v2] IntelFsp2WrapperPkg: Remove microcode related PCDs
REF: https
Hi Chasel & Star,
So far, the PCD usage has been removed from all of these platforms:
1. Server platforms("ServerGen2")
2. Client platforms("ClientMaster")
3. Edk2-platform opensource
Thanks
Jason Lou
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Rep
Agree with you Chasel.
Deprecating them step by step, it will benefit further code sync for platforms
consuming the PCDs.
Thanks,
Star
-Original Message-
From: Chiu, Chasel
Sent: Thursday, April 15, 2021 3:26 PM
To: Lou, Yun ; devel@edk2.groups.io
Cc: Desimone, Nathaniel L ; Zeng, Star
Hi Yun,
I would recommend that we split this patch and remove PCD from DEC as last step
after all involved platforms not consuming them, what do you think?
Thanks,
Chasel
> -Original Message-
> From: Lou, Yun
> Sent: Thursday, April 15, 2021 2:49 PM
> To: devel@edk2.groups.io
> Cc:
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3334
IntelFsp2WrapperPkg defines following PCDs:
PcdCpuMicrocodePatchAddress
PcdCpuMicrocodePatchRegionSize
PcdFlashMicrocodeOffset
But the PCD name caused confusion because UefiCpuPkg defines:
PcdCpuMicrocodePatchAddress
PcdCpuMicroco