Re: [edk2] [PATCH 17/37] IntelFrameworkModulePkg: Removing ipf from edk2.

2018-06-12 Thread Gao, Liming
Reviewed-by: Liming Gao 

>-Original Message-
>From: Chen, Chen A
>Sent: Wednesday, June 13, 2018 11:44 AM
>To: edk2-devel@lists.01.org
>Cc: Chen, Chen A ; Gao, Liming
>; Kinney, Michael D 
>Subject: [PATCH 17/37] IntelFrameworkModulePkg: Removing ipf from edk2.
>
>Removing rules for Ipf sources file:
>* Remove the source file which path with "ipf" and also listed in
>  [Sources.IPF] section of INF file.
>* Remove the source file which listed in [Components.IPF] section
>  of DSC file and not listed in any other [Components] section.
>* Remove the embedded Ipf code for MDE_CPU_IPF.
>
>Removing rules for Inf file:
>* Remove IPF from VALID_ARCHITECTURES comments.
>* Remove DXE_SAL_DRIVER from LIBRARY_CLASS in [Defines] section.
>* Remove the INF which only listed in [Components.IPF] section in DSC.
>* Remove statements from [BuildOptions] that provide IPF specific flags.
>* Remove any IPF sepcific sections.
>
>Removing rules for Dec file:
>* Remove [Includes.IPF] section from Dec.
>
>Removing rules for Dsc file:
>* Remove IPF from SUPPORTED_ARCHITECTURES in [Defines] section of DSC.
>* Remove any IPF specific sections.
>* Remove statements from [BuildOptions] that provide IPF specific flags.
>
>Cc: Liming Gao 
>Cc: Michael D Kinney 
>Signed-off-by: chenc2 
>Contributed-under: TianoCore Contribution Agreement 1.1
>---
> .../Bus/Isa/IsaBusDxe/IsaBusDxe.inf|   2 +-
> .../Bus/Isa/IsaFloppyDxe/IsaFloppyDxe.inf  |   2 +-
> .../Bus/Isa/IsaFloppyPei/IsaFloppyPei.inf  |   2 +-
> .../Bus/Isa/IsaIoDxe/IsaIoDxe.inf  |   2 +-
> .../Bus/Isa/IsaSerialDxe/IsaSerialDxe.inf  |   2 +-
> .../Bus/Isa/Ps2KeyboardDxe/Ps2keyboardDxe.inf  |   2 +-
> .../Ps2MouseAbsolutePointerDxe.inf |   2 +-
> .../Bus/Isa/Ps2MouseDxe/Ps2MouseDxe.inf|   2 +-
> .../Bus/Pci/IdeBusDxe/IdeBusDxe.inf|   2 +-
> .../Bus/Pci/VgaMiniPortDxe/VgaMiniPortDxe.inf  |   2 +-
> .../Csm/BiosThunk/KeyboardDxe/KeyboardDxe.inf  |   2 +-
> .../Csm/BiosThunk/Snp16Dxe/Snp16Dxe.inf|   2 +-
> .../Csm/BiosThunk/VideoDxe/VideoDxe.inf|   2 +-
> .../Csm/LegacyBiosDxe/Ipf/IpfBootSupport.c | 277 ---
> .../Csm/LegacyBiosDxe/Ipf/IpfThunk.h   | 102 
> .../Csm/LegacyBiosDxe/Ipf/IpfThunk.i   |  89 
> .../Csm/LegacyBiosDxe/Ipf/IpfThunk.s   | 524 
> .../Csm/LegacyBiosDxe/Ipf/Thunk.c  | 550 -
> .../Csm/LegacyBiosDxe/LegacyBiosDxe.inf|  14 +-
> .../Csm/LegacyBiosDxe/LegacyBiosInterface.h|  79 ---
> .../IntelFrameworkModulePkg.dsc|  32 +-
> .../BaseUefiTianoCustomDecompressLib.inf   |   2 +-
> .../Library/DxeCapsuleLib/DxeCapsuleLib.inf|   2 +-
> .../DxeReportStatusCodeLib.inf |   4 +-
> .../Library/GenericBdsLib/GenericBdsLib.inf|   2 +-
> .../LegacyBootMaintUiLib/LegacyBootMaintUiLib.inf  |   2 +-
> .../LzmaCustomDecompressLib.inf|   2 +-
> .../PeiDxeDebugLibReportStatusCode.inf |   4 +-
> .../Library/PeiRecoveryLib/PeiRecoveryLib.inf  |   4 +-
> .../Library/PeiS3Lib/PeiS3Lib.inf  |   2 +-
> .../PlatformBdsLibNull/PlatformBdsLibNull.inf  |   4 +-
> .../Acpi/AcpiSupportDxe/AcpiSupportDxe.inf |   2 +-
> .../Universal/BdsDxe/BdsDxe.inf|   2 +-
> .../Universal/Console/VgaClassDxe/VgaClassDxe.inf  |   2 +-
> .../Universal/DataHubDxe/DataHubDxe.inf|   2 +-
> .../DataHubStdErrDxe/DataHubStdErrDxe.inf  |   2 +-
> .../Universal/FirmwareVolume/FwVolDxe/FwVolDxe.inf |   2 +-
> .../UpdateDriverDxe/UpdateDriverDxe.inf|   2 +-
> .../Universal/LegacyRegionDxe/LegacyRegionDxe.inf  |   2 +-
> .../SectionExtractionDxe/SectionExtractionDxe.inf  |   2 +-
> .../DatahubStatusCodeHandlerDxe.inf|   2 +-
> .../Universal/StatusCode/Pei/StatusCodePei.inf |   2 +-
> 42 files changed, 41 insertions(+), 1702 deletions(-)
> delete mode 100644
>IntelFrameworkModulePkg/Csm/LegacyBiosDxe/Ipf/IpfBootSupport.c
> delete mode 100644
>IntelFrameworkModulePkg/Csm/LegacyBiosDxe/Ipf/IpfThunk.h
> delete mode 100644
>IntelFrameworkModulePkg/Csm/LegacyBiosDxe/Ipf/IpfThunk.i
> delete mode 100644
>IntelFrameworkModulePkg/Csm/LegacyBiosDxe/Ipf/IpfThunk.s
> delete mode 100644
>IntelFrameworkModulePkg/Csm/LegacyBiosDxe/Ipf/Thunk.c
>
>diff --git a/IntelFrameworkModulePkg/Bus/Isa/IsaBusDxe/IsaBusDxe.inf
>b/IntelFrameworkModulePkg/Bus/Isa/IsaBusDxe/IsaBusDxe.inf
>index be6dd43ea6..af9342ecd1 100644
>--- a/IntelFrameworkModulePkg/Bus/Isa/IsaBusDxe/IsaBusDxe.inf
>+++ b/IntelFrameworkModulePkg/Bus/Isa/IsaBusDxe/IsaBusDxe.inf
>@@ -27,7 +27,7 @@
>   ENTRY_POINT= InitializeIsaBus
>
> #
>-#  VALID_ARCHITECTURES   = IA32 X64 IPF EBC
>+#  VALID_ARCHITECTURES   = IA32 X64 EBC
> #  DRIVER_BINDING= gIsaBusControllerDriver
> #  CO

