[edk2] [patch] MdePkg/HiiImageEx: Fix incorrect parameter type in GetImageInfo()

ted-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdePkg/Include/Protocol/HiiImageEx.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/MdePkg/Include/Protocol/HiiImageEx.h b/MdePkg/Include/Protocol/HiiImageEx.h index 9905ecd..b92fc3a 100644 --- a/Mde

[edk2] [PATCH v2 2/2] IntelFrameworkModulePkg/LegacyUi: Get legacy options when open legacy form

forminstead in its the constructor function. Because when opening the legacy form, the LegacyBootManagerLib must have been initialized. Cc: Liming Gao Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi Reviewed-by: Eric Dong

[edk2] [PATCH v2 0/2] MdeModulePkg/Ui: Remove the implicit dependency for libraries

orm is open in the callback function. -Patch 2:compare the question id when the form is open in the callback function. Cc: Liming Gao Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi Dandan Bi (2): MdeModulePkg/BootMaintUiLib: Update menus when o

[edk2] [PATCH v2 1/2] MdeModulePkg/BootMaintUiLib: Update menus when open BMM form

option) when opening the BMM form. We think when opening BMM form, the LeagcyBootMaintUiLib must have been initialized. Cc: Liming Gao Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi Reviewed-by: Eric Dong --- .../BootMaintenanceManagerUiLib

[edk2] [patch] MdeModulePkg/HiiDB: Record fail info if fail to save data for EfiVarStore

w the reason of failure. Cc: Liming Gao Cc: Eric Dong Cc: Chao Zhang Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi Reviewed-by: Eric Dong --- MdeModulePkg/Universal/HiiDatabaseDxe/ConfigRouting.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/MdeModu

[edk2] [patch] MdeModulePkg/BootMaintUi: Add error handling codes when AllocatePool fail

Agreement 1.0 Signed-off-by: Dandan Bi Reviewed-by: Eric Dong --- .../BootMaintenanceManagerUiLib/BootMaintenance.c | 5 ++- .../BootMaintenanceManagerUiLib/BootOption.c | 42 -- .../BootMaintenanceManagerUiLib/UpdatePage.c | 14 3 files changed, 35 insertions

[edk2] [patch 2/2] MdeModulePkg/Ui: Add ASSERT before using the pointer "Buffer"

Pointer "Buffer" is returned by function HiiGetFormSetFromHiiHandle, when the function HiiGetFormSetFromHiiHandle return success, the "Buffer" can not be NULL, so we add the ASSET codes. Cc: Eric Dong Cc: Hao Wu Contributed-under: TianoCore Contribution Agreement 1.0 Sign

[edk2] [patch 0/2]MdeModulePkg: Add error handling codes in function HiiGetFormSetFromHiiHandle

Patch 1 is to enhance the error handling codes in function HiiGetFormSetFromHiiHandle Patch 2 is enhance the logic in the caller functions of HiiGetFormSetFromHiiHandle. Cc: Eric Dong Cc: Hao Wu Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi Dandan Bi (2

[edk2] [patch 1/2] MdeModulePkg/UefiHiiLib: Add error handling codes when AllocatePool fail

Cc: Eric Dong Cc: Hao Wu Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Library/UefiHiiLib/HiiLib.c | 14 -- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/MdeModulePkg/Library/UefiHiiLib/HiiLib.c b/MdeModulePkg

[edk2] [patch 2/2] IntelFrameworkModulePkg/LegacyUi: Get legacy options when open legacy form

legacy forminstead in its the constructor function. Because when opening the legacy form, the LegacyBootManagerLib must have been initialized. Cc: Liming Gao Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi Reviewed-by: Eric Dong

[edk2] [patch 1/2] MdeModulePkg/BootMaintUiLib: Update menus when open BMM form

option) when opening the BMM form. When opening BMM form, the LeagcyBootMaintUiLib must have been initialized. Cc: Liming Gao Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi Reviewed-by: Eric Dong --- .../BootMaintenanceManagerUiLib

[edk2] [patch 0/2] MdeModulePkg/Ui: Remove the implicit dependency for libraries

