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

2015-08-19 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.Compared with previous patch,mainly update the console page. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan

[edk2] [patch] MdeModulePkg:Refine the code in HiiDatabaseDxe

2015-08-23 Thread Dandan Bi
Check the local variable 'NameSpace' before use it. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi dandan...@intel.com --- MdeModulePkg/Universal/HiiDatabaseDxe/ConfigKeywordHandler.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git

[edk2] [patch 1/2] IntelFrameworkModulePkg:Refine the code comments in LegacyBootMaintUiLib

2015-08-03 Thread Dandan Bi
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi dandan...@intel.com --- .../Library/LegacyBootMaintUiLib/LegacyBootMaintUi.c | 4 +--- .../Library/LegacyBootMaintUiLib/LegacyBootMaintUiLib.inf| 12 ++-- 2 files changed, 7 insertions(+), 9

[edk2] [patch] MdeModulePkg:Fix the issue that refresh the question fail in DriverSample

2015-08-03 Thread Dandan Bi
Once the question is refreshed,the processing should happen in the callback function of EFI_BROWSER_ACTION_RETRIEVE case,rather than EFI_BROWSER_ACTION_CHANGING in DriverSample. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi dandan...@intel.com

[edk2] [patch] MdeModulePkg:Use safe string functions in UiApp.

2015-08-10 Thread Dandan Bi
Replace the unsafe string functions with the safe one in UiApp. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi dandan...@intel.com --- .../Application/UiApp/BootMaint/BootOption.c | 30 ++ .../Application/UiApp/BootMaint/FormGuid.h

[edk2] [patch 1/2] IntelFrameworkModulePkg:Refine the code in LegacyBootMaintUiLib

2015-08-10 Thread Dandan Bi
Refine the code in LegacyBootMaintUiLib to prevent the potential risk. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi dandan...@intel.com --- .../Library/LegacyBootMaintUiLib/LegacyBootMaintUi.c| 13 + 1 file changed, 9 insertions(+), 4

[edk2] [patch] MdeModulePkg:SetupBrowser output debug message when DisplayEngineDxe is not installed

2015-07-24 Thread Dandan Bi
Enhance the setupbrowserdxe to output debug message when DisplayEngineDxe is not installed and this will be easy for user to find the reason why can not enter Setup page. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi dandan...@intel.com --- MdeModulePkg

[edk2] [patch] MdeModulePkg:Modify the pointer type.

2015-11-08 Thread Dandan Bi
Previously the pointer type is EFI_IFR_FORM_SET, it is incorrect when do pointer addition without conversion.Now change it to UINT8 type. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> Reviewed-by: Eric Dong <eric.d...@

[edk2] [patch 5/7] MdeModulePkg/Include/Guid:Add Guid file

2015-11-11 Thread Dandan Bi
Add the FrontPage Guid and SetupEnter Guid used by new added libraries. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> --- MdeModulePkg/Include/Guid/HiiFrontPageFormset.h | 26 + MdeModulePkg/Includ

[edk2] [patch 6/7] MdeModulePkg: To support new UiApp.

2015-11-11 Thread Dandan Bi
Add the new added Guid and libraries to the dec and dsc file in MdeModulePkg. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> --- MdeModulePkg/MdeModulePkg.dec | 6 ++ MdeModulePkg/MdeModulePkg.dsc | 4 2 files chang

[edk2] [patch 2/7] MdeModulePkg:Create Device Manager Library.

2015-11-11 Thread Dandan Bi
Split the device manager library from UiApp in MdeModulePkg/Application and put the library in MdeModulePkg/Library. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> --- .../Library/DeviceManagerLib/DeviceManager.c

[edk2] [patch 1/7] MdeModulePkg:Create Boot Manager Library

2015-11-11 Thread Dandan Bi
Split the boot manager library from UiApp in MdeModulePkg/Application and put the library in MdeModulePkg/Library. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> --- MdeModulePkg/Library/BootManagerLib/BootManager.c

[edk2] [patch 7/7] Nt32Pk: To support new UiApp.

2015-11-11 Thread Dandan Bi
Add the new added libraries to the Nt32 platform. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> --- Nt32Pkg/Nt32Pkg.dsc | 4 1 file changed, 4 insertions(+) diff --git a/Nt32Pkg/Nt32Pkg.dsc b/Nt32Pkg/Nt32Pkg.dsc index 2

