[edk2] [patch 1/3] MdeModulePkg/FileExplorer: Remove the codes of searching load file protocol

https://bugzilla.tianocore.org/show_bug.cgi?id=323 FileExplorer no need to search load file protocol to show files. Now remove the codes. Cc: Eric Dong Cc: Liming Gao Cc: Jiewen Yao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- .../Library

[edk2] [patch] MdePkg: Avoid Non-Boolean type used as Boolean

Cc: Brijesh Singh Cc: Leo Duran Cc: Liming Gao Signed-off-by: Leo Duran Reviewed-by: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdePkg/Library/BaseIoLibIntrinsic/IoLibIpf.c | 12 ++-- MdePkg/Library/DxeIoLibCpuIo2

[edk2] [patch] OvmfPkg/QemuBootOrderLib: Fix NOOPT build failure

Cc: Jordan Justen Cc: Laszlo Ersek Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- OvmfPkg/Library/QemuBootOrderLib/ExtraRootBusMap.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/OvmfPkg/Library/QemuBootOrderLib

[edk2] [PATCH v2] OvmfPkg/QemuBootOrderLib: Fix NOOPT build failure

This patch is to fix the IA32/NOOPT/VS Toolchain build failure. Cc: Jordan Justen Cc: Laszlo Ersek Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi Notes: V2: Change the cast type form UINT32 to UINTN. Add commit message to cover the

[edk2] [PATCH v3] OvmfPkg/QemuBootOrderLib: Fix NOOPT build failure

error LNK1120: 1 unresolved externals NMAKE : fatal error U1077: '"C:\Program Files\Microsoft Visual Studio 14.0\Vc\bin\link.exe"' : return code '0x460' Stop. Cc: Jordan Justen Cc: Laszlo Ersek Cc: Liming Gao Contributed-under: TianoCore Contribution Agreemen

[edk2] [patch] MdeMoudlePkg/DisplayEngine: Fix incorrect index used in array "InputText"

/show_bug.cgi?id=358 Cc: Eric Dong Cc: Liming Gao Cc: Wang Cloud Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- .../Universal/DisplayEngineDxe/InputHandler.c | 33 -- 1 file changed, 18 insertions(+), 15 deletions(-) diff

[edk2] [patch] MdeModulePkg/HiiDatabase: clean the value before setting default string

1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Universal/HiiDatabaseDxe/ConfigRouting.c | 12 ++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/MdeModulePkg/Universal/HiiDatabaseDxe/ConfigRouting.c b/MdeModulePkg/Universal/HiiDatabaseDxe/ConfigRouting.c index d547f42

[edk2] [patch] MdeModulePkg/BMMUiLib: Remove old useless data before new save action

: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- .../BootMaintenanceManagerUiLib/BootMaintenance.c | 35 +- .../BootMaintenanceManager.h | 14 - 2 files changed, 47 insertions(+), 2 deletions

[edk2] [PATCH v2] MdeModulePkg/HiiDatabase: clean the value before setting default string

-by: Dandan Bi --- MdeModulePkg/Universal/HiiDatabaseDxe/ConfigRouting.c | 17 +++-- 1 file changed, 15 insertions(+), 2 deletions(-) diff --git a/MdeModulePkg/Universal/HiiDatabaseDxe/ConfigRouting.c b/MdeModulePkg/Universal/HiiDatabaseDxe/ConfigRouting.c index d547f42..ccf4b5a

[edk2] [patch] MdeModulePkg/BMMUiLib: Replace same logic with API in UefiBootManagerLib

Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- .../BootMaintenanceManager.h | 37 +--- .../Library/BootMaintenanceManagerUiLib/Variable.c | 202 + 2 files changed, 11 insertions(+), 228 deletions(-) diff --git a

[edk2] [patch] MdeModulePkg: Fix coding style issues

Core Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- .../SmiHandlerProfileInfo/SmiHandlerProfileInfo.c | 2 +- .../SmmSmiHandlerProfileLib/SmmSmiHandlerProfileLib.inf| 2 +- MdeModulePkg/MdeModulePkg.uni | 14 ++ 3 files changed

[edk2] [patch] MdeModulePkg: Add PROMPT&HELP string of pcd to UNI file

Cc: Brijesh Singh Cc: Leo Duran Cc: Ard Biesheuvel Cc: Star Zeng Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/MdeModulePkg.dec | 2 +- MdeModulePkg/MdeModulePkg.uni | 38 ++ 2 files changed, 39

[edk2] [patch] MdeModulePkg/DxeCore: Fix coding style issues

Add comments for functions. Cc: Ard Biesheuvel Cc: Star Zeng Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Core/Dxe/Mem/Pool.c | 19 +++ MdeModulePkg/Core/Dxe/Misc/MemoryProtection.c | 6 -- 2 files changed

[edk2] [patch] EmulatorPkg: Add the UefiBootManagerLib library

Since shell has consumed the APIs in UefiBootManagerLib. Cc: Jordan Justen Cc: Andrew Fish Cc: Ruiyu Ni Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- EmulatorPkg/EmulatorPkg.dsc | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a

[edk2] [patch] MdePkg/HiiImageEx: Fix incorrect comments of functions

Cc: Liming Gao Cc: Ruiyu Ni Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdePkg/Include/Protocol/HiiImageEx.h | 80 +++- 1 file changed, 43 insertions(+), 37 deletions(-) diff --git a/MdePkg/Include

[edk2] [patch] ShellPkg/UefiShellBcfgCommandLib: Fix VS2012 build failure

Cc: Ruiyu Ni Cc: Jaben Carsey Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- ShellPkg/Library/UefiShellBcfgCommandLib/UefiShellBcfgCommandLib.c | 1 + 1 file changed, 1 insertion(+) diff --git a/ShellPkg/Library/UefiShellBcfgCommandLib

[edk2] [patch] OvmfPkg/QemuFwCfgS3Lib: Fix VS tool chain build failure

Cc: Jordan Justen Cc: Laszlo Ersek Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- OvmfPkg/Include/Library/QemuFwCfgS3Lib.h| 10 +- OvmfPkg/Library/QemuFwCfgS3Lib/QemuFwCfgS3Base.c| 3 ++- OvmfPkg/Library/QemuFwCfgS3Lib

[edk2] [patch] MdeModulePkg/DxeCapsuleLibFmp: Fix build failure issues

Cc: Jiewen Yao Cc: Star Zeng Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Library/DxeCapsuleLibFmp/DxeCapsuleLib.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/MdeModulePkg/Library/DxeCapsuleLibFmp/DxeCapsuleLib.c b/MdeModulePkg

[edk2] [patch] MdeModulePkg/DxeCapsuleLibFmp: Fix build failure

Move the definition of variable "mEsrtTable" and "mIsVirtualAddrConverted" to DxeCapsuleLib.c. Cc: Jiewen Yao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Library/DxeCapsuleLibFmp/DxeCapsuleLib.c | 4 ++--

[edk2] [patch] DuetPkg: Add SortLib/UefiBootManagerLib

ConPlatformDxe uses the UefiBootManagerLib， and UefiBootManagerLib consumes SortLib. Cc: Ruiyu Ni Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- DuetPkg/DuetPkgIa32.dsc | 4 +++- DuetPkg/DuetPkgX64.dsc | 4 +++- 2 files changed, 6 insertions(+), 2

[edk2] [patch] EmulatorPkg: Add SortLib/UefiBootManagerLib

ConPlatformDxe uses the UefiBootManagerLib， and UefiBootManagerLib consumes SortLib. Cc: Ruiyu Ni Contributed-under: TianoCore Contribution Agreement Signed-off-by: Dandan Bi --- EmulatorPkg/EmulatorPkg.dsc | 2 ++ 1 file changed, 2 insertions(+) diff --git a/EmulatorPkg/EmulatorPkg.dsc b

[edk2] [patch] MdeModulePkg:Fix the issue that Commit changes and Exit fail when add driver option

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/

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

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

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

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

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

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

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

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

[edk2] [patch 2/2] MdeModulePkg:Set default string value

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,

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

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

[edk2] [patch] MdeModulePkg:Refine the code comments in RegularExpressionDxe.

Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- .../RegularExpressionDxe/RegularExpressionDxe.c| 146 + .../RegularExpressionDxe/RegularExpressionDxe.h| 108 +++ .../RegularExpressionDxe/RegularExpressionDxe.inf | 10

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

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

[edk2] [patch] MdeModulePkg:Add HiiGetFormSetFromHiiHandle function into UefiHiiLib

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

[edk2] [patch] CryptoPkg/BaseCryptLib: Make comments consistent with the function

Cc: Qin Long Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- CryptoPkg/Library/BaseCryptLib/Hmac/CryptHmacSha256.c | 2 +- CryptoPkg/Library/BaseCryptLib/Hmac/CryptHmacSha256Null.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a

[edk2] [patch] ShellPkg: Fix the coding style issue

Cc: Ruiyu Ni Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- ShellPkg/Application/Shell/FileHandleWrappers.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/ShellPkg/Application/Shell/FileHandleWrappers.c b/ShellPkg/Application/Shell

[edk2] [patch] MdeModulePkg/BMMUI: add comments for function parameter

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

[edk2] [PATCH v2] CryptoPkg/BaseCryptLib: Make comments consistent with the function

V2: Correct the unaligned parameter name in BaseCryptLib.h. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- CryptoPkg/Include/Library/BaseCryptLib.h | 4 ++-- CryptoPkg/Library/BaseCryptLib/Hmac/CryptHmacSha256.c | 2 +- CryptoPkg

[edk2] [patch 0/4] MdeModulePkg: Don't support password without interactive flag

code in the DriverSampleDxe. Cc: Liming Gao Cc: Eric Dong Cc: Jiewen Yao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi Dandan Bi (4): MdeModulePkg/DisplayEngine: Popup dialogue when password is not supported MdeModulePkg/SetupBrowser:Don't su

[edk2] [patch 1/4] MdeModulePkg/DisplayEngine: Popup dialogue when password is not supported

when the password is not supported, pop up a dialogue to let user know the reason. Cc: Liming Gao Cc: Eric Dong Cc: Jiewen Yao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Universal/DisplayEngineDxe/FormDisplay.c | 3 +++ MdeModulePkg

[edk2] [patch 2/4] MdeModulePkg/SetupBrowser:Don't support password without interactive flag

Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Universal/SetupBrowserDxe/Setup.c | 23 ++- 1 file changed, 2 insertions(+), 21 deletions(-) diff --git a/MdeModulePkg/Universal/SetupBrowserDxe/Setup.c b/MdeModulePkg/Universal/SetupBrowserDxe/Setup.c index 7547814

[edk2] [patch 4/4] MdeModulePkg/DriverSampleDxe: Remove the non-interactive password

Cc: Liming Gao Cc: Eric Dong Cc: Jiewen Yao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Universal/DriverSampleDxe/NVDataStruc.h | 1 - MdeModulePkg/Universal/DriverSampleDxe/Vfr.vfr | 10 -- 2 files changed, 11 deletions

[edk2] [patch 3/4] MdeModulePkg: Clear the buffer after using it

Cc: Liming Gao Cc: Eric Dong Cc: Jiewen Yao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Universal/DisplayEngineDxe/ProcessOptions.c | 3 ++- MdeModulePkg/Universal/SetupBrowserDxe/Presentation.c| 1 + 2 files changed, 3 insertions

[edk2] [patch 1/4] MdeModulePkg: Add BaseCryptLib/OpensslLib/IntrinsicLib

The DriverSample will consume BaseCryptLib/OpensslLib/IntrinsicLib, so add these libraries into the dsc files. Cc: Feng Tian Cc: Star Zeng Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/MdeModulePkg.dsc | 4 1 file changed, 4 insertions

[edk2] [patch 4/4] MdeModulePkg/DriverSample: Use SHA256 algorithm for password encoding

Use the SHA256 algorithm to hash the password instead of using XOR operation to encode the password. And remove the string opcode that echo the password. Cc: Liming Gao Cc: Eric Dong Cc: Jiewen Yao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi

[edk2] [patch 0/4] MdeModulePkg: Use SHA256 algorithm for password encoding

: Liming Gao Cc: Eric Dong Cc: Jiewen Yao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi Dandan Bi (4): MdeModulePkg: Add BaseCryptLib/OpensslLib/IntrinsicLib Nt32Pkg: Add BaseCryptLib/OpensslLib/IntrinsicLib EmulatorPkg: Add BaseCryptLib/OpensslLib

[edk2] [patch 2/4] Nt32Pkg: Add BaseCryptLib/OpensslLib/IntrinsicLib

DriverSampleDxe will consume BaseCryptLib/OpensslLib/IntrinsicLib, so remove the SECURE_BOOT_ENABLE check before these libraries now. Cc: Ruiyu Ni Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- Nt32Pkg/Nt32Pkg.dsc | 7 +++ 1 file changed, 3 insertions

[edk2] [patch 3/4] EmulatorPkg: Add BaseCryptLib/OpensslLib/IntrinsicLib

DriverSampleDxe will consume BaseCryptLib/OpensslLib/IntrinsicLib, so add these libraries to the dsc file. Cc: Jordan Justen Cc: Andrew Fish Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- EmulatorPkg/EmulatorPkg.dsc | 5 - 1 file changed, 4 insertions

[edk2] [patch] MdeModulePkg/DriverSample: Remove the password related codes

In current DriverSampleDxe, the sample code of password is not a good example, so remove it now. Cc: Liming Gao Cc: Eric Dong Cc: Jiewen Yao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- .../Universal/DriverSampleDxe/DriverSample.c | 261

[edk2] [patch] MdeModulePkg/DxeCapsuleLibFmp: Use AllocatePool for CapsuleResultVariable

Cc: Jiewen Yao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- .../Library/DxeCapsuleLibFmp/DxeCapsuleReportLib.c | 16 ++-- 1 file changed, 10 insertions(+), 6 deletions(-) diff --git a/MdeModulePkg/Library/DxeCapsuleLibFmp

[edk2] [PATCH v2] MdeModulePkg/DxeCapsuleLibFmp: Use AllocatePool for CapsuleResultVariable

V2: Add check for the result of AllocatePool function. Cc: Jiewen Yao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- .../Library/DxeCapsuleLibFmp/DxeCapsuleReportLib.c | 20 ++-- 1 file changed, 14 insertions(+), 6 deletions(-) diff

[edk2] [patch] MdeModulePkg/DisplayEngine: Return the selectable menu correctly

ng Gao Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Universal/DisplayEngineDxe/FormDisplay.c | 13 - 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/MdeModulePkg/Universal/DisplayEngineDxe/FormDisplay.c

[edk2] [PATCH v2] MdeModulePkg/DriverSample: Remove the password related codes

V2: Remove the PasswordState which is not used anymore. In current DriverSampleDxe, the sample code of password is not a good example, so we plan to remove it. Cc: Liming Gao Cc: Eric Dong Cc: Jiewen Yao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi

[edk2] [patch] MdeModulePkg: Fix GCC build failure

Remove the variables which are set but not used. Cc: Pete Batard Cc: Jiewen Yao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Universal/EbcDxe/EbcDebugger/Edb.c | 3 --- MdeModulePkg/Universal/EbcDxe/EbcDebugger

[edk2] [patch] MdeModulePkg: Fix GCC build failure

Remove the variables which are set but not used. Cc: Pete Batard Cc: Jiewen Yao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Universal/EbcDxe/EbcDebugger/Edb.c | 3 --- MdeModulePkg/Universal/EbcDxe/EbcDebugger

[edk2] [PATCH v2] MdeModulePkg: Fix GCC build failure

V2: Add check for CreateEvent and CloseEvent. Cc: Jiewen Yao Cc: Pete Batard Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Universal/EbcDxe/EbcDebugger/Edb.c | 3 --- .../Universal/EbcDxe/EbcDebugger/EdbCmdBreakpoint.c | 5

[edk2] [patch 0/5] MdeModulePkg/EbcDxe: Fix coding style issues

These patch series make the coding style of EbcDxe follow the EDKII coding rules. Cc: Jiewen Yao Cc: Pete Batard Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi Dandan Bi (5): MdeModulePkg/EbcDxe: Make the comments align with EDKII coding style

[edk2] [patch 2/5] MdeModulePkg/EbcDxe: Initialize variable after declaration

Cc: Jiewen Yao Cc: Pete Batard Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Universal/EbcDxe/EbcDebugger/Edb.c | 4 +++- MdeModulePkg/Universal/EbcDxe/EbcDebugger/EdbSymbol.c | 4 +++- 2 files changed, 6 insertions(+), 2 deletions

[edk2] [patch 3/5] MdeModulePkg/EbcDxe: Make the variable name follow rules

Cc: Jiewen Yao Cc: Pete Batard Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- .../Universal/EbcDxe/EbcDebugger/EdbSupport.h | 10 +- .../EbcDxe/EbcDebugger/EdbSupportString.c | 216 ++--- 2 files changed, 113 insertions

[edk2] [patch 5/5] MdeModulePkg/EbcDxe: Add comments for functions

Cc: Jiewen Yao Cc: Pete Batard Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- .../EbcDxe/EbcDebugger/EbcDebuggerConfig.c | 21 ++ .../EbcDxe/EbcDebugger/EdbDisasmSupport.h | 338 + .../Universal/EbcDxe/EbcDebugger

[edk2] [patch 4/5] MdeModulePkg/EbcDxe: Avoid Non-Boolean type used as Boolean

Cc: Jiewen Yao Cc: Pete Batard Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- .../Universal/EbcDxe/EbcDebugger/EdbDisasm.c | 130 ++--- .../EbcDxe/EbcDebugger/EdbDisasmSupport.c | 4 +- .../EbcDxe/EbcDebugger

[edk2] [patch] MdeModulePkg/SetupBrowserDxe: Fix incorrect local variable used

https://bugzilla.tianocore.org/show_bug.cgi?id=280 Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Universal/SetupBrowserDxe/Setup.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/MdeModulePkg

[edk2] [patch 3/8] FatPkg\EnhancedFatDxe: Make function prototype align with definition

Cc: Ruiyu Ni Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- FatPkg/EnhancedFatDxe/Info.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/FatPkg/EnhancedFatDxe/Info.c b/FatPkg/EnhancedFatDxe/Info.c index 858b3f4..aa1b0f2 100644 --- a

[edk2] [patch 1/8] FatPkg\EnhancedFatDxe: Avoid Non-Boolean type uses as Boolean

Cc: Ruiyu Ni Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- FatPkg/EnhancedFatDxe/FileName.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/FatPkg/EnhancedFatDxe/FileName.c b/FatPkg/EnhancedFatDxe/FileName.c index 5df4036..f393aa6

[edk2] [patch 4/8] FatPkg\EnhancedFatDxe: Make the variable name follow rule

Cc: Ruiyu Ni Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- FatPkg/EnhancedFatDxe/DirectoryManage.c | 16 - FatPkg/EnhancedFatDxe/DiskCache.c | 50 +-- FatPkg/EnhancedFatDxe/Fat.h | 28

[edk2] [patch 5/8] FatPkg\EnhancedFatDxe: Use typedef for complex type

Cc: Ruiyu Ni Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- FatPkg/EnhancedFatDxe/Fat.h | 36 --- FatPkg/EnhancedFatDxe/FatFileSystem.h | 10 ++ 2 files changed, 27 insertions(+), 19 deletions(-) diff --git

[edk2] [patch 0/8] FatPkg: Fix coding style issues

The patch series make the coding style of FatPkg align with EDKII rules. Cc: Ruiyu Ni Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi Dandan Bi (8): FatPkg\EnhancedFatDxe: Avoid Non-Boolean type uses as Boolean FatPkg\EnhancedFatDxe: Initialize variable

[edk2] [patch 2/8] FatPkg\EnhancedFatDxe: Initialize variable after declaration

Cc: Ruiyu Ni Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- FatPkg/EnhancedFatDxe/Misc.c | 14 +- 1 file changed, 13 insertions(+), 1 deletion(-) diff --git a/FatPkg/EnhancedFatDxe/Misc.c b/FatPkg/EnhancedFatDxe/Misc.c index f91759c..6ad688c

[edk2] [patch 7/8] FatPkg\EnhancedFatDxe: Add comments for functions

Cc: Ruiyu Ni Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- FatPkg/EnhancedFatDxe/Fat.c | 64 FatPkg/EnhancedFatDxe/Fat.h | 804 +++ FatPkg/EnhancedFatDxe/Info.c | 40 +++ 3 files changed, 908 insertions

[edk2] [patch 8/8] FatPkg: Fix format issues in dec/inf/dsc files

Cc: Ruiyu Ni Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- FatPkg/EnhancedFatDxe/Fat.inf | 2 +- FatPkg/FatPei/FatPei.inf | 6 +++--- FatPkg/FatPkg.dec | 1 - FatPkg/FatPkg.dsc | 3 +-- 4 files changed, 5 insertions(+), 7

[edk2] [PATCH v2 2/8] FatPkg/EnhancedFatDxe: Use global variable replace static local variable

Create global variable "mMonthDays" to replace static local variable "MonthDays" in function FatIsValidTime() in Misc.c. Cc: Ruiyu Ni Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- FatPkg/EnhancedFatDxe/Misc.c | 4 ++-- 1 file changed

[edk2] [patch] FatPkg/EnhancedFatDxe: Remove the file which is not used

Cc: Ruiyu Ni Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- FatPkg/EnhancedFatDxe/Debug.c | 62 --- 1 file changed, 62 deletions(-) delete mode 100644 FatPkg/EnhancedFatDxe/Debug.c diff --git a/FatPkg/EnhancedFatDxe

[edk2] [patch 01/18] ShellPkg/UefiShellDebug1CommandsLib: Fix coding style issues

Cc: Ruiyu Ni Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- ShellPkg/Library/UefiShellDebug1CommandsLib/DmpStore.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/ShellPkg/Library/UefiShellDebug1CommandsLib/DmpStore.c b/ShellPkg

[edk2] [patch 00/18] Fix coding style issues

These patch series make the coding style follow EDKII coding rules. Dandan Bi (18): ShellPkg/UefiShellDebug1CommandsLib: Fix coding style issues ShellPkg/UefiShellNetwork1CommandsLib: Fix incorrect Protocol format ShellPkg/UefiShellNetwork2CommandsLib: Fix incorrect Protocol format

[edk2] [patch 03/18] ShellPkg/UefiShellNetwork2CommandsLib: Fix incorrect Protocol format

Cc: Fu Siyuan Cc: Ruiyu Ni Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- .../UefiShellNetwork2CommandsLib/UefiShellNetwork2CommandsLib.inf | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ShellPkg/Library

[edk2] [patch 02/18] ShellPkg/UefiShellNetwork1CommandsLib: Fix incorrect Protocol format

Cc: Fu Siyuan Cc: Ruiyu Ni Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- .../UefiShellNetwork1CommandsLib/UefiShellNetwork1CommandsLib.inf | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ShellPkg/Library

[edk2] [patch 04/18] Security:Add info string to UNI file for PcdPkcs7CertBuffer

Cc: Jiewen Yao Cc: Chao Zhang Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- SecurityPkg/SecurityPkg.uni | 5 + 1 file changed, 5 insertions(+) diff --git a/SecurityPkg/SecurityPkg.uni b/SecurityPkg/SecurityPkg.uni index 17a6842..6d99638 100644 --- a

[edk2] [patch 05/18] SecurityPkg/FmpAuthenticationLibPkcs7: Fix incorrect Guid format

Cc: Jiewen Yao Cc: Chao Zhang Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- .../Library/FmpAuthenticationLibPkcs7/FmpAuthenticationLibPkcs7.inf | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/SecurityPkg/Library

[edk2] [patch 08/18] SignedCapsulePkg/EdkiiSystemCapsuleLib: Fix coding style issues in INF file

Fix incorrect Guid/Pcd format issues in INF file. Cc: Jiewen Yao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- .../EdkiiSystemCapsuleLib/EdkiiSystemCapsuleLib.inf| 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a

[edk2] [patch 07/18] SecurityPkg/OpalPasswordDxe: Make comments align with the function

Cc: Eric Dong Cc: Chao Zhang Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- SecurityPkg/Tcg/Opal/OpalPasswordDxe/OpalHii.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/SecurityPkg/Tcg/Opal/OpalPasswordDxe/OpalHii.c b/SecurityPkg/Tcg

[edk2] [patch 06/18] SecurityPkg/FmpAuthenticationLibRsa2048Sha256: Fix Guid/pcd format issues

Cc: Jiewen Yao Cc: Chao Zhang Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- .../FmpAuthenticationLibRsa2048Sha256.inf | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/SecurityPkg/Library

[edk2] [patch 09/18] SignedCapsulePkg/RecoveryModuleLoadPei: Add source file to INF file

Add RecoveryModuleLoadPei.h to [Sources] section of INF file. Cc: Jiewen Yao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- .../Universal/RecoveryModuleLoadPei/RecoveryModuleLoadPei.inf| 1 + 1 file changed, 1 insertion(+) diff --git a

[edk2] [patch 10/18] SignedCapsulePkg/SystemFirmwareUpdate: Add source file to INF file

Add SystemFirmwareDxe.h to [Sources] section of INF file. Cc: Jiewen Yao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- .../Universal/SystemFirmwareUpdate/SystemFirmwareReportDxe.inf | 1 + .../Universal/SystemFirmwareUpdate

[edk2] [patch 15/18] UefiCpuPkg/Capsule: Fix coding style issues

Cc: Jiewen Yao Cc: Jeff Fan Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- UefiCpuPkg/Feature/Capsule/MicrocodeCapsulePdb/MicrocodeCapsulePdb.dsc | 1 + UefiCpuPkg/Feature/Capsule/MicrocodeCapsuleTxt/MicrocodeCapsuleTxt.dsc | 1 + 2 files changed, 2

[edk2] [patch 17/18] MdeModulePkg: Add missing info string for PCDs

Some Pcds are added to the dec file, but miss to add the prompt&&help info to the uni file, now add them. Cc: Jiewen Yao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/MdeModulePkg.dec | 1 + MdeModulePkg/MdeModulePkg.

[edk2] [patch 11/18] SignedCapsulePkg/IniParsingLib: Update struct name

Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- SignedCapsulePkg/Library/IniParsingLib/IniParsingLib.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/SignedCapsulePkg/Library/IniParsingLib/IniParsingLib.c b/SignedCapsulePkg/Library/IniParsingLib/IniParsingLib.c index

[edk2] [patch 14/18] UefiCpuPkg/MicrocodeUpdateDxe: Fix coding style issues in INF file

1. Add MicrocodeUpdate.h to the [Sources] section. 2. Fix the incorrect Guid/Protocol format issues. Cc: Jiewen Yao Cc: Jeff Fan Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- UefiCpuPkg/Feature/Capsule/MicrocodeUpdateDxe/MicrocodeUpdateDxe.inf | 5

[edk2] [patch 12/18] UefiCpuPkg: Make the comments align with the functions

Cc: Jeff Fan Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- UefiCpuPkg/CpuDxe/CpuMp.c | 2 +- UefiCpuPkg/Include/Library/MpInitLib.h | 4 ++-- UefiCpuPkg/Library/MpInitLib/DxeMpLib.c | 4 ++-- UefiCpuPkg/Library/MpInitLib/MpLib.c| 11

[edk2] [patch 16/18] MdeModulePkg: Make the comments align with the function

Cc: Jiewen Yao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Application/CapsuleApp/AppSupport.c | 10 +- MdeModulePkg/Application/CapsuleApp/CapsuleDump.c | 6 +++--- MdeModulePkg/Core/PiSmmCore/MemoryAttributesTable.c

[edk2] [patch 18/18] MdeModulePkg/CapsuleApp: Fix GUID/protocol format issues

Cc: Jiewen Yao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Application/CapsuleApp/CapsuleApp.inf | 28 +++--- 1 file changed, 14 insertions(+), 14 deletions(-) diff --git a/MdeModulePkg/Application/CapsuleApp/CapsuleApp.inf

[edk2] [patch 13/18] UefiCpuPkg: Add info string to UNI file for PcdCpuSmmStaticPageTable

Cc: Jiewen Yao Cc: Jeff Fan Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- UefiCpuPkg/UefiCpuPkg.uni | 8 1 file changed, 8 insertions(+) diff --git a/UefiCpuPkg/UefiCpuPkg.uni b/UefiCpuPkg/UefiCpuPkg.uni index 13f0e41..298ed7e 100644 --- a

[edk2] [PATCH v2 11/18] SignedCapsulePkg/IniParsingLib: Update struct name

Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- SignedCapsulePkg/Library/IniParsingLib/IniParsingLib.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/SignedCapsulePkg/Library/IniParsingLib/IniParsingLib.c b/SignedCapsulePkg/Library/IniParsingLib/IniParsingLib.c

[edk2] [patch 2/2] MdeModulePkg/NonDiscoverablePciDevice: Make variable definition follow rule

Cc: Ard Biesheuvel Cc: Ruiyu Ni Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- .../Bus/Pci/NonDiscoverablePciDeviceDxe/NonDiscoverablePciDeviceDxe.c | 2 +- .../Bus/Pci/NonDiscoverablePciDeviceDxe/NonDiscoverablePciDeviceIo.c | 3 ++- 2 files changed, 3

[edk2] [patch 1/2] MdeModulePkg/NonDiscoverablePciDeviceDxe: Add comments for functions

Cc: Ard Biesheuvel Cc: Ruiyu Ni Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- .../NonDiscoverablePciDeviceDxe/ComponentName.c| 47 .../NonDiscoverablePciDeviceDxe.c | 50 +++- .../NonDiscoverablePciDeviceDxe.inf

[edk2] [patch 1/3] MdeModulePkg/NonDiscoverablePciDeviceDxe: Fix VS2010/2012 build failure

Cc: Ard Biesheuvel Cc: Ruiyu Ni Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- .../Bus/Pci/NonDiscoverablePciDeviceDxe/NonDiscoverablePciDeviceIo.c | 1 + 1 file changed, 1 insertion(+) diff --git a/MdeModulePkg/Bus/Pci/NonDiscoverablePciDeviceDxe

[edk2] [patch 2/3] MdeModulePkg/NonDiscoverablePciDeviceDxe: Add comments for functions

Cc: Ard Biesheuvel Cc: Ruiyu Ni Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- .../NonDiscoverablePciDeviceDxe/ComponentName.c| 47 +++ .../NonDiscoverablePciDeviceDxe.c | 50 ++- .../NonDiscoverablePciDeviceDxe.inf

[edk2] [patch 3/3] MdeModulePkg/NonDiscoverablePciDevice: Make variable definition follow rule

Cc: Ard Biesheuvel Cc: Ruiyu Ni Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- .../Bus/Pci/NonDiscoverablePciDeviceDxe/NonDiscoverablePciDeviceDxe.c | 2 +- .../Bus/Pci/NonDiscoverablePciDeviceDxe/NonDiscoverablePciDeviceIo.c | 3 ++- 2 files changed, 3

[edk2] [patch 0/8] Fix coding style issues

Dandan Bi (8): SecurityPkg/FmpAuthenticationLib: Fix Guid format issues IntelSiliconPkg: Fix format issues MdeModulePkg/CapsuleApp: Fix Guid format issues MdeModulePkg/EbcDxe: Make variable name follow rules MdeModulePkg/EmmcDxe: Avoid Non-Boolean type used as Boolean MdeModulePkg

[edk2] [patch 1/8] SecurityPkg/FmpAuthenticationLib: Fix Guid format issues

Cc: Jiewen Yao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- .../Library/FmpAuthenticationLibPkcs7/FmpAuthenticationLibPkcs7.inf | 2 +- .../FmpAuthenticationLibRsa2048Sha256.inf | 4 ++-- 2 files changed, 3 insertions(+), 3

[edk2] [patch 3/8] MdeModulePkg/CapsuleApp: Fix Guid format issues

Cc: Jiewen Yao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Application/CapsuleApp/CapsuleApp.inf | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/MdeModulePkg/Application/CapsuleApp/CapsuleApp.inf b

[edk2] [patch 2/8] IntelSiliconPkg: Fix format issues

1. Fix doxygen formt 2. Fix Guid format Cc: Jiewen Yao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- IntelSiliconPkg/Include/IndustryStandard/FirmwareInterfaceTable.h | 2 +- IntelSiliconPkg/Include/IndustryStandard/FirmwareVersionInfo.h | 2

[edk2] [patch 4/8] MdeModulePkg/EbcDxe: Make variable name follow rules

For variable name, it should contain lower case characters. Cc: Jiewen Yao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- .../Universal/EbcDxe/EbcDebugger/EdbCmdSymbol.c| 8 MdeModulePkg/Universal/EbcDxe/EbcDebugger/EdbCommon.h | 2

[edk2] [patch 5/8] MdeModulePkg/EmmcDxe: Avoid Non-Boolean type used as Boolean

Cc: Feng Tian Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Bus/Sd/EmmcDxe/EmmcBlockIo.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/MdeModulePkg/Bus/Sd/EmmcDxe/EmmcBlockIo.c b/MdeModulePkg/Bus/Sd/EmmcDxe/EmmcBlockIo.c

<    1   2   3   4   5   6   7   >