public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Michael D Kinney" <michael.d.kinney@intel.com>
To: "devel@edk2.groups.io" <devel@edk2.groups.io>,
	"Kinney, Michael D" <michael.d.kinney@intel.com>,
	"Kinney, Michael D" <michael.d.kinney@intel.com>
Cc: "Sun, Zailiang" <zailiang.sun@intel.com>,
	"Qian, Yi" <yi.qian@intel.com>, Gary Lin <glin@suse.com>
Subject: Re: [edk2-devel] [edk2-platforms Patch 00/28] Vlv2TbltDevicePkg: Fix Linux build issues
Date: Sat, 13 Jul 2019 00:18:18 +0000	[thread overview]
Message-ID: <E92EE9817A31E24EB0585FDF735412F5B9D73E94@ORSMSX113.amr.corp.intel.com> (raw)
In-Reply-To: <20190710190516.6012-1-michael.d.kinney@intel.com>

For all patches in this series authored by Gary Lin

Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com>

Mike


> -----Original Message-----
> From: devel@edk2.groups.io [mailto:devel@edk2.groups.io]
> On Behalf Of Michael D Kinney
> Sent: Wednesday, July 10, 2019 12:05 PM
> To: devel@edk2.groups.io
> Cc: Sun, Zailiang <zailiang.sun@intel.com>; Qian, Yi
> <yi.qian@intel.com>; Gary Lin <glin@suse.com>
> Subject: [edk2-devel] [edk2-platforms Patch 00/28]
> Vlv2TbltDevicePkg: Fix Linux build issues
> 
> * Update Readme.md for multiple repos
> * Clean up all build scripts and make Linux and Windows
> support consistent
> * Add IA32 GCC build support and only build capsules for
> X64
> * Clean up library mappings, add TlsLib for HTTPS Boot,
> and add SD/MMC drivers.
> * Re-enable BIOS ID feature using new GenBiosId.py tool
> * Remove redundant declarations from DEC file and remove
> unused content
> * Add missing header files in [Sources] sections of INF
> files.
> * Vlv2DeviceRefCodePlg/AcpiTablesPCAT: Fix ASL build
> error using newer iasl.
> * Vlvt2TbltDevicePkg: Fix NULL pointer access in
> SmbiosMiscDxe
> * Vlv2TbltDevicePkg/AcpiPlatform: Fix the error from
> InstallAcpiTable and
>   remove unused local variable.
> 
> Cc: Zailiang Sun <zailiang.sun@intel.com>
> Cc: Yi Qian <yi.qian@intel.com>
> Signed-off-by: Michael D Kinney
> <michael.d.kinney@intel.com>
> Signed-off-by: Gary Lin <glin@suse.com>
> 
> Gary Lin (18):
>   Vlv2TbltDevicePkg/Build_IFWI.sh: Add "set -e" to exit
> on error
>   Vlv2TbltDevicePkg/Build_IFWI.sh: Change the root
> directory path
>   Vlv2TbltDevicePkg/bld_vlv.sh: Fix the log name
>   Vlv2TbltDevicePkg/bld_vlv.sh: Update the gcc version
> detection
>   Vlv2TbltDevicePkg/bld_vlv.sh: Remove ECP completely
>   Vlv2TbltDevicePkg/bld_vlv.sh: Update the script to fit
> edk2-platforms
>   Vlv2TbltDevicePkg: Add the missing headers to inf files
>   Vlv2TbltDevicePkg/bld_vlv.sh: Remove BiosId.env
> completely
>   Vlv2TbltDevicePkg/bld_vlv.sh: Correct the path to
>     PlatformCapsuleGcc.dsc
>   Vlv2TbltDevicePkg/bld_vlv.sh: Create Vlv.ROM
>   Vlv2TbltDevicePkg/GenerateCapsule: Fix the bash scripts
>   Vlv2TbltDevicePkg/AcpiPlatform: Remove a unused
> variable
>   Vlv2TbltDevicePkg/AcpiPlatform: Fix the error from
> InstallAcpiTable
>   Vlv2TbltDevicePkg/SmBiosMiscDxe: Fix the indentation
>   Vlv2TbltDevicePkg: Import SD/MMC drivers
>   Vlv2TbltDevicePkg/Build_IFWI.sh: Check the WORKSPACE
> env variable
>   Vlv2TbltDevicePkg: Import TlsLib for HTTPS Boot
>   Vlv2TbltDevicePkg: Reorganize the libraries
> 
> Michael D Kinney (10):
>   Vlv2TbltDevicePkg: Update Linux build scripts
>   Vlvt2TbltDevicePkg: Fix NULL pointer access in
> SmbiosMiscDxe
>   Vlv2DeviceRefCodePkg: Remove redundant
> gEfiSpiProtocolGuid
>   Vlv2TbltDevicePkg: Remove redundant gEfiSpiProtocolGuid
>   Vlv2TbltDevicePkg: Re-enable BIOS ID feature
>   Vlv2TbltDevicePkg: Add GCC IA32 build support
>   Vlv2DeviceRefCodePlg/AcpiTablesPCAT: Fix ASL error
>   Vlv2TbltDevicePkg: Clean up all build scripts
>   Vlv2TbltDevicePkg: Only build capsules for X64
>   Vlv2TbltDevicePkg: Update Readme.md for multiple repos
> 
>  Platform/Intel/Vlv2TbltDevicePkg/.gitignore   |   5 -
>  .../AcpiPlatform/AcpiPlatform.c               |   5 +-
>  .../Intel/Vlv2TbltDevicePkg/Build_IFWI.bat    |  75 +---
>  .../Intel/Vlv2TbltDevicePkg/Build_IFWI.sh     |  76 ++--
>  .../Capsule/GenerateCapsule/GenCapsuleAll.sh  |  37 +-
>  .../GenerateCapsule/GenCapsuleMinnowMax.sh    |  24 +-
>  .../GenCapsuleMinnowMaxRelease.sh             |  19 +-
>  .../Include/Guid/PlatformCpuInfo.h            |  16 +-
>  .../Include/Library/BiosIdLib.h               |  86 ----
> -
>  .../Include/Library/CpuIA32.h                 | 345 ----
> --------------
>  .../Include/PlatformDefinitions.h             |  43 ---
>  .../Library/BiosIdLib/BiosIdLib.c             |  98 ----
> -
>  .../Library/BiosIdLib/BiosIdLib.inf           |  33 --
>  .../Library/CpuIA32Lib/CpuIA32Lib.inf         |  41 ---
>  .../Library/CpuIA32Lib/EfiCpuVersion.c        |  70 ----
>  .../Library/CpuIA32Lib/IA32/CpuIA32.S         | 223 ----
> -------
>  .../Library/CpuIA32Lib/IA32/CpuIA32.asm       | 206 ----
> -------
>  .../Library/CpuIA32Lib/IA32/CpuIA32.c         | 177 ----
> -----
>  .../Library/CpuIA32Lib/X64/Cpu.S              | 207 ----
> -------
>  .../Library/CpuIA32Lib/X64/Cpu.asm            | 222 ----
> -------
>  .../Library/FlashDeviceLib/FlashDeviceLib.inf |   2 +-
>  .../Library/PchSmmLib/PchSmmLib.inf           |   8 +-
>  .../MonoStatusCode/MonoStatusCode.inf         |   2 +-
>  .../PlatformDxe/PlatformDxe.inf               |   5 +-
>  .../PlatformInitPei/MemoryCallback.c          |  11 +-
>  .../PlatformInitPei/PlatformEarlyInit.h       |   4 +-
>  .../PlatformInitPei/PlatformInitPei.inf       |   1 -
>  .../PlatformPei/PlatformPei.inf               |   1 -
>  .../Intel/Vlv2TbltDevicePkg/PlatformPkg.dec   |   1 -
>  .../Intel/Vlv2TbltDevicePkg/PlatformPkg.fdf   |  14 +
>  .../Vlv2TbltDevicePkg/PlatformPkgConfig.dsc   |   2 -
>  .../Vlv2TbltDevicePkg/PlatformPkgGcc.fdf      |  14 +
>  ...formPkgIA32.dsc => PlatformPkgGccIA32.dsc} | 122 ++--
> ---
>  .../Vlv2TbltDevicePkg/PlatformPkgGccX64.dsc   | 112 ++--
> --
>  .../Vlv2TbltDevicePkg/PlatformPkgIA32.dsc     | 109 ++--
> --
>  .../Vlv2TbltDevicePkg/PlatformPkgX64.dsc      | 113 ++--
> --
>  .../PlatformSetupDxe/PlatformSetupDxe.h       |   1 -
>  .../PlatformSetupDxe/PlatformSetupDxe.inf     |   5 +-
>  .../PlatformSetupDxe/SetupInfoRecords.c       |  42 ++-
>  .../Vlv2TbltDevicePkg/PpmPolicy/PpmPolicy.c   |  12 +-
>  .../Vlv2TbltDevicePkg/PpmPolicy/PpmPolicy.inf |   1 +
>  Platform/Intel/Vlv2TbltDevicePkg/Readme.md    | 145
> ++++----
>  .../MiscBaseBoardManufacturerFunction.c       |  14 +-
>  .../SmBiosMiscDxe/MiscBiosVendorFunction.c    |  49 +--
>  .../SmBiosMiscDxe/MiscOemType0x90Function.c   |  37 +-
>  .../SmBiosMiscDxe/MiscOemType0x94Function.c   |  10 +-
>  .../MiscProcessorInformationFunction.c        |   3 +-
>  .../MiscSystemManufacturerFunction.c          |  65 ++--
>  .../SmBiosMiscDxe/SmBiosMiscDxe.inf           |   9 +-
>  Platform/Intel/Vlv2TbltDevicePkg/bld_vlv.bat  | 151
> ++++----
>  Platform/Intel/Vlv2TbltDevicePkg/bld_vlv.sh   | 251
> +++++++------
>  Platform/Intel/Vlv2TbltDevicePkg/cln.sh       |  62 ----
>  .../AcpiTablesPCAT/IgdOGBDA.ASL               |  14 +-
>  .../AcpiTablesPCAT/IgdOpRn.ASL                |  18 +-
>  .../Vlv2DeviceRefCodePkg.dec                  |   1 -
>  55 files changed, 670 insertions(+), 2749 deletions(-)
> delete mode 100644
> Platform/Intel/Vlv2TbltDevicePkg/.gitignore
>  mode change 100644 => 100755
> Platform/Intel/Vlv2TbltDevicePkg/Build_IFWI.sh
>  mode change 100644 => 100755
> Platform/Intel/Vlv2TbltDevicePkg/Feature/Capsule/Generate
> Capsule/GenCapsuleAll.sh
>  delete mode 100644
> Platform/Intel/Vlv2TbltDevicePkg/Include/Library/BiosIdLi
> b.h
>  delete mode 100644
> Platform/Intel/Vlv2TbltDevicePkg/Include/Library/CpuIA32.
> h
>  delete mode 100644
> Platform/Intel/Vlv2TbltDevicePkg/Include/PlatformDefiniti
> ons.h
>  delete mode 100644
> Platform/Intel/Vlv2TbltDevicePkg/Library/BiosIdLib/BiosId
> Lib.c
>  delete mode 100644
> Platform/Intel/Vlv2TbltDevicePkg/Library/BiosIdLib/BiosId
> Lib.inf
>  delete mode 100644
> Platform/Intel/Vlv2TbltDevicePkg/Library/CpuIA32Lib/CpuIA
> 32Lib.inf
>  delete mode 100644
> Platform/Intel/Vlv2TbltDevicePkg/Library/CpuIA32Lib/EfiCp
> uVersion.c
>  delete mode 100644
> Platform/Intel/Vlv2TbltDevicePkg/Library/CpuIA32Lib/IA32/
> CpuIA32.S
>  delete mode 100644
> Platform/Intel/Vlv2TbltDevicePkg/Library/CpuIA32Lib/IA32/
> CpuIA32.asm
>  delete mode 100644
> Platform/Intel/Vlv2TbltDevicePkg/Library/CpuIA32Lib/IA32/
> CpuIA32.c
>  delete mode 100644
> Platform/Intel/Vlv2TbltDevicePkg/Library/CpuIA32Lib/X64/C
> pu.S
>  delete mode 100644
> Platform/Intel/Vlv2TbltDevicePkg/Library/CpuIA32Lib/X64/C
> pu.asm
>  copy
> Platform/Intel/Vlv2TbltDevicePkg/{PlatformPkgIA32.dsc =>
> PlatformPkgGccIA32.dsc} (91%)  mode change 100644 =>
> 100755 Platform/Intel/Vlv2TbltDevicePkg/bld_vlv.sh
>  delete mode 100644
> Platform/Intel/Vlv2TbltDevicePkg/cln.sh
> 
> --
> 2.21.0.windows.1
> 
> 
> 


      parent reply	other threads:[~2019-07-13  0:18 UTC|newest]