[edk2] [patch 0/7]MdeModulePkg:UiApp remodeling task

2015-11-11 Thread Dandan Bi
Our current UiApp code is huge and hard to understand and maintain. So we split it to several independent libraries:device manager,boot manager, boot maintenance manager.Make the code clear and easy to maintain. Dandan Bi (7): MdeModulePkg:Create Boot Manager Library MdeModulePkg:Create

[edk2] [patch 1/2] BaseTools:To generate string default type correctly in VfrCompiler

2015-08-26 Thread Dandan Bi
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi dandan...@intel.com --- BaseTools/Source/C/VfrCompile/VfrFormPkg.h | 1 + BaseTools/Source/C/VfrCompile/VfrSyntax.g | 6 -- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/BaseTools/Source/C

[edk2] [patch] MdeModulePkg:Support orderedList with default value

2015-09-01 Thread Dandan Bi
Our tool can support OrderedList which has default value,but doesn't update the source code in HiiDatabase when parse the ifr data.Now update the code and add test case in DriverSample. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.

[edk2] [patch] MdeModulePkg:Fix the bug the incorrect change of StrCpyS function

2015-09-16 Thread Dandan Bi
The pointer to the destination string changed,the max length also changed.Previous change neglect this point. And base on the code logic,we can use StrCatS to replace StrCpyS.Now this patch is to fix this bug. Signed-off-by: Dandan Bi <dandan...@intel.com> --- MdeModulePkg/Library/Uefi

[edk2] [patch] MdeModulePkg:Fix bug that get the password width info incorrectly

2015-12-16 Thread Dandan Bi
Cc: Liming Gao <liming@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> --- MdeModulePkg/Universal/HiiDatabaseDxe/ConfigKeywordHandler.c | 2 ++ 1 file changed, 2 inserti

[edk2] [patch] MdeModulePkg:Update Boot Manager form when enter it.

2015-12-17 Thread Dandan Bi
: Liming Gao <liming@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> --- MdeModulePkg/Library/BootManagerLib/BootManager.c | 16 MdeModulePkg/Libr

[edk2] [patch] MdeModulePkg:Clear the screen before booting the boot option

2015-12-17 Thread Dandan Bi
greement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> --- MdeModulePkg/Library/BootManagerLib/BootManager.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/MdeModulePkg/Library/BootManagerLib/BootManager.c b/MdeModulePkg/Library/BootManagerLib/BootManager.c index a38f1

[edk2] [patch] MdeModulePkg:Avoid ASSERT in HiiConfigRoutingRouteConfig

2016-01-04 Thread Dandan Bi
Add error handling code to enhance the code,the driver may not install the ConfigAccess protocol,so should not just ASSERT here. Cc: Liming Gao <liming@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Danda

[edk2] [patch] MdeModulePkg:Fix the potential memory leak issue in Display Engine

2016-01-04 Thread Dandan Bi
The MenuOption insert to gMenuOption allocate memory everytime,but not free. Now add the code to free it. Cc: Liming Gao <liming@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dand

[edk2] [PATCH v2] MdeModulePkg:Fix the potential memory leak issue in Display Engine

2016-01-06 Thread Dandan Bi
: Liming Gao <liming@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> --- .../Universal/DisplayEngineDxe/FormDisplay.c | 46 ++ 1 file changed, 38 insertio

[edk2] [patch] MdeModulePkg:Refine the comments in INF files

2015-11-25 Thread Dandan Bi
Add comments in INF file in BootMaintenanceManagerLib.inf. Refine the comments in INF file in FileExplorerLib.inf Cc: Liming Gao <liming@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dand

[edk2] [patch] MdeModulePkg:Correct the file name in UiApp.inf

2015-11-24 Thread Dandan Bi
Cc: Liming Gao <liming@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> --- MdeModulePkg/Application/UiApp/UiApp.inf | 2 +- 1 file changed, 1 insertion(+), 1 delet

[edk2] [PATCH v2] MdeModulePkg:Replace unsafe string function with safe one.

2015-11-26 Thread Dandan Bi
V2: Update the Copyright info. Use StrCpyS instead of StrCatS when copy str1 to str. Cc: Liming Gao <liming@intel.com> Cc: Eric Dong <eric.d...@intel.com> Cc: Qiu Shumin <shumin@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by:

[edk2] [PATCH v2 6/6] Nt32Pkg:To support new UiApp

2015-11-19 Thread Dandan Bi
Add the new added libraries to the Nt32 platform. Cc: Liming Gao <liming@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> Reviewed-by: Eric Dong <eric.d...@intel.com>

[edk2] [PATCH v2 5/6] MdeModulePkg: To support new UiApp

2015-11-19 Thread Dandan Bi
Add the new added Guid and libraries to the dec and dsc file in MdeModulePkg. Cc: Liming Gao <liming@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> Reviewed-by:

[edk2] [PATCH v2 1/6] MdeModulePkg:Create Boot Manager Library

2015-11-19 Thread Dandan Bi
Split the boot manager library from UiApp in MdeModulePkg/Application and put the library in MdeModulePkg/Library. Cc: Liming Gao <liming@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dand

[edk2] [PATCH v2 0/6] UiApp remodeling task

2015-11-19 Thread Dandan Bi
directly in Library constructor Only define the gEfiIfrFrontPageGuid in MdeModulePkg.dec file, not add HiiFrontPageFormset.h file. Update MdeModulePkg.dsc and Nt32Pkg.dsc file to specify the NULL class Library instance.in UiApp.inf LibraryClass section. Dandan Bi (6): MdeModulePkg:Create Boot

[edk2] [PATCH v2 2/6] MdeModulePkg:Create Device Manager Library

2015-11-19 Thread Dandan Bi
Split the device manager library from UiApp in MdeModulePkg/Application and put the library in MdeModulePkg/Library. Cc: Liming Gao <liming@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dand

[edk2] [patch 8/8] Nt32Pkg:Modify the dsc file because of the rename action

2016-01-12 Thread Dandan Bi
Cc: Liming Gao <liming@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> Reviewed-by: Eric Dong <eric.d...@intel.com> --- Nt32Pkg/Nt32Pkg.dsc | 6 +++--- 1 file cha

[edk2] [patch 0/8] MdeModulePkg:Rename the new libraries separated from UiApp

2016-01-12 Thread Dandan Bi
nagerUiLib and DeviceManagerUiLib. And modify related c files,inf files and dsc files because of the rename action. Cc: Liming Gao <liming@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> Re

[edk2] [patch 7/8] MdeModulePkg:Modify the dsc file because of the rename action

2016-01-12 Thread Dandan Bi
Cc: Liming Gao <liming@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> Reviewed-by: Eric Dong <eric.d...@intel.com> --- MdeModulePkg/MdeModulePkg.dsc | 6 +++--- 1 fi

[edk2] [patch 5/8] MdeModulePkg:Rename DeviceManagerLib to DeviceManagerUiLib

2016-01-12 Thread Dandan Bi
Cc: Liming Gao <liming@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> Reviewed-by: Eric Dong <eric.d...@intel.com> --- .../Library/DeviceManagerLib/Dev

[edk2] [patch 6/8] MdeModulePkg:Modify related files because of rename action

2016-01-12 Thread Dandan Bi
Cc: Liming Gao <liming@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> Reviewed-by: Eric Dong <eric.d...@intel.com> --- MdeModulePkg/Library/DeviceManagerUiLib/De

[edk2] [patch 4/8] MdeModulePkg:Modify related files because of rename action

2016-01-12 Thread Dandan Bi
Cc: Liming Gao <liming@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> Reviewed-by: Eric Dong <eric.d...@intel.com> --- MdeModulePkg/Library/BootManagerUiLib/

[edk2] [patch] MdeModulePkg/Database: Fix incorrect calculation of baseline in FontPackage

2016-06-08 Thread Dandan Bi
the baseline when the glyph block type is EFI_HII_GIBT_GLYPH_DEFAULT or EFI_HII_GIBT_GLYPHS_DEFAULT. Cc: Liming Gao <liming@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com>

[edk2] [patch] MdeModulePkg/UiApp: Fix the incorrect use of the HiiHandle

2016-06-12 Thread Dandan Bi
In current code, when adding string package, it will return 'gStringPackHandle'. But the code use the 'gHiiHandle' to get string. It is incorrect. This patch is to fix this issue. Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dan

[edk2] [patch] MdeModulePkg/UiApp: Get current language when open front page

