[edk2] [Patch 2/2] LegacyBootMaintUi: Update copyright info, cover old code existed in old BdsDxe driver.

2015-08-02 Thread Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Eric Dong Cc: Ruiyu Ni Cc: Daryl McDaniel --- .../Library/LegacyBootMaintUiLib/LegacyBootMaintUi.c | 2 +- .../Library/LegacyBootMaintUiLib/LegacyBootMaintUi.h | 2 +- .../LegacyBootMaintUiLib/LegacyBootMaintUiStrings.

[edk2] [Patch 1/2] UiApp: Update copyright info, cover old code existed in old BdsDxe driver.

2015-08-02 Thread Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Eric Dong Cc: Ruiyu Ni Cc: Daryl McDaniel --- MdeModulePkg/Application/UiApp/BootMaint/Bm.vfr | 2 +- MdeModulePkg/Application/UiApp/BootMaint/BmLib.c| 2 +- .../Application/UiApp/BootMaint/Bmstring.uni|

[edk2] [Patch 0/2] UiApp/LegacyBootMaintUi: Update Copyright info for

2015-08-02 Thread Eric Dong
Although these two modules are new added, but this code is come from IntelFrameworkModulePkg/Universal/BdsDxe driver, so update copyright info to cover these old code history. Eric Dong (2): UiApp: Update copyright info, cover old code existed in old BdsDxe driver. LegacyBootMaintUi: Up

[edk2] [Patch 3/3] OvmfPkg: Remove unused PlatformBdsLib after using new BDS.

2015-08-02 Thread Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Jordan Justen Cc: Laszlo Ersek --- OvmfPkg/Library/PlatformBdsLib/BdsPlatform.c | 1597 - OvmfPkg/Library/PlatformBdsLib/BdsPlatform.h | 292 OvmfPkg/Library/PlatformBdsLib/Pl

[edk2] [Patch 2/3] OvmfPkg: use new BDS and UiApp in MdeModulePkg

2015-08-02 Thread Ruiyu Ni
Compare to the old BDS, the new BDS separates the UI part to a standalone application UiApp. QemuBootOrderLib was changed to depend on the UefiBootManagerLib. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Jordan Justen Cc: Laszlo Ersek --- .../Library/Plat

[edk2] [Patch 1/3] ArmVirtPkg: Copy QemuBootOrderLib to ArmVirtPkg

2015-08-02 Thread Ruiyu Ni
Because OvmfPkg will change to use new BDS in MdeModulePkg. Keep the QemuBootOrderLib which depends on old BDS library in ArmVirtPkg because ArmVirtPkg are still using old BDS in IntelFrameworkModulePkg. We can unify the QemuBootOrderLib in future if ArmVirtPkg starts to use new BDS as well. Contr

[edk2] [Patch 0/3] Use new BDS and UiApp for OvmfPkg

2015-08-02 Thread Ruiyu Ni
A new BDS and UiApp was created in MdeModulePkg and are already used in Nt32Pkg. The patch changes the OvmfPkg to use the new BDS and UiApp as well. Ruiyu Ni (3): ArmVirtPkg: Copy QemuBootOrderLib to ArmVirtPkg OvmfPkg: use new BDS and UiApp in MdeModulePkg OvmfPkg: Remove unused PlatformBds

Re: [edk2] BaseTools features: multiple workspaces

2015-08-02 Thread Gao, Liming
Ray: Good question. I would like to keep consistent style with OS. ';' in windows, ':' is Linux. Thanks Liming -Original Message- From: Ni, Ruiyu Sent: Monday, August 3, 2015 1:00 PM To: Gao, Liming; edk2-devel@lists.01.org Subject: RE: BaseTools features: multiple workspaces Path se

Re: [edk2] BaseTools features: multiple workspaces

2015-08-02 Thread Ni, Ruiyu
Path separator in Linux seems to be ":". Will you always use ";"? Thanks, Ray > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Gao, Liming > Sent: Monday, August 3, 2015 11:57 AM > To: edk2-devel@lists.01.org > Subject: [edk2] BaseTools featu

Re: [edk2] [Patch] Nt32Pkg: Do the connect all and Refresh boot option after console is connected.

2015-08-02 Thread Ni, Ruiyu
Reviewed-by: Ruiyu Ni > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Eric Dong > Sent: Monday, August 3, 2015 10:57 AM > To: Ni, Ruiyu ; edk2-devel@lists.01.org > Subject: [edk2] [Patch] Nt32Pkg: Do the connect all and Refresh boot option >

Re: [edk2] [Patch] Nt32Pkg: Remove old useless bds platform lib.

2015-08-02 Thread Ni, Ruiyu
Reviewed-by: Ruiyu Ni > -Original Message- > From: Dong, Eric > Sent: Monday, August 3, 2015 10:57 AM > To: Ni, Ruiyu ; edk2-devel@lists.01.org > Subject: [Patch] Nt32Pkg: Remove old useless bds platform lib. > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Er

[edk2] BaseTools features: multiple workspaces

2015-08-02 Thread Gao, Liming
Hi, all We will update BaseTools feature to allow more than one workspaces. The detail design in the below. Please help review it. If you have any comments, please let me know. 1. Keep $(WORKSPACE) environment as is a. $(WORKSPACE) determines location of Build and Conf directory.

Re: [edk2] [PATCH] MdeModulePkg DxeCore: Move ProcessLibraryConstructorList()

2015-08-02 Thread Gao, Liming
Reviewed-by: Liming Gao -Original Message- From: Zeng, Star Sent: Monday, August 03, 2015 10:57 AM To: edk2-devel@lists.01.org Cc: Gao, Liming Subject: [PATCH] MdeModulePkg DxeCore: Move ProcessLibraryConstructorList() to right after CoreInitializeGcdServices(). Why? Some platforms may

[edk2] [PATCH] MdeModulePkg DxeCore: Move ProcessLibraryConstructorList()

2015-08-02 Thread Star Zeng
to right after CoreInitializeGcdServices(). Why? Some platforms maybe report both below 4G and above 4G memory resource hob as tested, then CoreInitializeMemoryServices() will find and add the tested above 4G memory resource hob to memory descriptor for early memory allocation services, then if

[edk2] [Patch] Nt32Pkg: Remove old useless bds platform lib.

2015-08-02 Thread Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Eric Dong --- Nt32Pkg/Library/Nt32BdsLib/BdsPlatform.c | 561 -- Nt32Pkg/Library/Nt32BdsLib/BdsPlatform.h | 119 --- Nt32Pkg/Library/Nt32BdsLib/Nt32BdsLib.inf | 63 Nt32Pkg/Library/Nt32B

[edk2] [Patch] Nt32Pkg: Do the connect all and Refresh boot option after console is connected.

2015-08-02 Thread Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Eric Dong --- Nt32Pkg/Library/PlatformBootManagerLib/PlatformBootManager.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Nt32Pkg/Library/PlatformBootManagerLib/PlatformBootManager.c b/Nt32Pkg/Library/PlatformBootManag

Re: [edk2] [patch] MdeModulePkg:Fix the issue FindQuestionFromProgress in SetupBrowserDxe is broken

2015-08-02 Thread Gao, Liming
Reviewed-by: Liming Gao -Original Message- From: Bi, Dandan Sent: Friday, July 31, 2015 11:11 AM To: Dong, Eric; Gao, Liming; edk2-devel@lists.01.org Subject: [patch] MdeModulePkg:Fix the issue FindQuestionFromProgress in SetupBrowserDxe is broken If the storage of the question is EFI

Re: [edk2] [Patch] BaseTools/build_rule: Added --trim-long option.

2015-08-02 Thread Gao, Liming
Reviewed-by: Liming Gao -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Yingke Liu Sent: Monday, August 03, 2015 10:46 AM To: edk2-devel@lists.01.org Subject: [edk2] [Patch] BaseTools/build_rule: Added --trim-long option. asm and asm16 need to t

[edk2] [Patch] BaseTools/build_rule: Added --trim-long option.

2015-08-02 Thread Yingke Liu
asm and asm16 need to trim ULL suffix number. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yingke Liu --- BaseTools/Conf/build_rule.template | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/BaseTools/Conf/build_rule.template b/BaseTools/Conf/buil

Re: [edk2] [PATCH] IntelFspPkg: Remove the const condition if statement to refine the code.

2015-08-02 Thread Yao, Jiewen
Reviewed by: Yao, Jiewen -Original Message- From: Qiu, Shumin Sent: Monday, August 03, 2015 9:54 AM To: edk2-devel@lists.01.org Cc: Qiu, Shumin; Yao, Jiewen Subject: [PATCH] IntelFspPkg: Remove the const condition if statement to refine the code. Cc: Yao Jiewen Contributed-under: Tian

[edk2] [PATCH] IntelFspPkg: Remove the const condition if statement to refine the code.

2015-08-02 Thread Qiu Shumin
Cc: Yao Jiewen Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Qiu Shumin --- IntelFspPkg/FspDxeIpl/DxeIpl.c | 101 +++-- 1 file changed, 46 insertions(+), 55 deletions(-) diff --git a/IntelFspPkg/FspDxeIpl/DxeIpl.c b/IntelFspPkg/FspDxeI

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

2015-08-02 Thread Liu, Yingke D
Hi Ard, Sorry for delaying reply, please go ahead, If Liming agreed BaseTools' patches, you could commit them without my feedback. Thanks, Dennis -Original Message- From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] Sent: Friday, July 31, 2015 19:05 To: edk2-devel@lists.01.org; Ju

Re: [edk2] [PATCH] MdeModulePkg DxeIpl: Add stack NX support

2015-08-02 Thread Yao, Jiewen
Reviewed by: Yao, Jiewen -Original Message- From: Zeng, Star Sent: Friday, July 31, 2015 8:23 PM To: edk2-devel@lists.01.org Cc: Yao, Jiewen Subject: [PATCH] MdeModulePkg DxeIpl: Add stack NX support This feature is added for UEFI spec that says "Stack may be marked as non-executable i