Thread overview: 69+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-07-10 19:04 [edk2-platforms Patch 00/28] Vlv2TbltDevicePkg: Fix Linux build issues Michael D Kinney
2019-07-10 19:04 ` [edk2-platforms Patch 01/28] Vlv2TbltDevicePkg/Build_IFWI.sh: Add "set -e" to exit on error Michael D Kinney
2019-07-11  4:46   ` Sun, Zailiang
2019-07-12  2:59   ` [edk2-devel] " rebecca
2019-07-10 19:04 ` [edk2-platforms Patch 02/28] Vlv2TbltDevicePkg/Build_IFWI.sh: Change the root directory path Michael D Kinney
2019-07-11  4:47   ` Sun, Zailiang
2019-07-12  3:03   ` [edk2-devel] " rebecca
2019-07-12 16:23     ` Michael D Kinney
2019-07-12 16:32       ` rebecca
2019-07-10 19:04 ` [edk2-platforms Patch 03/28] Vlv2TbltDevicePkg/bld_vlv.sh: Fix the log name Michael D Kinney
2019-07-11  4:47   ` Sun, Zailiang
2019-07-10 19:04 ` [edk2-platforms Patch 04/28] Vlv2TbltDevicePkg/bld_vlv.sh: Update the gcc version detection Michael D Kinney
2019-07-11  4:47   ` Sun, Zailiang
2019-07-10 19:04 ` [edk2-platforms Patch 05/28] Vlv2TbltDevicePkg/bld_vlv.sh: Remove ECP completely Michael D Kinney
2019-07-11  4:47   ` Sun, Zailiang
2019-07-10 19:04 ` [edk2-platforms Patch 06/28] Vlv2TbltDevicePkg/bld_vlv.sh: Update the script to fit edk2-platforms Michael D Kinney
2019-07-11  4:48   ` Sun, Zailiang
2019-07-10 19:04 ` [edk2-platforms Patch 07/28] Vlv2TbltDevicePkg: Add the missing headers to inf files Michael D Kinney
2019-07-11  4:48   ` Sun, Zailiang
2019-07-10 19:04 ` [edk2-platforms Patch 08/28] Vlv2TbltDevicePkg/bld_vlv.sh: Remove BiosId.env completely Michael D Kinney
2019-07-11  4:48   ` Sun, Zailiang
2019-07-10 19:04 ` [edk2-platforms Patch 09/28] Vlv2TbltDevicePkg/bld_vlv.sh: Correct the path to PlatformCapsuleGcc.dsc Michael D Kinney
2019-07-11  4:48   ` Sun, Zailiang
2019-07-10 19:04 ` [edk2-platforms Patch 10/28] Vlv2TbltDevicePkg/bld_vlv.sh: Create Vlv.ROM Michael D Kinney
2019-07-11  4:52   ` Sun, Zailiang
2019-07-11  6:08     ` [edk2-devel] " Gary Lin
2019-07-12 23:42       ` Michael D Kinney
2019-07-13  4:17         ` Sun, Zailiang
2019-07-10 19:04 ` [edk2-platforms Patch 11/28] Vlv2TbltDevicePkg/GenerateCapsule: Fix the bash scripts Michael D Kinney
2019-07-12  2:37   ` Sun, Zailiang
2019-07-10 19:05 ` [edk2-platforms Patch 12/28] Vlv2TbltDevicePkg/AcpiPlatform: Remove a unused variable Michael D Kinney
2019-07-12  2:37   ` Sun, Zailiang
2019-07-10 19:05 ` [edk2-platforms Patch 13/28] Vlv2TbltDevicePkg/AcpiPlatform: Fix the error from InstallAcpiTable Michael D Kinney
2019-07-12  2:37   ` Sun, Zailiang
2019-07-10 19:05 ` [edk2-platforms Patch 14/28] Vlv2TbltDevicePkg/SmBiosMiscDxe: Fix the indentation Michael D Kinney
2019-07-12  2:37   ` Sun, Zailiang
2019-07-10 19:05 ` [edk2-platforms Patch 15/28] Vlv2TbltDevicePkg: Import SD/MMC drivers Michael D Kinney
2019-07-12  2:37   ` [edk2-devel] " Sun, Zailiang
2019-07-10 19:05 ` [edk2-platforms Patch 16/28] Vlv2TbltDevicePkg: Update Linux build scripts Michael D Kinney
2019-07-11  4:55   ` [edk2-devel] " Sun, Zailiang
2019-07-10 19:05 ` [edk2-platforms Patch 17/28] Vlvt2TbltDevicePkg: Fix NULL pointer access in SmbiosMiscDxe Michael D Kinney
2019-07-11  4:53   ` Sun, Zailiang
2019-07-10 19:05 ` [edk2-platforms Patch 18/28] Vlv2TbltDevicePkg/Build_IFWI.sh: Check the WORKSPACE env variable Michael D Kinney
2019-07-12  2:37   ` Sun, Zailiang
2019-07-10 19:05 ` [edk2-platforms Patch 19/28] Vlv2DeviceRefCodePkg: Remove redundant gEfiSpiProtocolGuid Michael D Kinney
2019-07-11  4:53   ` Sun, Zailiang
2019-07-10 19:05 ` [edk2-platforms Patch 20/28] Vlv2TbltDevicePkg: " Michael D Kinney
2019-07-11  4:53   ` Sun, Zailiang
2019-07-10 19:05 ` [edk2-platforms Patch 21/28] Vlv2TbltDevicePkg: Re-enable BIOS ID feature Michael D Kinney
2019-07-11  4:54   ` Sun, Zailiang
2019-07-10 19:05 ` [edk2-platforms Patch 22/28] Vlv2TbltDevicePkg: Add GCC IA32 build support Michael D Kinney
2019-07-11  4:54   ` Sun, Zailiang
2019-07-10 19:05 ` [edk2-platforms Patch 23/28] Vlv2TbltDevicePkg: Import TlsLib for HTTPS Boot Michael D Kinney
2019-07-12  2:38   ` [edk2-devel] " Sun, Zailiang
2019-07-12  3:34   ` rebecca
2019-07-10 19:05 ` [edk2-platforms Patch 24/28] Vlv2TbltDevicePkg: Reorganize the libraries Michael D Kinney
2019-07-12  2:38   ` Sun, Zailiang
2019-07-10 19:05 ` [edk2-platforms Patch 25/28] Vlv2DeviceRefCodePlg/AcpiTablesPCAT: Fix ASL error Michael D Kinney
2019-07-11  4:54   ` Sun, Zailiang
2019-07-10 19:05 ` [edk2-platforms Patch 26/28] Vlv2TbltDevicePkg: Clean up all build scripts Michael D Kinney
2019-07-11  4:55   ` Sun, Zailiang
2019-07-12  3:39   ` [edk2-devel] " rebecca
2019-07-12 18:16     ` Michael D Kinney
2019-07-10 19:05 ` [edk2-platforms Patch 27/28] Vlv2TbltDevicePkg: Only build capsules for X64 Michael D Kinney
2019-07-11  4:55   ` Sun, Zailiang
2019-07-10 19:05 ` [edk2-platforms Patch 28/28] Vlv2TbltDevicePkg: Update Readme.md for multiple repos Michael D Kinney
2019-07-11  4:55   ` Sun, Zailiang
2019-07-12  2:38 ` [edk2-platforms Patch 00/28] Vlv2TbltDevicePkg: Fix Linux build issues Sun, Zailiang
2019-07-13  0:18 ` Michael D Kinney [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=E92EE9817A31E24EB0585FDF735412F5B9D73E94@ORSMSX113.amr.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