[edk2] [PATCH 17/37] IntelFrameworkModulePkg: Removing ipf from edk2.

2018-06-12 Thread chenc2
Removing rules for Ipf sources file:
* Remove the source file which path with "ipf" and also listed in
  [Sources.IPF] section of INF file.
* Remove the source file which listed in [Components.IPF] section
  of DSC file and not listed in any other [Components] section.
* Remove the embedded Ipf code for MDE_CPU_IPF.

Removing rules for Inf file:
* Remove IPF from VALID_ARCHITECTURES comments.
* Remove DXE_SAL_DRIVER from LIBRARY_CLASS in [Defines] section.
* Remove the INF which only listed in [Components.IPF] section in DSC.
* Remove statements from [BuildOptions] that provide IPF specific flags.
* Remove any IPF sepcific sections.

Removing rules for Dec file:
* Remove [Includes.IPF] section from Dec.

Removing rules for Dsc file:
* Remove IPF from SUPPORTED_ARCHITECTURES in [Defines] section of DSC.
* Remove any IPF specific sections.
* Remove statements from [BuildOptions] that provide IPF specific flags.

Cc: Liming Gao 
Cc: Michael D Kinney 
Signed-off-by: chenc2 
Contributed-under: TianoCore Contribution Agreement 1.1
---
 .../Bus/Isa/IsaBusDxe/IsaBusDxe.inf|   2 +-
 .../Bus/Isa/IsaFloppyDxe/IsaFloppyDxe.inf  |   2 +-
 .../Bus/Isa/IsaFloppyPei/IsaFloppyPei.inf  |   2 +-
 .../Bus/Isa/IsaIoDxe/IsaIoDxe.inf  |   2 +-
 .../Bus/Isa/IsaSerialDxe/IsaSerialDxe.inf  |   2 +-
 .../Bus/Isa/Ps2KeyboardDxe/Ps2keyboardDxe.inf  |   2 +-
 .../Ps2MouseAbsolutePointerDxe.inf |   2 +-
 .../Bus/Isa/Ps2MouseDxe/Ps2MouseDxe.inf|   2 +-
 .../Bus/Pci/IdeBusDxe/IdeBusDxe.inf|   2 +-
 .../Bus/Pci/VgaMiniPortDxe/VgaMiniPortDxe.inf  |   2 +-
 .../Csm/BiosThunk/KeyboardDxe/KeyboardDxe.inf  |   2 +-
 .../Csm/BiosThunk/Snp16Dxe/Snp16Dxe.inf|   2 +-
 .../Csm/BiosThunk/VideoDxe/VideoDxe.inf|   2 +-
 .../Csm/LegacyBiosDxe/Ipf/IpfBootSupport.c | 277 ---
 .../Csm/LegacyBiosDxe/Ipf/IpfThunk.h   | 102 
 .../Csm/LegacyBiosDxe/Ipf/IpfThunk.i   |  89 
 .../Csm/LegacyBiosDxe/Ipf/IpfThunk.s   | 524 
 .../Csm/LegacyBiosDxe/Ipf/Thunk.c  | 550 -
 .../Csm/LegacyBiosDxe/LegacyBiosDxe.inf|  14 +-
 .../Csm/LegacyBiosDxe/LegacyBiosInterface.h|  79 ---
 .../IntelFrameworkModulePkg.dsc|  32 +-
 .../BaseUefiTianoCustomDecompressLib.inf   |   2 +-
 .../Library/DxeCapsuleLib/DxeCapsuleLib.inf|   2 +-
 .../DxeReportStatusCodeLib.inf |   4 +-
 .../Library/GenericBdsLib/GenericBdsLib.inf|   2 +-
 .../LegacyBootMaintUiLib/LegacyBootMaintUiLib.inf  |   2 +-
 .../LzmaCustomDecompressLib.inf|   2 +-
 .../PeiDxeDebugLibReportStatusCode.inf |   4 +-
 .../Library/PeiRecoveryLib/PeiRecoveryLib.inf  |   4 +-
 .../Library/PeiS3Lib/PeiS3Lib.inf  |   2 +-
 .../PlatformBdsLibNull/PlatformBdsLibNull.inf  |   4 +-
 .../Acpi/AcpiSupportDxe/AcpiSupportDxe.inf |   2 +-
 .../Universal/BdsDxe/BdsDxe.inf|   2 +-
 .../Universal/Console/VgaClassDxe/VgaClassDxe.inf  |   2 +-
 .../Universal/DataHubDxe/DataHubDxe.inf|   2 +-
 .../DataHubStdErrDxe/DataHubStdErrDxe.inf  |   2 +-
 .../Universal/FirmwareVolume/FwVolDxe/FwVolDxe.inf |   2 +-
 .../UpdateDriverDxe/UpdateDriverDxe.inf|   2 +-
 .../Universal/LegacyRegionDxe/LegacyRegionDxe.inf  |   2 +-
 .../SectionExtractionDxe/SectionExtractionDxe.inf  |   2 +-
 .../DatahubStatusCodeHandlerDxe.inf|   2 +-
 .../Universal/StatusCode/Pei/StatusCodePei.inf |   2 +-
 42 files changed, 41 insertions(+), 1702 deletions(-)
 delete mode 100644 
