Re: [edk2] [PATCH] BaseTools: Fix GCC49 build failure

2015-08-08 Thread Ard Biesheuvel
option to satisfy this requirement and avoid a GCC44 build failure. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Scott Duplichan sc...@notabs.org For consistency, we should probably invert the order for all versions, but it can wait until later. Reviewed-by: Ard

[edk2] [PATCH 2/5] ArmPlatformPkg/PlatformIntelBdsLib: fix error handling

2015-08-08 Thread Ard Biesheuvel
In InitializeConsolePipe (), we clobber the Status variable in the error handling path that reports Status in its output. Instead, use a NULL check on the LocateProtocol () output argument. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu

[edk2] [PATCH 0/5] secure boot support for ARM FVP

2015-08-08 Thread Ard Biesheuvel
be disabled on the build command line to use the Intel BDS instead. Patch #5 enables the Secure Boot build of ArmVExpress-FVP, by introducing a define 'SECURE_BOOT_ENABLE' which defaults to FALSE but can be enabled on the build command line. Ard Biesheuvel (5): ArmPlatformPkg/PlatformIntelBdsLib

[edk2] [PATCH 5/5] ArmPlatformPkg/ArmVExpressPkg: enable UEFI Secure Boot

2015-08-08 Thread Ard Biesheuvel
. The FVP Fast model is recommended in this case, since the certificate store is kept in NOR flash. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- ArmPlatformPkg/ArmVExpressPkg/ArmVExpress-FVP-AArch64.dsc | 12

[edk2] [PATCH 1/5] ArmPlatformPkg/PlatformIntelBdsLib: remove ARM BDS dependency

2015-08-08 Thread Ard Biesheuvel
The Intel BDS platform library still depends on the ARM BDS specific BdsLib. So replace its invocations with GenericBdsLib counterparts, and fix up where needed, so that we can drop the dependency. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu

[edk2] [PATCH 4/5] ArmPlatformPkg/ArmVExpressPkg: add support for the Intel BDS

2015-08-08 Thread Ard Biesheuvel
This adds support for the Intel BDS, by introducing a define 'USE_ARM_BDS' which defaults to TRUE, and can be overridden on the build command line. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- ArmPlatformPkg/ArmVExpressPkg

Re: [edk2] [RFC PATCH 0/4] unify GCC command line options

2015-08-13 Thread Ard Biesheuvel
On 13 August 2015 at 08:27, Ard Biesheuvel ard.biesheu...@linaro.org wrote: On 12 August 2015 at 23:48, David Woodhouse dw...@infradead.org wrote: On Wed, 2015-08-12 at 09:08 +0200, Ard Biesheuvel wrote: Is there any reason these are kept out of sync? Are UNIXGCC and CYGGCC known to be widely

[edk2] [PATCH v2] BaseTools IA32/X64: prevent .eh_frame sections from being generated

2015-08-12 Thread Ard Biesheuvel
called .debug_frame instead of into .eh_frame. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org Reviewed-by: Laszlo Ersek ler...@redhat.com Build-tested-by: Laszlo Ersek ler...@redhat.com --- v2: give IA32 the same treatment as X86 add

Re: [edk2] [PATCH v2] BaseTools IA32/X64: prevent .eh_frame sections from being generated

2015-08-13 Thread Ard Biesheuvel
On 13 August 2015 at 07:44, Gao, Liming liming@intel.com wrote: Reviewed-by: Liming Gao liming@intel.com Thanks Committed as SVN r18217 -- Ard. -Original Message- From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] Sent: Thursday, August 13, 2015 1:19 PM To: edk2

Re: [edk2] [PATCH v2 00/16] unify GCC command line options

2015-08-18 Thread Ard Biesheuvel
On 17 August 2015 at 21:16, David Woodhouse dw...@infradead.org wrote: See http://www.infradead.org/rpr.html X-SRS-Rewrite: SMTP reverse-path rewritten from dw...@infradead.org by twosheds.infradead.org See http://www.infradead.org/rpr.html On 2015-08-17 11:25:41, David Woodhouse wrote:

Re: [edk2] [PATCH v2 00/16] unify GCC command line options

2015-08-18 Thread Ard Biesheuvel
On 17 August 2015 at 20:53, Jordan Justen jordan.l.jus...@intel.com wrote: On 2015-08-17 11:25:56, Ard Biesheuvel wrote: On 17 August 2015 at 20:22, Jordan Justen jordan.l.jus...@intel.com wrote: Can't you use an elf-based GCC4.9 with the GCC49 toolchain instead? I'm not sure it makes

Re: [edk2] [PATCH 00/15] unify GCC command line options

2015-08-17 Thread Ard Biesheuvel
On 16 August 2015 at 23:48, Scott Duplichan sc...@notabs.org wrote: ]Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] wrote: . . . ] Thanks for this much needed tool chain definition consolidation. I ran ] a build test with and without the patch. The build test uses GCC44-49

Re: [edk2] [PATCH] ArmPkg: Bug fix for UncachedMemoryAllocationLib

2015-08-17 Thread Ard Biesheuvel
leif.lindh...@linaro.org Cc: Ard Biesheuvel ard.biesheu...@linaro.org --- .../UncachedMemoryAllocationLib/UncachedMemoryAllocationLib.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) Reviewed-by: Ard Biesheuvel ard.biesheu...@linaro.org diff --git a/ArmPkg/Library

[edk2] [PATCH v2 07/16] BaseTools GCC: use leading underscore for symbol names where appropriate

