Re: [edk2] [patch] SecurityPkg: Fixed build error due to FixedAtBuild PcdTcg2HashAlgorithmBitmap

2015-08-17 Thread Yao, Jiewen
Reviewed-by: Yao, Jiewen jiewen@intel.com At same time, I suggest we move PcdTpm2HashMask to Dynamic section too, because now Tcg2Pei will set this PCD according to TPM2 device capability. If you agree, I will check in both. Thank you Yao Jiewen -Original Message- From:

[edk2] [patch] MdeModulePkg:Full support F10 hot key in UiApp.

2015-08-17 Thread Dandan Bi
In current UiApp/Boot Maintenance manager,some pages don't support F10, they use Commit Changes and Exit menu to save changes.Now support F10 in these pages. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi dandan...@intel.com ---

Re: [edk2] [PATCH 00/15] Separate variable check service to library

2015-08-17 Thread Ni, Ruiyu
Star, The NT32Pkg changes are good. Reviewed-by: Ruiyu Ni ruiyu...@intel.com -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Star Zeng Sent: Monday, August 17, 2015 4:24 PM To: edk2-devel@lists.01.org Subject: [edk2] [PATCH 00/15] Separate

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 ] and

Re: [edk2] [PATCH 00/15] Separate variable check service to library

2015-08-17 Thread Wei, David
The patch for Vlv2TbltDevicePkg is good. Thanks. Thanks, David | SSG BIOS -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Star Zeng Sent: Monday, August 17, 2015 4:24 PM To: edk2-devel@lists.01.org Subject: [edk2] [PATCH 00/15] Separate

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

2015-08-17 Thread Ard Biesheuvel
On 13 August 2015 at 16:37, Heyi Guo heyi@linaro.org wrote: NewNode is the node we found, while Node is the last node in the list. Also update mFreedBufferSize. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Heyi Guo heyi@linaro.org Cc: Leif Lindholm

Re: [edk2] [Patch 6/8] OptionRomPkg/OvmfPkg: Remove BltLib::BltConfigure API

2015-08-17 Thread Ni, Ruiyu
Jordan, Sorry I missed your previous questions. Let me try to answer all your questions in this email. Q1: Why merging Ex and non-Ex APIs? Providing Non-Ex APIs was to make simpler functions available for blt operations. Merging the Ex and non-EX APIs is to avoid the library APIs provide

Re: [edk2] Section Alignment of elf binaries compiled with GCC(Linux)

2015-08-17 Thread Gao, Liming
Jiewen: The updated message is useful. I suggest to change error level from EFI_D_ERROR to EFI_D_INFO. Thanks Liming -Original Message- From: Yao, Jiewen Sent: Tuesday, August 18, 2015 10:46 AM To: Gao, Liming; Michael Zimmermann; edk2-devel@lists.01.org Subject: RE: [edk2] Section

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

2015-08-17 Thread Gao, Liming
Ard: I think this patch needs to update VS tool chain link flag to enable 4K for DXE_RUNTIME modules, because Ovmf platform also supports VS tool chain. Thanks Liming -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Ard Biesheuvel Sent:

Re: [edk2] Section Alignment of elf binaries compiled with GCC(Linux)

2015-08-17 Thread Gao, Liming
Hi, This is a warning message that describes the runtime driver alignment is not 4K. UEFI PropertiesTable table feature expects all runtime driver alignment is 4K. When DxeCore loads Runtime driver, it will check its alignment and report such warning message if it doesn't meet with the

Re: [edk2] Section Alignment of elf binaries compiled with GCC(Linux)

2015-08-17 Thread Yao, Jiewen
OK -Original Message- From: Gao, Liming Sent: Tuesday, August 18, 2015 10:48 AM To: Yao, Jiewen; Michael Zimmermann; edk2-devel@lists.01.org Subject: RE: [edk2] Section Alignment of elf binaries compiled with GCC(Linux) Jiewen: The updated message is useful. I suggest to change error

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

