Re: [edk2] [PATCH V2 24/50] BaseTool/Pkcs7: Add TestRoot.cer.

2016-10-07 Thread Zhu, Yonghong
Reviewed-by: Yonghong Zhu Best Regards, Zhu Yonghong -Original Message- From: Yao, Jiewen Sent: Friday, September 30, 2016 8:22 PM To: edk2-devel@lists.01.org Cc: Zhu, Yonghong ; Gao, Liming ; Kinney, Michael D

Re: [edk2] [PATCH 0/3] Add PcdRecoveryFileName PCD.

2016-10-07 Thread Tian, Feng
Reviewed-by: Feng Tian -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Jiewen Yao Sent: Saturday, October 8, 2016 11:00 AM To: edk2-devel@lists.01.org Subject: [edk2] [PATCH 0/3] Add PcdRecoveryFileName PCD. This series

[edk2] [PATCH 0/3] Add PcdRecoveryFileName PCD.

2016-10-07 Thread Jiewen Yao
This series patch adds PcdRecoveryFileName PCD. This PCD is used to indicated the recovery file name. The previous name - FvMain.Fv is hardcoded in FatPei and CdExpressPei. It does not make sense to force the name. Now a platform may use any recovery file name. Jiewen Yao (3):

[edk2] [PATCH 3/3] FatPkg/FatPei: Use PcdRecoveryFileName PCD.

2016-10-07 Thread Jiewen Yao
This PCD is used to indicated the recovery file name. The previous name - FvMain.Fv is hardcoded in FatPei. It does not make sense to force the name. Now a platform may use any recovery file name. Cc: Ruiyu Ni Cc: Feng Tian Cc: Star Zeng

[edk2] [PATCH 2/3] MdeModulePkg/CdExpressPei: Use PcdRecoveryFileName PCD.

2016-10-07 Thread Jiewen Yao
This PCD is used to indicated the recovery file name. The previous name - FvMain.Fv is hardcoded in CdExpressPei. It does not make sense to force the name. Now a platform may use any recovery file name. Cc: Feng Tian Cc: Star Zeng Cc: Liming Gao

[edk2] [PATCH 1/3] MdeModulePkg/dec: Add PcdRecoveryFileName PCD.

2016-10-07 Thread Jiewen Yao
This PCD is used to indicated the recovery file name. The previous name - FvMain.Fv is hardcoded in FatPei and CdExpressPei. It does not make sense to force the name. Now a platform may use any recovery file name. Cc: Feng Tian Cc: Star Zeng Cc: Liming

Re: [edk2] [PATCH] MdePkg: Fix some typing errors in the header files

2016-10-07 Thread Kinney, Michael D
Thomas, Thanks for fixing these typos. I have also reviewed both of your patches and have pushed them to edk2/master. https://github.com/tianocore/edk2/commit/00b7cc0fe327fcd00d70a8d30f16245cca0c55f2 https://github.com/tianocore/edk2/commit/3868d06d8209d4907c0c6b6f5ec690aebb8b5971 Mike >

Re: [edk2] [PATCH] MdePkg: Fix some typing errors in the header files

2016-10-07 Thread Kinney, Michael D
Reviewed-by: Michael Kinney > -Original Message- > From: Gao, Liming > Sent: Monday, September 12, 2016 2:10 AM > To: Thomas Huth ; edk2-de...@ml01.01.org > Cc: Kinney, Michael D > Subject: RE: [PATCH] MdePkg: Fix

Re: [edk2] [PATCH] MdePkg: Fix some typing errors

2016-10-07 Thread Kinney, Michael D
Reviewed-by: Michael Kinney > -Original Message- > From: Gao, Liming > Sent: Monday, September 12, 2016 2:10 AM > To: Thomas Huth ; edk2-de...@ml01.01.org > Cc: Kinney, Michael D > Subject: RE: [PATCH] MdePkg: Fix

[edk2] SetCurDir() doesn't allow '\\' in Dir?

2016-10-07 Thread Tim Lewis
I did a simple break apart of a path into map-name and directory. fs0:\MdeModulePkg So FileSystem was "fs0:" and Dir was "\MdeModulePkg" and the resulting working directory was: "fs0:MdeModulePkg" (with escape characters).. It seems the culprit is in EfiShellSetCurDir: if

[edk2] [Patch 2/2] BaseTools/GenFds: Skip parse time OUTPUT_DIRECTORY file verify

2016-10-07 Thread Michael Kinney
https://bugzilla.tianocore.org/show_bug.cgi?id=132 This patch relaxes the file verification for all FILE statements to not verify the file exists during initial FDF file parsing if the file specified is in $(OUTPUT_DIRECTORY). If a file is specified does not exist when a section is processed,

[edk2] [Patch 0/2] BaseTools/GenFds: Make FDF parser mode flexible

2016-10-07 Thread Michael Kinney
This patch series addresses the following issues: https://bugzilla.tianocore.org/show_bug.cgi?id=132 https://bugzilla.tianocore.org/show_bug.cgi?id=141 This patch series replaces the following patch email: https://lists.01.org/pipermail/edk2-devel/2016-October/002433.html The logic for the

[edk2] [Patch 1/2] BaseTools/GenFds: Support FDF sections in any order

2016-10-07 Thread Michael Kinney
https://bugzilla.tianocore.org/show_bug.cgi?id=141 This patch updates EDK II FDF parser in GenFds to allow sections to be placed in any order in the FDF file. Cc: Kelly Steele Cc: Yonghong Zhu Cc: Liming Gao

Re: [edk2] [Patch 0/5] QuarkPlatformPkg: Fix BOOT_IN_RECOVERY_MODE issues

2016-10-07 Thread Steele, Kelly
Entire patch reviewed. Looks good to me. Reviewed-by: Kelly Steele > -Original Message- > From: Steele, Kelly > Sent: October 07, 2016 12:27 > To: Kinney, Michael D ; edk2-devel@lists.01.org > Subject: RE: [Patch 0/5] QuarkPlatformPkg:

Re: [edk2] [Patch 0/2] QuarkSocPkg/QncSmmDispatcher: Fix SMI Handler ASSERTs()

2016-10-07 Thread Steele, Kelly
Entire patch reviewed. Looks good to me. Reviewed-by: Kelly Steele > -Original Message- > From: Steele, Kelly > Sent: October 07, 2016 12:09 > To: Kinney, Michael D ; edk2-devel@lists.01.org > Subject: RE: [Patch 0/2]

[edk2] [PATCH] ShellPkg:‘cd \’ command fails to go back to the root directory of a file system

2016-10-07 Thread Tapan Shah
Allows cd command to go back to the root directory when 'cd \' executed in system. This change prevents last PathRemoveLastItem() call which truncates '\' from 'fs0:\' in desired root path which is required to set CWD to the root directory. Contributed-under: TianoCore Contribution Agreement

Re: [edk2] [Patch 0/5] QuarkPlatformPkg: Fix BOOT_IN_RECOVERY_MODE issues

2016-10-07 Thread Steele, Kelly
Reviewed-by: Kelly Steele > -Original Message- > From: Kinney, Michael D > Sent: October 06, 2016 12:12 > To: edk2-devel@lists.01.org > Cc: Steele, Kelly > Subject: [Patch 0/5] QuarkPlatformPkg: Fix BOOT_IN_RECOVERY_MODE issues > >

Re: [edk2] [Patch 0/2] QuarkSocPkg/QncSmmDispatcher: Fix SMI Handler ASSERTs()

2016-10-07 Thread Steele, Kelly
Reviewed-by: Kelly Steele > -Original Message- > From: Kinney, Michael D > Sent: October 05, 2016 11:29 > To: edk2-devel@lists.01.org > Cc: Steele, Kelly > Subject: [Patch 0/2] QuarkSocPkg/QncSmmDispatcher: Fix SMI Handler > ASSERTs()

Re: [edk2] [PATCH] ShellPkg: Fix erroneous Status returned by ShellOpenFileByName()

2016-10-07 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey And pushed. > -Original Message- > From: Shah, Tapan [mailto:tapands...@hpe.com] > Sent: Thursday, October 06, 2016 3:04 PM > To: Vladimir Olovyannikov ; edk2- > de...@lists.01.org; Carsey, Jaben

Re: [edk2] [PATCH 0/3] MdeModulePkg/TerminalDxe: TtyTerm improvements

2016-10-07 Thread Laszlo Ersek
Roy, Ryan, On 10/07/16 16:53, Brian J. Johnson wrote: > This patch series implements some improvements to the TtyTerm terminal > type in the TerminalDxe driver. It fixes an end case with cursor > position tracking, and uses that to optimize cursor motion escape > sequences. It also adds support

Re: [edk2] [PATCH 0/3] MdeModulePkg/TerminalDxe: TtyTerm improvements

2016-10-07 Thread Leif Lindholm
Roy can now be found at Roy Franz (cc:d). On Fri, Oct 07, 2016 at 05:56:26PM +0200, Laszlo Ersek wrote: > Roy, Ryan, > > On 10/07/16 16:53, Brian J. Johnson wrote: > > This patch series implements some improvements to the TtyTerm terminal > > type in the TerminalDxe driver.

Re: [edk2] [PATCH] ArmPlatformPkg/PlatformPeiLib: allow patch PCDs for FV base address/size

2016-10-07 Thread Leif Lindholm
On Fri, Oct 07, 2016 at 04:15:36PM +0100, Ard Biesheuvel wrote: > On 7 October 2016 at 16:14, Ard Biesheuvel wrote: > > Allow patchable or dynamic PCDs to be used for PcdFvBaseAddress and > > PcdFvBaseSize, so that platforms whose UEFI image may be loaded anywhere > >

Re: [edk2] [PATCH] ArmPlatformPkg/PlatformPeiLib: allow patch PCDs for FV base address/size

2016-10-07 Thread Ard Biesheuvel
On 7 October 2016 at 16:14, Ard Biesheuvel wrote: > Allow patchable or dynamic PCDs to be used for PcdFvBaseAddress and > PcdFvBaseSize, so that platforms whose UEFI image may be loaded anywhere > in memory can use PlatformPeiLib. > ... and also, drop some references

[edk2] [PATCH] ArmPlatformPkg/PlatformPeiLib: allow patch PCDs for FV base address/size

2016-10-07 Thread Ard Biesheuvel
Allow patchable or dynamic PCDs to be used for PcdFvBaseAddress and PcdFvBaseSize, so that platforms whose UEFI image may be loaded anywhere in memory can use PlatformPeiLib. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ---

Re: [edk2] OVMF.fd and placement of EfiBootServicesData

2016-10-07 Thread Laszlo Ersek
On 10/07/16 05:11, spam collector wrote: > - Original Message - >> From: "Laszlo Ersek" >> To: "spam collector" >> Cc: edk2-de...@ml01.01.org >> Sent: Thursday, October 6, 2016 12:39:39 AM >> Subject: Re: [edk2] OVMF.fd and placement of

[edk2] [PATCH 2/3] MdeModulePkg/TerminalDxe: Optimize TtyTerm cursor motion

2016-10-07 Thread Brian J. Johnson
For TtyTerm terminals, output a shorter escape sequence when possible to move the cursor within the current line, and don't print any escape sequence if the cursor is already at the correct position. This removes extra cursor motion activity at the EFI shell prompt, improving performance. It