2015-08-17 Thread Ard Biesheuvel
-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- BaseTools/Conf/tools_def.template | 22 +++- 1 file changed, 12 insertions(+), 10 deletions(-) diff --git a/BaseTools/Conf/tools_def.template b/BaseTools/Conf/tools_def.template

[edk2] [PATCH v2 08/16] BaseTools GCC: refactor tools_def internal GCC defines for [AS]DLINK

2015-08-17 Thread Ard Biesheuvel
Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- BaseTools/Conf/tools_def.template | 30 +++- 1 file changed, 16 insertions(+), 14 deletions(-) diff --git a/BaseTools/Conf/tools_def.template b/BaseTools/Conf/tools_def.template index 1304f514cd37

[edk2] [PATCH v2 04/16] BaseTools GCC: merge warning flags for all GCC versions

2015-08-17 Thread Ard Biesheuvel
The warning flags -Wno-address -Wno-unused-but-set-variable are added for version 4.6 and up, but since they are happily accepted by version 4.4 and 4.5, add them there as well. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org

[edk2] [PATCH v2 14/16] BaseTools GCC: unify ASM flags for all GCC versions

2015-08-17 Thread Ard Biesheuvel
Use the same GCC options for assembling regardless of the exact GCC version. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- BaseTools/Conf/tools_def.template | 67 1 file changed, 27 insertions(+), 40

[edk2] [PATCH v2 15/16] BaseTools GCC: align ELFGCC with GCC4x toolchains

2015-08-17 Thread Ard Biesheuvel
This aligns the remaining configuration options for ELFGCC with the other ELF based toolchains. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- BaseTools/Conf/tools_def.template | 18 +- 1 file changed, 9

[edk2] [PATCH v2 16/16] OvmfPkg/X64: enable 4 KB alignment for DXE_RUNTIME modules

2015-08-17 Thread Ard Biesheuvel
This enables 4 KB section alignment for DXE_RUNTIME modules, for ELF based toolchains and for the UNIXGCC PE/COFF toolchain. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- OvmfPkg/OvmfPkgX64.dsc | 7 +++ 1 file changed, 7

[edk2] [PATCH v2 11/16] BaseTools GCC: unify all IA32 and X64 [AS]DLINK[2] flags for ELF based GCC

2015-08-17 Thread Ard Biesheuvel
GCC4x and ELFGCC are both ELF based GCC toolchains, so there is no justification for allowing the command line options to deviate. So align them. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- BaseTools/Conf/tools_def.template

[edk2] [PATCH v2 10/16] BaseTools GCC: remove GCC 4.9 specific linker alignment override

2015-08-17 Thread Ard Biesheuvel
a need for special treatment of GCC 4.9, and the linker command line override can be removed. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- BaseTools/Conf/tools_def.template | 8 1 file changed, 4 insertions(+), 4

[edk2] [PATCH v2 05/16] BaseTools GCC: refactor tools_def internal GCC defines for CC flags

2015-08-17 Thread Ard Biesheuvel
not modify the options that are ultimately passed to GCC, although it does deduplicate ARM and AARCH64 since those toolchains were passing both GCC_ALL_CC_FLAGS and GCC44_ALL_CC_FLAGS, which overlap to a great extent. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel

[edk2] [PATCH v2 09/16] BaseTools GCC: don't set .data address explicitly

2015-08-17 Thread Ard Biesheuvel
, but advance the location pointer by the same amount. This way, the linker will be free to choose the start address, but will ultimately end up using the exact same value. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- BaseTools/Scripts

[edk2] [PATCH v2 13/16] BaseTools GCC: unify ARM and AARCH64 DLINK flags for all GCC versions

2015-08-17 Thread Ard Biesheuvel
Use the same GCC options for linking regardless of the exact GCC version. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- BaseTools/Conf/tools_def.template | 63 +++- 1 file changed, 23 insertions(+), 40

[edk2] [PATCH v2 00/16] unify GCC command line options

2015-08-17 Thread Ard Biesheuvel
Branch can be found here https://git.linaro.org/people/ard.biesheuvel/uefi-next.git/shortlog/refs/heads/unify-gcc-v2 Ard Biesheuvel (15): BaseTools/GenFv: use PE/COFF virtual section size if raw size is larger BaseTools GCC: remove unused definition of GCC_WINDRES_FLAGS BaseTools GCC: merge

[edk2] [PATCH v2 03/16] BaseTools GCC: remove unused definition of GCC_WINDRES_FLAGS

2015-08-17 Thread Ard Biesheuvel
The definition of GCC_WINDRES_FLAGS is never referenced again so remove it. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- BaseTools/Conf/tools_def.template | 1 - 1 file changed, 1 deletion(-) diff --git a/BaseTools/Conf

Re: [edk2] [PATCH v2 16/16] OvmfPkg/X64: enable 4 KB alignment for DXE_RUNTIME modules