: Dandan Bi Dandan Bi (2): MdeModulePkg/BootMaintUiLib: Update menus when open BMM form IntelFrameworkModulePkg/LegacyUi: Get legacy options when open legacy form .../LegacyBootMaintUiLib/LegacyBootMaintUi.c | 42 ++ .../BootMaintenanceManagerUiLib

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

odes. Cc: Eric Dong Cc: Hao Wu Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Universal/HiiDatabaseDxe/ConfigKeywordHandler.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/MdeModulePkg/Universal/HiiDatabaseDxe/Confi

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

ic Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi Reviewed-by: Eric Dong --- .../Application/UiApp/FrontPageCustomizedUiSupport.c| 13 + 1 file changed, 13 insertions(+) diff --git a/MdeModulePkg/Application/UiApp/FrontPageCustomizedU

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

The connect all action has been done in UiApp, no need to do it when update boot manager form. Cc: Liming Gao Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi Reviewed-by: Eric Dong --- MdeModulePkg/Library/BootManagerUiLib/BootManager.c | 7

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

If the connect all action has not been performed before. We do it in UiApp now. Cc: Liming Gao Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi Reviewed-by: Eric Dong --- MdeModulePkg/Application/UiApp/FrontPage.c | 6 +++--- MdeModulePkg

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

Do the connect all action in UiApp if needed, and no need to do the connect action in other UI library. Cc: Liming Gao Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi Reviewed-by: Eric Dong Dandan Bi (2): MdeModulePkg/UiApp: Connect all

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

ode when open the front page. Then the language menu can show the language info correctly. Cc: Liming Gao Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi Reviewed-by: Eric Dong --- .../UiApp/FrontPageCustomizedUiSupport.c |

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

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 Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan B

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

ating the baseline when the glyph block type is EFI_HII_GIBT_GLYPH_DEFAULT or EFI_HII_GIBT_GLYPHS_DEFAULT. Cc: Liming Gao Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Universal/HiiDatabaseDxe/Font.c | 16 1

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

not necessary. So remove it now. Cc: Liming Gao Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi Reviewed-by: Eric Dong --- MdeModulePkg/Universal/HiiDatabaseDxe/Font.c | 4 1 file changed, 4 deletions(-) diff --git a/MdeModulePkg/Universal

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

Signed-off-by: Dandan Bi Reviewed-by: Eric Dong --- MdeModulePkg/Universal/HiiDatabaseDxe/Font.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/MdeModulePkg/Universal/HiiDatabaseDxe/Font.c b/MdeModulePkg/Universal/HiiDatabaseDxe/Font.c index 56b30ff..5d31f75 100644 --- a/MdeModulePkg

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

Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi Reviewed-by: Eric Dong --- MdeModulePkg/Universal/SetupBrowserDxe/Expression.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/MdeModulePkg/Universal/SetupBrowserDxe

[edk2] [patch] MdeModulePkg: Make function comments and function match in UI codes

Cc: Qiu Shumin Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- .../Application/UiApp/FrontPageCustomizedUiSupport.c| 5 + .../Application/UiApp/FrontPageCustomizedUiSupport.h| 10

[edk2] [PATCH v2] MdeModulePkg/DisplayEngine: Fix memory leak issues in DisplayEngine

The following codes are useless and cause memory leak issues. So now remove them. Cc: Cecil Sheng Cc: Qiu Shumin Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Universal/DisplayEngineDxe/InputHandler.c | 5

[edk2] [patch] MdeModulePkg/DisplayEngine: Fix memory leak issues in DisplayEngine

Cc: Qiu Shumin Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi Reviewed-by: Eric Dong --- MdeModulePkg/Universal/DisplayEngineDxe/InputHandler.c | 5 + MdeModulePkg/Universal/DisplayEngineDxe/ProcessOptions.c | 3 ++- 2 files changed, 3

[edk2] [patch] MdeModulePkg/UiApp: Enhance the error handling

When the parameter is unsupported or invalid, should exit the function. Cc: Qiu Shumin Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi Reviewed-by: Eric Dong --- MdeModulePkg/Application/UiApp/FrontPageCustomizedUiSupport.c | 2 ++ 1 file

[edk2] [patch] MdeModulePkg/SetupBrowser: Clean the BufferValue for string before use

When copy new string content to BufferValue, need to clean the BufferValue firstly, or the BufferValue may contain some content that doesn't belong to the new string. Cc: Liming Gao Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModu

[edk2] [patch] MdeModulePkg/SetupBrowser: Should free ConfigResp when it no longer be used

When submit form fail, the progress point to the first fail part in ConfigResp, so should free the ConfigResp after Progrss has been processed. Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Universal/SetupBrowserDxe/Setup.c | 3

[edk2] [PATCH v2] Security/OpalPasswordDxe: Enhance the logic in RouteConfig/ExtractConfig

Make the implementation of RouteConfig/ExtractConfig function follow the UEFI spec. Notes: v1->v2: - Correct the storage name. Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- SecurityPkg/Tcg/Opal/OpalPasswordDxe/OpalHii.c |

[edk2] [patch] Security/OpalPasswordDxe: Enhance the logic in RouteConfig/ExtractConfig

Make the implementation of RouteConfig/ExtractConfig function follow the UEFI spec. Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- SecurityPkg/Tcg/Opal/OpalPasswordDxe/OpalHii.c | 15 +++ 1 file changed, 15 insertions(+) diff

[edk2] [patch] MdeModulePkg/FileExplore: Make LibraryClass & Depex module type consistent

Cc: Liming Gao Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Library/FileExplorerLib/FileExplorerLib.inf | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/MdeModulePkg/Library/FileExplorerLib

[edk2] [patch] MdeModulePkg/HiiDatabaseDxe: Fix memory leak issues in HiiDatabaseDxe

Cc: Qiu Shumin Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi Reviewed-by: Eric Dong --- .../Universal/HiiDatabaseDxe/ConfigRouting.c| 21 - 1 file changed, 20 insertions(+), 1 deletion(-) diff --git a/MdeModulePkg

[edk2] [patch 2/2] MdeModulePkg/SetupBrowser: Call submit callback function when no failure

In current code will always call submit callback function, but we should call submit callback function when has no failure in the submit action. This patch to fix this issues. Cc: Liming Gao Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi Reviewed

[edk2] [patch 1/2] MdeModulePkg: Restore question base on the fail info when submit fail

1.0 Signed-off-by: Dandan Bi Reviewed-by: Eric Dong --- MdeModulePkg/Universal/SetupBrowserDxe/Setup.c | 152 - MdeModulePkg/Universal/SetupBrowserDxe/Setup.h | 6 +- 2 files changed, 151 insertions(+), 7 deletions(-) diff --git a/MdeModulePkg/Universal

[edk2] [patch 0/2] MdeModulePkg/SetupBrowser: Fix two issues in submit form/formset

: Dandan Bi Dandan Bi (2): MdeModulePkg: Restore question base on the fail info when submit fail MdeModulePkg/SetupBrowser: Call submit callback function when no failure MdeModulePkg/Universal/SetupBrowserDxe/Setup.c | 167 +++-- MdeModulePkg/Universal/SetupBrowserDxe

[edk2] [patch] MdeModulePkg: BOOLEAN type needn't to compare to TRUE/FALSE explicitly

Fix this issue to follow the coding style. Cc: Qiu Shumin Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Universal/HiiDatabaseDxe/ConfigRouting.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/MdeModulePkg

[edk2] [PATCH v2] MdeModulePkg: Export ConfigResp only for form Package after ReadyToBoot

ect. - Close the ReadyToBoot event. - Add the check (whether need to export ConfigResp) in HiiGetConfigurationSetting(), doesn't update the caller logic. Cc: Liming Gao Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi Reviewed-by: E

[edk2] [patch] MdeModulePkg/SetupBrowserDxe: Remove the useless check

When the type of HiiValue is BUFFER Type, the BufferValue of the related question can not be NULL, so can remove the check. Cc: Qiu Shumin Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Universal/SetupBrowserDxe/Setup.c | 2

[edk2] [patch] MdeModulePkg/HiiDatabaseDxe: Refine the logic for Hii runtime support code

Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Universal/HiiDatabaseDxe/Database.c | 72 +- .../Universal/HiiDatabaseDxe/HiiDatabase.h | 4 +- .../Universal/HiiDatabaseDxe/HiiDatabaseEntry.c| 3 +- 3

[edk2] [patch] MdeModulePkg/HiiDatabaseDxe: Refine the coding style.

Cc: Liming Gao Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Universal/HiiDatabaseDxe/Database.c | 42 MdeModulePkg/Universal/HiiDatabaseDxe/Image.c| 4 +-- MdeModulePkg/Universal/HiiDatabaseDxe

[edk2] [patch] MdeModulePkg/HiiDatabaseDxe: Fix the VS2010/VS2012 build fail

Cc: Liming Gao Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Universal/HiiDatabaseDxe/ConfigRouting.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/MdeModulePkg/Universal/HiiDatabaseDxe/ConfigRouting.c b/MdeModulePkg

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

ction FindSameBlockElement (). Cc: Liming Gao Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- .../Universal/HiiDatabaseDxe/ConfigRouting.c | 541 + 1 file changed, 541 insertions(+) diff --git a/MdeModulePkg/Universal/HiiDataba

[edk2] [PATCH v2 2/2] MdeModulePkg/DriverSampleDxe: Add a sample case

Add the sample case for orderedlist to get standard default value from Callback function. Notes: V2: - Refine the sample case. Cc: Liming Gao Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- .../Universal/DriverSampleDxe/DriverSample.c

[edk2] [PATCH v2 1/2] MdeModulePkg/SetupBrowserDxe: Get default from callback for orderedList

1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Universal/SetupBrowserDxe/Setup.c | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/MdeModulePkg/Universal/SetupBrowserDxe/Setup.c b/MdeModulePkg/Universal/SetupBrowserDxe/Setup.c index b357e29..c36588e 100644 --- a

[edk2] [PATCH v2 0/2] Get default value from CallBack function for OrderedList

: Dandan Bi Dandan Bi (2): MdeModulePkg/SetupBrowserDxe: Get default from callback for orderedList MdeModulePkg/DriverSampleDxe: Add a sample case .../Universal/DriverSampleDxe/DriverSample.c | 53 +- .../Universal/DriverSampleDxe/NVDataStruc.h| 1

[edk2] [patch 1/2] MdeModulePkg/SetupBrowserDxe: Get default from callback for orderedList

1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Universal/SetupBrowserDxe/Setup.c | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/MdeModulePkg/Universal/SetupBrowserDxe/Setup.c b/MdeModulePkg/Universal/SetupBrowserDxe/Setup.c index b357e29..c36588e 100644 --- a

[edk2] [patch 2/2] MdeModulePkg/DriverSampleDxe: Add a sample case

Add the sample case for orderedlist to get standard default value from Callback function. Cc: Liming Gao Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- .../Universal/DriverSampleDxe/DriverSample.c | 53

[edk2] [patch 0/2] Get default value from CallBack function for OrderedList

SetupBrowser to fix the issue described above. Patch 2 is to add a sample case for orderedlist to get default value through Callback() function. Cc: Liming Gao Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi Dandan Bi (2): MdeModulePkg

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

. Cc: Liming Gao Cc: Eric Dong Cc: Laszlo Ersek Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- .../Universal/HiiDatabaseDxe/ConfigRouting.c | 534 + 1 file changed, 534 insertions(+) diff --git a/MdeModulePkg/Universal/HiiDataba

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

under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- .../Universal/HiiDatabaseDxe/ConfigRouting.c | 517 + 1 file changed, 517 insertions(+) diff --git a/MdeModulePkg/Universal/HiiDatabaseDxe/ConfigRouting.c b/MdeModulePkg/Universal/HiiDatabaseDxe/ConfigRout

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

