Re: [edk2] [PATCH 07/37] CorebootPayloadPkg: Removing ipf from edk2.

2018-07-11 Thread You, Benjamin
Reviewed-by: Benjamin You 

> -Original Message-
> From: Chen, Chen A
> Sent: Wednesday, June 13, 2018 11:44 AM
> To: edk2-devel@lists.01.org
> Cc: Chen, Chen A ; Ma, Maurice
> ; Agyeman, Prince ; You,
> Benjamin ; Kinney, Michael D
> 
> Subject: [PATCH 07/37] CorebootPayloadPkg: 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: Maurice Ma 
> Cc: Prince Agyeman 
> Cc: Benjamin You 
> Cc: Michael D Kinney 
> Signed-off-by: chenc2 
> Contributed-under: TianoCore Contribution Agreement 1.1
> ---
>  CorebootPayloadPkg/FbGop/FbGop.inf   | 2 +-
>  CorebootPayloadPkg/Library/AcpiTimerLib/AcpiTimerLib.inf | 4 ++--
>  CorebootPayloadPkg/Library/PciHostBridgeLib/PciHostBridgeLib.inf | 2 +-
>  CorebootPayloadPkg/Library/ResetSystemLib/ResetSystemLib.inf | 2 +-
>  4 files changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/CorebootPayloadPkg/FbGop/FbGop.inf
> b/CorebootPayloadPkg/FbGop/FbGop.inf
> index 0d97387679..b95add240b 100644
> --- a/CorebootPayloadPkg/FbGop/FbGop.inf
> +++ b/CorebootPayloadPkg/FbGop/FbGop.inf
> @@ -29,7 +29,7 @@
>  #
>  # The following information is for reference only and not required by the 
> build
> tools.
>  #
> -#  VALID_ARCHITECTURES   = IA32 X64 IPF EBC
> +#  VALID_ARCHITECTURES   = IA32 X64 EBC
>  #
>  #  DRIVER_BINDING=  gBiosVideoDriverBinding
>  #  COMPONENT_NAME=  gBiosVideoComponentName
> diff --git a/CorebootPayloadPkg/Library/AcpiTimerLib/AcpiTimerLib.inf
> b/CorebootPayloadPkg/Library/AcpiTimerLib/AcpiTimerLib.inf
> index 2bff61094a..4c5684cb09 100644
> --- a/CorebootPayloadPkg/Library/AcpiTimerLib/AcpiTimerLib.inf
> +++ b/CorebootPayloadPkg/Library/AcpiTimerLib/AcpiTimerLib.inf
> @@ -25,7 +25,7 @@
>  #
>  # The following information is for reference only and not required by the 
> build
> tools.
>  #
> -#  VALID_ARCHITECTURES   = IA32 X64 IPF EBC
> +#  VALID_ARCHITECTURES   = IA32 X64 EBC
>  #
> 
>  [Sources]
> @@ -43,4 +43,4 @@
>DebugLib
> 
>  [Guids]
> -  gUefiAcpiBoardInfoGuid
> \ No newline at end of file
> +  gUefiAcpiBoardInfoGuid
> diff --git a/CorebootPayloadPkg/Library/PciHostBridgeLib/PciHostBridgeLib.inf
> b/CorebootPayloadPkg/Library/PciHostBridgeLib/PciHostBridgeLib.inf
> index 15ffb45f84..7fc1326a70 100644
> --- a/CorebootPayloadPkg/Library/PciHostBridgeLib/PciHostBridgeLib.inf
> +++ b/CorebootPayloadPkg/Library/PciHostBridgeLib/PciHostBridgeLib.inf
> @@ -27,7 +27,7 @@
>  # The following information is for reference only and not required by the 
> build
>  # tools.
>  #
> -#  VALID_ARCHITECTURES   = IA32 X64 IPF EBC
> +#  VALID_ARCHITECTURES   = IA32 X64 EBC
>  #
> 
>  [Sources]
> diff --git a/CorebootPayloadPkg/Library/ResetSystemLib/ResetSystemLib.inf
> b/CorebootPayloadPkg/Library/ResetSystemLib/ResetSystemLib.inf
> index 1af96c6e23..9e025ecee2 100644
> --- a/CorebootPayloadPkg/Library/ResetSystemLib/ResetSystemLib.inf
> +++ b/CorebootPayloadPkg/Library/ResetSystemLib/ResetSystemLib.inf
> @@ -23,7 +23,7 @@
>  #
>  # The following information is for reference only and not required by the 
> build
> tools.
>  #
> -#  VALID_ARCHITECTURES   = IA32 X64 IPF
> +#  VALID_ARCHITECTURES   = IA32 X64
>  #
> 
>  [Sources]
> --
> 2.16.2.windows.1