2015-08-17 Thread Ard Biesheuvel
[mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Ard Biesheuvel Sent: Monday, August 17, 2015 10:25 PM To: edk2-devel@lists.01.org; Liu, Yingke D Cc: wp...@windriver.com; sc...@notabs.org; Ard Biesheuvel; Justen, Jordan L; Gao, Liming; dw...@infradead.org Subject: [edk2] [PATCH v2 16/16

Re: [edk2] [PATCH 14/15] ArmPlatformPkg: Link separated VarCheckUefiLib NULL class library instance

2015-08-18 Thread Ard Biesheuvel
On 17 August 2015 at 10:24, Star Zeng star.z...@intel.com wrote: Cc: Leif Lindholm leif.lindh...@linaro.org Cc: Ard Biesheuvel ard.biesheu...@linaro.org Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Star Zeng star.z...@intel.com Reviewed-by: Ard Biesheuvel ard.biesheu

Re: [edk2] [PATCH 3/3] ArmPlatformPkg: PL061: support multiple controller

2015-08-18 Thread Ard Biesheuvel
On 18 August 2015 at 14:30, Haojian Zhuang haojian.zhu...@linaro.org wrote: On Tue, 2015-08-18 at 11:43 +0200, Ard Biesheuvel wrote: On 18 August 2015 at 11:31, Haojian Zhuang haojian.zhu...@linaro.org wrote: On Tue, 2015-08-18 at 11:22 +0200, Ard Biesheuvel wrote: On 18 August 2015 at 11:04

Re: [edk2] [PATCH 07/15] ArmPlatformPkg: Add VarCheckLib library mapping

2015-08-18 Thread Ard Biesheuvel
On 17 August 2015 at 10:24, Star Zeng star.z...@intel.com wrote: Since Variable driver has been updated to consume the separated VarCheckLib. Cc: Leif Lindholm leif.lindh...@linaro.org Cc: Ard Biesheuvel ard.biesheu...@linaro.org Contributed-under: TianoCore Contribution Agreement 1.0 Signed

Re: [edk2] [PATCH 06/15] ArmVirtPkg: Add VarCheckLib library mapping

2015-08-18 Thread Ard Biesheuvel
On 17 August 2015 at 10:24, Star Zeng star.z...@intel.com wrote: Since Variable driver has been updated to consume the separated VarCheckLib. Cc: Laszlo Ersek ler...@redhat.com Cc: Ard Biesheuvel ard.biesheu...@linaro.org Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off

Re: [edk2] [PATCH 0/5] secure boot support for ARM FVP

2015-08-18 Thread Ard Biesheuvel
On 8 August 2015 at 14:00, Ard Biesheuvel ard.biesheu...@linaro.org wrote: This series adds support for using the Intel BDS with ArmVExpress-FVP, and for building it with UEFI Secure Boot enabled. Note that the former is a prerequisite of the latter, since the ARM BDS has no GUI for enrolling

Re: [edk2] [PATCH 0/3] unify FVP Base and Foundation model support

2015-08-18 Thread Ard Biesheuvel
On 18 August 2015 at 14:12, Leif Lindholm leif.lindh...@linaro.org wrote: On Tue, Aug 18, 2015 at 01:15:54PM +0200, Ard Biesheuvel wrote: On 12 August 2015 at 08:45, Ard Biesheuvel ard.biesheu...@linaro.org wrote: Instead of omitting some drivers that are known to break the Foundation model

Re: [edk2] [PATCH 0/3] unify FVP Base and Foundation model support

2015-08-18 Thread Ard Biesheuvel
On 12 August 2015 at 08:45, Ard Biesheuvel ard.biesheu...@linaro.org wrote: Instead of omitting some drivers that are known to break the Foundation model when ARM_FOUNDATION_FVP is defined, fix those drivers so that they simply fail to load without interfering with the boot. This way, we can

Re: [edk2] [PATCH 0/2] AARCH64 tiny code model support

2015-08-18 Thread Ard Biesheuvel
On 10 August 2015 at 12:27, Ard Biesheuvel ard.biesheu...@linaro.org wrote: The AARCH64 GCC builds use the GCC large code model by default, simply because it is the code model that requires the least amount of hacking to produce code that supports the PE/COFF conversion applied by Tianocore

Re: [edk2] [PATCH v2 00/16] unify GCC command line options

2015-08-17 Thread Ard Biesheuvel
On 17 August 2015 at 19:53, Jordan Justen jordan.l.jus...@intel.com wrote: On 2015-08-17 07:24:57, Ard Biesheuvel wrote: This got a bit out of hand after I noticed the ELFGCC and UNIXGCC toolchains that needed some tlc as well. Anyway, this series aims to refactor the toolchains definitions

[edk2] [PATCH] ShellBinPkg: Arm/AArch64 Shell binary update.

2015-08-19 Thread Ard Biesheuvel
The binaries of ShellBinPkg are generated with ShellPkg project 18222. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- These are built with the tiny code model, note the code size reduction for AARCH64. Branch: https

Re: [edk2] [PATCH v2 00/16] unify GCC command line options

2015-08-19 Thread Ard Biesheuvel
On 19 August 2015 at 09:53, Ard Biesheuvel ard.biesheu...@linaro.org wrote: On 18 August 2015 at 22:29, Ard Biesheuvel ard.biesheu...@linaro.org wrote: On 18 August 2015 at 22:03, Ard Biesheuvel ard.biesheu...@linaro.org wrote: On 18 August 2015 at 19:35, David Woodhouse dw...@infradead.org

Re: [edk2] [PATCH v2] ArmPkg: remove ARMv6 support code

2015-08-19 Thread Ard Biesheuvel
On 19 August 2015 at 12:47, Leif Lindholm leif.lindh...@linaro.org wrote: On Wed, Aug 19, 2015 at 11:51:46AM +0200, Ard Biesheuvel wrote: No platforms use the ARMv6 (ARM11) support code anymore. In fact, the only reference to it in ArmPkg.dsc was commented out by Andrew in SVN r11298 (2011-02

Re: [edk2] [PATCH 3/3] ArmPlatformPkg: PL061: support multiple controller

2015-08-18 Thread Ard Biesheuvel
On 18 August 2015 at 11:04, Haojian Zhuang haojian.zhu...@linaro.org wrote: Support multiple PL061 controllers. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Haojian Zhuang haojian.zhu...@linaro.org --- ArmPlatformPkg/Drivers/PL061GpioDxe/PL061Gpio.c| 107

Re: [edk2] [PATCH 3/3] ArmPlatformPkg: PL061: support multiple controller

2015-08-18 Thread Ard Biesheuvel
On 18 August 2015 at 11:31, Haojian Zhuang haojian.zhu...@linaro.org wrote: On Tue, 2015-08-18 at 11:22 +0200, Ard Biesheuvel wrote: On 18 August 2015 at 11:04, Haojian Zhuang haojian.zhu...@linaro.org wrote: Support multiple PL061 controllers. Contributed-under: TianoCore Contribution

Re: [edk2] [PATCH v2 00/16] unify GCC command line options

2015-08-24 Thread Ard Biesheuvel
On 24 August 2015 at 19:20, Bill Paul wp...@windriver.com wrote: Of all the gin joints in all the towns in all the world, Ard Biesheuvel had to walk into mine at 10:06:10 on Monday 24 August 2015 and say: On 24 August 2015 at 19:02, Bill Paul wp...@windriver.com wrote: Of all the gin joints

Re: [edk2] [PATCH v2 00/16] unify GCC command line options

2015-08-24 Thread Ard Biesheuvel
On 19 August 2015 at 00:27, Laszlo Ersek ler...@redhat.com wrote: On 08/18/15 22:04, Paolo Bonzini wrote: On 18/08/2015 08:52, Ard Biesheuvel wrote: Personally, I would not mind deprecating GCC44, but the biggest question I would have is what toolchains do the latest UDK releases claim

[edk2] [PATCH v2 0/3] unify FVP Base and Foundation model support

2015-08-18 Thread Ard Biesheuvel
the PrimeCell ID registers. This is currently not required, since the Foundation model deals gracefully with the reads to the unpopulated range, but this may change in future versions - Added Leif's R-b - Rebased onto latest upstream Ard Biesheuvel (3): ArmPlatformPkg/PL180MciDxe: check PrimeCell ID

[edk2] [PATCH v2 1/3] ArmPlatformPkg/PL180MciDxe: check PrimeCell ID before initializing

2015-08-18 Thread Ard Biesheuvel
To deal gracefully with the absence of the PL180 hardware on the Foundation model, check the PrimeCell ID before proceeding with the installation. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org Reviewed-by: Leif Lindholm leif.lindh

[edk2] [PATCH v2 1/2] ShellPkg: force use of AARCH64 small model when building DEBUG shell

2015-08-19 Thread Ard Biesheuvel
-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org Reviewed-by: Leif Lindholm leif.lindh...@linaro.org Tested-by: Leif Lindholm leif.lindh...@linaro.org --- ShellPkg/Application/Shell/Shell.inf | 6 ++ 1 file changed, 6 insertions(+) diff --git

Re: [edk2] [PATCH v2 00/16] unify GCC command line options

2015-08-18 Thread Ard Biesheuvel
On 18 August 2015 at 22:03, Ard Biesheuvel ard.biesheu...@linaro.org wrote: On 18 August 2015 at 19:35, David Woodhouse dw...@infradead.org wrote: On Tue, 2015-08-18 at 17:52 +0200, Ard Biesheuvel wrote: On 18 August 2015 at 17:19, Jordan Justen jordan.l.jus...@intel.com wrote: Last time I

Re: [edk2] [PATCH v2 00/16] unify GCC command line options

2015-08-18 Thread Ard Biesheuvel
On 18 August 2015 at 17:19, Jordan Justen jordan.l.jus...@intel.com wrote: On 2015-08-18 03:57:51, Ard Biesheuvel wrote: On 17 August 2015 at 20:53, Jordan Justen jordan.l.jus...@intel.com wrote: On 2015-08-17 11:25:56, Ard Biesheuvel wrote: MinGW generates PE/COFF not ELF, so much

Re: [edk2] [PATCH v2 00/16] unify GCC command line options

2015-08-19 Thread Ard Biesheuvel
On 18 August 2015 at 22:29, Ard Biesheuvel ard.biesheu...@linaro.org wrote: On 18 August 2015 at 22:03, Ard Biesheuvel ard.biesheu...@linaro.org wrote: On 18 August 2015 at 19:35, David Woodhouse dw...@infradead.org wrote: On Tue, 2015-08-18 at 17:52 +0200, Ard Biesheuvel wrote: On 18 August

Re: [edk2] [PATCH v2 00/16] unify GCC command line options

2015-08-17 Thread Ard Biesheuvel
On 17 August 2015 at 20:37, Scott Duplichan sc...@notabs.org wrote: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] wrote: ]Sent: Monday, August 17, 2015 09:25 AM ]To: edk2-de...@ml01.01.org; yingke.d@intel.com ]Cc: wp...@windriver.com; sc...@notabs.org; Ard Biesheuvel ard.biesheu