2015-08-17 Thread Ard Biesheuvel
The MS ABI uses leading underscores to decorate symbol names only when generating 32-bit code. Due to a bug in GCC prior to version 4.3, it used the same decoration for 64-bit code but this has been fixed since. So uses the leading underscore for IA32 only, and remove it for X64.

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

2015-08-17 Thread Ard Biesheuvel
Disentangle the arguments passed to the various flavors of GCC we support, by refactoring the [AS]DLINK flags so that we distinguish more clearly between toolchains that generate PE/COFF directly (for IA32 and X86 only) and toolchains that generate ELF only. Contributed-under: TianoCore

[edk2] [PATCH v2 06/16] BaseTools GCC: unify all IA32 and X64 CC 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 option 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 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 8/8] MdeModulePkg: Add GraphicsOutputDxe driver

2015-08-17 Thread Ruiyu Ni
The driver consumes the GraphicsInfo HOB and produces GOP protocol. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni ruiyu...@intel.com Cc: Feng Tian feng.t...@intel.com --- MdeModulePkg/MdeModulePkg.dsc | 4 +

[edk2] [Patch 0/8] Move BltLib to MdeModulePkg and create GraphicsOutputDxe driver

2015-08-17 Thread Ruiyu Ni
The patch serials refined the BltLib and moved it to MdeModulePkg. Based on the BltLib, the patch created the GraphicsOutputDxe driver which consumes the GraphicsInfo HOB. Ruiyu Ni (8): OptionRomPkg: Refine FrameBufferBltLib to use UINT8* instead of VOID* OptionRomPkg: Add video move test

[edk2] [Patch 4/8] OptionRomPkg: Remove BltLibGetSizes() interface from BltLib

2015-08-17 Thread Ruiyu Ni
Since the library consumes the GOP mode structure which is provided by library caller, library caller doesn't need to ask the library about the screen resolution, instead, it can directly get from the GOP mode structure. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu

[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 02/16] BaseTools/GenFv: use PE/COFF virtual section size if raw size is larger

2015-08-17 Thread Ard Biesheuvel
When copying the relocated sections into the FFS file, we need to take care that we don't overrun the end of the file. Since, unlike the virtual size, the PE/COFF raw section size must be a multiple of the file alignment, which means its size may exceed the virtual size. So use the minimum of the

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

2015-08-17 Thread Ard Biesheuvel
If any version of GCC emits any object whose actual alignment requirement exceeds 32 bytes, this actual alignment value will automatically become the PE/COFF section alignment value after PE/COFF conversion, now that GenFw propagates the alignment of the ELF input sections. So there is no longer a

[edk2] [PATCH v2 12/16] BaseTools GCC: unify ARM and AARCH64 GCC compiler flags

2015-08-17 Thread Ard Biesheuvel
Use the same GCC options for compiling regardless of the exact GCC version. The only option that needs special treatment is -mno-unaligned-access on ARM, which is not supported by upstream GCC v4.6. Everything else can be shared by all versions. Contributed-under: TianoCore Contribution Agreement

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

2015-08-17 Thread Ard Biesheuvel
As a first step towards disentangling the arguments passed to the various flavors of GCC we support, refactor the CC flags so that we distinguish more clearly between toolchains that generate PE/COFF directly (for IA32 and X86 only) and toolchains that generate ELF only. Note that this does not

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

2015-08-17 Thread Ard Biesheuvel
When a section's start address is set explicitly and is not aligned to the alignment of its contents, the linker will rearrange them so that everything lines up provided that the misalignment is preserved. Since we cannot do the same in PE/COFF, don't set the .data start address directly, but

