public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Wu, Hao A" <hao.a.wu@intel.com>
To: "Chen, Chen A" <chen.a.chen@intel.com>,
	"edk2-devel@lists.01.org" <edk2-devel@lists.01.org>
Cc: "Ni, Ruiyu" <ruiyu.ni@intel.com>,
	"Kinney, Michael D" <michael.d.kinney@intel.com>
Subject: Re: [PATCH 23/37] Nt32Pkg: Removing ipf which is no longer supported from edk2.
Date: Wed, 13 Jun 2018 07:56:55 +0000	[thread overview]
Message-ID: <B80AF82E9BFB8E4FBD8C89DA810C6A0931E0876D@SHSMSX104.ccr.corp.intel.com> (raw)
In-Reply-To: <20180613034458.2408-1-chen.a.chen@intel.com>

Reviewed-by: Hao Wu <hao.a.wu@intel.com>

Best Regards,
Hao Wu

> -----Original Message-----
> From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of
> chenc2
> Sent: Wednesday, June 13, 2018 11:45 AM
> To: edk2-devel@lists.01.org
> Cc: Ni, Ruiyu; Wu, Hao A; Kinney, Michael D
> Subject: [edk2] [PATCH 23/37] Nt32Pkg: Removing ipf which is no longer
> supported 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: Ruiyu Ni <ruiyu.ni@intel.com>
> Cc: Hao Wu <hao.a.wu@intel.com>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Signed-off-by: chenc2 <chen.a.chen@intel.com>
> Contributed-under: TianoCore Contribution Agreement 1.1
> ---
>  Nt32Pkg/BootModePei/BootModePei.inf                                   | 4 ++--
>  .../DxeNt32OemHookStatusCodeLib/DxeNt32OemHookStatusCodeLib.inf       |
> 2 +-
>  Nt32Pkg/Library/PlatformSecureLib/PlatformSecureLib.inf               | 2 +-
>  Nt32Pkg/SnpNt32Dxe/SnpNt32Dxe.inf                                     | 2 +-
>  Nt32Pkg/StallPei/StallPei.inf                                         | 2 +-
>  Nt32Pkg/WinNtFirmwareVolumePei/WinNtFirmwareVolumePei.inf             | 4
> ++--
>  Nt32Pkg/WinNtFlashMapPei/WinNtFlashMapPei.inf                         | 2 +-
>  7 files changed, 9 insertions(+), 9 deletions(-)
> 
> diff --git a/Nt32Pkg/BootModePei/BootModePei.inf
> b/Nt32Pkg/BootModePei/BootModePei.inf
> index 43d6476734..0d3357b204 100644
> --- a/Nt32Pkg/BootModePei/BootModePei.inf
> +++ b/Nt32Pkg/BootModePei/BootModePei.inf
> @@ -26,7 +26,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]
> @@ -47,4 +47,4 @@
>    gEfiPeiBootInRecoveryModePpiGuid              # PPI SOMETIMES_PRODUCED
> 
>  [depex]
> -  TRUE
> \ No newline at end of file
> +  TRUE
> diff --git
> a/Nt32Pkg/Library/DxeNt32OemHookStatusCodeLib/DxeNt32OemHookStatusC
> odeLib.inf
> b/Nt32Pkg/Library/DxeNt32OemHookStatusCodeLib/DxeNt32OemHookStatusC
> odeLib.inf
> index 6d2a33b44d..5874ea651c 100644
> ---
> a/Nt32Pkg/Library/DxeNt32OemHookStatusCodeLib/DxeNt32OemHookStatusC
> odeLib.inf
> +++
> b/Nt32Pkg/Library/DxeNt32OemHookStatusCodeLib/DxeNt32OemHookStatusC
> odeLib.inf
> @@ -21,7 +21,7 @@
>    FILE_GUID                      = EF172A23-C7C5-47b7-B24E-D10DFE15540F
>    MODULE_TYPE                    = DXE_DRIVER
>    VERSION_STRING                 = 1.0
> -  LIBRARY_CLASS                  = OemHookStatusCodeLib|DXE_DRIVER
> DXE_RUNTIME_DRIVER DXE_SAL_DRIVER DXE_SMM_DRIVER
> UEFI_APPLICATION UEFI_DRIVER
> +  LIBRARY_CLASS                  = OemHookStatusCodeLib|DXE_DRIVER
> DXE_RUNTIME_DRIVER DXE_SMM_DRIVER UEFI_APPLICATION UEFI_DRIVER
> 
>  #
>  # The following information is for reference only and not required by the build
> tools.
> diff --git a/Nt32Pkg/Library/PlatformSecureLib/PlatformSecureLib.inf
> b/Nt32Pkg/Library/PlatformSecureLib/PlatformSecureLib.inf
> index a6891dd5f3..aef0ff421a 100644
> --- a/Nt32Pkg/Library/PlatformSecureLib/PlatformSecureLib.inf
> +++ b/Nt32Pkg/Library/PlatformSecureLib/PlatformSecureLib.inf
> @@ -23,7 +23,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/Nt32Pkg/SnpNt32Dxe/SnpNt32Dxe.inf
> b/Nt32Pkg/SnpNt32Dxe/SnpNt32Dxe.inf
> index 70449015fa..4f421927ee 100644
> --- a/Nt32Pkg/SnpNt32Dxe/SnpNt32Dxe.inf
> +++ b/Nt32Pkg/SnpNt32Dxe/SnpNt32Dxe.inf
> @@ -26,7 +26,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/Nt32Pkg/StallPei/StallPei.inf b/Nt32Pkg/StallPei/StallPei.inf
> index 31020ed7ed..b6b0588eef 100644
> --- a/Nt32Pkg/StallPei/StallPei.inf
> +++ b/Nt32Pkg/StallPei/StallPei.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]
> diff --git a/Nt32Pkg/WinNtFirmwareVolumePei/WinNtFirmwareVolumePei.inf
> b/Nt32Pkg/WinNtFirmwareVolumePei/WinNtFirmwareVolumePei.inf
> index 721f1b374c..dee7b8769e 100644
> --- a/Nt32Pkg/WinNtFirmwareVolumePei/WinNtFirmwareVolumePei.inf
> +++ b/Nt32Pkg/WinNtFirmwareVolumePei/WinNtFirmwareVolumePei.inf
> @@ -26,7 +26,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]
> @@ -58,4 +58,4 @@
>    gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageVariableSize
> 
>  [Depex]
> -  gNtFwhPpiGuid AND gEfiPeiMemoryDiscoveredPpiGuid
> \ No newline at end of file
> +  gNtFwhPpiGuid AND gEfiPeiMemoryDiscoveredPpiGuid
> diff --git a/Nt32Pkg/WinNtFlashMapPei/WinNtFlashMapPei.inf
> b/Nt32Pkg/WinNtFlashMapPei/WinNtFlashMapPei.inf
> index f5a14af519..9d9fe54d04 100644
> --- a/Nt32Pkg/WinNtFlashMapPei/WinNtFlashMapPei.inf
> +++ b/Nt32Pkg/WinNtFlashMapPei/WinNtFlashMapPei.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]
> --
> 2.16.2.windows.1
> 
> _______________________________________________
> edk2-devel mailing list
> edk2-devel@lists.01.org
> https://lists.01.org/mailman/listinfo/edk2-devel


      reply	other threads:[~2018-06-13  7:56 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-06-13  3:44 [PATCH 23/37] Nt32Pkg: Removing ipf which is no longer supported from edk2 chenc2
2018-06-13  7:56 ` Wu, Hao A [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=B80AF82E9BFB8E4FBD8C89DA810C6A0931E0876D@SHSMSX104.ccr.corp.intel.com \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox