Re: [edk2] [PATCH 2/9] Linux: Ignore *.patch and *~ files

2016-03-04 Thread David Woodhouse
On Fri, 2016-03-04 at 08:56 -0800, Lee Leahy wrote: > Have git ignore patch files and backup files (file names ending with > ~). > > TEST=git status no longer displays these files > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Lee Leahy

Re: [edk2] [PATCH 3/5] OvmfPkg: add DxePciLibI440FxQ35

2016-03-04 Thread Kinney, Michael D
Jordan, Given these libs are Base type, a Dynamic PCD could not be used. Only a fixed or patchable PCD could be used, and if you go through path, you might as well pick the right lib in the DSC at build time and get smaller size and better performance. If we wanted to restrict the use of this

Re: [edk2] [PATCH 1/2] MdePkg: Add ARM Serial Port Subtype definitions

2016-03-04 Thread Yao, Jiewen
Reviewed by: jiewen@intel.com -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of evan.ll...@arm.com Sent: Saturday, March 05, 2016 1:25 AM To: edk2-de...@ml01.01.org; Kinney, Michael D; Gao, Liming Cc: leif.lindh...@linaro.org Subject: [edk2]

Re: [edk2] [PATCH 2/2] MdePkg: Add ARM Serial Port Subtypes to DBG2

2016-03-04 Thread Yao, Jiewen
Reviewed by: jiewen@intel.com -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of evan.ll...@arm.com Sent: Saturday, March 05, 2016 1:25 AM To: edk2-de...@ml01.01.org; Kinney, Michael D; Gao, Liming Cc: leif.lindh...@linaro.org Subject: [edk2]

Re: [edk2] [PATCH 8/9] QuarkPkg: Move baud rate setting to top of .dsc file

2016-03-04 Thread Bjorge, Erik C
Reviewed-by: Erik Bjorge > -Original Message- > From: Leahy, Leroy P > Sent: Friday, March 4, 2016 2:19 PM > To: Kinney, Michael D ; Steele, Kelly > ; edk2-devel@lists.01.org; Bjorge, Erik C >

Re: [edk2] [PATCH v2 8/9] QuarkPkg: Move baud rate setting to top of .dsc file

2016-03-04 Thread Kinney, Michael D
Hi Lee, Thanks for the quick update to the DSC file. Reviewed-by: Michael Kinney Can you also look at updating the Build Features section of QuarkPlatformPkg/Readme.md to describe the new BAUD_RATE setting. Thanks, Mike > -Original Message- >

Re: [edk2] [PATCH v2 8/9] QuarkPkg: Move baud rate setting to top of .dsc file

2016-03-04 Thread Steele, Kelly
Reviewed-by: Kelly Steele > -Original Message- > From: Leahy, Leroy P > Sent: March 04, 2016 14:31 > To: Kinney, Michael D ; Steele, Kelly > ; edk2-devel@lists.01.org; Bjorge, Erik C >

[edk2] [PATCH v2 8/9] QuarkPkg: Move baud rate setting to top of .dsc file

2016-03-04 Thread Lee Leahy
Move the baud rate setting to the top of the .dsc file. Use a single setting for each board. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Lee Leahy --- QuarkPlatformPkg/Quark.dsc| 25 - QuarkPlatformPkg/QuarkMin.dsc

Re: [edk2] [PATCH 8/9] QuarkPkg: Move baud rate setting to top of .dsc file

2016-03-04 Thread Steele, Kelly
Reviewed-by: Kelly Steele > -Original Message- > From: Leahy, Leroy P > Sent: March 04, 2016 14:19 > To: Kinney, Michael D ; Steele, Kelly > ; edk2-devel@lists.01.org; Bjorge, Erik C >

[edk2] [PATCH 8/9] QuarkPkg: Move baud rate setting to top of .dsc file

2016-03-04 Thread Lee Leahy
Move the baud rate setting to the top of the .dsc file. Use a single setting for each board. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Lee Leahy --- QuarkPlatformPkg/Quark.dsc| 25 - QuarkPlatformPkg/QuarkMin.dsc

[edk2] [PATCH 2/2] NetworkPkg: Use the New Functions from HttpLib

2016-03-04 Thread Ghazi Belaam
After submitting changes for HttpLib, other modules should be able to use those functions 1 remove the private function and their calls 2 update it with the functions from httpLib Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ghazi Belaam ---

[edk2] [PATCH 1/2] ModulePkg/DxeHttpLib: Adding Functions to HttpLib

2016-03-04 Thread Ghazi Belaam
There some usefull functions in edk2 private modules that could be used, so we added them to the httpLib Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ghazi Belaam --- MdeModulePkg/Include/Library/HttpLib.h | 129 ++

[edk2] allocation of PMem64 BARs

2016-03-04 Thread Laszlo Ersek
Hi Ray, on top of my pending series [edk2] [PATCH 0/5] OvmfPkg: enable PCIe on Q35 http://thread.gmane.org/gmane.comp.bios.edk2.devel/8707 I also have some patches (not posted yet) that enable 64-bit MMIO. I tested it and it seems to work, but something is strange to me. Namely, there is

Re: [edk2] [PATCH 3/5] OvmfPkg: add DxePciLibI440FxQ35

2016-03-04 Thread Laszlo Ersek
On 03/04/16 22:05, Laszlo Ersek wrote: > On 03/04/16 21:38, Jordan Justen wrote: >> On 2016-03-04 06:46:32, Laszlo Ersek wrote: >>> This library is a trivial unification of the following two PciLib >>> instances (and the result is easily diffable against each): >>> - MdePkg/Library/BasePciLibCf8

Re: [edk2] [PATCH 3/5] OvmfPkg: add DxePciLibI440FxQ35

2016-03-04 Thread Laszlo Ersek
On 03/04/16 21:38, Jordan Justen wrote: > On 2016-03-04 06:46:32, Laszlo Ersek wrote: >> This library is a trivial unification of the following two PciLib >> instances (and the result is easily diffable against each): >> - MdePkg/Library/BasePciLibCf8 >> - MdePkg/Library/BasePciLibPciExpress >> >

Re: [edk2] [PATCH 3/5] OvmfPkg: add DxePciLibI440FxQ35

2016-03-04 Thread Jordan Justen
On 2016-03-04 06:46:32, Laszlo Ersek wrote: > This library is a trivial unification of the following two PciLib > instances (and the result is easily diffable against each): > - MdePkg/Library/BasePciLibCf8 > - MdePkg/Library/BasePciLibPciExpress > Mike, what would you think about a

Re: [edk2] [PATCH 0/5] OvmfPkg: enable PCIe on Q35

2016-03-04 Thread Laszlo Ersek
On 03/04/16 20:42, Michał Zegan wrote: > Confirmed, PCI hotplug seems to work now. Thank you, I'll take that as Tested-by: Michał Zegan Cheers Laszlo > W dniu 04.03.2016 o 20:13, Laszlo Ersek pisze: >> On 03/04/16 19:56, Michał Zegan wrote: >>> How are you supposed

Re: [edk2] [PATCH 0/5] OvmfPkg: enable PCIe on Q35

2016-03-04 Thread Laszlo Ersek
On 03/04/16 19:56, Michał Zegan wrote: > How are you supposed to apply those patches? git am says the first one > is empty. It's best to fetch the public branch I linked in the blurb. Thanks! Laszlo > W dniu 04.03.2016 o 15:46, Laszlo Ersek pisze: >> The first two patches should fix >>

Re: [edk2] [PATCH 8/9] QuarkPkg: Move baud rate setting to top of .dsc file

2016-03-04 Thread Kinney, Michael D
Hi Lee, I like the idea of setting the BAUD rate in one place. Do we really need the extra defines for GEN1_BAUD_RATE and GEN2_BAUD_RATE? With BAUD_RATE define, I can set the GALILEO board type and the BAUD_RATE from the command line using /D flags for just those two if I want to override. I

Re: [edk2] [PATCH 7/9] QuarkPlatformPkg/PlatformBootManagerLib: No non-countdown msgs

2016-03-04 Thread Kinney, Michael D
Reviewed-by: Michael Kinney Mike > -Original Message- > From: Leahy, Leroy P > Sent: Friday, March 4, 2016 9:03 AM > To: Kinney, Michael D ; Steele, Kelly > ; edk2-devel@lists.01.org; Bjorge, Erik C >

Re: [edk2] [PATCH 1/9] edksetup.sh: Create the Conf directory if necessary

2016-03-04 Thread Andrew Fish
> On Mar 4, 2016, at 8:54 AM, Lee Leahy wrote: > > Edit the shell script to determine if the Conf directory is present. If > not then create the Conf directory. > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Lee Leahy

[edk2] [PATCH] ArmPlatformPkg: Initialize the UART Debug Port

2016-03-04 Thread evan . lloyd
From: Sami Mujawar The Operating system debugger relies on the firmware to configure and initialize the UART debug port. This patch initializes the PL011 UART to be used as a Debug Port. Code at:

[edk2] [PATCH] ArmPlatformPkg: Add PCD for Pl011 UART Interrupt

2016-03-04 Thread evan . lloyd
From: Sami Mujawar This patch adds a PCD for the PL011 UART Interrupt as this is needed by the Serial Port Console Redirection Table. Code at: https://github.com/EvanLloyd/tianocore/commit/68f26f23a236501d9e4294077da0d5070bbdab80 Contributed-under: TianoCore Contribution

Re: [edk2] [PATCH 1/9] edksetup.sh: Create the Conf directory if necessary

2016-03-04 Thread Jordan Justen
On 2016-03-04 08:54:44, Lee Leahy wrote: > Edit the shell script to determine if the Conf directory is present. If > not then create the Conf directory. > With PACKAGES_PATH, I think it might be possible that WORKSPACE is not defined. Mike mentioned that in some scenario my usage of WORKSPACE

[edk2] [PATCH] ArmPlatformPkg: Add HDLCD Support for Juno

2016-03-04 Thread evan . lloyd
From: Sami Mujawar The HDLCD Juno library implements the platform specific interfaces for configuring the Juno HDLCD controller. This patch is required to enable Graphics Output Protocol on Juno. Note: The HDLCD Juno Library does not configure the pixel clock and relies

Re: [edk2] [PATCH 8/9] QuarkPkg: Move baud rate setting to top of .dsc file

2016-03-04 Thread Steele, Kelly
Reviewed-by: Kelly Steele > -Original Message- > From: Leahy, Leroy P > Sent: March 04, 2016 09:03 > To: Kinney, Michael D ; Steele, Kelly > ; edk2-devel@lists.01.org; Bjorge, Erik C >

Re: [edk2] [PATCH 7/9] QuarkPlatformPkg/PlatformBootManagerLib: No non-countdown msgs

2016-03-04 Thread Steele, Kelly
Reviewed-by: Kelly Steele > -Original Message- > From: Leahy, Leroy P > Sent: March 04, 2016 09:03 > To: Kinney, Michael D ; Steele, Kelly > ; edk2-devel@lists.01.org; Bjorge, Erik C >

Re: [edk2] [PATCH 8/9] QuarkPkg: Move baud rate setting to top of .dsc file

2016-03-04 Thread Bjorge, Erik C
Reviewed-by: Erik Bjorge > -Original Message- > From: Leahy, Leroy P > Sent: Friday, March 4, 2016 9:03 AM > To: Kinney, Michael D ; Steele, Kelly > ; edk2-devel@lists.01.org; Bjorge, Erik C >

Re: [edk2] [PATCH 7/9] QuarkPlatformPkg/PlatformBootManagerLib: No non-countdown msgs

2016-03-04 Thread Bjorge, Erik C
Reviewed-by: Erik Bjorge > -Original Message- > From: Leahy, Leroy P > Sent: Friday, March 4, 2016 9:03 AM > To: Kinney, Michael D ; Steele, Kelly > ; edk2-devel@lists.01.org; Bjorge, Erik C >

Re: [edk2] [PATCH 1/9] edksetup.sh: Create the Conf directory if necessary

2016-03-04 Thread Andrew Fish
> On Mar 4, 2016, at 8:54 AM, Lee Leahy wrote: > > Edit the shell script to determine if the Conf directory is present. If > not then create the Conf directory. > Lee, What use case is this fixing? The Conf directory is checked into source control?

Re: [edk2] [PATCH 6/9] MdeModulePkg/BdsDxe: Eliminate non-countdown messages

2016-03-04 Thread Bjorge, Erik C
Reviewed-by: Erik Bjorge > -Original Message- > From: Leahy, Leroy P > Sent: Friday, March 4, 2016 9:01 AM > To: Tian, Feng ; Zeng, Star ; > edk2-devel@lists.01.org; Bjorge, Erik C > Cc: Leahy,

Re: [edk2] [PATCH 5/9] MdeModulePkg/PciSioSerialDxe: Set RTS and DTR by default

2016-03-04 Thread Bjorge, Erik C
Reviewed-by: Erik Bjorge > -Original Message- > From: Leahy, Leroy P > Sent: Friday, March 4, 2016 9:01 AM > To: Tian, Feng ; Zeng, Star ; > edk2-devel@lists.01.org; Bjorge, Erik C > Cc: Leahy,

Re: [edk2] [PATCH 4/9] DuetPkg/PciBusNoEnumerationDxe: Skip disabled devices

2016-03-04 Thread Bjorge, Erik C
Reviewed-by: Erik Bjorge > -Original Message- > From: Leahy, Leroy P > Sent: Friday, March 4, 2016 8:59 AM > To: Ni, Ruiyu ; edk2-devel@lists.01.org; Bjorge, > Erik C > Cc: Leahy, Leroy P >

Re: [edk2] [PATCH 3/9] CorebootPayloadPkg/PlatformBdsLib: Fix spelling error

2016-03-04 Thread Bjorge, Erik C
Reviewed-by: Erik Bjorge > -Original Message- > From: Leahy, Leroy P > Sent: Friday, March 4, 2016 8:58 AM > To: Ni, Ruiyu ; edk2-devel@lists.01.org; Bjorge, > Erik C > Cc: Leahy, Leroy P >

Re: [edk2] [PATCH 2/9] Linux: Ignore *.patch and *~ files

2016-03-04 Thread Bjorge, Erik C
Not sure this is needed but will not hurt anything. Reviewed-by: Erik Bjorge > -Original Message- > From: Leahy, Leroy P > Sent: Friday, March 4, 2016 8:57 AM > To: Justen, Jordan L ; edk2- > de...@lists.01.org; Bjorge, Erik C

Re: [edk2] [PATCH] ArmPlatformPkg: Allocate VRAM as RuntimeServicesData

2016-03-04 Thread Ard Biesheuvel
On 4 March 2016 at 18:40, wrote: > From: Sami Mujawar > > The UEFI specification allows the operating system (OS) to use the > Graphics Output Protocol (GOP) in the following scenarios: > a. as part of the startup process and > b. prior to loading of

Re: [edk2] [PATCH 1/9] edksetup.sh: Create the Conf directory if necessary

2016-03-04 Thread Bjorge, Erik C
Reviewed-by: Erik Bjorge > -Original Message- > From: Leahy, Leroy P > Sent: Friday, March 4, 2016 8:55 AM > To: Justen, Jordan L ; edk2- > de...@lists.01.org; Bjorge, Erik C > Cc: Leahy, Leroy P

[edk2] [PATCH 1/2] MdePkg: Add ARM Serial Port Subtype definitions

2016-03-04 Thread evan . lloyd
From: Sami Mujawar The Serial Port Console Redirection Table specification Version 1.03 - August 10, 2015 adds support for Serial Port Subtypes for ARM. These Subtypes are described in the Table 3 of the Microsoft Debug Port Table 2 (DBG2) Specification - December 10, 2015.

Re: [edk2] [PATCH 0/2] MdePkg: Add ARM Serial Port definitions

2016-03-04 Thread evan . lloyd
Second stab at Serial port id updates, responding to request from Jiewen Yao [PATCH 1/2] MdePkg: Add ARM Serial Port Subtype definitions [PATCH 2/2] MdePkg: Add ARM Serial Port Subtypes to DBG2 ___ edk2-devel mailing list

[edk2] [PATCH 8/9] QuarkPkg: Move baud rate setting to top of .dsc file

2016-03-04 Thread Lee Leahy
Move the baud rate setting to the top of the .dsc file. Use a single setting for each board. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Lee Leahy --- QuarkPlatformPkg/Quark.dsc | 28 +++- 1 file changed, 15

[edk2] [PATCH 7/9] QuarkPlatformPkg/PlatformBootManagerLib: No non-countdown msgs

2016-03-04 Thread Lee Leahy
Disable the countdown messages which are displayed every second when the timeout is specified as infinite (0x). Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Lee Leahy --- QuarkPlatformPkg/Library/PlatformBootManagerLib/PlatformBootManager.c

[edk2] [PATCH 6/9] MdeModulePkg/BdsDxe: Eliminate non-countdown messages

2016-03-04 Thread Lee Leahy
Eliminate the message every one second when the timeout is specified as infinite (0x). Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Lee Leahy --- MdeModulePkg/Universal/BdsDxe/BdsEntry.c | 4 +++- 1 file changed, 3 insertions(+), 1

[edk2] [PATCH 5/9] MdeModulePkg/PciSioSerialDxe: Set RTS and DTR by default

2016-03-04 Thread Lee Leahy
Usually the debug serial connection only uses two wires. However when the controller at the other end is honoring hardware flow control signals, it is necessary for the PciSioSerialDxe to properly generate the necessary signals. Turn on DTR and RTS during reset. Allow the higher level

[edk2] [PATCH 4/9] DuetPkg/PciBusNoEnumerationDxe: Skip disabled devices

2016-03-04 Thread Lee Leahy
Skip non-bridge devices which are not enabled either for memory or I/O access. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Lee Leahy --- DuetPkg/PciBusNoEnumerationDxe/PciEnumeratorSupport.c | 9 + 1 file changed, 9 insertions(+) diff

[edk2] [PATCH 3/9] CorebootPayloadPkg/PlatformBdsLib: Fix spelling error

2016-03-04 Thread Lee Leahy
Change vender to vendor Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Lee Leahy --- CorebootPayloadPkg/Library/PlatformBdsLib/BdsPlatform.c | 2 +- CorebootPayloadPkg/Library/PlatformBdsLib/BdsPlatform.h | 4 ++--

[edk2] [PATCH 1/9] edksetup.sh: Create the Conf directory if necessary

2016-03-04 Thread Lee Leahy
Edit the shell script to determine if the Conf directory is present. If not then create the Conf directory. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Lee Leahy --- edksetup.sh | 4 1 file changed, 4 insertions(+) diff --git

[edk2] (no subject)

2016-03-04 Thread evan . lloyd
Code at: https://github.com/EvanLloyd/tianocore/commit/ From: Sami Mujawar Date: Thu, 25 Feb 2016 15:07:40 + Subject: [PATCH] ArmPlatformPkg: Allocate VRAM as RuntimeServicesData The UEFI specification allows the operating system (OS) to use the Graphics Output Protocol

Re: [edk2] [PATCH 0/5] OvmfPkg: enable PCIe on Q35

2016-03-04 Thread Laszlo Ersek
On 03/04/16 16:37, Gabriel L. Somlo wrote: > On Fri, Mar 04, 2016 at 03:46:29PM +0100, Laszlo Ersek wrote: >> The first two patches should fix >> , based on Marcel's >> analysis in that item. (Thanks a lot for that again!) These patches are >> (and

Re: [edk2] [PATCH 2/2] MdePkg: Add ARM Serial Port Subtypes to DBG2

2016-03-04 Thread Yao, Jiewen
HI Sami Your update seems good to me. I checked spec and found more fields missing. Can we also add ARM DCC and BCM 2835, to make it complete? 0x000d (deprecated) ARM SBSA (2.x only) Generic UART supporting only 32-bit accesses 0x000e ARM SBSA Generic UART 0x000f ARM DCC 0x0010 BCM2835

Re: [edk2] [PATCH 0/5] OvmfPkg: enable PCIe on Q35

2016-03-04 Thread Gabriel L. Somlo
On Fri, Mar 04, 2016 at 03:46:29PM +0100, Laszlo Ersek wrote: > The first two patches should fix > , based on Marcel's > analysis in that item. (Thanks a lot for that again!) These patches are > (and should be) independent of -D USE_OLD_PCI_HOST; that

Re: [edk2] [PATCH 2/3] ArmPkg: ARM/AArch64 implementation of CpuExceptionHandlerLib

2016-03-04 Thread Cohen, Eugene
Ard, > > + // > > + // Patch in the common Assembly exception handler > > + // > > + Offset = (UINTN)CommonExceptionEntry - > (UINTN)ExceptionHandlersStart; > > + *(UINTN *)((UINT8 *)(UINTN)(BaseAddress + Offset)) = > (UINTN)AsmCommonExceptionEntry; > > + > > Why is this needed?

[edk2] [PATCH 1/2] MdePkg: Add ARM Serial Port Subtype definitions

2016-03-04 Thread evan . lloyd
From: Sami Mujawar The Serial Port Console Redirection Table specification Version 1.03 - August 10, 2015 adds support for Serial Port Subtypes for ARM. These Subtypes are described in the Table 3 of the Microsoft Debug Port Table 2 (DBG2) Specification - December 10, 2015.

[edk2] [PATCH 0/2] MdePkg: Add ARM Serial Port definitions

2016-03-04 Thread evan . lloyd
These patches add macro definitions for Serial Port identifiers to header files. The values are used in platform specific code. Note: the values were previously confidential, but are now made public in the specifications involved. [PATCH 1/2] MdePkg: Add ARM Serial Port Subtype

[edk2] [PATCH 4/5] OvmfPkg: match PCI config access to machine type (if not USE_OLD_PCI_HOST)

2016-03-04 Thread Laszlo Ersek
If USE_OLD_PCI_HOST is FALSE, then we switch all executable module types supported by DxePciLibI440FxQ35 to the following library instance stack: BasePciSegmentLibPci [class: PciSegmentLib] DxePciLibI440FxQ35 [class: PciLib] BasePciCf8Lib [class: PciCf8Lib]

[edk2] [PATCH 5/5] OvmfPkg: PciHostBridgeLib: permit access to the full extended config space

2016-03-04 Thread Laszlo Ersek
By now OVMF makes MdeModulePkg/Bus/Pci/PciHostBridgeDxe go through MMCONFIG (when running on Q35). Enable the driver to address each B/D/F's config space up to and including offset 0xFFF. Cc: Gabriel Somlo Cc: Jordan Justen Cc: Marcel Apfelbaum

[edk2] [PATCH 1/5] OvmfPkg: IndustryStandard/Q35MchIch9.h: add PCIEXBAR macros

2016-03-04 Thread Laszlo Ersek
Section 5.1.16 ("PCIEXBAR -- PCI Express Register Range Base Address") in Intel document #316966-002 (already referenced near the top of this header file) describes the Q35 DRAM Controller register that configures the memory-mapped PCI config space (also known as MMCONFIG, and ECAM / Enhanced

[edk2] [PATCH 2/5] OvmfPkg: PlatformPei: enable PCIEXBAR (aka MMCONFIG / ECAM) on Q35

2016-03-04 Thread Laszlo Ersek
The comments in the code should speak for themselves; here we note only two facts: - The PCI config space writes (to the PCIEXBAR register) are performed using the 0xCF8 / 0xCFC IO ports, by virtue of PciLib being resolved to BasePciLibCf8. (This library resolution will permanently remain in

[edk2] [PATCH] EmbeddedPkg: Fix SBSA Generic Watchdog helper

2016-03-04 Thread evan . lloyd
From: Sami Mujawar The Reserved field in the SBSA Generic Watchdog Structure is 1 byte in length. Refer Table 5-123 in the ACPI 5.1 Specification Errata A. The EFI_ACPI_5_1_SBSA_GENERIC_WATCHDOG_STRUCTURE_INIT() helper macro was initializing this field as

Re: [edk2] [PATCH 2/3] ArmPkg: ARM/AArch64 implementation of CpuExceptionHandlerLib

2016-03-04 Thread Ard Biesheuvel
On 4 March 2016 at 00:05, Cohen, Eugene wrote: > Introduce ARM and AArch64 instances of the CpuExceptionHandlerLib > which provides exception handling and registration of handlers > regardless of execution phase. > > Two variants of the ArmExceptionLib are provided: one where >

Re: [edk2] [PATCH 1/3] ArmPkg: Add ArmReadHcr to enable read-modify-write of HCR

2016-03-04 Thread Ard Biesheuvel
On 4 March 2016 at 00:04, Cohen, Eugene wrote: > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Eugene Cohen Reviewed-by: Ard Biesheuvel > --- > ArmPkg/Include/Chipset/AArch64.h | 5 + >

[edk2] [Patch 2/2] MdeModulePkg/Bds: More user-friendly network boot option description

2016-03-04 Thread Ruiyu Ni
The patch enhances the UefiBootManagerLib to use more user-friendly network boot option description. It builds description like below: "PXEv6 (MAC:112233445566 VLAN1)" "HTTPv4 (MAC:112233445566)" Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni

[edk2] [Patch 0/2] MdeModulePkg/Bds: More user-friendly network boot option description

2016-03-04 Thread Ruiyu Ni
The patch enhances the UefiBootManagerLib to use more user-friendly network boot option description. It builds description like below: "PXEv6 (MAC:112233445566 VLAN1)" "HTTPv4 (MAC:112233445566)" Ruiyu Ni (2): MdeModulePkg/UefiBootManagerLib: Separate boot description functions.

[edk2] [Patch 1/2] MdeModulePkg/UefiBootManagerLib: Separate boot description functions.

2016-03-04 Thread Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Feng Tian Cc: Siyuan Fu --- MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c | 576

Re: [edk2] [Patch] MdeModulePkg/PciHostBridge: Don't assert when setting UC to MMIO fails

2016-03-04 Thread Laszlo Ersek
On 03/04/16 10:31, Ni, Ruiyu wrote: > Laszlo, > > Thank you for committing patches for meJ. Sure -- I gladly help committing stuff outside of OvmfPkg too, especially if I'm reviewing and/or testing the patch anyway. Thanks! Laszlo > > > > Regards, > > Ray > > > > *From:*Laszlo Ersek

Re: [edk2] [Patch] MdeModulePkg/PciHostBridge: Don't assert when setting UC to MMIO fails

2016-03-04 Thread Ni, Ruiyu
Laszlo, Thank you for committing patches for me:). Regards, Ray From: Laszlo Ersek [mailto:ler...@redhat.com] Sent: Friday, March 4, 2016 5:12 PM To: Ni, Ruiyu Cc: edk2-de...@ml01.01.org; Yao, Jiewen Subject: Re: [edk2] [Patch]

Re: [edk2] [Patch] MdeModulePkg/PciHostBridge: Don't assert when setting UC to MMIO fails

2016-03-04 Thread Laszlo Ersek
On 03/04/16 06:04, Ruiyu Ni wrote: > Failing to set EFI_MEMORY_UC to MMIO aperture is not a fatal error. Particularly on a 32-bit processor that doesn't support MTRR ;) Reviewed-by: Laszlo Ersek Tested-by: Laszlo Ersek Committed as git 13be90faafdf.

Re: [edk2] [PATCH] MdeModulePkg: Add DEBUG and ASSERT on failed BDS entry

2016-03-04 Thread Zeng, Star
On 2016/3/4 11:15, El-Haj-Mahmoud, Samer wrote: Star, It has been a while since I have seen this, so I will need to go back to some old notes to get the details.. But I am pretty sure we didn't see the ASSERT below. The code ended up accessing gBds which was NULL and causing an exception.

Re: [edk2] [PATCH v3] NetworkPkg: Add URI configuration form to HTTP boot driver.

2016-03-04 Thread Ye, Ting
Looks good to me. Reviewed-by: Ye Ting -Original Message- From: Fu, Siyuan Sent: Friday, March 04, 2016 4:40 PM To: Fu, Siyuan ; edk2-devel@lists.01.org Cc: Ye, Ting ; Wu, Jiaxin Subject: RE: [edk2] [PATCH

[edk2] [PATCH v3] NetworkPkg: Add URI configuration form to HTTP boot driver.

2016-03-04 Thread Fu Siyuan
This patch updates the HTTP boot driver to produce a setup page for the boot file URI configuration. A new boot option will be created for the manual configured URI address. This change is made to support the HTTP boot usage in home environment. Contributed-under: TianoCore Contribution Agreement