Re: [edk2] [PATCH v2 00/16] unify GCC command line options

2015-08-17 Thread Ard Biesheuvel
On 17 August 2015 at 20:22, Jordan Justen jordan.l.jus...@intel.com wrote: On 2015-08-17 11:10:57, Bill Paul wrote: Of all the gin joints in all the towns in all the world, David Woodhouse had to walk into mine at 11:00:23 on Monday 17 August 2015 and say: On Mon, 2015-08-17 at 10:53 -0700,

[edk2] [PATCH] ArmPkg: remove ARMv6 support code

2015-08-17 Thread Ard Biesheuvel
No platforms use the ARMv6 (ARM11) support code anymore. In fact, the only reference to it in ArmPkg.dsc was commented out by Andrew in SVN r11298 (2011-02-03) so it may well be broken. So remove it. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu

[edk2] [PATCH 2/2] BaseTools GCC: update mingw-gcc-build.py to GCC 4.9.3

2015-08-17 Thread Ard Biesheuvel
This updates mingw-gcc-build.py to the newest version of GCC currently supported by the EDK2 build system, which is 4.9.3. At the same time, binutils is updated to version 2.24.51.0.2. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org

Re: [edk2] Some pages on new TianoCore wiki not being indexed (no results on Google)?

2015-08-24 Thread Ard Biesheuvel
(+ Leif) On 24 August 2015 at 06:27, Bruce Cran br...@cran.org.uk wrote: It seems indexing of the new wiki at http://tianocore.org is rather poor: I was looking for some information about the BeagleBoard/BeagleBone, Those instructions are slightly outdated after some recent cleanup work (I was

Re: [edk2] [PATCH v2 0/3] unify FVP Base and Foundation model support

2015-08-24 Thread Ard Biesheuvel
On 18 August 2015 at 16:10, Ard Biesheuvel ard.biesheu...@linaro.org wrote: Instead of omitting some drivers that are known to break the Foundation model when ARM_FOUNDATION_FVP is defined, fix those drivers so that they simply fail to load without interfering with the boot. Changes since v1

[edk2] [PATCH v3 06/12] BaseTools/GCC: align start of .data to .text alignment

2015-07-29 Thread Ard Biesheuvel
layout of the PE/COFF binary match the layout of the ELF version more closely. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- BaseTools/Scripts/GccBase.lds | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[edk2] [PATCH v3 10/12] ArmVirtPkg: move to unified GCC linker script

2015-07-29 Thread Ard Biesheuvel
Move to the parametrised generic GCC linker script and set 64 KB alignment, instead of using the AARCH64 specific incremental linker script for 64 KB alignment which is about to be removed. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu

[edk2] [PATCH v3 09/12] ArmPlatformPkg/ArmVExpressPkg: move to unified GCC linker script

2015-07-29 Thread Ard Biesheuvel
Move to the parametrised generic GCC linker script and set 64 KB alignment, instead of using the AARCH64 specific incremental linker script for 64 KB alignment which is about to be removed. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu

[edk2] [PATCH v3 02/12] BaseTools/GCC: move .rodata to PE/COFF .text section

2015-07-29 Thread Ard Biesheuvel
- .data section. So move it there. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- BaseTools/Scripts/gcc-4K-align-ld-script | 2 +- BaseTools/Scripts/gcc4.4-ld-script | 2 +- BaseTools/Scripts/gcc4.9-ld-script | 2 +- 3

[edk2] [PATCH v3 04/12] BaseTools/GCC: move .got contents to the PE/COFF .text section

2015-07-29 Thread Ard Biesheuvel
from being overwritten inadvertently. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- BaseTools/Scripts/gcc-4K-align-ld-script | 5 + BaseTools/Scripts/gcc4.4-ld-script | 5 + BaseTools/Scripts/gcc4.9-ld-script

[edk2] [PATCH v2] ArmPlatformPkg: remove obsolete ARM and AARCH64 platforms

2015-07-29 Thread Ard Biesheuvel
-RTSM-A15_MPCore) Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- Branch is here: https://git.linaro.org/people/ard.biesheuvel/uefi-next.git/shortlog/refs/heads/remove-obsolete-platforms ArmPlatformPkg/ArmRealViewEbPkg

Re: [edk2] [PATCH] ArmPkg: Move FDT offset higher in RAM

