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
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
---
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
---
OvmfPkg/OvmfPkgIa32.dsc| 3 ++-
OvmfPkg
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
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
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
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
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
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
---
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/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/Library/BootManagerUiLib/BootManager.c| 6 +++---
MdeModulePkg/Library/BootManagerUiLib/BootManagerUiLib.inf | 8
2 files
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
Reviewed-by: Eric Dong
---
.../Library/BootMaintenanceManagerUiLib/BootMaintenance.c | 6 +++---
.../BootMaintenanceManagerUiLib/BootMaintenanceManagerUiLib.inf | 8
rUiLib and DeviceManagerUiLib.
And modify related c files,inf files and dsc files because of the rename action.
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
Reviewed-by: Eric Dong
Dandan Bi (8):
Rename BootMaintenanceManagerLib
MdeModulePkg:
: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
.../Universal/DisplayEngineDxe/FormDisplay.c | 46 ++
1 file changed, 38 insertions(+), 8 deletions(-)
diff --git a/MdeModulePkg/Universal/DisplayEngineDxe
The MenuOption insert to gMenuOption allocate memory everytime,but not free.
Now add the code to free it.
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
.../Universal/DisplayEngineDxe/FormDisplay.c | 34
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
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
MdeModulePkg/Universal/HiiDatabaseDxe
Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
.../BootMaintenanceManagerLib/BootMaintenance.c| 2 +-
.../BootMaintenanceManagerLib/BootMaintenanceManager.h | 2 ++
.../Library/BootMaintenanceManagerLib/FormGuid.h | 2
: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
MdeModulePkg/Universal/HiiDatabaseDxe/Database.c | 102 -
.../Universal/HiiDatabaseDxe/HiiDatabase.h | 5 +
.../Universal/HiiDatabaseDxe/HiiDatabaseEntry.c| 51
When do UiApp remodeling task,miss clearing the screen before booting
the selected boot option,so cause some behaviors change.Now add the code
to fix this issue.
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
MdeModulePkg
: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
MdeModulePkg/Library/BootManagerLib/BootManager.c | 16
MdeModulePkg/Library/BootManagerLib/BootManagerVfr.Vfr | 11 +++
2 files changed, 23 insertions(+), 4
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
MdeModulePkg/Universal/HiiDatabaseDxe/ConfigKeywordHandler.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/MdeModulePkg/Universal/HiiDatabaseDxe/ConfigKeywordHandler.c
b
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
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: 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
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
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 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
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 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
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 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
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
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
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
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
---
MdeModulePkg/Library/UefiHiiLib/HiiLib.c
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
.../RegularExpressionDxe/RegularExpressionDxe.c| 146 +
.../RegularExpressionDxe/RegularExpressionDxe.h| 108 +++
.../RegularExpressionDxe/RegularExpressionDxe.inf | 10
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
---
MdeModulePkg/Univ
Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
MdeModulePkg/Universal/DriverSampleDxe/Vfr.vfr | 1 +
.../Universal/DriverSampleDxe/VfrStrings.uni | Bin 63554 -> 63834 bytes
.../Universal/HiiDatabaseDxe/ConfigRouting.c | 23 ++---
3 files changed,
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
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/VfrCompile/VfrFormPkg.h
Check the local variable 'NameSpace' before use it.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
MdeModulePkg/Universal/HiiDatabaseDxe/ConfigKeywordHandler.c | 9 ++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/Md
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: D
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.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
.../Application/UiApp/BootMaint/Bmstrin
TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
MdeModulePkg/Application/UiApp/BootMaint/Variable.c | 20
1 file changed, 8 insertions(+), 12 deletions(-)
diff --git a/MdeModulePkg/Application/UiApp/BootMaint/Variable.c
b/MdeModulePkg/Application/UiApp/BootMaint/
Replace the unsafe string functions with the safe one in UiApp.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
.../Application/UiApp/BootMaint/BootOption.c | 30 ++
.../Application/UiApp/BootMaint/FormGuid.h | 2
Refine the code in UiApp to prevent the potential risk.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
MdeModulePkg/Application/UiApp/BootMaint/FileExplorer.c | 1 -
MdeModulePkg/Application/UiApp/BootMaint/Variable.c | 12 ++--
2 files changed
Refine the code in LegacyBootMaintUiLib to prevent the potential risk.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
.../Library/LegacyBootMaintUiLib/LegacyBootMaintUi.c| 13 +
1 file changed, 9 insertions(+), 4 deletions(-)
diff --git
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
---
.../Universal/DriverSampleDxe
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
.../Library/LegacyBootMaintUiLib/LegacyBootMaintUi.c | 4 +---
.../Library/LegacyBootMaintUiLib/LegacyBootMaintUiLib.inf| 12 ++--
2 files changed, 7 insertions(+), 9 deletions(-)
diff
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
.../Application/UiApp/BootMaint/BootMaint.c| 1 -
.../Application/UiApp/DeviceMngr/DeviceManager.h | 4 +-
MdeModulePkg/Application/UiApp/FrontPage.c | 4 ++
MdeModulePkg/Application/UiApp
the configuration string in SetupBrowserDxe to lowercase to fix
this issue.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
MdeModulePkg/Universal/SetupBrowserDxe/IfrParse.c | 1 +
MdeModulePkg/Universal/SetupBrowserDxe/Setup.h| 13 +
2
;ESC' to ignore."
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
.../Universal/DisplayEngineDxe/FormDisplay.c| 3 ++-
.../Universal/DisplayEngineDxe/FormDisplayStr.uni | Bin 17032 -> 17056 bytes
2 files changed, 2 insertions(+
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
---
MdeModulePkg/Universal/SetupBrowserDxe
to ignore."
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
.../Universal/DisplayEngineDxe/FormDisplay.c| 3 ++-
.../Universal/DisplayEngineDxe/FormDisplayStr.uni | Bin 17032 -> 17048 bytes
2 files changed, 2 insertions(+), 1 deletion(-)
di
When mismatch happens,there exists one case that exit current
form and display last form.Assert code don't cover this case.
Now add check to handle this situation.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
MdeModulePkg/Universal/DisplayEngi
The first parameter of match2 opcode should be the pattern
and the second one should be the string.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
MdeModulePkg/Universal/DriverSampleDxe/Vfr.vfr | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff
601 - 664 of 664 matches
Mail list logo