Re: [edk2-devel] [edk2-platforms/devel-MinPlatform][PATCH v1 0/6] Some cleanup for platforms under ClevoOpenBoardPkg & KabylakeOpenBoardPkg

["Kubacki, Michael A" <michael.a.kubacki@intel.com>]

Reviewed-by: Michael Kubacki <michael.a.kubacki@intel.com>

> -----Original Message-----
> From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of
> Wu, Hao A
> Sent: Wednesday, April 3, 2019 10:57 PM
> To: devel@edk2.groups.io
> Cc: Wu, Hao A <hao.a.wu@intel.com>; Kubacki, Michael A
> <michael.a.kubacki@intel.com>; Chiu, Chasel <chasel.chiu@intel.com>;
> Sinha, Ankit <ankit.sinha@intel.com>; Desimone, Nathaniel L
> <nathaniel.l.desimone@intel.com>
> Subject: [edk2-devel] [edk2-platforms/devel-MinPlatform][PATCH v1 0/6]
> Some cleanup for platforms under ClevoOpenBoardPkg &
> KabylakeOpenBoardPkg
> 
> The series will do the below cleanups for platforms under
> ClevoOpenBoardPkg and KabylakeOpenBoardPkg:
> 
> A) Remove unused PCD 'PcdUseHpetTimer';
> B) Remove the unnecessary consumption of the 8259-related PCDs;
> C) Remove the unused duplicated Legacy8259 protocol GUID.
> 
> 
> Test done for the series:
> Build pass for N1xxWU & KabyLakeRvp3 platforms.
> 
> Cc: Michael A Kubacki <michael.a.kubacki@intel.com>
> Cc: Chasel Chiu <chasel.chiu@intel.com>
> Cc: Ankit Sinha <ankit.sinha@intel.com>
> Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
> 
> Hao Wu (6):
>   ClevoOpenBoardPkg/N1xxWU: Remove unused PCD 'PcdUseHpetTimer'
>   ClevoOpenBoardPkg/N1xxWU: Remove the consume of 8259-related PCD
>   KabylakeOpenBoardPkg/KBLRvp3: Remove unused PCD 'PcdUseHpetTimer'
>   KabylakeOpenBoardPkg/KBLRvp3: Remove the consume of 8259-related
> PCD
>   KabylakeSiliconPkg: Remove unused PCD 'PcdUseHpetTimer'
>   KabylakeSiliconPkg/SiPkg.dec: Remove Legacy8259 protocol GUID
> 
>  Silicon/Intel/KabylakeSiliconPkg/SiPkg.dec                              | 4 +---
>  Platform/Intel/ClevoOpenBoardPkg/N1xxWU/OpenBoardPkgConfig.dsc
> | 1 -
>  Platform/Intel/ClevoOpenBoardPkg/N1xxWU/OpenBoardPkgPcd.dsc
> | 2 --
> 
> Platform/Intel/KabylakeOpenBoardPkg/KabylakeRvp3/OpenBoardPkgConfig.
> dsc | 3 +--
> 
> Platform/Intel/KabylakeOpenBoardPkg/KabylakeRvp3/OpenBoardPkgPcd.ds
> c    | 5 +----
>  Silicon/Intel/KabylakeSiliconPkg/KabylakeSiliconPkg.dsc                 | 3 +--
>  6 files changed, 4 insertions(+), 14 deletions(-)
> 
> --
> 2.12.0.windows.1
> 
> 
>