2015-07-28 Thread Ard Biesheuvel
On 28 July 2015 at 11:41, Ryan Harkin ryan.har...@linaro.org wrote: On 28 July 2015 at 10:26, Ard Biesheuvel ard.biesheu...@linaro.org wrote: On 28 July 2015 at 11:01, Ryan Harkin ryan.har...@linaro.org wrote: [+ Tixy as he's interested in making sure UEFI follows the Linux requirements

Re: [edk2] [PATCH] ArmPkg: Move FDT offset higher in RAM

2015-07-28 Thread Ard Biesheuvel
On 28 July 2015 at 11:01, Ryan Harkin ryan.har...@linaro.org wrote: [+ Tixy as he's interested in making sure UEFI follows the Linux requirements] On 28 July 2015 at 07:39, Ard Biesheuvel ard.biesheu...@linaro.org wrote: On 27 July 2015 at 22:42, Ryan Harkin ryan.har...@linaro.org wrote

Re: [edk2] [PATCH v3 12/12] BaseTools/X86|IA32: move to unified GCC linker script

2015-07-30 Thread Ard Biesheuvel
Biesheuvel; edk2-devel@lists.01.org; Liu, Yingke D; Gao, Liming Cc: ler...@redhat.com; leif.lindh...@linaro.org; Ard Biesheuvel Subject: Re: [PATCH v3 12/12] BaseTools/X86|IA32: move to unified GCC linker script Subject prefix: BaseTools/X86|IA32 = BaseTools IA32/X64 What about 1 more step

Re: [edk2] [PATCH 1/2] ArmPkg/ArmGicLibArchLib: common check for GICv3

2015-07-31 Thread Ard Biesheuvel
On 31 July 2015 at 14:21, Ryan Harkin ryan.har...@linaro.org wrote: On 31 July 2015 at 13:13, Ard Biesheuvel ard.biesheu...@linaro.org wrote: On 31 July 2015 at 14:06, Ryan Harkin ryan.har...@linaro.org wrote: Make Arm and Aarch64 both use the same code, conditionally compiled, to check

Re: [edk2] [PATCH] ArmVirtPkg: align ARM BDS build with LinuxLoader changes

2015-08-04 Thread Ard Biesheuvel
On 4 August 2015 at 11:21, Sharma Bhupesh bhupesh.sha...@freescale.com wrote: -Original Message- From: Laszlo Ersek [mailto:ler...@redhat.com] Sent: Tuesday, August 04, 2015 2:36 PM On 08/04/15 10:38, Sharma Bhupesh wrote: -Original Message- From: Ard Biesheuvel

Re: [edk2] [PATCH] ArmVirtPkg: align ARM BDS build with LinuxLoader changes

2015-08-04 Thread Ard Biesheuvel
On 4 August 2015 at 10:54, Laszlo Ersek ler...@redhat.com wrote: On 08/04/15 09:57, Ard Biesheuvel wrote: LinuxLoader has been split off from the ARM BDS into a separate EFI application. Because we never included this application into the ArmVirtPkg platforms, its ARM BDS builds have

Re: [edk2] [PATCH] ArmVirtPkg: align ARM BDS build with LinuxLoader changes

2015-08-04 Thread Ard Biesheuvel
Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Ard Biesheuvel Sent: Tuesday, August 04, 2015 1:27 PM To: edk2-devel@lists.01.org; leif.lindh...@linaro.org; ler...@redhat.com Cc: ryan.har...@linaro.org; Ard Biesheuvel Subject: [edk2] [PATCH] ArmVirtPkg: align ARM

Re: [edk2] [PATCH] ArmVirtPkg/ArmVirtQemu: drop ARM BDS and make Intel BDS the default

2015-08-04 Thread Ard Biesheuvel
On 4 August 2015 at 16:50, Laszlo Ersek ler...@redhat.com wrote: On 08/04/15 16:34, Ard Biesheuvel wrote: On 4 August 2015 at 15:35, Laszlo Ersek ler...@redhat.com wrote: [...] I propose the following: - The Xen hit should be removed as a separate patch, as it has never been useful

[edk2] [PATCH 3/3] ArmVirtPkg/ArmVirtQemu: add LinuxLoader UEFI app to ARM build

2015-08-04 Thread Ard Biesheuvel
:zImage -c console=ttyAMA0 Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org Reviewed-by: Laszlo Ersek ler...@redhat.com --- ArmVirtPkg/ArmVirt.dsc.inc | 6 +++--- ArmVirtPkg/ArmVirtQemu.dsc | 9 + ArmVirtPkg/ArmVirtQemu.fdf | 5

[edk2] [PATCH 0/3] ArmVirtPkg: drop support for the ARM BDS

2015-08-04 Thread Ard Biesheuvel
build of ArmVirtQemu when running the Intel BDS. The LinuxLoader for 32-bit ARM implements the boot protocol correctly (although some patches are pending to improve it in some areas), and is actually required to boot a Linux kernel at all since the ARM/Linux kernel has no UEFI stub support (yet). Ard

[edk2] [PATCH 2/3] ArmVirtPkg/ArmVirtXen: remove unused PcdFirmwareVendor PCD

2015-08-04 Thread Ard Biesheuvel
The PcdFirmwareVendor PCD was never used on this platform, since it has never supported the ARM BDS. So remove it. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- ArmVirtPkg/ArmVirtXen.dsc | 1 - 1 file changed, 1 deletion

Re: [edk2] [PATCH 4/9] ArmVirtPkg: revert ArmVirtPkg: add QemuFwCfgToPcdDxe

2015-08-04 Thread Ard Biesheuvel
is dispatched. should be hereafter solved similarly to how OvmfPkg/Library/SmbiosVersionLib is plugged into MdeModulePkg/Universal/SmbiosDxe now (originally suggested by Jordan Justen jordan.l.jus...@intel.com). Cc: Ard Biesheuvel ard.biesheu...@linaro.org Cc: Wei Huang w...@redhat.com

Re: [edk2] [PATCH v2 0/4] FFS/FV aligment optimization (was: [RFC] small C model and LLVM/clang support for AARCH64)

2015-07-30 Thread Ard Biesheuvel
On 31 July 2015 at 01:17, Laszlo Ersek ler...@redhat.com wrote: On 07/27/15 15:52, Ard Biesheuvel wrote: On 27 July 2015 at 15:34, Liu, Yingke D yingke.d@intel.com wrote: Reviewed-by: Yingke Liu yingke.d@intel.com Thank you Committed as SVN r18077 ... r18080 I do have another

Re: [edk2] [PATCH v2 0/4] FFS/FV aligment optimization

2015-07-31 Thread Ard Biesheuvel
On 31 July 2015 at 02:02, Laszlo Ersek ler...@redhat.com wrote: On 07/31/15 01:49, Ard Biesheuvel wrote: On 31 July 2015 at 01:17, Laszlo Ersek ler...@redhat.com wrote: On 07/27/15 15:52, Ard Biesheuvel wrote: On 27 July 2015 at 15:34, Liu, Yingke D yingke.d@intel.com wrote: Reviewed

[edk2] [PATCH v2 6/7] BaseTools/GenFw: allow AArch64 tiny and small code model relocations

2015-08-03 Thread Ard Biesheuvel
the relative section offset to be equal in the ELF and PE/COFF versions of the binary. This affects both the 'tiny' and 'small' GCC code models. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- BaseTools/Source/C/GenFw/Elf64Convert.c

[edk2] [PATCH v2 1/7] ArmPkg/GicV3: use GICv3 generic sysreg names only for GNU as

2015-08-03 Thread Ard Biesheuvel
Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- ArmPkg/Drivers/ArmGic/GicV3/AArch64/ArmGicV3.S | 10 ++ 1 file changed, 10 insertions(+) diff --git a/ArmPkg/Drivers/ArmGic/GicV3/AArch64/ArmGicV3.S b/ArmPkg/Drivers/ArmGic/GicV3/AArch64/ArmGicV3.S index

[edk2] [PATCH v2 4/7] ArmPlatformPkg/ArmJunoPkg: use TE 'auto' alignment for SEC modules

2015-08-03 Thread Ard Biesheuvel
. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- ArmPlatformPkg/ArmJunoPkg/ArmJuno.fdf | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/ArmPlatformPkg/ArmJunoPkg/ArmJuno.fdf b/ArmPlatformPkg/ArmJunoPkg

[edk2] [PATCH v2 2/7] ArmPkg/GenericWatchdogDxe: add missing VOID* cast

2015-08-03 Thread Ard Biesheuvel
Use an explicit VOID* cast when passing a static char array into a function taking a void pointer. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- ArmPkg/Drivers/GenericWatchdogDxe/GenericWatchdogDxe.c | 2 +- 1 file changed, 1

Re: [edk2] [PATCH v4 00/13] BaseTools: unify all GCC linker scripts

2015-08-03 Thread Ard Biesheuvel
Message- From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] Sent: Friday, July 31, 2015 19:05 To: edk2-devel@lists.01.org; Justen, Jordan L; Liu, Yingke D; Gao, Liming Cc: Laszlo Ersek; Leif Lindholm; Ard Biesheuvel Subject: Re: [PATCH v4 00/13] BaseTools: unify all GCC linker scripts

Re: [edk2] [Patch 2/2] Update copyright info, use BDS license.

2015-07-30 Thread Ard Biesheuvel
On 29 July 2015 at 10:59, Eric Dong eric.d...@intel.com wrote: Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Eric Dong eric.d...@intel.com The license is called 'BSD' not 'BDS' Could you fix up the commit titles please? -- Ard. ---

Re: [edk2] [PATCH 2/4] ArmVirtPkg: use 'auto' alignment and FIXED placement for XIP modules

2015-07-31 Thread Ard Biesheuvel
On 31 July 2015 at 02:10, Laszlo Ersek ler...@redhat.com wrote: On 07/28/15 18:42, Ard Biesheuvel wrote: Now that GenFw correctly propagates the minimum alignment of the ELF input sections to the PE/COFF binary, we can simply select 'auto' alignment in the FDF Rule section instead of tweaking

Re: [edk2] [PATCH v2 0/4] FFS/FV aligment optimization (was: [RFC] small C model and LLVM/clang support for AARCH64)

2015-07-31 Thread Ard Biesheuvel
. -Original Message- From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] Sent: Monday, July 27, 2015 9:53 PM To: Liu, Yingke D Cc: Gao, Liming; edk2-devel@lists.01.org; leif.lindh...@linaro.org; eugene.co...@hp.com Subject: Re: [PATCH v2 0/4] FFS/FV aligment optimization