2016-06-12 Thread Dandan Bi
ode when open the front page. Then the language menu can show the language info correctly. Cc: Liming Gao <liming@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> Reviewed-

[edk2] [patch] MdeModulePkg/HiiDatabase: Preinit background when render string to screen

2016-06-01 Thread Dandan Bi
ibuted-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> Reviewed-by: Eric Dong <eric.d...@intel.com> --- MdeModulePkg/Universal/HiiDatabaseDxe/Font.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/MdeModulePkg/Universal/HiiDatabaseDx

[edk2] [patch] MdeModulePkg/HiiDatabase: Remove the incorrect calculation of font baseline

2016-06-01 Thread Dandan Bi
is not necessary. So remove it now. Cc: Liming Gao <liming@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> Reviewed-by: Eric Dong <eric.d...@intel.com> --- MdeModulePkg/

[edk2] [patch] MdeModulePkg/SetupBrowser: Correct the incorrect variable name

2016-05-29 Thread Dandan Bi
Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> Reviewed-by: Eric Dong <eric.d...@intel.com> --- MdeModulePkg/Universal/SetupBrowserDxe/Expression.c | 4 ++-- 1 file changed, 2 insertio

[edk2] [patch 0/2] Connect all devices in UiApp if needed

2016-06-15 Thread Dandan Bi
Do the connect all action in UiApp if needed, and no need to do the connect action in other UI library. Cc: Liming Gao <liming@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@inte

[edk2] [patch 2/2] MdeModulePkg/BootManagerUiLib: No need to connect all devices

2016-06-15 Thread Dandan Bi
The connect all action has been done in UiApp, no need to do it when update boot manager form. Cc: Liming Gao <liming@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> Revi

[edk2] [patch 1/2] MdeModulePkg/UiApp: Connect all devices in UiApp if needed

2016-06-15 Thread Dandan Bi
If the connect all action has not been performed before. We do it in UiApp now. Cc: Liming Gao <liming@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> Reviewed-by:

[edk2] [PATCH v2] MdeModulePkg/UiApp: Retrieve the value of language menu

2016-06-15 Thread Dandan Bi
c: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> Reviewed-by: Eric Dong <eric.d...@intel.com> --- .../Application/UiApp/FrontPageCustomizedUiSupport.c| 13 + 1 file changed

[edk2] [patch] MdeModulePkg/HiiDatabaseDxe: Add ASSERT before using the pointer 'String'

2016-06-23 Thread Dandan Bi
<eric.d...@intel.com> Cc: Hao Wu <hao.a...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> --- MdeModulePkg/Universal/HiiDatabaseDxe/ConfigKeywordHandler.c | 8 1 file changed, 4 insertions(+), 4 deleti

[edk2] [patch 3/6] OvmfPkg:Add FileExplorerLib.inf to the dsc file

2016-02-01 Thread Dandan Bi
Because SecureBootConfigDxe use FileExplorerLib now,but FileExplorerLib is not in the dsc file of the package which use SecureBootConfigDxe.Now add it to pass build. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> --- OvmfPkg/OvmfP

[edk2] [patch 5/6] SecurityPkg:Add FileExplorerLib.inf to the dsc file

2016-02-01 Thread Dandan Bi
Because SecureBootConfigDxe use FileExplorerLib now,but FileExplorerLib is not in the dsc file of the package which use SecureBootConfigDxe.Now add it to pass build. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> --- Secur

[edk2] [patch 2/6] ArmVirPkg:Add FileExplorerLib.inf to the dsc file

2016-02-01 Thread Dandan Bi
Because SecureBootConfigDxe use FileExplorerLib now,but FileExplorerLib is not in the dsc file of the package which use SecureBootConfigDxe.Now add it to pass build. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> --- ArmV

[edk2] [patch 4/6] Vlv2TbltDevicePkg:Add FileExplorerLib.inf to the dsc file

2016-02-01 Thread Dandan Bi
Because SecureBootConfigDxe use FileExplorerLib now,but FileExplorerLib is not in the dsc file of the package which use SecureBootConfigDxe.Now add it to pass build. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> --- Vlv2TbltDev

[edk2] [patch 1/6] ArmPlatformPkg:Add FileExplorerLib.inf to the dsc file

2016-02-01 Thread Dandan Bi
Because SecureBootConfigDxe use FileExplorerLib now,but FileExplorerLib is not in the dsc file of the package which use SecureBootConfigDxe.Now add it to pass build. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> --- ArmPlatf

[edk2] [patch 0/6] Use FileExplorerLib in SecureBootConfigDxe

2016-02-01 Thread Dandan Bi
Using existing library FileExplorerLib to replace the same logic in SecureBootConfigDxe.And add FileExplorerLib to dsc files of related packages which use SecureBootConfigDxe. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> Dandan

[edk2] [patch 6/6] SecurityPkg:Use FileExplorerLib in SecureBootConfigDxe

2016-02-01 Thread Dandan Bi
to FileExplorer. Now using FileExplorerLib the behavior will keep same with enroll KEK/DB/...,commit/discard changes will return to Custom Secure Boot Options form and ESC will return to PK options form. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.

[edk2] [PATCH v3 5/6] SecurityPkg: Add FileExplorerLib.inf to the dsc file

2016-02-02 Thread Dandan Bi
on Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> --- SecurityPkg/SecurityPkg.dsc | 1 + 1 file changed, 1 insertion(+) diff --git a/SecurityPkg/SecurityPkg.dsc b/SecurityPkg/SecurityPkg.dsc index 0f1fc0f..bec58f0 100644 --- a/SecurityPkg/SecurityPkg.dsc +++ b/SecurityPkg/Securit

[edk2] [PATCH v3 1/6] ArmPlatformPkg: Add FileExplorerLib.inf to the dsc file

2016-02-02 Thread Dandan Bi
ndh...@linaro.org> Cc: Ard Biesheuvel <ard.biesheu...@linaro.org> Cc: Ryan.Harkin <ryan.har...@linaro.org> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> --- ArmPlatformPkg/

[edk2] [PATCH v3 6/6] SecurityPkg: Use FileExplorerLib in SecureBootConfigDxe

2016-02-02 Thread Dandan Bi
o Ersek <ler...@redhat.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> Acked-by: Laszlo Ersek <ler...@redhat.com> --- .../SecureBootConfigDxe/SecureBootConfig.vfr | 109 +- .../SecureBootConfigDxe/Sec

[edk2] [PATCH v3 3/6] OvmfPkg: Add FileExplorerLib.inf to the dsc file

2016-02-02 Thread Dandan Bi
<jordan.l.jus...@intel.com> Cc: Laszlo Ersek <ler...@redhat.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> --- OvmfPkg/OvmfPkgIa32.dsc| 5 - OvmfPkg/OvmfPkgIa32X64.dsc | 5 -

[edk2] [PATCH v3 4/6] Vlv2TbltDevicePkg: Add FileExplorerLib.inf to the dsc file

2016-02-02 Thread Dandan Bi
vid@intel.com> Cc: Tim He <tim...@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> --- Vlv2TbltDevicePkg/PlatformPkgGccX64.dsc | 3 ++- Vlv2TbltDevicePkg/PlatformPkgIA32.dsc |

[edk2] [PATCH v3 2/6] ArmVirPkg: Add FileExplorerLib.inf to the dsc file

2016-02-02 Thread Dandan Bi
ler...@redhat.com> Cc: Ard Biesheuvel <ard.biesheu...@linaro.org> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> --- ArmVirtPkg/ArmVirtQemu.dsc | 5 - 1 file changed, 4 insertions(+), 1

[edk2] [PATCH v3 0/6] Use FileExplorerLib in SecureBootConfigDxe

2016-02-02 Thread Dandan Bi
t;david@intel.com> Cc: Tim He <tim...@intel.com> Cc: Chao Zhang <chao.b.zh...@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> Dandan Bi (6): ArmPlatformP

[edk2] [patch] SecurityPkg:Use FileExplorerLib in SecureBootConfigDxe

2016-02-01 Thread Dandan Bi
Using existing library FileExplorerLib to replace the same logic in SecureBootConfigDxe to make the code clear. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> --- .../SecureBootConfigDxe/SecureBootConfig.vfr

[edk2] [patch 6/6] SecurityPkg:Use FileExplorerLib in SecureBootConfigDxe

2016-02-01 Thread Dandan Bi
: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> --- .../SecureBootConfigDxe/SecureBootConfig.vfr | 109 +- .../SecureBootConfigDxe/SecureBootConfigDxe.inf|3 +- .../SecureBootConfigFileExplorer.c

[edk2] [PATCH v2 5/6] SecurityPkg: Add FileExplorerLib.inf to the dsc file

2016-02-02 Thread Dandan Bi
<chao.b.zh...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> --- SecurityPkg/SecurityPkg.dsc | 1 + 1 file changed, 1 insertion(+) diff --git a/SecurityPkg/SecurityPkg.dsc b/SecurityPkg/SecurityPkg.dsc index 0f1fc0f..be

[edk2] [patch] MdeModulePkg: Add ASSERT to make sure pointer 'OptionalData' not be NULL

2016-02-25 Thread Dandan Bi
Cc: Liming Gao <liming@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> --- MdeModulePkg/Library/BootMaintenanceManagerUiLib/Variable.c | 2 ++ 1 file changed, 2 inserti

[edk2] [patch] ArmVirPkg: Add FileExplorerLib.inf to the dsc file

2016-02-24 Thread Dandan Bi
.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> --- ArmVirtPkg/ArmVirtQemuKernel.dsc | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/ArmVirtPkg/ArmVirtQemuKernel.dsc b/ArmVirtPkg/ArmVirtQemuKernel.ds

[edk2] [PATCH v4 5/5] SecurityPkg: Use FileExplorerLib in SecureBootConfigDxe

2016-02-25 Thread Dandan Bi
o Ersek <ler...@redhat.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> Acked-by: Laszlo Ersek <ler...@redhat.com> Reviewed-by: Eric Dong <eric.d...@intel.com> Reviewed-by: Chao Zhang <chao.b.zh...@

[edk2] [PATCH v4 0/5] Use FileExplorerLib in SecureBootConfigDxe

2016-02-25 Thread Dandan Bi
ent 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> Dandan Bi (5): ArmVirPkg: Add FileExplorerLib.inf to the QEMU dsc files OvmfPkg: Add FileExplorerLib.inf to the dsc file Vlv2TbltDevicePkg: Add FileExplorerLib.inf to the dsc file SecurityPkg: Add FileExplorerLib.inf to the dsc

[edk2] [PATCH v4 3/5] Vlv2TbltDevicePkg: Add FileExplorerLib.inf to the dsc file

2016-02-25 Thread Dandan Bi
d-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> Reviewed-by: Eric Dong <eric.d...@intel.com> Reviewed-by: David Wei <david@intel.com> --- Vlv2TbltDevicePkg/PlatformPkgGccX64.dsc | 3 ++- Vlv2TbltDevicePkg/PlatformPkgIA32.dsc | 3 +

[edk2] [PATCH v4 4/5] SecurityPkg: Add FileExplorerLib.inf to the dsc file

2016-02-25 Thread Dandan Bi
on Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> Reviewed-by: Eric Dong <eric.d...@intel.com> Reviewed-by: Chao Zhang <chao.b.zh...@intel.com> --- SecurityPkg/SecurityPkg.dsc | 1 + 1 file changed, 1 insertion(+) diff --git a/SecurityPkg/SecurityPkg.dsc b/SecurityPkg

[edk2] [PATCH v2 3/3] MdeModulePkg: Add submitted callback test case in DriverSample

2016-02-22 Thread Dandan Bi
r: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> --- .../Universal/DriverSampleDxe/DriverSample.c | 24 +- .../Universal/DriverSampleDxe/NVDataStruc.h| 3 ++- MdeModulePkg/Universal/DriverSampleDxe/Vfr.vfr | 10 ++

[edk2] [PATCH v2 0/3] Add new HII action type EFI_BROWSER_ACTION_SUBMITTED

2016-02-22 Thread Dandan Bi
the change that not related to submitted callback issue. Add test case in DriverSample. Cc: Liming Gao <liming@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> Dandan Bi (3): M

[edk2] [PATCH v2 2/3] MdeModulePkg: Add new HII action type EFI_BROWSER_ACTION_SUBMITTED

2016-02-22 Thread Dandan Bi
Add new HII action type EFI_BROWSER_ACTION_SUBMITTED to notify HII driver when its question values are submitted. Cc: Liming Gao <liming@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dand

[edk2] [PATCH v2 1/3] MdePkg: Add new HII action type EFI_BROWSER_ACTION_SUBMITTED

2016-02-22 Thread Dandan Bi
Base on the UEFI2.6, Add the new HII action type EFI_BROWSER_ACTION_SUBMITTED to notify HII driver when its question values are submitted. Cc: Liming Gao <liming@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-b

[edk2] [patch] MdeModulePkg:Make the logic in ConfigRouting.c clear and safe

2016-01-24 Thread Dandan Bi
are not consistent,now refine the code to make the logic clear. Cc: Liming Gao <liming@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> Reviewed-by: Eric Dong <eric.d...@intel.co

[edk2] [patch 0/2] Refine function comments in Keyword Handler Protocol

2016-02-14 Thread Dandan Bi
The follow two patches mainly to refine the function comments in EFI Configuration Keyword Handler Protocol. Cc: Liming Gao <liming@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.

[edk2] [patch 2/2] MdePkg: Refine the function comments in Keyword Handler Protocol

2016-02-14 Thread Dandan Bi
Refine the function comments in HiiConfigKeyword.h. Cc: Liming Gao <liming@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> Reviewed-by: Eric Dong <eric.d...@intel.com

[edk2] [patch] MdeModulePkg: Make HII configuration settings available to OS runtime

2016-02-15 Thread Dandan Bi
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> --- MdeModulePkg/Universal/HiiDatabaseDxe/Database.c | 135 - .../Universal/HiiDatabaseDxe/HiiDatabase.h | 16 ++- MdeModulePkg/Universal/HiiDatabaseDxe/

[edk2] [patch 1/2] MdePkg: Add new HII action type EFI_BROWSER_ACTION_SUBMITTED

2016-02-16 Thread Dandan Bi
Base on the UEFI2.6, Add the new HII action type EFI_BROWSER_ACTION_SUBMITTED to notify HII driver when its question values are submitted. Cc: Liming Gao <liming@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-b

[edk2] [patch 2/2] MdeModulePkg: Add new HII action type EFI_BROWSER_ACTION_SUBMITTED

2016-02-16 Thread Dandan Bi
Add new HII action type EFI_BROWSER_ACTION_SUBMITTED to notify HII driver when its question values are submitted. Cc: Liming Gao <liming@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dand

[edk2] [patch 0/2] Add new HII action type EFI_BROWSER_ACTION_SUBMITTED

2016-02-16 Thread Dandan Bi
0 Signed-off-by: Dandan Bi <dandan...@intel.com> Dandan Bi (2): MdePkg: Add new HII action type EFI_BROWSER_ACTION_SUBMITTED MdeModulePkg: Add new HII action type EFI_BROWSER_ACTION_SUBMITTED MdeModulePkg/Universal/SetupBrowserDxe/Setup.c | 170 - MdePk

[edk2] [patch] MdeModulePkg: Refine the code in BootMaintenanceManagerUiLib

2016-02-17 Thread Dandan Bi
<liming@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> --- .../BootMaintenanceManager.h | 11 +- .../BootMaintenanceManagerUiLib/BootOption.c |

[edk2] [patch] MdeModulePkg: Should not overwrite the BltX

2016-02-18 Thread Dandan Bi
when has next line to draw, should not overwrite the BltX to 0, instead should keep the BltX value that pass into StringToImage function. Cc: Liming Gao <liming@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-b

[edk2] [PATCH v2] MdeModulePkg: HiiDatabaseDxe: HiiStringToImage() should not overwrite BltX

2016-02-21 Thread Dandan Bi
o Ersek <ler...@redhat.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> --- MdeModulePkg/Universal/HiiDatabaseDxe/Font.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/MdeModulePkg/Universal/HiiDatabaseDx

[edk2] [patch] MdeModulePkg: Add ASSERT to make sure pointers are not NULL

2016-02-29 Thread Dandan Bi
Cc: Qiu Shumin <shumin@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> --- MdeModulePkg/Library/FileExplorerLib/FileExplorer.c | 2 ++ 1 file changed, 2 inserti

[edk2] [patch] SecurityPkg: SecureBootConfigDxe: Enhance the code logic

2016-03-01 Thread Dandan Bi
ribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> --- .../SecureBootConfigDxe/SecureBootConfigFileExplorer.c| 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfigFileEx

[edk2] [patch] MdeModulePkg: Refine the UI code

2016-03-14 Thread Dandan Bi
Remove the ASSERT in UI code that may be triggered, and clean up the useless code. Cc: Liming Gao <liming@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> --- MdeModulePkg