Cc: Liming Gao Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Universal/HiiDatabaseDxe/ConfigRouting.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/MdeModulePkg/Universal/HiiDatabaseDxe/ConfigRouting.c b

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

handlers: CreateBootOptionFromFile, CreateDriverOptionFromFile, BootFromFile. Now specifies the calling convention for those functions. Cc: Liming Gao Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- .../Library/BootMaintenanceManagerUiLib

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

The type casting is not necessary and now remove it. Cc: Liming Gao Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Library/BootMaintenanceManagerUiLib/BootMaintenance.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions

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

In DriverSampleInit, has installed the configAccess protocol to the DriverHandle[1], but don't uninstall it in DriverSampleUnload. Now uninstall the configAccess protocol. Cc: Liming Gao Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dand

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

Remove the ASSERT in UI code that may be triggered, and clean up the useless code. Cc: Liming Gao Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Application/UiApp/FrontPage.c | 7 ++- .../BootMaintenanceManagerUiLib

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

Remove the ASSERT code that may be triggered in LegacyBootMaintUiLib. Cc: Liming Gao Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- .../LegacyBootMaintUiLib/LegacyBootMaintUi.c | 30 ++ 1 file changed, 25 insertions

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

Since PcdHiiOsRuntimeSupport has added in the MdeModulePkg.dec file, now add the usage information in the uni file. Cc: Qiu Shumin Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/MdeModulePkg.uni | 8 1 file changed, 8

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

Remove the empty line to conform with current coding style. Cc: Qiu Shumin Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Universal/DisplayEngineDxe/FormDisplay.c | 1 - 1 file changed, 1 deletion(-) diff --git a/MdeModulePkg

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

: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/MdeModulePkg.dec | 8 ++ MdeModulePkg/Universal/HiiDatabaseDxe/Database.c | 144 - .../Universal/HiiDatabaseDxe/HiiDatabase.h | 30 + .../Universal

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

previous version: - Add a Pcd to control this feature, default is enabled. - Export the date after ReadyToBoot. Cc: Liming Gao Cc: Eric Dong Cc: Brian J. Johnson Cc: Andrew Fish Cc: El-Haj-Mahmoud, Samer Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi

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

Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- .../SecureBootConfigFileExplorer.c | 30 +++--- 1 file changed, 21 insertions(+), 9 deletions(-) diff --git a/SecurityPkg/VariableAuthenticated/SecureBootConfigDxe

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

The function AllocateCopyPool may return NULL, so need to do check after calling it. This patch is to enhance the related logic. Cc: Chao Zhang Cc: Qiu Shumin Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- .../SecureBootConfigDxe

[edk2] [patch] MdeModulePkg: FileExplorerLib: Modify the EFI_FILE_EXPLORE_FORMSET_GUID

Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Library/FileExplorerLib/FormGuid.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/MdeModulePkg/Library/FileExplorerLib/FormGuid.h b/MdeModulePkg/Library/FileExplorerLib/FormGuid.h index 011ae22..d0a43a9

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

Cc: Qiu Shumin Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Library/FileExplorerLib/FileExplorer.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/MdeModulePkg/Library/FileExplorerLib/FileExplorer.c b/MdeModulePkg

[edk2] [patch] MdeModulePkg: FileExplorerLib: Add check when to call FreePool function

when free up resource allocated for a MenuEntry, there exists the case that the DevicePath and DisplayString are NULL, so before calling FreePool function, need to check. Cc: Liming Gao Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi

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

Agreement 1.0 Signed-off-by: Dandan Bi Acked-by: Laszlo Ersek Reviewed-by: Eric Dong Reviewed-by: Chao Zhang --- .../SecureBootConfigDxe/SecureBootConfig.vfr | 109 +- .../SecureBootConfigDxe/SecureBootConfigDxe.inf|3 +- .../SecureBootConfigFileExplorer.c | 1127

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

Because SecureBootConfigDxe use FileExplorerLib now, but FileExplorerLib is not in the dsc file of the package that use SecureBootConfigDxe. Now add it to pass build. Cc: Chao Zhang Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi Reviewed-by: Eric

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

x27;t need previous patch. - ArmVirtPkg/ArmVirtQemuKernel.dsc has been added, so add FileExplorerLib.inf to it. Cc: Laszlo Ersek Cc: Ard Biesheuvel Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi Dandan Bi (5): ArmVirPkg: Add FileExplorerLib.i

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

Because SecureBootConfigDxe use FileExplorerLib now, but FileExplorerLib is not in the dsc file of the package that use SecureBootConfigDxe. Now add it to pass build. Cc: David Wei Cc: Tim He Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi

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

Because SecureBootConfigDxe use FileExplorerLib now, but FileExplorerLib is not in the dsc file of the package that use SecureBootConfigDxe. Now add it to pass build. Cc: Jordan Justen Cc: Laszlo Ersek Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan

[edk2] [PATCH v4 1/5] ArmVirPkg: Add FileExplorerLib.inf to the QEMU dsc files

: Laszlo Ersek Cc: Ard Biesheuvel Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- ArmVirtPkg/ArmVirtQemu.dsc | 5 - ArmVirtPkg/ArmVirtQemuKernel.dsc | 5 - 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/ArmVirtPkg

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

Cc: Liming Gao Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Library/BootMaintenanceManagerUiLib/Variable.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/MdeModulePkg/Library/BootMaintenanceManagerUiLib/Variable.c b

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

: Dandan Bi --- ArmVirtPkg/ArmVirtQemuKernel.dsc | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/ArmVirtPkg/ArmVirtQemuKernel.dsc b/ArmVirtPkg/ArmVirtQemuKernel.dsc index 3bd0cbd..bfa4dbe 100644 --- a/ArmVirtPkg/ArmVirtQemuKernel.dsc +++ b/ArmVirtPkg/ArmVirtQemuKernel.dsc @@ -1,9

[edk2] [patch] MdeModulePkg: Fix the build fail issue on VS2008

: Dandan Bi --- .../BootMaintenanceManagerUiLib/BootOption.c | 32 ++ 1 file changed, 8 insertions(+), 24 deletions(-) diff --git a/MdeModulePkg/Library/BootMaintenanceManagerUiLib/BootOption.c b/MdeModulePkg/Library/BootMaintenanceManagerUiLib/BootOption.c index 5665d04

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

Now we add new HII action type EFI_BROWSER_ACTION_SUBMITTED in EFI HII Configuration Access Protocol, so add the test case in DriverSample to show how to use it and whether it works. Cc: Liming Gao Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi

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

Add new HII action type EFI_BROWSER_ACTION_SUBMITTED to notify HII driver when its question values are submitted. Cc: Liming Gao Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Universal/SetupBrowserDxe/Setup.c | 98

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

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 Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdePkg/Include/Protocol

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

change that not related to submitted callback issue. Add test case in DriverSample. Cc: Liming Gao Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi Dandan Bi (3): MdePkg: Add new HII action type EFI_BROWSER_ACTION_SUBMITTED MdeModulePkg: Add

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

Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Universal/HiiDatabaseDxe/Font.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/MdeModulePkg/Universal/HiiDatabaseDxe/Font.c b/MdeModulePkg/Universal/HiiDatabaseDxe/Font.c index 4b70b99..56b30ff 100644 --- a/MdeModulePkg

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

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 Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Universal/HiiDatabaseDxe

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

Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- .../BootMaintenanceManager.h | 11 +- .../BootMaintenanceManagerUiLib/BootOption.c | 21 +- .../Library/BootMaintenanceManagerUiLib/Variable.c | 226

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

The following two patches mainly add the new HII action type EFI_BROWSER_ACTION_SUBMITTED to notify HII driver when its question values are submitted. Cc: Liming Gao Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi Dandan Bi (2): MdePkg: Add

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

Add new HII action type EFI_BROWSER_ACTION_SUBMITTED to notify HII driver when its question values are submitted. Cc: Liming Gao Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Universal/SetupBrowserDxe/Setup.c | 170

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

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 Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdePkg/Include/Protocol

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

1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Universal/HiiDatabaseDxe/Database.c | 135 - .../Universal/HiiDatabaseDxe/HiiDatabase.h | 16 ++- MdeModulePkg/Universal/HiiDatabaseDxe/Image.c | 10 ++ MdeModulePkg/Universal/HiiDatabaseDxe/String.c | 12 +- 4

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

Refine the function comments in HiiConfigKeyword.h. Cc: Liming Gao Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi Reviewed-by: Eric Dong --- MdePkg/Include/Protocol/HiiConfigKeyword.h | 12 +++- 1 file changed, 7 insertions(+), 5

[edk2] [patch 1/2] MdeModulePkg: Refine function comments in Keyword Handler Protocol

Refine the function comments in ConfigKeywordHandler.c and HiiDatabase.h. Cc: Liming Gao Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi Reviewed-by: Eric Dong --- MdeModulePkg/Universal/HiiDatabaseDxe/ConfigKeywordHandler.c | 12

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

The follow two patches mainly to refine the function comments in EFI Configuration Keyword Handler Protocol. Cc: Liming Gao Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi Dandan Bi (2): MdeModulePkg: Refine function comments in Keyword Handler

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

Ersek Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- OvmfPkg/OvmfPkgIa32.dsc| 5 - OvmfPkg/OvmfPkgIa32X64.dsc | 5 - OvmfPkg/OvmfPkgX64.dsc | 5 - 3 files changed, 12 insertions(+), 3 deletions(-) diff --git a/OvmfPkg

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

Because SecureBootConfigDxe use FileExplorerLib now, but FileExplorerLib is not in the dsc file of the package that use SecureBootConfigDxe. Now add it to pass build. Cc: Chao Zhang Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- SecurityPkg

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

Biesheuvel Cc: Ryan.Harkin Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- ArmPlatformPkg/ArmVExpressPkg/ArmVExpress-FVP-AArch64.dsc | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/ArmPlatformPkg/ArmVExpressPkg

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

Agreement 1.0 Signed-off-by: Dandan Bi Acked-by: Laszlo Ersek --- .../SecureBootConfigDxe/SecureBootConfig.vfr | 109 +- .../SecureBootConfigDxe/SecureBootConfigDxe.inf|3 +- .../SecureBootConfigFileExplorer.c | 1127 +++- .../SecureBootConfigDxe

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

Biesheuvel Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- ArmVirtPkg/ArmVirtQemu.dsc | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/ArmVirtPkg/ArmVirtQemu.dsc b/ArmVirtPkg/ArmVirtQemu.dsc index e6440ec..0a21bf1 100644 --- a

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

: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- Vlv2TbltDevicePkg/PlatformPkgGccX64.dsc | 3 ++- Vlv2TbltDevicePkg/PlatformPkgIA32.dsc | 3 ++- Vlv2TbltDevicePkg/PlatformPkgX64.dsc| 3 ++- 3 files changed, 6 insertions(+), 3 deletions(-) diff

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

packages that use SecureBootConfigDxe. Cc: Leif Lindholm Cc: Ard Biesheuvel Cc: Ryan.Harkin Cc: Laszlo Ersek Cc: Jordan Justen Cc: David Wei Cc: Tim He Cc: Chao Zhang Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi Dandan Bi (6): ArmPlatformPkg

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

Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- 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

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

: Laszlo Ersek Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- OvmfPkg/OvmfPkgIa32.dsc| 3 ++- OvmfPkg/OvmfPkgIa32X64.dsc | 3 ++- OvmfPkg/OvmfPkgX64.dsc | 3 ++- 3 files changed, 6 insertions(+), 3 deletions(-) diff --git a/OvmfPkg/OvmfPkgIa32.dsc b

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

: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- .../SecureBootConfigDxe/SecureBootConfig.vfr | 109 +- .../SecureBootConfigDxe/SecureBootConfigDxe.inf|3 +- .../SecureBootConfigFileExplorer.c | 1127 +++- .../SecureBootConfigDxe

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

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

<    1   2   3   4   5   6   7   >