Re: [edk2] [PATCH] ArmPlatformPkg: PL061: fix accessing gpio value

2015-08-11 Thread Ard Biesheuvel
On 11 August 2015 at 09:11, Haojian Zhuang haojian.zhu...@linaro.org wrote: The way of accessing PL061 GPIODATA register is wrong. The spec said in below. In order to write to GPIODATA, the corresponding bits in the mask, resulting from the address bus, PADDR[9:2], must be HIGH. Otherwise

[edk2] [PATCH v2 6/6] BaseTools: remove ARMGCC and ARMLINUXGCC toolchains

2015-08-11 Thread Ard Biesheuvel
these different architectures aligned, remove the ARMGCC and ARMLINUXGCC toolchains entirely. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- BaseTools/Conf/build_rule.template | 16 +- BaseTools/Conf/tools_def.template | 200

[edk2] [PATCH v2 4/6] BeagleBoardPkg: fold configuration header into FD build

2015-08-11 Thread Ard Biesheuvel
to add it to the flash image. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- BeagleBoardPkg/BeagleBoardPkg.fdf | 13 +++-- BeagleBoardPkg/ConfigurationHeader.bin | Bin 0 - 512 bytes 2 files changed, 11 insertions

Re: [edk2] [PATCH 3/3] OvmfPkg/Xen: use lower case x in hex immediate value

2015-08-11 Thread Ard Biesheuvel
On 11 August 2015 at 14:26, Laszlo Ersek ler...@redhat.com wrote: On 08/10/15 16:00, Ard Biesheuvel wrote: CLANG chokes on the value 0XEA1 since it expects the 0x prefix to use a lower case x. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu

Re: [edk2] some new BFD warnings