IntelFrameworkModulePkg/Csm/LegacyBiosDxe/Ipf/IpfBootSupport.c
 delete mode 100644 IntelFrameworkModulePkg/Csm/LegacyBiosDxe/Ipf/IpfThunk.h
 delete mode 100644 IntelFrameworkModulePkg/Csm/LegacyBiosDxe/Ipf/IpfThunk.i
 delete mode 100644 IntelFrameworkModulePkg/Csm/LegacyBiosDxe/Ipf/IpfThunk.s
 delete mode 100644 IntelFrameworkModulePkg/Csm/LegacyBiosDxe/Ipf/Thunk.c

diff --git a/IntelFrameworkModulePkg/Bus/Isa/IsaBusDxe/IsaBusDxe.inf 
b/IntelFrameworkModulePkg/Bus/Isa/IsaBusDxe/IsaBusDxe.inf
index be6dd43ea6..af9342ecd1 100644
--- a/IntelFrameworkModulePkg/Bus/Isa/IsaBusDxe/IsaBusDxe.inf
+++ b/IntelFrameworkModulePkg/Bus/Isa/IsaBusDxe/IsaBusDxe.inf
@@ -27,7 +27,7 @@
   ENTRY_POINT= InitializeIsaBus
 
 #
-#  VALID_ARCHITECTURES   = IA32 X64 IPF EBC
+#  VALID_ARCHITECTURES   = IA32 X64 EBC
 #  DRIVER_BINDING= gIsaBusControllerDriver
 #  COMPONENT_NAME= gIsaBusComponentName;
 #  COMPONENT_NAME2   = gIsaBusComponentName2;
diff --git a/IntelFrameworkModulePkg/Bus/Isa/IsaFloppyDxe/IsaFloppyDxe.inf 
b/IntelFrameworkModulePkg/Bus/Isa/IsaFloppyDxe/IsaFloppyDxe.inf
index 65756209f5..f911ffe15c 100644
--- a/IntelFrameworkModulePkg/Bus/Isa/IsaFloppyDxe/IsaFloppyDxe.inf
+++ b/Int