[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
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 for UNIXGCC, GCC44, GCC45, GCC46, GCC47, GCC48, GCC49, CLANG35, ELFGCC, CYGGCC and CYGGCCxASL so that they share as much of

Re: [edk2] [Patch 0/8] Move BltLib to MdeModulePkg and create GraphicsOutputDxe driver

2015-08-17 Thread Ni, Ruiyu
The code was also checked in to below URL: https://github.com/niruiyu/edk2/tree/GOP -Original Message- From: Ni, Ruiyu Sent: Monday, August 17, 2015 9:45 PM To: edk2-devel@lists.01.org Cc: Ni, Ruiyu ruiyu...@intel.com Subject: [Patch 0/8] Move BltLib to MdeModulePkg and create

[edk2] [Patch 5/8] OptionRomPkg/OvmfPkg: BltLib API refinement

2015-08-17 Thread Ruiyu Ni
API removal: BltLibVideoToBltBuffer BltLibBufferToVideo BltLibGopBlt API rename: BltLibVideoToBltBufferEx - BltVideoToBuffer BltLibBufferToVideoEx - BltBufferToVideo BltLibVideoToVideo - BltVideoToVideo BltLibVideoFill - BltVideoFill BltLibConfigure - BltConfigure The 3 APIs in

[edk2] [Patch 3/8] OptionRomPkg: Fix a bug in BltVideoToVideo operation

2015-08-17 Thread Ruiyu Ni
The bug can be reproduced by the BltLibSample application TestMove1() test case. The old code cannot handle the case when the source and destination video is overlapped. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni ruiyu...@intel.com Cc: Laszlo Ersek

[edk2] [Patch 6/8] OptionRomPkg/OvmfPkg: Remove BltLib::BltConfigure API

2015-08-17 Thread Ruiyu Ni
The BltConfigure() API caches the video frame buffer meta data which forbids the library to be implemented to support re-entry. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni ruiyu...@intel.com Cc: Laszlo Ersek ler...@redhat.com Cc: Jordan Justen

[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

Re: [edk2] Help debugging PEIM on Minnowboard Max

2015-08-17 Thread Eric Wittmayer
Hi Feng, Now I understand the concept. I was expecting that PEIM would already be available for the Minnowboard Max. Is Usb in the PEI phase not supported in Minnowboard? From my digging in the code today, it seems to enable the xhci controller, it's PCI BAR needs to be set and enabled for

Re: [edk2] [patch] SecurityPkg: Fixed build error due to FixedAtBuild PcdTcg2HashAlgorithmBitmap

2015-08-17 Thread El-Haj-Mahmoud, Samer
Please see attached patch file. Can you help review it and check it in please? Thanks, --Samer -Original Message- From: Yao, Jiewen [mailto:jiewen@intel.com] Sent: Saturday, August 15, 2015 1:10 AM To: El-Haj-Mahmoud, Samer samer.el-haj-mahm...@hp.com; edk2-devel@lists.01.org Cc:

Re: [edk2] [patch] SecurityPkg: Fixed build error due to FixedAtBuild PcdTcg2HashAlgorithmBitmap

2015-08-17 Thread El-Haj-Mahmoud, Samer
Yes that makes sense. I missed it since I have not tried to build the new Tcg2 drivers yet. Thanks, --Samer -Original Message- From: Yao, Jiewen [mailto:jiewen@intel.com] Sent: Tuesday, August 18, 2015 12:15 AM To: El-Haj-Mahmoud, Samer samer.el-haj-mahm...@hp.com;

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

2015-08-17 Thread Ard Biesheuvel
On 18 August 2015 at 04:35, Gao, Liming liming@intel.com wrote: Ard: I think this patch needs to update VS tool chain link flag to enable 4K for DXE_RUNTIME modules, because Ovmf platform also supports VS tool chain. Good point, thanks. -Original Message- From: edk2-devel

Re: [edk2] [uswg] Shell 2.0 and 2.1

2015-08-17 Thread Carsey, Jaben
The UEFI Shell 2.1 has been available for a long time. Any UDK release after Aug last year should have it. I am not an expert on the UDK releases... -Jaben -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Daniel Samuelraj Sent: Monday,

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

Re: [edk2] [uswg] Shell 2.0 and 2.1

2015-08-17 Thread Daniel Samuelraj
Thank you, Jaben. I have added my comments inline *From:* Carsey, Jaben [mailto:jaben.car...@intel.com] *Sent:* Monday, August 17, 2015 11:12 AM *To:* Daniel Samuelraj; uswg *Cc:* edk2-devel@lists.01.org; Carsey, Jaben *Subject:* RE: [uswg] Shell 2.0 and 2.1 (note: I changed the edk2

Re: [edk2] [Patch 6/8] OptionRomPkg/OvmfPkg: Remove BltLib::BltConfigure API

2015-08-17 Thread Jordan Justen
On 2015-08-17 06:45:27, Ruiyu Ni wrote: The BltConfigure() API caches the video frame buffer meta data which forbids the library to be implemented to support re-entry. How does this help? GraphicsOutputDxe will set a mode, and then use BltLib with that mode. I already asked this, and I had

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

2015-08-17 Thread Jordan Justen
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 for UNIXGCC, GCC44, GCC45, GCC46, GCC47, GCC48, GCC49, CLANG35, ELFGCC,

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

2015-08-17 Thread David Woodhouse
On Mon, 2015-08-17 at 10:53 -0700, Jordan Justen wrote: UNIXGCC and CYGGCC are GCC 4.3 mingw based. Did this get tested? I think ELFGCC is unused at this point. (And has been since UnixPkg was deprecated.) I think we should deprecate all three of these toolchains. I would like to see

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

2015-08-17 Thread Bill Paul
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, Jordan Justen wrote: UNIXGCC and CYGGCC are GCC 4.3 mingw based. Did this get tested? I think ELFGCC is unused

Re: [edk2] [uswg] Shell 2.0 and 2.1

2015-08-17 Thread Carsey, Jaben
(note: I changed the edk2 mailing list) Daniel, A UEFI Shell 2.0 application will work fine in UEFI Shell 2.1 with no changes. To create/build a UEFI Shell 2.1 application would need to have a version of the UDK that has the updated protocol to get easy access to the new API’s that were added

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

2015-08-17 Thread Jordan Justen
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, Jordan Justen wrote: UNIXGCC and CYGGCC are GCC 4.3 mingw based. Did

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

2015-08-17 Thread Scott Duplichan
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...@linaro.org; jordan.l.jus...@intel.com; ]liming@intel.com;

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

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

2015-08-17 Thread Bill Paul
Of all the gin joints in all the towns in all the world, Jordan Justen had to walk into mine at 11:22:15 on Monday 17 August 2015 and say: 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

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

2015-08-17 Thread Jordan Justen
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 sense to 'upgrade' the UNIXGCC toolchain to be based on GCC 4.9 rather than

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

2015-08-17 Thread Jordan Justen
On 2015-08-17 11:25:41, David Woodhouse wrote: On Mon, 2015-08-17 at 11:22 -0700, Jordan Justen wrote: Can't you use an elf-based GCC4.9 with the GCC49 toolchain instead? Not for testing LLP64, no. How/who is this helping? I'm not sure it makes sense to 'upgrade' the UNIXGCC toolchain to

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,

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

2015-08-17 Thread David Woodhouse
On Mon, 2015-08-17 at 11:22 -0700, Jordan Justen wrote: Can't you use an elf-based GCC4.9 with the GCC49 toolchain instead? Not for testing LLP64, no. I'm not sure it makes sense to 'upgrade' the UNIXGCC toolchain to be based on GCC 4.9 rather than 4.3. I think GCC 4.3 was implicitly part of

[edk2] System is restarting instead of shut down when ResetSystem is called

2015-08-17 Thread Fernando Arpini
Hi, I have an application that runs in DXE phase. It executes some code and then call ResetSystem() from RunTimeServices to shut down the system (consider that the OS has put the system in S4 state previously, and this app is being called right after system resumes from S4). The system goes down,

[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

[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] [PATCH v2 00/16] unify GCC command line options

2015-08-17 Thread David Woodhouse
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: On Mon, 2015-08-17 at 11:22 -0700, Jordan Justen wrote: Can't you use an

[edk2] PCI driver issue

2015-08-17 Thread Leekha Shaveta
Hi, I was trying to run network using PCI. Have written PCI hostBridge driver for my controller, using PCI bus driver from MdeModulePkg(as it is). PCI NIC card I am using is from Intel, and I have picked its driver from Intel's source. But after integrating it in UEFI, I am facing issues.

Re: [edk2] [PATCH v2] MdeModulePkg: IP4 should re-initiate a DHCP while network reconnection

2015-08-17 Thread Ye, Ting
Reviewed-by: Ye Ting ting...@intel.com -Original Message- From: Wu, Jiaxin Sent: Monday, August 17, 2015 11:25 AM To: edk2-devel@lists.01.org Cc: Ye, Ting; Zhang, Lubo Subject: [PATCH v2] MdeModulePkg: IP4 should re-initiate a DHCP while network reconnection v2: * Update the

[edk2] Emulator not spawning window, triggering breakpoint instead

2015-08-17 Thread David Van Arnem
Hi all, About a week or two ago I was able to start the emulator + shell (Fedora 22 64-bit host) using: $ sh EmulatorPkg/build.sh run However, after re-cloning the git repository today and trying the same process, a breakpoint is instead triggered somewhere in PeiCore (I believe). If I

[edk2] [MinnowBoard] i2cdetect inconsistency/problem

2015-08-17 Thread Gerard Bucas
Hi All, I have some problems and inconsistencies on i2c bus on my minnowBoard MAX. Two problems: 1. When I run i2cdetect -l on the same board with my yocto project build and then again with running Ubuntu 14.04.3 LTS I get different listing for i2c buses. 2. Even though I have no lures

[edk2] [PATCH v2 2/2] OvmfPkg: Add HttpBoot support

2015-08-17 Thread Gary Ching-Pang Lin
This commit introdues a new build option to OvmfPkg: HTTP_BOOT_ENABLE. When HttpBoot is enabled, a new Network boot option will show in the boot manager menu with the device path like this: PciRoot(0x0)/Pci(0x3,0x0)/MAC(525400123456,0x1)/IPv4(0.0.0.0)/Uri() It works like the PXE one but fetches

[edk2] [PATCH v2 1/2] NetworkPkg: Remove the hostname from the http request url

2015-08-17 Thread Gary Ching-Pang Lin
The hostname is already set in the header of the http request. The url shouldn't contain the hostname since the hostname will be prepended to the url when the server interprets the request. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Gary Ching-Pang Lin g...@suse.com

[edk2] [PATCH v2 0/2] Add HttpBoot support to OvmfPkg

2015-08-17 Thread Gary Ching-Pang Lin
V2: Updated my git config to generate the reviewer-friendly diff. This patch series fixes a http request bug in HttpDxe and adds the HttpBoot support to OvmfPkg. I've tested the HttpBoot implementation with a simple environment: [QEMU] ---(tap0)--- [HOST] Ovmf DHCP server

[edk2] [PATCH 00/15] Separate variable check service to library

2015-08-17 Thread Star Zeng
NOTICE: To keep git bisect, the update to platform package Nt32Pkg, OvmfPkg, EmulatorPkg, ArmVirtPkg, ArmPlatformPkg and Vlv2TbltDevicePkg has been split to two patches. For your easy review, the forked code is at g...@github.com:lzeng14/edk2.git branch VariableCheckService. What to do: 1. Add

[edk2] [PATCH 02/15] MdeModulePkg: Add VarCheckUefiLib NULL class library

2015-08-17 Thread Star Zeng
What to do: Implement VarCheckUefiLib NULL class library instance. The code logic are separated from Variable driver, and it will consume VarCheckLib to register var check handler and variable property set for UEFI defined variables. Why to do: Share code. Separate variable check UEFI code from

[edk2] [PATCH 09/15] MdeModulePkg Variable: Consume the separated VarCheckLib

2015-08-17 Thread Star Zeng
Since the variable check service has be separated to VarCheckLib from Variable driver, so update Variable driver to consume the separated VarCheckLib. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Star Zeng star.z...@intel.com Reviewed-by: Jiewen Yao jiewen@intel.com

[edk2] [PATCH 04/15] OvmfPkg: Add VarCheckLib library mapping

2015-08-17 Thread Star Zeng
Since Variable driver has been updated to consume the separated VarCheckLib. Cc: Jordan Justen jordan.l.jus...@intel.com Cc: Laszlo Ersek ler...@redhat.com Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Star Zeng star.z...@intel.com --- OvmfPkg/OvmfPkgIa32.dsc| 1 +

[edk2] [PATCH 01/15] MdeModulePkg: Add VarCheckLib library

2015-08-17 Thread Star Zeng
What to do: 1. Add VarCheckLib LibraryClass definitions. 2. Implement VarCheckLib library instance. The code logic are separated from Variable driver. Why to do: Share code. Separate variable check service from Variable driver in MdeModulePkg. We are going to separate generic software logic code

[edk2] [PATCH 13/15] ArmVirtPkg: Link separated VarCheckUefiLib NULL class library instance

2015-08-17 Thread Star Zeng
Cc: Laszlo Ersek ler...@redhat.com Cc: Ard Biesheuvel ard.biesheu...@linaro.org Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Star Zeng star.z...@intel.com --- ArmVirtPkg/ArmVirtQemu.dsc | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git

[edk2] [PATCH 11/15] OvmfPkg: Link separated VarCheckUefiLib NULL class library instance

2015-08-17 Thread Star Zeng
Cc: Jordan Justen jordan.l.jus...@intel.com Cc: Laszlo Ersek ler...@redhat.com Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Star Zeng star.z...@intel.com --- OvmfPkg/OvmfPkgIa32.dsc| 5 - OvmfPkg/OvmfPkgIa32X64.dsc | 5 - OvmfPkg/OvmfPkgX64.dsc | 5 -

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

2015-08-17 Thread Star Zeng
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-off-by: Star Zeng star.z...@intel.com ---

[edk2] [PATCH 03/15] Nt32Pkg: Add VarCheckLib library mapping

2015-08-17 Thread Star Zeng
Since Variable driver has been updated to consume the separated VarCheckLib. Cc: Ruiyu Ni ruiyu...@intel.com Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Star Zeng star.z...@intel.com --- Nt32Pkg/Nt32Pkg.dsc | 1 + 1 file changed, 1 insertion(+) diff --git

[edk2] [PATCH 15/15] Vlv2TbltDevicePkg: Link separated VarCheckUefiLib NULL class library instance

2015-08-17 Thread Star Zeng
Cc: David Wei david@intel.com Cc: Tim He tim...@intel.com Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Star Zeng star.z...@intel.com --- Vlv2TbltDevicePkg/PlatformPkgGccX64.dsc | 1 + Vlv2TbltDevicePkg/PlatformPkgIA32.dsc | 1 + Vlv2TbltDevicePkg/PlatformPkgX64.dsc

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

2015-08-17 Thread Star Zeng
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-by: Star Zeng star.z...@intel.com --- ArmVirtPkg/ArmVirt.dsc.inc | 1 +

[edk2] [PATCH 10/15] Nt32Pkg: Link separated VarCheckUefiLib NULL class library instance

2015-08-17 Thread Star Zeng
Cc: Ruiyu Ni ruiyu...@intel.com Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Star Zeng star.z...@intel.com --- Nt32Pkg/Nt32Pkg.dsc | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/Nt32Pkg/Nt32Pkg.dsc b/Nt32Pkg/Nt32Pkg.dsc index a95f4e9..90a7b75

[edk2] [PATCH 12/15] EmulatorPkg: Link separated VarCheckUefiLib NULL class library instance

2015-08-17 Thread Star Zeng
Cc: Jordan Justen jordan.l.jus...@intel.com Cc: Andrew Fish af...@apple.com Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Star Zeng star.z...@intel.com --- EmulatorPkg/EmulatorPkg.dsc | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git

Re: [edk2] [PATCH 7/10] ShellPkg: Support format string argument checking

2015-08-17 Thread Qiu, Shumin
Reviewed-by: Qiu Shumin shumin@intel.com This patch for ShellPkg depends on the ' EFIFORMAT ' change in MdePkg. So block the check in after the MdePkg has been updated. -Shumin -Original Message- From: Scott Duplichan [mailto:sc...@notabs.org] Sent: Saturday, August 08, 2015 2:04