2015-08-07 Thread Ard Biesheuvel
On 5 August 2015 at 18:00, Laszlo Ersek ler...@redhat.com wrote: On 08/05/15 17:17, Ard Biesheuvel wrote: On 5 aug. 2015, at 15:53, Laszlo Ersek ler...@redhat.com wrote: Hi, I assume that it is in response to the recent BaseTools changes that now I'm getting the following warnings during

Re: [edk2] [PATCH 3/3] ArmVirtPkg/ArmVirtQemu: add LinuxLoader UEFI app to ARM build

2015-08-04 Thread Ard Biesheuvel
On 4 August 2015 at 17:21, Ard Biesheuvel ard.biesheu...@linaro.org wrote: The ARM build still needs an intermediate loader to boot Linux, since ARM/Linux has no builtin UEFI boot stub (yet). So add the LinuxLoader UEFI application to the FV, and enable the FvSimpleFileSystemDxe driver so

Re: [edk2] [PATCH 0/3] ArmVirtPkg: drop support for the ARM BDS

2015-08-04 Thread Ard Biesheuvel
On 4 August 2015 at 17:21, Ard Biesheuvel ard.biesheu...@linaro.org wrote: This series is a followup to the patches 'ArmVirtPkg/ArmVirtQemu: drop ARM BDS and make Intel BDS the default' and 'ArmVirtPkg/ArmVirtQemu: add LinuxLoader UEFI app to ARM build' that I sent out earlier today

[edk2] [PATCH] ArmVirtPkg/ArmVirtXen: add missing BdsLib instance

2015-08-04 Thread Ard Biesheuvel
Now that the ARM BDS has been removed, there is a remaining BdsLib dependency in ArmVirtXen that has now become unresolved. So re-add the BdsLib resolution that we removed from ArmVirt.dsc.inc to ArmVirtXen.dsc Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel

Re: [edk2] [PATCH v3 7/7] BaseTools: add support for CLANG compiler to GCC family

2015-08-07 Thread Ard Biesheuvel
On 7 August 2015 at 16:59, Ard Biesheuvel ard.biesheu...@linaro.org wrote: On 7 August 2015 at 16:51, Jordan Justen jordan.l.jus...@intel.com wrote: I think the subject should say 'Add CLANG toolchain with AARCH64 support' to highlight that a new toolchain name is being defined. OK. I can

[edk2] [PATCH 1/3] ArmPlatformPkg/ArmJunoDxe: drop unused BdsLib dependency

2015-08-07 Thread Ard Biesheuvel
The ArmJunoDxe driver declares a dependency on BdsLib, but never actually uses anything it provides. So drop it from the .inf Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- ArmPlatformPkg/ArmJunoPkg/Drivers/ArmJunoDxe

[edk2] [RFC PATCH 1/4] BaseTools GCC: remove 4.9 specific linker alignment override

2015-08-07 Thread Ard Biesheuvel
a need for special treatment of GCC 4.9, and the linker command line override can be removed. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- BaseTools/Conf/tools_def.template | 8 1 file changed, 4 insertions(+), 4

[edk2] [RFC PATCH 3/4] BaseTools GCC: unify ARM CC flags for all GCC versions

2015-08-07 Thread Ard Biesheuvel
There is no reason to pass -mno-unaligned-access only to ARM GCC versions 4.7 and up, since version 4.6 understands it just fine. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- BaseTools/Conf/tools_def.template | 4 ++-- 1 file

[edk2] [RFC PATCH 4/4] BaseTools GCC: unify GCC toolchain command line options

2015-08-07 Thread Ard Biesheuvel
This unifies the command line options passed to the various versions of the GCC toolchain. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- BaseTools/Conf/tools_def.template | 306 1 file changed, 117

[edk2] [RFC PATCH 0/4] unify GCC command line options

2015-08-07 Thread Ard Biesheuvel
This unifies all command line option defines in tools_def.txt, in order to reduce the maintenance burden. Note that this does not add or remove any GCC4x toolchains, it just folds the common DEFINEs into a single series of GCC4X defines. Ard Biesheuvel (4): BaseTools GCC: remove 4.9 specific

[edk2] [PATCH 2/3] ArmPlatformPkg/ArmJunoDxe: drop sky2.mac_address kernel argument

2015-08-07 Thread Ard Biesheuvel
in doing this. So drop it. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- ArmPlatformPkg/ArmJunoPkg/Drivers/ArmJunoDxe/ArmJunoDxe.c | 28 ++-- 1 file changed, 2 insertions(+), 26 deletions(-) diff --git

[edk2] [PATCH 3/3] ArmPlatformPkg/ArmJunoDxe: use single DTB for Juno rev1 boot options

2015-08-07 Thread Ard Biesheuvel
manually through sysfs. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- ArmPlatformPkg/ArmJunoPkg/ArmJuno.dec | 3 +-- ArmPlatformPkg/ArmJunoPkg/Drivers/ArmJunoDxe/ArmJunoDxe.c | 16

Re: [edk2] [PATCH v2 0/7] small model and clang support for AARCH64

2015-08-07 Thread Ard Biesheuvel
On 7 August 2015 at 16:05, Leif Lindholm leif.lindh...@linaro.org wrote: Thanks Ard, Tested on Juno/FVP. I can see two issues: - While building, clang gives a fair amount of clang: warning: argument unused during compilation: '-I some dir'. Yes, this turns out to be the assembler driver

  1   2   3   4   5   6   7   8   9   10   >