Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
Reviewed-by: Eric Dong
---
MdeModulePkg/Library/BootManagerLib/BootManager.c | 816 -
MdeModulePkg/Library/BootManagerLib/BootManager.h | 170 -
.../Library
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
Reviewed-by: Eric Dong
---
MdeModulePkg/MdeModulePkg.dsc | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/MdeModulePkg/MdeModulePkg.dsc b/MdeModulePkg
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
Reviewed-by: Eric Dong
---
.../Library/DeviceManagerLib/DeviceManager.c | 925 -
.../Library/DeviceManagerLib/DeviceManager.h | 194 -
.../Library
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
Reviewed-by: Eric Dong
---
MdeModulePkg/Library/DeviceManagerUiLib/DeviceManager.c| 6 +++---
MdeModulePkg/Library/DeviceManagerUiLib/DeviceManagerUiLib.inf | 8
2
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
Reviewed-by: Eric Dong
---
Nt32Pkg/Nt32Pkg.dsc | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/Nt32Pkg/Nt32Pkg.dsc b/Nt32Pkg/Nt32Pkg.dsc
index e8edb9e
consistent,now refine the code to make the logic clear.
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
Reviewed-by: Eric Dong
---
.../Universal/HiiDatabaseDxe/ConfigRouting.c | 145 ++---
1 file changed, 128
Contribution Agreement 1.0
Signed-off-by: Dandan Bi
Reviewed-by: Eric Dong
---
MdeModulePkg/Universal/SetupBrowserDxe/Setup.c | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/MdeModulePkg/Universal/SetupBrowserDxe/Setup.c
b/MdeModulePkg/Universal/SetupBrowserDxe/Setup.c
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
---
.../SecureBootConfigDxe/SecureBootConfig.vfr | 109 +-
.../SecureBootConfigDxe
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 Bi (6):
ArmPlatformPkg:Add
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
---
OvmfPkg/OvmfPkgIa32.dsc| 3 ++-
OvmfPkg
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
---
SecurityPkg/SecurityPkg.dsc | 1 +
1 file
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
---
ArmPlatformPkg/ArmVExpressPkg/ArmVExpress
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
---
ArmVirtPkg/ArmVirtQemu.dsc | 3 ++-
1 file
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
---
Vlv2TbltDevicePkg/PlatformPkgGccX64.dsc | 3
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
: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
.../SecureBootConfigDxe/SecureBootConfig.vfr | 109 +-
.../SecureBootConfigDxe/SecureBootConfigDxe.inf|3 +-
.../SecureBootConfigFileExplorer.c | 1127 +++-
.../SecureBootConfigDxe
: 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
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
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
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
: 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
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
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
Agreement 1.0
Signed-off-by: Dandan Bi
Acked-by: Laszlo Ersek
---
.../SecureBootConfigDxe/SecureBootConfig.vfr | 109 +-
.../SecureBootConfigDxe/SecureBootConfigDxe.inf|3 +-
.../SecureBootConfigFileExplorer.c | 1127 +++-
.../SecureBootConfigDxe
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
: 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
This function is mainly through the Hii handle to get the formset opcode.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
MdeModulePkg/Include/Library/HiiLib.h| 23 +++
MdeModulePkg/Library/UefiHiiLib/HiiLib.c | 111
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
Reviewed-by: Eric Dong
---
MdeModulePkg/Library/UefiHiiLib/HiiLib.c | 6
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
Add the new added libraries to the Nt32 platform.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
Nt32Pkg/Nt32Pkg.dsc | 4
1 file changed, 4 insertions(+)
diff --git a/Nt32Pkg/Nt32Pkg.dsc b/Nt32Pkg/Nt32Pkg.dsc
index 21e1f69..931cb0c 100644
--- a
Add the FrontPage Guid and SetupEnter Guid used by new added libraries.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
MdeModulePkg/Include/Guid/HiiFrontPageFormset.h | 26 +
MdeModulePkg/Include/Guid/HiiSetupEnter.h | 26
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
---
MdeModulePkg/MdeModulePkg.dec | 6 ++
MdeModulePkg/MdeModulePkg.dsc | 4
2 files changed, 10 insertions(+)
diff --git a
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
---
.../Library/DeviceManagerLib/DeviceManager.c | 953 +
.../Library
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
---
MdeModulePkg/Library/BootManagerLib/BootManager.c | 820 +
MdeModulePkg
Split the boot manager library from UiApp in MdeModulePkg/Application
and put the library in MdeModulePkg/Library.
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
Reviewed-by: Eric Dong
Reviewed-by: Liming Gao
---
MdeModulePkg
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
Split the device manager library from UiApp in MdeModulePkg/Application
and put the library in MdeModulePkg/Library.
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
Reviewed-by: Eric Dong
Reviewed-by: Liming Gao
---
.../Library
Add the new added libraries to the Nt32 platform.
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
Reviewed-by: Eric Dong
Reviewed-by: Liming Gao
---
Nt32Pkg/Nt32Pkg.dsc | 7 ++-
1 file changed, 6 insertions(+), 1 deletion
Add the new added Guid and libraries to the dec and dsc file in MdeModulePkg.
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
Reviewed-by: Eric Dong
Reviewed-by: Liming Gao
---
MdeModulePkg/MdeModulePkg.dec | 3 +++
MdeModulePkg
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
MdeModulePkg/Application/UiApp/UiApp.inf | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/MdeModulePkg/Application/UiApp/UiApp.inf
b/MdeModulePkg/Application
Add comments in INF file in BootMaintenanceManagerLib.inf.
Refine the comments in INF file in FileExplorerLib.inf
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
.../BootMaintenanceManagerLib.inf | 14
Cc: Liming Gao
Cc: Eric Dong
Cc: Qiu Shumin
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
MdeModulePkg/Library/FileExplorerLib/FileExplorer.c | 20 +++-
1 file changed, 11 insertions(+), 9 deletions(-)
diff --git a/MdeModulePkg/Library
V2:
Update the Copyright info.
Use StrCpyS instead of StrCatS when copy str1 to str.
Cc: Liming Gao
Cc: Eric Dong
Cc: Qiu Shumin
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
.../Library/FileExplorerLib/FileExplorer.c | 22
Cc: Eric Dong
Cc: Star Zeng
Cc: Qiu Shumin
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
.../BootMaintenanceManagerLib/BootMaintenance.c| 18 --
MdeModulePkg/Library/BootManagerLib/BootManager.c | 18 --
2
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
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
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
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
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
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
: 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
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
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
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
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
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
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 |
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
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
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
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
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
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
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
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
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
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
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
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 |
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
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
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
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
Initialize the variable "OptionNumber".
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
Nt32Pkg/Library/PlatformBootManagerLib/PlatformBootManager.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/Nt32Pkg/Library/PlatformBoot
For a question, its question id can not be zero.
This patch is to fix the issue that using zero as question id.
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
MdeModulePkg/Application/UiApp/FrontPageCustomizedUiSupport.c | 3
&READONLY" tag must be consistent with the result of
"ExtractReadOnlyFromOpCode" function.
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
.../Universal/HiiDatabaseDxe/ConfigKeywordHandler.c | 17 ++
Cc: Jordan Justen
Cc: Laszlo Ersek
Cc: Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
OvmfPkg/VirtioGpuDxe/Commands.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/OvmfPkg/VirtioGpuDxe/Commands.c b/OvmfPkg/VirtioGpuDxe
V2: add the assert codes.
Cc: Jordan Justen
Cc: Laszlo Ersek
Cc: Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
OvmfPkg/VirtioGpuDxe/Commands.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/OvmfPkg/VirtioGpuDxe
Remove the variable which is set but not used to pass GCC build.
Cc: Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
OptionRomPkg/Bus/Usb/UsbNetworking/Ax88772/Ax88772.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/OptionRomPkg/Bus/Usb
The type of variable "SmallestIdFromFlag" should be BOOLEAN,
this patch fix the incorrect type.
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
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
.../BootMaintenanceManagerUiLib/BootMaintenance.c | 206 ++---
.../Library/BootMaintenanceManagerUiLib/Variable.c | 28 ++-
2 files changed, 207 insertions(+), 27
This patch is mainly to:
1. Enhance the error handling codes when set variable fail.
2. Enhance the logic to fix some incorrect UI behaviors.
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
.../BootMaintenanceManagerUiLib
When fail to submit data and user discard the change, we should send
the discard info to river with EFI_BROWSER_ACTION_CHANGED callback.
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
MdeModulePkg/Universal/SetupBrowserDxe
1.0
Signed-off-by: Dandan Bi
---
.../BootMaintenanceManagerUiLib/BootMaintenance.c | 390 -
.../BootMaintenanceManagerUiLib/UpdatePage.c | 42 ++-
.../Library/BootMaintenanceManagerUiLib/Variable.c | 28 +-
3 files changed, 357 insertions(+), 103 deletions(-)
diff
The function which handles the "Boot", "BootOrder" ...
may return failure. This patch adds the error handling codes.
return the failure info to browser.
Cc: Laszlo Ersek
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Sign
Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
.../BootMaintenanceManagerUiLib/BootMaintenance.c | 177 ++---
.../BootMaintenanceManagerUiLib/UpdatePage.c | 16 +-
2 files changed, 126 insertions(+), 67 deletions
401 - 500 of 664 matches
Mail list logo