From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp.nue.novell.com (smtp.nue.novell.com [195.135.221.5]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id D2FD91A1E20 for ; Thu, 28 Jul 2016 20:25:53 -0700 (PDT) Received: from GaryWorkstation.apac.novell.com (ip-203-192-156-9.asianetcom.net [203.192.156.9]) by smtp.nue.novell.com with ESMTP (NOT encrypted); Fri, 29 Jul 2016 05:25:50 +0200 From: Gary Lin To: edk2-devel@lists.01.org Date: Fri, 29 Jul 2016 11:25:19 +0800 Message-Id: <20160729032538.17730-1-glin@suse.com> X-Mailer: git-send-email 2.9.2 MIME-Version: 1.0 Subject: [PATCH v2 00/19] Fix Vlv2TbltDevicePkg gcc build X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 29 Jul 2016 03:25:54 -0000 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit This series amend the build script of Vlv2TbltDevicePkg and fix the error messages from gcc. I've built the firmware with gcc 6.1.1 on openSUSE Tumbleweed, and the firmware binary[*] was generated successfully (but not guaranteed to work). [*] Vlv2TbltDevicePkg/Stitch/MNW2MAX_X64_D_0093_01_GCC.bin The patches are also available in my github branch: https://github.com/lcp/edk2/tree/Vlv2-gcc-fix-v2-platforms Changes in v2: 1. Instead of removing MDEPKG_NDEBUG completely, amend the CFLAG to define MDEPKG_NDEBUG only for the gcc release build. 2. Remove the variable Hob and the related code from IchRcrbInit() in Vlv2TbltDevicePkg/PlatformInitPei/PchInitPeim.c. 3. Include UefiBootServicesTableLib.h in PlatformGopPolicy/PlatformGopPolicy.c so that we don't have to declare gBS. 4. Rebase the patchset on minnowboard-max-udk2015 of https://github.com/tianocore/edk2-platforms.git instead of the master of https://github.com/tianocore/edk2.git. Besides the patches of this series, minnowboard-max-udk2015 needs a few more commits from edk2.git to make it work with gcc6: * NetworkPkg: fix ASSERT_EFI_ERROR() typos * EdkCompatibilityPkg: fix ASSERT_EFI_ERROR() typos * Vlv2TbltDevicePkg: fix ASSERT_EFI_ERROR() typos * CryptoPkg/BaseCryptLib: Add missing OpenSSL includes * CryptoPkg/OpensslLib: Undefine NO_BUILTIN_VA_FUNCS to fix varargs breakage * CryptoPkg: Fix OpenSSL BN wordsize and OPENSSL_SYS_UEFI handling For convenience, I merged all the patches before and include openssl 1.0.2h update in my branch. Gary Lin (19): Vlv2TbltDevicePkg: Amend the build script Vlv2TbltDevicePkg: Only define MDEPKG_NDEBUG for gcc release build Vlv2TbltDevicePkg/MultiPlatformLib: Remove the unused variables Vlv2TbltDevicePkg/PlatformPei: Remove the unused variable Vlv2TbltDevicePkg/PlatformBdsLib: Remove the unused variable Vlv2TbltDevicePkg/PlatformInitPei: Remove the unused variables Vlv2TbltDevicePkg/PlatformInitPei: Suppress the gcc error message Vlv2TbltDevicePkg/AcpiPlatform: Remove unused variables Vlv2TbltDevicePkg/AcpiPlatform: Amend the declaration of mGlobalNvsArea Vlv2TbltDevicePkg/PlatformGopPolicy: include UefiBootServicesTableLib.h Vlv2DeviceRefCodePkg/ValleyView2Soc: Remove the unused code Vlv2TbltDevicePkg/FvbRuntimeDxe: Remove unused variables Vlv2TbltDevicePkg/PlatformSetupDxe: Remove the unused variables Vlv2TbltDevicePkg/VlvPlatformInitDxe: Remove unused variables Vlv2TbltDevicePkg/PlatformDxe: Remove the unused variables Vlv2TbltDevicePkg/Wpce791: Remove the unused variable Vlv2TbltDevicePkg/SmBiosMiscDxe: Remove unused variables Vlv2TbltDevicePkg/PpmPolicy: Remove the usage of global variables Vlv2TbltDevicePkg/PpmPolicy: Remove the unused variable .../SouthCluster/Include/PchAccess.h | 49 ----- Vlv2TbltDevicePkg/AcpiPlatform/AcpiPlatform.c | 9 +- Vlv2TbltDevicePkg/AcpiPlatform/AcpiPlatform.h | 2 +- Vlv2TbltDevicePkg/AcpiPlatform/AcpiPlatformHooks.c | 4 +- Vlv2TbltDevicePkg/FvbRuntimeDxe/FvbService.c | 11 -- .../MultiPlatformLib/BoardClkGens/BoardClkGens.c | 5 +- .../Library/MultiPlatformLib/MultiPlatformLib.c | 53 ++--- .../Library/PlatformBdsLib/BdsPlatform.c | 3 +- Vlv2TbltDevicePkg/PlatformDxe/IchRegTable.c | 2 - Vlv2TbltDevicePkg/PlatformDxe/PciDevice.c | 214 ++++++++++----------- Vlv2TbltDevicePkg/PlatformDxe/Platform.c | 40 ++-- .../PlatformGopPolicy/PlatformGopPolicy.c | 4 +- Vlv2TbltDevicePkg/PlatformInitPei/MemoryCallback.c | 2 - Vlv2TbltDevicePkg/PlatformInitPei/PchInitPeim.c | 12 -- .../PlatformInitPei/PlatformEarlyInit.c | 1 + Vlv2TbltDevicePkg/PlatformPei/MemoryCallback.c | 9 +- Vlv2TbltDevicePkg/PlatformPkgGccX64.dsc | 2 +- .../PlatformSetupDxe/SetupInfoRecords.c | 41 ++-- Vlv2TbltDevicePkg/PpmPolicy/PpmPolicy.c | 13 +- Vlv2TbltDevicePkg/PpmPolicy/PpmPolicy.h | 6 +- .../SmBiosMiscDxe/MiscOemType0x90Function.c | 5 - .../SmBiosMiscDxe/MiscProcessorCacheFunction.c | 2 - .../MiscProcessorInformationFunction.c | 2 - Vlv2TbltDevicePkg/VlvPlatformInitDxe/IgdOpRegion.c | 4 - Vlv2TbltDevicePkg/Wpce791/LpcDriver.c | 4 - Vlv2TbltDevicePkg/bld_vlv.sh | 24 ++- 26 files changed, 203 insertions(+), 320 deletions(-) -- 2.9.2