[edk2] [patch] IntelFrameworkModulePkg/LegacyBootMaintUiLib: Refine the code

2016-03-14 Thread Dandan Bi
Remove the ASSERT code that may be triggered in LegacyBootMaintUiLib. Cc: Liming Gao <liming@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> --- .../LegacyBootMaintUiLib/L

[edk2] [patch] MdeModulePkg: Add missing pcd description to MdeModulePkg.uni

2016-03-08 Thread Dandan Bi
Since PcdHiiOsRuntimeSupport has added in the MdeModulePkg.dec file, now add the usage information in the uni file. Cc: Qiu Shumin <shumin@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dand

[edk2] [patch] MdeModulePkg/DriverSampleDxe: Uninstall the ConfigAccess protocol

2016-03-14 Thread Dandan Bi
on Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> --- MdeModulePkg/Universal/DriverSampleDxe/DriverSample.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/MdeModulePkg/Universal/DriverSampleDxe/DriverSample.c b/MdeModulePkg/Universal/DriverSampleDxe/DriverSample.c inde

[edk2] [PATCH v2] MdeModulePkg/HiiDatabaseDxe: Support EfiVarStore to get AltCfg from Driver

2016-04-13 Thread Dandan Bi
: Liming Gao <liming@intel.com> Cc: Eric Dong <eric.d...@intel.com> Cc: Laszlo Ersek <ler...@redhat.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> --- .../Universal/HiiDatabaseDxe/ConfigRouting.c | 534

[edk2] [patch] MdeModulePkg/HiiDatabaseDxe: Correct the ReallocatePool size

2016-04-12 Thread Dandan Bi
Cc: Liming Gao <liming@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> --- MdeModulePkg/Universal/HiiDatabaseDxe/ConfigRouting.c | 3 ++- 1 file changed, 2 insertions(+), 1

[edk2] [patch] MdeModulePkg/HiiDatabaseDxe: Support EfiVarStore to get AltCfg from Driver

2016-04-12 Thread Dandan Bi
c Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> --- .../Universal/HiiDatabaseDxe/ConfigRouting.c | 517 + 1 file changed, 517 insertions(+) diff --git a/MdeModulePkg/Univer

[edk2] [patch 2/2] MdeModulePkg/BootMaintenanceManagerUiLib: Remove type casting in ChooseFile

2016-03-21 Thread Dandan Bi
The type casting is not necessary and now remove it. Cc: Liming Gao <liming@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> --- MdeModulePkg/Library/BootMaint

[edk2] [patch 1/2] MdeModulePkg/BootMaintManagerUiLib: Declare EFIAPI for ChooseFile handler

2016-03-21 Thread Dandan Bi
handlers: CreateBootOptionFromFile, CreateDriverOptionFromFile, BootFromFile. Now specifies the calling convention for those functions. Cc: Liming Gao <liming@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Danda

[edk2] [PATCH v2] MdeModulePkg: Make HII configuration settings available to OS runtime

2016-03-01 Thread Dandan Bi
oud, Samer <samer.el-haj-mahm...@hpe.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> --- MdeModulePkg/MdeModulePkg.dec | 8 ++ MdeModulePkg/Universal/HiiDatabaseDxe/Database.c | 144 +++

[edk2] [PATCH v2] SecurityPkg: SecureBootConfigDxe: Enhance the code logic

2016-03-01 Thread Dandan Bi
c: Qiu Shumin <shumin@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> --- .../SecureBootConfigFileExplorer.c | 30 +++--- 1 file changed, 21 in

[edk2] [PATCH v3] MdeModulePkg: Make HII configuration settings available to OS runtime

2016-03-02 Thread Dandan Bi
john...@sgi.com> Cc: Andrew Fish <af...@apple.com> Cc: El-Haj-Mahmoud Samer <samer.el-haj-mahm...@hpe.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> --- MdeModulePkg/MdeModulePkg.dec | 8 ++ M

[edk2] [patch] MdeModulePkg/DisplayEngineDxe: Refine the coding style

2016-03-03 Thread Dandan Bi
Remove the empty line to conform with current coding style. Cc: Qiu Shumin <shumin@intel.com> Cc: Eric Dong <eric.d...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan...@intel.com> --- MdeModulePkg/Universal/DisplayEngin

  1   2   3   4   5   6   7   >