___
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel


[edk2] [PATCH 07/37] CorebootPayloadPkg: 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: Maurice Ma 
Cc: Prince Agyeman 
Cc: Benjamin You 
Cc: Michael D Kinney 
Signed-off-by: chenc2 
Contributed-under: TianoCore Contribution Agreement 1.1
---
 CorebootPayloadPkg/FbGop/FbGop.inf   | 2 +-
 CorebootPayloadPkg/Library/AcpiTimerLib/AcpiTimerLib.inf | 4 ++--
 CorebootPayloadPkg/Library/PciHostBridgeLib/PciHostBridgeLib.inf | 2 +-
 CorebootPayloadPkg/Library/ResetSystemLib/ResetSystemLib.inf | 2 +-
 4 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/CorebootPayloadPkg/FbGop/FbGop.inf 
b/CorebootPayloadPkg/FbGop/FbGop.inf
index 0d97387679..b95add240b 100644
--- a/CorebootPayloadPkg/FbGop/FbGop.inf
+++ b/CorebootPayloadPkg/FbGop/FbGop.inf
@@ -29,7 +29,7 @@
 #
 # The following information is for reference only and not required by the 
build tools.
 #
-#  VALID_ARCHITECTURES   = IA32 X64 IPF EBC
+#  VALID_ARCHITECTURES   = IA32 X64 EBC
 #
 #  DRIVER_BINDING=  gBiosVideoDriverBinding
 #  COMPONENT_NAME=  gBiosVideoComponentName
diff --git a/CorebootPayloadPkg/Library/AcpiTimerLib/AcpiTimerLib.inf 
b/CorebootPayloadPkg/Library/AcpiTimerLib/AcpiTimerLib.inf
index 2bff61094a..4c5684cb09 100644
--- a/CorebootPayloadPkg/Library/AcpiTimerLib/AcpiTimerLib.inf
+++ b/CorebootPayloadPkg/Library/AcpiTimerLib/AcpiTimerLib.inf
@@ -25,7 +25,7 @@
 #
 # The following information is for reference only and not required by the 
build tools.
 #
-#  VALID_ARCHITECTURES   = IA32 X64 IPF EBC
+#  VALID_ARCHITECTURES   = IA32 X64 EBC
 #
 
 [Sources]
@@ -43,4 +43,4 @@
   DebugLib
   
 [Guids]  
-  gUefiAcpiBoardInfoGuid
\ No newline at end of file
+  gUefiAcpiBoardInfoGuid
diff --git a/CorebootPayloadPkg/Library/PciHostBridgeLib/PciHostBridgeLib.inf 
b/CorebootPayloadPkg/Library/PciHostBridgeLib/PciHostBridgeLib.inf
index 15ffb45f84..7fc1326a70 100644
--- a/CorebootPayloadPkg/Library/PciHostBridgeLib/PciHostBridgeLib.inf
+++ b/CorebootPayloadPkg/Library/PciHostBridgeLib/PciHostBridgeLib.inf
@@ -27,7 +27,7 @@
 # The following information is for reference only and not required by the build
 # tools.
 #
-#  VALID_ARCHITECTURES   = IA32 X64 IPF EBC
+#  VALID_ARCHITECTURES   = IA32 X64 EBC
 #
 
 [Sources]
diff --git a/CorebootPayloadPkg/Library/ResetSystemLib/ResetSystemLib.inf 
b/CorebootPayloadPkg/Library/ResetSystemLib/ResetSystemLib.inf
index 1af96c6e23..9e025ecee2 100644
--- a/CorebootPayloadPkg/Library/ResetSystemLib/ResetSystemLib.inf
+++ b/CorebootPayloadPkg/Library/ResetSystemLib/ResetSystemLib.inf
@@ -23,7 +23,7 @@
 #
 # The following information is for reference only and not required by the 
build tools.
 #
-#  VALID_ARCHITECTURES   = IA32 X64 IPF
+#  VALID_ARCHITECTURES   = IA32 X64
 #
 
 [Sources]
-- 
2.16.2.windows.1

___
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel