Re: [edk2] [PATCH v2 0/5] Add Bluetooth LE definitions per UEFI Spec 2.7

Reviewed-by: jiewen@intel.com > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Ruiyu > Ni > Sent: Tuesday, June 6, 2017 11:36 AM > To: edk2-devel@lists.01.org > Subject: [edk2] [PATCH v2 0/5] Add Bluetooth LE definitions per UEFI Spec 2.7 >

Re: [edk2] [PATCH v6 00/17] x86: Secure Encrypted Virtualization (AMD)

Hi Jordan, On 6/5/17 9:08 PM, Zeng, Star wrote: > I was not tracking this thread. > Jiewen will help give comments about the potential change in MdeModulePkg. > > Thanks, > Star > -Original Message- > From: Justen, Jordan L > Sent: Tuesday, June 6, 2017 9:12 AM > To: Brijesh Singh

[edk2] [PATCH v2 5/5] MdePkg: Add BluetoothAttribute.h and BluetoothLeConfig.h

UEFI Spec 2.7 introduces BluetoothAttribute and BluetoothLeConfig protocols. The patch adds the definitions for them. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Jiewen Yao ---

[edk2] [PATCH v2 4/5] MdePkg/BluetoothIo: Formalize function header comments.

Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Jiewen Yao --- MdePkg/Include/Protocol/BluetoothIo.h | 287 +- 1 file changed, 144 insertions(+), 143 deletions(-) diff --git

[edk2] [PATCH v2 0/5] Add Bluetooth LE definitions per UEFI Spec 2.7

v2: Change 5/5 to add GUID definitions to MdePkg.dec Ruiyu Ni (5): MdePkg/DevicePath: Add BluetoothLe device path node support MdePkg/BluetoothConfig: Add new EFI_BLUETOOTH_CONFIG_DATA_TYPE types MdePkg/BluetoothHc: Add detailed function header comments MdePkg/BluetoothIo: Formalize

[edk2] [PATCH v2 2/5] MdePkg/BluetoothConfig: Add new EFI_BLUETOOTH_CONFIG_DATA_TYPE types

UEFI spec 2.7 adds new EFI_BLUETOOTH_CONFIG_DATA_TYPE types. The patch adds them to the header file. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Jiewen Yao --- MdePkg/Include/Protocol/BluetoothConfig.h | 25

[edk2] [PATCH v2 1/5] MdePkg/DevicePath: Add BluetoothLe device path node support

Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Hao A Wu --- MdePkg/Include/IndustryStandard/Bluetooth.h| 17 +- MdePkg/Include/Protocol/DevicePath.h | 11 ++-

[edk2] [PATCH v2 3/5] MdePkg/BluetoothHc: Add detailed function header comments

Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Jiewen Yao --- MdePkg/Include/Protocol/BluetoothHc.h | 510 -- 1 file changed, 303 insertions(+), 207 deletions(-) diff --git

[edk2] [PATCH 2/5] MdePkg/BluetoothConfig: Add new EFI_BLUETOOTH_CONFIG_DATA_TYPE types

UEFI spec 2.7 adds new EFI_BLUETOOTH_CONFIG_DATA_TYPE types. The patch adds them to the header file. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Jiewen Yao --- MdePkg/Include/Protocol/BluetoothConfig.h | 25

[edk2] [PATCH 3/5] MdePkg/BluetoothHc: Add detailed function header comments

Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Jiewen Yao --- MdePkg/Include/Protocol/BluetoothHc.h | 510 -- 1 file changed, 303 insertions(+), 207 deletions(-) diff --git

[edk2] [PATCH 1/5] MdePkg/DevicePath: Add BluetoothLe device path node support

Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Hao A Wu --- MdePkg/Include/IndustryStandard/Bluetooth.h| 17 +- MdePkg/Include/Protocol/DevicePath.h | 11 ++-

[edk2] [PATCH 5/5] MdePkg: Add BluetoothAttribute.h and BluetoothLeConfig.h

UEFI Spec 2.7 introduces BluetoothAttribute and BluetoothLeConfig protocols. The patch adds the definitions for them. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Jiewen Yao ---

[edk2] [PATCH 4/5] MdePkg/BluetoothIo: Formalize function header comments.

Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Jiewen Yao --- MdePkg/Include/Protocol/BluetoothIo.h | 287 +- 1 file changed, 144 insertions(+), 143 deletions(-) diff --git

[edk2] [PATCH 0/5] Add Bluetooth LE definitions per UEFI Spec 2.7

Ruiyu Ni (5): MdePkg/DevicePath: Add BluetoothLe device path node support MdePkg/BluetoothConfig: Add new EFI_BLUETOOTH_CONFIG_DATA_TYPE types MdePkg/BluetoothHc: Add detailed function header comments MdePkg/BluetoothIo: Formalize function header comments. MdePkg: Add

[edk2] [RFC v3 2/2] MdeModulePkg/DriverSample: Add sample questions to refer union type

Cc: Eric Dong Cc: Liming Gao Cc: Ruiyu Ni Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- .../Universal/DriverSampleDxe/DriverSample.c | 57 ++

[edk2] [RFC v3 0/2] Support Union type in VFR

V3: Avoid adding new member in current stucture to indicate indicate whether current date type is Union or not. We can directly paas the data type info to sub-statement/function during parsing phase. This serie is the POC to update VfrCompiler to support Union type in VarStore and construct the

[edk2] [RFC v3 1/2] BaseTool/VfrCompile: Support Union type in VFR

V3: Avoid adding new member in current stucture to indicate indicate whether current date type is Union or not. We can directly paas the data type info to sub-statement/function during parsing phase. Cc: Eric Dong Cc: Liming Gao Cc: Ruiyu Ni

Re: [edk2] [PATCH v6 00/17] x86: Secure Encrypted Virtualization (AMD)

I was not tracking this thread. Jiewen will help give comments about the potential change in MdeModulePkg. Thanks, Star -Original Message- From: Justen, Jordan L Sent: Tuesday, June 6, 2017 9:12 AM To: Brijesh Singh ; Laszlo Ersek ;

Re: [edk2] [PATCH v6 00/17] x86: Secure Encrypted Virtualization (AMD)

On 2017-06-05 14:56:04, Brijesh Singh wrote: > On 06/01/2017 04:10 AM, Laszlo Ersek wrote: > > On 06/01/17 09:40, Jordan Justen wrote: > >> In https://lists.01.org/pipermail/edk2-devel/2017-April/009883.html > >> Leo said that DxeIpl won't work because new I/O ranges might be added. > >> I don't

[edk2] [PATCH] MdePkg/Ftp4: Fix wrong function pointer declaration

EFI_FTP4_DATA_CALLBACK is a function pointer and defined as follows: > typedef > EFI_STATUS > (EFIAPI *EFI_FTP4_DATA_CALLBACK)( > IN EFI_FTP4_PROTOCOL *This, > IN EFI_FTP4_COMMAND_TOKEN *Token > ); And EFI_FTP4_COMMAND_TOKEN structure declared it as: > EFI_FTP4_DATA_CALLBACK

Re: [edk2] [PATCH v6 00/17] x86: Secure Encrypted Virtualization (AMD)

Hi Jordan, On 06/01/2017 04:10 AM, Laszlo Ersek wrote: On 06/01/17 09:40, Jordan Justen wrote: On 2017-05-29 14:59:46, Brijesh Singh wrote: On 5/29/17 3:38 PM, Jordan Justen wrote: On 2017-05-29 04:16:15, Laszlo Ersek wrote: (looks like I was the one to comment as second reviewer after

[edk2] [PATCH 07/10] BaseTools: Tools change to support PI v1.5 Specification.

This patch verifies SMM_CORE_STANDALONE module compatibility with PI specification version. Also, it registers SMM_STANDALONE/SMM_CORE_STANDALONE modules with FdfParser class and provides mapping between SMM_STANDALONE and SMM_CORE_STANDALONE module type in FDF with EFI_FV_FILETYPE_SMM_STANDALONE

[edk2] [PATCH 09/10] BaseTools: Tools change to support PI v1.5 Specification.

This patch checks SUP_MODULE_SMM_CORE_STANDALONE and SUP_MODULE_SMM_STANDALONE module compatibility with PI specification version. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jiewen Yao Signed-off-by: Supreeth Venkatesh

[edk2] [PATCH 08/10] BaseTools: Tools change to support PI v1.5 Specification.

This patch registers SMM_STANDALONE and SMM_CORE_STANDALONE module type with python build tools. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jiewen Yao Signed-off-by: Supreeth Venkatesh ---

[edk2] [PATCH 10/10] BaseTools: Tools change to support PI v1.5 Specification.

This patch registers SMM_STANDALONE with Ffs class. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jiewen Yao Signed-off-by: Supreeth Venkatesh --- BaseTools/Source/Python/Eot/FvImage.py | 1 + 1 file changed, 1

[edk2] [PATCH 06/10] BaseTools: Tools change to support PI v1.5 Specification.

This patch registers SMM_STANDALONE and SMM_CORE_STANDALONE module types with CommonClass and PackageIncludePkgHeaderClass in CommonDataClass. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jiewen Yao Signed-off-by: Supreeth Venkatesh

[edk2] [PATCH 03/10] BaseTools: Tools change to support PI v1.5 Specification.

This patch adds changes to auto generate SMM_CORE_STANDALONE and SMM_STANDALONE Entry Point templates. Also, it adds changes to help auto generate dependency expressions for SMM_STANDALONE modules. PI Specification v1.6 specifies Management Mode System Table (MMST) which is a collection of

[edk2] [PATCH 04/10] BaseTools: Tools change to support PI v1.5 Specification.

This patch adds SUP_MODULE_SMM_STANDALONE and SUP_MODULE_SMM_CORE_STANDALONE data types and includes it in SUP_MODULE_LIST. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jiewen Yao Signed-off-by: Supreeth Venkatesh ---

[edk2] [PATCH 00/10] *** BaseTools: Tools change to support PI v1.5 Specification. ***

*** PI v1.5 Specification Volume 4 defines Management Mode Core Interface. In order to support Management Mode Core Interface, Module Types MM_STANDALONE, MM_CORE_STANDALONE are needed. PI specification v1.5 defines the following new file types: #define EFI_FV_FILETYPE_MM 0x0A #define

[edk2] [PATCH 02/10] BaseTools: Tools change to support PI v1.5 Specification.

PI v1.6 Specification Volume 4 defines Management Mode Core Interface. In order to support Management Mode Core Interface, Module Types MM_STANDALONE, MM_CORE_STANDALONE are needed. This patch ensures that SMM_STANDALONE, SMM_CORE_STANDALONE Modules are treated as EFI Boot Service Driver in GenFw

[edk2] [PATCH 05/10] BaseTools: Tools change to support PI v1.5 Specification.

This patch adds support for FdfParser tool to parse SMM_STANDALONE and SMM_CORE_STANDALONE modules. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jiewen Yao Signed-off-by: Supreeth Venkatesh ---

[edk2] [PATCH 01/10] BaseTools: Tools change to support PI v1.5 Specification.

PI specification v1.6 defines the following new file types: This patch adds the new file type EFI_FV_FILETYPE_SMM_STANDALONE in GenFfs tool. *_SMM_* regex instead of *_MM_* is used to preserve backwards compatibility. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jiewen

Re: [edk2] EDK2 build using Visual Studio

On 06/05/2017 01:23 AM, GN Keshava wrote: > I was able to build EDK2 using VS2008. Did anyone used VS2015 and built > the EDK2? If yes, I want to know how I can do it. You might check out how the VisualUEFI project builds EDK2. According to it's readme, it works with VS2015.

Re: [edk2] [PATCH 1/2] EmulatorPkg/Unix/Host: Add GCC5 CC/DLINK commands (for GCC >= 5)

On 2017-06-02 14:48:14, Laszlo Ersek wrote: > On 06/02/17 11:03, Ard Biesheuvel wrote: > > On 1 June 2017 at 23:56, Jordan Justen wrote: > > > > A commit log would be nice > > I agree. FWIW: How about this? EmulatorPkg/Unix/Host: Add GCC5 CC/DLINK commands (for GCC

Re: [edk2] [PATCH 1/1] OvmfPkg/AcpiPlatformDxe: alloc blobs from 64-bit space unless restricted

On 2017-06-03 08:42:03, Laszlo Ersek wrote: > ... by narrower than 8-byte ADD_POINTER references. > > Introduce the CollectRestrictedAllocations() function, which iterates over How about Collect32BitRestrictedAllocations and similar treatment for other names that just say 'restricted'? With

Re: [edk2] [staging/cadence-aarch64 PATCH v2 1/6] CadencePkg: Add libraries for Cadence CSP platform.

On Mon, Jun 05, 2017 at 03:12:34PM +, Ard Biesheuvel wrote: > Hi Scott, > > On 5 June 2017 at 10:50, Scott Telford wrote: > > Add libraries (CadenceCspLib, CadenceCspSecLib, > > CadenceCspResetSystemLib) to support the Cadence Configurable System > > Platform (CSP)

Re: [edk2] [staging/cadence-aarch64 PATCH v2 1/6] CadencePkg: Add libraries for Cadence CSP platform.

On Mon, Jun 05, 2017 at 11:50:23AM +0100, Scott Telford wrote: > Add libraries (CadenceCspLib, CadenceCspSecLib, > CadenceCspResetSystemLib) to support the Cadence Configurable System > Platform (CSP) configured with a single ARM Cortex-A53, GIC-500, > Cadence UART and Cadence PCIe Root Complex. >

Re: [edk2] [staging/cadence-aarch64 PATCH v2 3/6] CadencePkg: Add PCI host bridge library for Cadence PCIe Root Complex.

On Mon, Jun 05, 2017 at 11:50:25AM +0100, Scott Telford wrote: > Add PciHostBridgeLib implementation for the Cadence PCIe Root Complex. > This library is derived from > Platforms/ARM/Juno/Library/JunoPciHostBridgeLib in OpenPlatformPkg. > > Contributed-under: TianoCore Contribution Agreement 1.0

Re: [edk2] Format SD card while on UEFI shell

> -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Shubham Mittal > Sent: June-03-17 10:28 PM > To: Andrew Fish; Blibbet > Cc: edk2-devel@lists.01.org > Subject: Re: [edk2] Format SD card while on UEFI shell > > Thanks Blibbet for your

Re: [edk2] [staging/cadence-aarch64 PATCH v2 5/6] CadencePkg: Add ACPI tables for Cadence CSP platform.

On 5 June 2017 at 10:50, Scott Telford wrote: > Add ACPI tables for Cadence CSP platform configured with a single > Cortex-A53, GIC-500, Cadence UART and Cadence PCIe Root Complex. > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Scott Telford

Re: [edk2] How to make my application automatically launch like uefi shell ?

> On Jun 5, 2017, at 4:42 AM, Amit kumar wrote: > > > Hi, > > I have created a uefi application and i want to directly launch from the Pen > Drive, just like uefi shell. > I made a EFI/BOOT directory in USB and copied my application in this director > and name it

Re: [edk2] [staging/cadence-aarch64 PATCH v2 3/6] CadencePkg: Add PCI host bridge library for Cadence PCIe Root Complex.

On 5 June 2017 at 10:50, Scott Telford wrote: > Add PciHostBridgeLib implementation for the Cadence PCIe Root Complex. > This library is derived from > Platforms/ARM/Juno/Library/JunoPciHostBridgeLib in OpenPlatformPkg. > > Contributed-under: TianoCore Contribution Agreement

Re: [edk2] [PATCH] ShellPkg/alias: Fix bug to support upper-case alias

It's a bug and should be fixed. Bugzilla: https://bugzilla.tianocore.org/show_bug.cgi?id=589 Thanks, Tapan -Original Message- From: Ni, Ruiyu [mailto:ruiyu...@intel.com] Sent: Saturday, June 03, 2017 7:48 PM To: Shah, Tapan ; Carsey, Jaben ;

Re: [edk2] [staging/cadence-aarch64 PATCH v2 1/6] CadencePkg: Add libraries for Cadence CSP platform.

Hi Scott, On 5 June 2017 at 10:50, Scott Telford wrote: > Add libraries (CadenceCspLib, CadenceCspSecLib, > CadenceCspResetSystemLib) to support the Cadence Configurable System > Platform (CSP) configured with a single ARM Cortex-A53, GIC-500, > Cadence UART and Cadence

Re: [edk2] EDK2 build using Visual Studio

> On Jun 5, 2017, at 7:04 AM, Gao, Liming wrote: > > _fltused symbol is generated by compiler. You can just define this symbol in > one C file to satisfy the linker like below: > > int _fltused = 1; > Keshava, Looks like _fltused is emitted by the MSVC to tell the C

Re: [edk2] EDK2 build using Visual Studio

_fltused symbol is generated by compiler. You can just define this symbol in one C file to satisfy the linker like below: int _fltused = 1; Thanks Liming From: GN Keshava [mailto:keshava...@gmail.com] Sent: Monday, June 5, 2017 9:04 PM To: Gao, Liming ;

Re: [edk2] EDK2 build using Visual Studio

Hi Liming, Thanks for the reply. I had tried this, but was getting errors that I didn't get on VS2008, mainly, error LNK2001 unresolved external symbol __fltused I didnt get this error on VS2008, but the same workspace is giving this error on VS2015. :( Thanks. With regards, Keshava On Mon, 5

Re: [edk2] EDK2 build using Visual Studio

Keshava: You can build platform with -t VS2015 or -t VS2015x86 to enable VS2015 tool chain. > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of GN > Keshava > Sent: Monday, June 5, 2017 4:23 PM > To: edk2-devel@lists.01.org > Cc: Carsey,

[edk2] How to make my application automatically launch like uefi shell ?

Hi, I have created a uefi application and i want to directly launch from the Pen Drive, just like uefi shell. I made a EFI/BOOT directory in USB and copied my application in this director and name it BOOTX64.efi it seems to works in some PCs, But there are some PCs where it doesn't work even

[edk2] [staging/cadence-aarch64 PATCH v2 0/6] CadencePkg: Add package for Cadence hardware IP support.

Add CadencePkg, which includes support for the Cadence Configurable System Platform (CSP) with a single ARM Cortex-A53 and GIC-500. Also include driver libraries for the Cadence PCIe Root Complex and Cadence UART. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Scott

[edk2] [staging/cadence-aarch64 PATCH v2 1/6] CadencePkg: Add libraries for Cadence CSP platform.

Add libraries (CadenceCspLib, CadenceCspSecLib, CadenceCspResetSystemLib) to support the Cadence Configurable System Platform (CSP) configured with a single ARM Cortex-A53, GIC-500, Cadence UART and Cadence PCIe Root Complex. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by:

[edk2] [staging/cadence-aarch64 PATCH v2 3/6] CadencePkg: Add PCI host bridge library for Cadence PCIe Root Complex.

Add PciHostBridgeLib implementation for the Cadence PCIe Root Complex. This library is derived from Platforms/ARM/Juno/Library/JunoPciHostBridgeLib in OpenPlatformPkg. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Scott Telford ---

[edk2] [staging/cadence-aarch64 PATCH v2 4/6] CadencePkg: Add SEC phase implementation for Cadence CSP platform.

Add SEC phase implementation for Cadence CSP platform configured with a single Cortex-A53 processor and GIC-500. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Scott Telford --- CadencePkg/Sec/AArch64/Arch.c | 25 +++

[edk2] [staging/cadence-aarch64 PATCH v2 5/6] CadencePkg: Add ACPI tables for Cadence CSP platform.

Add ACPI tables for Cadence CSP platform configured with a single Cortex-A53, GIC-500, Cadence UART and Cadence PCIe Root Complex. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Scott Telford --- CadencePkg/AcpiTables/AcpiTables.inf | 49 +

[edk2] [staging/cadence-aarch64 PATCH v2 6/6] CadencePkg: Add .dsc, .fdf and .dec files for Cadence CSP platform.

Add .dsc, .fdf and .dec files for a Cadence CSP platform configured with a single Cortex-A53, GIC-500, Cadence UART and Cadence PCIe Root Complex. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Scott Telford --- CadencePkg/CadenceCsp.dsc| 711

[edk2] [staging/cadence-aarch64 PATCH v2 2/6] CadencePkg: Add library for Cadence UART.

Add serial port library to support the Cadence IP6528 UART used in the Cadence CSP platform. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Scott Telford --- CadencePkg/Include/Library/CspSerialPortLib.h | 86

Re: [edk2] [Qemu-devel] allocation zone extensions for the firmware linker/loader

On Sat, 3 Jun 2017 09:36:23 +0200 Laszlo Ersek wrote: > On 06/02/17 17:45, Laszlo Ersek wrote: > > > The patches can cause linker/loader breakage when old firmware is booted > > on new QEMU. However, that's no problem (it's nothing new), the next > > release of QEMU should

Re: [edk2] [RFC v2 1/2] BaseTool/VfrCompile: Support Union type in VFR

Hi Ray, Thanks for your comments! Yes, we also can avoid using member variable. Based on my understanding, we can parse value to sub-statement. So when parsing a Struct/Union type, it can pass this info to its member fields , then we can detect whether current Data Type is Struct or Union. In

[edk2] EDK2 build using Visual Studio

Hi, I was able to build EDK2 using VS2008. Did anyone used VS2015 and built the EDK2? If yes, I want to know how I can do it. Thanks. With regards, Keshava ___ edk2-devel mailing list edk2-devel@lists.01.org

Re: [edk2] [Patch 1/2] MdeModulePkg/UefiPxeBcDxe: Fix the PXE BootMenu selection issue

Reviewed-by: Fu Siyuan -Original Message- From: Ye, Ting Sent: 2017年6月5日 15:39 To: Wu, Jiaxin ; edk2-devel@lists.01.org Cc: Fu, Siyuan Subject: RE: [Patch 1/2] MdeModulePkg/UefiPxeBcDxe: Fix the PXE BootMenu selection

Re: [edk2] [Patch 2/2] MdeModulePkg/UefiPxeBcDxe: Refine the PXE boot displayed information

Reviewed-by: Fu Siyuan -Original Message- From: Wu, Jiaxin Sent: 2017年5月22日 15:36 To: edk2-devel@lists.01.org Cc: Ye, Ting ; Fu, Siyuan ; Wu, Jiaxin Subject: [Patch 2/2] MdeModulePkg/UefiPxeBcDxe:

Re: [edk2] [Patch 1/2] MdeModulePkg/UefiPxeBcDxe: Fix the PXE BootMenu selection issue

Reviewed-by: Ye Ting -Original Message- From: Wu, Jiaxin Sent: Monday, May 22, 2017 3:36 PM To: edk2-devel@lists.01.org Cc: Ye, Ting ; Fu, Siyuan ; Wu, Jiaxin Subject: [Patch 1/2]

Re: [edk2] [Patch 2/2] MdeModulePkg/UefiPxeBcDxe: Refine the PXE boot displayed information

Reviewed-by: Ye Ting -Original Message- From: Wu, Jiaxin Sent: Monday, May 22, 2017 3:36 PM To: edk2-devel@lists.01.org Cc: Ye, Ting ; Fu, Siyuan ; Wu, Jiaxin Subject: [Patch 2/2]

Re: [edk2] [patch] BaseTools/VfrCompile: Update MAX_PATH to a bigger one

Dandan: Please use LongFilePath() to convert file path. Then, MAX_PATH limitation can be removed. Thanks Liming >-Original Message- >From: Bi, Dandan >Sent: Monday, June 05, 2017 2:26 PM >To: edk2-devel@lists.01.org >Cc: Dong, Eric ; Gao, Liming

Re: [edk2] [PATCH v2] MdeModulePkg/DxeHttpLib: Fix potential memory leaks

Reviewed-by: Ye Ting -Original Message- From: Wu, Jiaxin Sent: Tuesday, May 23, 2017 10:11 AM To: Paulo Alcantara ; edk2-devel@lists.01.org Cc: Dong, Eric ; Zeng, Star Subject: RE: [edk2] [PATCH v2]

Re: [edk2] [PATCH] MdePkg SmmIoLib: Use NULL pointer check instead of useless Status check

Reviewed-by: jiewen@intel.com > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Star > Zeng > Sent: Monday, June 5, 2017 2:41 PM > To: edk2-devel@lists.01.org > Cc: Yao, Jiewen ; Zeng, Star >

Re: [edk2] [RFC v2 1/2] BaseTool/VfrCompile: Support Union type in VFR

Dandan, Is it possible to avoid adding IsUnion member variable? I remember .G syntax supports passing value to sub-statement. In this case, can you use: vfrDataStructFields [IsUnion]? Thanks/Ray > -Original Message- > From: Bi, Dandan > Sent: Monday, June 5, 2017 12:31 PM > To:

[edk2] [PATCH] MdePkg SmmIoLib: Use NULL pointer check instead of useless Status check

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=587 The Status check in "if (!EFI_ERROR (Status))" condition is useless, it should be NULL pointer check. And this patch also fixes a typo "continous" to "continuous". Cc: Jiewen Yao Contributed-under: TianoCore

[edk2] [patch] BaseTools/VfrCompile: Update MAX_PATH to a bigger one

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=579 Cc: Eric Dong Cc: Liming Gao Cc: Daniel Díaz Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi ---