[edk2] [Patch] BaseTools/GenFw: add new option to not zero PE/COFF optional header fields

Add new option --keepoptionalheader and that flag does not zero PE/COFF optional header fields including the version fields. It can support the case that the PE/COFF optional header would be kept. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- Source/C

[edk2] [Patch] BaseTools: Fix two warning reported in the make phase.

when we make BaseTools, it report warnings about VfrError.cpp and VolInfo, so this patch fix this warning. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- Source/C/VfrCompile/VfrError.cpp | 2 +- Source/C/VolInfo/VolInfo.c | 19

[edk2] [Patch] BaseTools/toolsetup.bat: fixed the error when the path contains space

when the path contains space, it will report error for PATH Environment update. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- toolsetup.bat | 21 - 1 file changed, 16 insertions(+), 5 deletions(-) diff --git a/toolsetup.bat b

[edk2] [Patch] BaseTools/toolsetup.bat: fixed the error when the path contains space

ff-by: Yonghong Zhu --- toolsetup.bat | 21 + 1 file changed, 5 insertions(+), 16 deletions(-) diff --git a/toolsetup.bat b/toolsetup.bat index 310ddd0..76fd8bb 100755 --- a/toolsetup.bat +++ b/toolsetup.bat @@ -320,28 +320,17 @@ goto end if not defined PYTHON_FREEZER

[edk2] [Patch] BaseTools: Add build error detection for Dynamic PCD name conflict

. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- Source/Python/AutoGen/GenC.py | 52 ++- 1 file changed, 36 insertions(+), 16 deletions(-) diff --git a/Source/Python/AutoGen/GenC.py b/Source/Python/AutoGen/GenC.py index

[edk2] [Patch] BaseTools: Add a VPD report subsection of FLASH to the Report

Build Spec already added a VPD report subsection of FLASH to the Report chapter, it provide a simple way for user to determine where the VPD region and VPD PCDs are located in the fd file. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- Source/Python

[edk2] [Patch] BaseTools: Fix a bug when apply patches to SEC use the FILE_GUID override

: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- Source/Python/GenFds/FfsInfStatement.py | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/Source/Python/GenFds/FfsInfStatement.py b/Source/Python/GenFds/FfsInfStatement.py index b9cb4f2..6d47a7a 100644

[edk2] [Patch] BaseTools: Add support for INF statement in FD region

FD region today can be file or data, but not a patched image.Add support for an INF statement in an FD region, so the binary from the INF can be patched prior to being added to the FD region. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- Source/Python

[edk2] [Patch] BaseTools: Fix a bug in the VPD report generation

Changed the if condition to check whether current Region is FD VPD region to fix a bug in the VPD report generation. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- Source/Python/build/BuildReport.py | 6 +++--- 1 file changed, 3 insertions(+), 3

[edk2] [Patch] BaseTools: Enhance GenFv Tool to report error message

When two vtf files in one FV image, no FV file can be generated, but it report the stack trace info. so we enhance the tool to report error message directly but not the stack trace info. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- Source/C/GenFv

[edk2] [Patch] BaseTools: process the files by the priority in BUILDRULEORDER

1.0 Signed-off-by: Yonghong Zhu --- Source/Python/AutoGen/AutoGen.py | 27 +++ 1 file changed, 27 insertions(+) diff --git a/Source/Python/AutoGen/AutoGen.py b/Source/Python/AutoGen/AutoGen.py index fe56574..263224b 100644 --- a/Source/Python/AutoGen/AutoGen.py +++ b/Source

[edk2] [Patch] BaseTools: Fix one bug in FD region for multiple workspace support

Update the os.path.join to mws.join for FILE/INF/CAPSULE/FV region type. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python/GenFds/Region.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/BaseTools/Source

[edk2] [Patch V2] BaseTools: Support \x####\ in UNI files to specify non-ascii characters

UNI spec updated to allow using \x\ to specify non-ascii characters, # is a hex digit. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python/AutoGen/UniClassObject.py | 18 ++ 1 file changed, 14

[edk2] [Patch] BaseTools: Fix bug in GenFds to handle FV image alignment

Cover the case that .fv file in the [Binaries] section. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python/GenFds/FvImageSection.py | 27 +++- 1 file changed, 26 insertions(+), 1 deletion(-) diff

[edk2] [Patch] BaseTools: fix a bug for uni file \x####\ format handling

It should start from the last '\x' position + 1 to find next '\x' character. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python/AutoGen/UniClassObject.py | 2 +- 1 file changed, 1 insertion(+),

[edk2] [Patch] BaseTools: support private package definition

. If a module or library instance outside of the package attempts to use the item, the build must fail with an appropriate error message. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python/AutoGen/AutoGen.py | 10

[edk2] [Patch] BaseTools/GenFds: enhance to get TOOL_CHAIN_TAG and TARGET value

e set by -D Flag; 2. Get the value by the -t/-b option. 3. get the value from target.txt file. Besides, this patch also remove the error checking for missing -t/-b option. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Source/P

[edk2] [Patch] BaseTools: Fix bug to not mix comment into Asbuilt inf Depex section

g Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python/AutoGen/AutoGen.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/BaseTools/Source/Python/AutoGen/AutoGen.py b/BaseTools/Source/Python/AutoGen/AutoGen.py

[edk2] [Patch] BaseTools/GenFds: enhance INF built arch filter

odules use FILE_GUIDmodule.inf as key which cause __GetPlatformArchList__ return empty. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python/GenFds/FfsInfStatement.py | 10 ++ 1 file changed, 10 insertions(+) diff --git a

[edk2] [Patch] BaseTools/GenFw: enhance to use Magic Field to identify the image

Original use the File Header Machine Field to identify EFI_IMAGE_OPTIONAL_HEADER32 or EFI_IMAGE_OPTIONAL_HEADER64, it cannot correctly handle EBC arch PE32 image. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Source/C/GenFw

[edk2] [Patch] BaseTools: add error condition for Variable PACKAGES_PATH

This patch adds error condition 1)if PACKAGES_PATH is not exist, 2) if space is exist in the PACKAGES_PATH. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python/build/build.py | 6 ++ 1 file changed, 6 insertions

[edk2] [Patch] BaseTools: Add error handling for current_dir is not exist

Add the error handling to cover the case that current_dir is not exist. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python/AutoGen/GenMake.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/BaseTools/Source

[edk2] [Patch] BaseTools: fix the bug to build a compressed ROM image via .INF file

Fix the bug that always use the '-e' as OPTROM_FLAGS even the .INF file has statement 'PCI_COMPRESS = TRUE'. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python/AutoGen/GenMake.py |

[edk2] [Patch V2] BaseTools: fix the bug to build a compressed ROM image via .INF file

Fix the bug that always use the '-e' as OPTROM_FLAGS even the .INF file has statement 'PCI_COMPRESS = TRUE'. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python/AutoGen/GenMake

[edk2] [Patch V2] BaseTools: Add error handling for current_dir is not exist

Add the error handling to cover the case that current_dir is not exist. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python/AutoGen/GenMake.py | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a

[edk2] [Patch] BaseTools: ignore the binary LIB file in gen_libs

For single module build, it would call gen_libs target. then if it use binary LIB file, it cause build failure. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python/AutoGen/GenMake.py | 3 ++- 1 file changed, 2

[edk2] [Patch] BaseTools: Add support for $(FAMILY) macro

Build spec mentions $(FAMILY) macro be used in DSC/FDF to specify the tool chain family, like GCC, MSFT. This patch add the support for this macro. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python/build/build.py | 23

[edk2] [Patch] BaseTools: Fix report generation failure on case sensitive file system

Since in GenFds phase, the generate FV all use upper letter. In the build report, we should use the upper letter FV name to combine the file path. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python/build/BuildReport.py

[edk2] [Patch] BaseTools: Update the FV region name as upper letter

Since in the GenFds phase, the FV is generated as upper letter. This patch update the FV region name as upper letter, it can fix the build report generate failure on case sensitive file system. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu

[edk2] [Patch] BaseTools: Add build info for binary modules that only list in FDF file

If the binary module is list in the FDF file but not list in the DSC file, current build report would not include these binary module's info in the report "Module section". The patch fix this issue. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-o

[edk2] [Patch] BaseTools: Fix a bug for FixedPcd value generation in AutoGen file

If the library is listed in [Components] section for build only, its used FixedPcd Value is not generated into AutoGen code. This patch cover this case to generate the FixedPcd Value in AutoGen file. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong

[edk2] [Patch] Update edksetup.bat to check NASM system environment variable

If the NASM_PREFIX variable is not set, it would report warning message. If there exist the C:\nasm\nasm.exe file, it would set the NASM_PREFIX variable to C:\nasm\. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- edksetup.bat | 10

[edk2] [Patch] BaseTools: report error if source module INF is only list in FDF file

Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python/AutoGen/AutoGen.py | 28 BaseTools/Source/Python/GenFds/FdfParser.py | 15 +++ 2 files changed, 43 insertions(+) diff --git a/BaseTools/Source/Python/AutoGen/AutoGen.py b/BaseTools/Source/Python

[edk2] [Patch] BaseTools: Keep the Pcd order in the Asbuilt Inf is same with Source

The original behavior is that in the Asbuilt inf Pcd's order is base on the Pcd's offset. Now we change the order to keep it is same with the Pcd order in the source inf file. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu ---

[edk2] [Patch V2] BaseTools: Add build info for binary modules that only list in FDF file

If the binary module is list in the FDF file but not list in the DSC file, current build report would not include these binary module's info in the report "Module section". The patch fix this issue. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-o

[edk2] [Patch] BaseTools: Allow string token identifier to use lower case letters

This patch is to align the code behavior with UNI spec that string token identifier can use upper case and lower case letters. Cc: Liming Gao Cc: Felix Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python/AutoGen/UniClassObject.py

[edk2] [Patch] BaseTools: Fix the bug when use FILE_GUID override the module in DSC

In last commit 2502b73, it doesn't cover the case that in the DSC file use FILE_GUID to override the module. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python/AutoGen/AutoGen.py | 13 +++-- 1 file change

[edk2] [Patch 2/3] BaseTools: Rsa2048Sha256Sign add new option to support Monotonic count

the EFI_FIRMWARE_IMAGE_AUTHENTICATION struct require the AuthInfo which is a signature across the image data and the Monotonic Count value, so we add the new option to support Monotonic count. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu

[edk2] [Patch 0/3] BaseTools: Add the support for FMP capsule generate auth info

Agreement 1.0 Signed-off-by: Yonghong Zhu Yonghong Zhu (3): BaseTools: Add the PKCS7 tool BaseTools: Rsa2048Sha256Sign add new option to support Monotonic count BaseTools: FMP capsule add the support to generate auth info BaseTools/Conf/tools_def.template | 6

[edk2] [Patch 1/3] BaseTools: Add the PKCS7 tool

Provide the PKCS7 Tool to support the CertType - EFI_CERT_TYPE_PKCS7_GUID, then user can use this tool to add EFI_FIRMWARE_IMAGE_AUTHENTICATION for a binary. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jiewen Yao --- BaseTools/Conf/tools_def.template

[edk2] [Patch 3/3] BaseTools: FMP capsule add the support to generate auth info

iming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python/GenFds/Capsule.py | 80 +-- BaseTools/Source/Python/GenFds/CapsuleData.py | 4 +- BaseTools/Source/Python/GenFds/FdfParser.py

[edk2] [Patch 1/3 V2] BaseTools: Add the PKCS7 tool

Provide the PKCS7 Tool to support the CertType - EFI_CERT_TYPE_PKCS7_GUID, then user can use this tool to add EFI_FIRMWARE_IMAGE_AUTHENTICATION for a binary. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jiewen Yao --- BaseTools/Conf/tools_def.template

[edk2] [Patch] BaseTools: Fix a bug use 'COMMON' as CodeBase in BuildOptions section

anoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python/Workspace/MetaFileTable.py | 8 +++- BaseTools/Source/Python/Workspace/WorkspaceDatabase.py | 3 ++- 2 files changed, 9 insertions(+), 2 deletions(-) diff --git a/BaseTools/Source/Python/Workspace

[edk2] [Patch] BaseTools: check CONF_PATH env to get the configure files

-off-by: Yonghong Zhu --- BaseTools/Source/Python/GenFds/GenFds.py | 7 +-- BaseTools/Source/Python/build/build.py | 9 ++--- 2 files changed, 11 insertions(+), 5 deletions(-) diff --git a/BaseTools/Source/Python/GenFds/GenFds.py b/BaseTools/Source/Python/GenFds/GenFds.py index 68232c5

[edk2] [Patch] BaseTools: add capsule image header for auth FMP capsule file

in last commit 91ae29, it missed to add the EFI_FIRMWARE_MANAGEMENT_CAPSULE_IMAGE_HEADER for the auth FMP capsule. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python/GenFds/Capsule.py | 15

[edk2] [Patch] BaseTools: update BinaryFiles.txt file to add Pkcs7Sign Tool

add Pkcs7Sign.exe and related pem file into BinaryFiles.txt for build server to automatically build the binary win32 files. Cc: Liming Gao Cc: Erik Bjorge Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Source/BinaryFiles.txt | 9 - 1

[edk2] [Patch] BaseTools: UpdateImageSize include Image auth info for FMP Auth capsule

Per UEFI spec UpdateImageSize may or may not include Firmware Image Authentication information. so for FMP auth capsule, UpdateImageSize should include the Image auth info. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Source

[edk2] [Patch] BaseTools: UpdateImageSize include Image auth info for FMP Auth capsule

Per UEFI spec UpdateImageSize may or may not include Firmware Image Authentication information. so for FMP auth capsule, UpdateImageSize should include the Image auth info. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Source

[edk2] [Patch] BaseTools: Change source files to DOS format

Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Source/C/GenFv/GenFvInternalLib.c| 12 +- BaseTools/Source/Python/Pkcs7Sign/Pkcs7Sign.py | 564 - 2 files changed, 288 insertions(+), 288 deletions

[edk2] [Patch] BaseTools: Extend the Macro used in the FDF !include statement

-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python/GenFds/FdfParser.py | 41 + 1 file changed, 30 insertions(+), 11 deletions(-) diff --git a/BaseTools/Source/Python/GenFds/FdfParser.py b/BaseTools/Source/Python/GenFds

[edk2] [Patch] BaseTools: Convert incomplete expression with dangling while()

From: Nikolai SAOUKH Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Nikolai SAOUKH Reviewed-by: Yonghong Zhu --- BaseTools/Source/C/VfrCompile/VfrSyntax.g | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/BaseTools/Source/C/VfrCompile

[edk2] [Patch] BaseTools: Fix the bug to parse the short varname in map file

current in the map file, there have two ways for var to save its offset, if the varname is short, then the offset will in the same line with varname, otherwise, it saved in the next line. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu

[edk2] [Patch] BaseTools: NULL pointer comparison to a char literal should be to NULL

From: Nikolai SAOUKH GCC7 complaint -- error: ISO C++ forbids comparison between pointer and integer. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Nikolai SAOUKH Reviewed-by: Yonghong Zhu --- BaseTools/Source/C/VfrCompile/VfrUtilityLib.cpp | 2 +- 1 file changed, 1

[edk2] [Patch] BaseTools: Enhance BaseTools supports FixedAtBuild usage in VFR file

always generated. Every FixedPcd _PCD_VALUE_PcdName will be generated without the postfix U or UL or ULL. Fixes: https://bugzilla.tianocore.org/show_bug.cgi?id=348 Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python/AutoGen

[edk2] [Patch V2] BaseTools: Enhance BaseTools supports FixedAtBuild usage in VFR file

always generated. Every FixedPcd _PCD_VALUE_PcdName will be generated without the postfix U or UL or ULL. Fixes: https://bugzilla.tianocore.org/show_bug.cgi?id=348 Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python/AutoGen

[edk2] [Patch] BaseTools: Fix bug for GUIDED tool path override by DSC [BuildOptions]

Current the GUIDED tool path can't be override to the different path in the [BuildOptions] of DSC file. This patch fix the bug. Fixes: https://bugzilla.tianocore.org/show_bug.cgi?id=283 Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghon

[edk2] [Patch V2] BaseTools: Fix bug for GUIDED tool path override by DSC [BuildOptions]

Current the GUIDED tool path can't be override to the different path in the [BuildOptions] of DSC file. This patch fix the bug. Fixes: https://bugzilla.tianocore.org/show_bug.cgi?id=283 Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghon

[edk2] [Patch] BaseTools: add error check for Macro usage in the INF file

Use of MACRO statements in the EDK II INF files is limited to local usage only; global or external macros are not permitted. This patch add the check for not defined macros. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Source

[edk2] [Patch] BaseTools: Fix the regression issue caused by commit dc4c77

In the last commit dc4c77, the _GetHeaderInfo will be called more than once, which cause the self._ConstructorList.append(Value) append some duplicate value. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python/Workspace

[edk2] [Patch] BaseTools: GenFds get the Size info for FV image in the FD region

When the FV size is specify in the FD region, Tool generate the FV file may not use the correct size. Fixes: https://bugzilla.tianocore.org/show_bug.cgi?id=387 Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python/GenFds

[edk2] [Patch] BaseTools: Update some tool with shell=True

Pkcs7Sign, Rsa2048Sha256Sign and Rsa2048Sha256GenerateKeys doesn't work on Linux. It needs to be changed with shell=True. Fixes: https://bugzilla.tianocore.org/show_bug.cgi?id=423 Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- Base

[edk2] [Patch] BaseTools: Fixed an error reported during generating report

Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- Source/Python/build/BuildReport.py | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/Source/Python/build/BuildReport.py b/Source/Python/build/BuildReport.py index 60e976c

[edk2] [Patch] BaseTools: added extern protocols/PPI/GUID definition in AutoGen header for Library

We already added the extern declaration for protocols/PPI/GUID in AutoGen.h file, but missing this feature for the Library. so this patch added this feature for library. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- Source/Python/AutoGen/GenC.py | 27

[edk2] [Patch] BaseTools:added extern protocol/PPI/GUID definition in AutoGen for Library

We already added the extern declaration for protocols/PPI/GUID in AutoGen.h file for driver, but missing this feature for the Library. so this patch add it. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- Source/Python/AutoGen/GenC.py | 27

[edk2] [Patch] BaseTools:remove the redundant directories for '-f' with absolute path.

when the absolute path is given to '-f', it would create some redundant empty directories. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- Source/C/Split/Split.c | 24 ++-- 1 file changed, 14 insertions(+), 10 deletions(-)

[edk2] [Patch] BaseTools:remove the redundant directories for '-f' with absolute path.

when the absolute path is given to '-f', it would create some redundant empty directories. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- Source/C/Split/Split.c | 25 +++-- 1 file changed, 15 insertions(+), 10 deletions(-)

[edk2] [Patch] BaseTools: Allow decimal values in the EDK II meta-data file.

Because the EDK II meta-data specifications already allow using decimal values in the EDK II Meta-data file [Defines] section, this patch update code to allow this usage. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- Source/Python/Workspace

[edk2] [Patch] BaseTools: Fix the Windows GCC Build Failure with too long path

-off-by: Yonghong Zhu --- BaseTools/Source/Python/AutoGen/GenMake.py | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/BaseTools/Source/Python/AutoGen/GenMake.py b/BaseTools/Source/Python/AutoGen/GenMake.py index c8c5fc5..51c5238 100644 --- a/BaseTools/Source/Python/AutoGen

[edk2] [Patch] BaseTools: Fix the bug to support generate FFS by Align = Auto

FDF spec support to use 'Auto' as , but current Tool report error about -a=Auto is invalid option. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python/GenFds/GenFdsGlobalVariable.py | 2 ++ 1 file changed, 2

[edk2] [Patch] BaseTools: Add the support for character '<' and '>' in the map file

: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python/GenPatchPcdTable/GenPatchPcdTable.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/BaseTools/Source/Python/GenPatchPcdTable/GenPatchPcdTabl

[edk2] [Patch] BaseTools: Fix the bug to support generate FFS by Align=Auto

FDF spec support to use 'Auto' as , but current Tool report error about -a=Auto is invalid option when we set Align=Auto for RAW File. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python/GenFds/FdfParser.py |

[edk2] [Patch] BaseTools: FILE DATA to support relative path under Multiple workspace

Fix the bug that FILE DATA to support relative path under Multiple workspace. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python/GenFds/FdfParser.py | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git

[edk2] [Patch] BaseTools: report error for same Guid's Private definition conflict

Add error check for the same Guid/Protocol/PPIs/Includes defined as both Private and non-Private attribute. Fixes: https://bugzilla.tianocore.org/show_bug.cgi?id=209 Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- .../Source/Python

[edk2] [Patch] BaseTools: Fix bug for decimal value of VPDPCD offset display in report

current if we set VPD PCD's offset to a decimal value, eg: 22, this value is displayed incorrectly in the "FD VPD Region" section in the report.txt. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Sour

[edk2] [Patch] BaseTools: fix the bug to add PaletteSize info into AutoGen

Fix the bug to add PaletteSize info into AutoGen.c when the flag UEFI_HII_RESOURCE_SECTION is set to FALSE. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python/AutoGen/GenC.py | 7 --- 1 file changed, 4 insertions

[edk2] [Patch] BaseTools: Fix the bug to parse the new map file format

Current the variable and Pcd format save in the map file is changed, so this patch is to support this new format. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python/Common/Misc.py | 11

[edk2] [Patch] BaseTools: Support QuotedString for PREBUILD/POSTBUILD in DSC file

If the prebuild/postbuild script statement start with double quotations, current tool report error, while DSC spec allow this usage. so update tool to support it. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python

[edk2] [Patch] BaseTools: add error check for "#image" for idf file format

Add new error check for "#image" keyword used in the image definition file. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python/AutoGen/IdfClassObject.py | 5 - 1 file changed, 4 insertions(+), 1 deletio

[edk2] [Patch] BaseTools: Correct bin wrappers for GenDepex.py path

1.0 Signed-off-by: Yonghong Zhu --- BaseTools/BinWrappers/PosixLike/GenDepex | 2 +- BaseTools/BinWrappers/WindowsLike/GenDepex.bat | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/BaseTools/BinWrappers/PosixLike/GenDepex b/BaseTools/BinWrappers/PosixLike/GenDepex index

[edk2] [Patch] BaseTools: Fix the bug for RAW file alignment value support

: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python/GenFds/GenFdsGlobalVariable.py | 7 +++ 1 file changed, 7 insertions(+) diff --git a/BaseTools/Source/Python/GenFds/GenFdsGlobalVariable.py b/BaseTools/Source/Python/GenFds/GenFdsGlobalVariable.py

[edk2] [Patch] BaseTools: fix the bug for Mixed Pcd display in the report

Fix the bug to not display the mixed PCD in the Global PCD section, and correct the Pcd display name. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python/build/BuildReport.py | 19 ++- 1 file changed, 14

[edk2] [Patch] BaseTools: not report error for the optional items in the FmpTokens

: Yonghong Zhu --- BaseTools/Source/Python/GenFds/CapsuleData.py | 8 ++-- BaseTools/Source/Python/GenFds/FdfParser.py | 13 + 2 files changed, 11 insertions(+), 10 deletions(-) diff --git a/BaseTools/Source/Python/GenFds/CapsuleData.py b/BaseTools/Source/Python/GenFds

[edk2] [Patch] BaseTools: Fix a bug for VOID* type Fixatbuild Pcd in Library

The case is a FixedAtBuild VOID* PCD is used from a lib, but is set to a different sized value in a module INF scope section. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python/AutoGen/GenC.py | 22 +- 1 file

[edk2] [Patch] BaseTools: Fix a bug for Size incorrect of Void* Fixatbuild Pcd

ution Agreement 1.1 Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python/AutoGen/AutoGen.py | 4 ++-- BaseTools/Source/Python/AutoGen/GenC.py| 2 ++ 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/BaseTools/Source/Python/AutoGen/AutoGen.py b/BaseTools/Source/Python/AutoGen/AutoG

[edk2] [Patch] BaseTools: Fix size override issue for Void* Patchable pcd

when multiple driver link same library, and the drivers override the pcd to different value in the DSC component section, it cause the pcd size incorrect. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python/AutoGen/GenC.py | 20

[edk2] [Patch] BaseTools: Fix the bug for VOID* pcd max size from component section

When the Pcd defined in components section, its value's size is larger than the value's size in [pcd] section, it cause build error, because original code use the size get in [pcd] section as max size. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yo

[edk2] [Patch] BaseTools: Fix two cases that use GUID CName as PCD Value

{GUID(TestGuid)} Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python/AutoGen/AutoGen.py| 9 +++-- BaseTools/Source/Python/Workspace/InfBuildData.py | 1 + 2 files changed, 4 insertions(+), 6 deletions(-) diff --git a/BaseTools/S

[edk2] [Patch] BaseTools: compare GUID value should not case-sensitive

From: Bin Wang build report error when the same Guid value in FDF file use lowercase, in tools_def.txt file use uppercase. The guid value's compare should not case-sensitive. Cc: Liming Gao Cc: Yonghong Zhu Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Bin

[edk2] [Patch] BaseTools: Pcd not used info should not in Module PCD section

Pcds in Conditional Directives and Pcds not used are Platform Level info, it should not display in Module PCD Section. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python/build/BuildReport.py | 11 ++- 1 file changed, 6

[edk2] [Patch] BaseTools: Pcds in [Components] are not display correct in the report

The Pcd used in [Components] section, the PCD value is displayed incorrect in the build report because the PCD default value was not override. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python/build/BuildReport.py | 1 + 1 file

[edk2] [Patch] Build spec: Add some clarification and clean up for build report

Add some clarification and clean up for build report section. Cc: Liming Gao Cc: Michael Kinney Cc: Kevin W Shaw Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yonghong Zhu --- 13_build_reports/134_platform_summary.md | 44

[edk2] [Patch] Build Spec: update chapter 2.7's format

This patch update chapter 2.7's format since we found the PDF of this section is ugly. Cc: Liming Gao Cc: Michael Kinney Cc: Kevin W Shaw Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yonghong Zhu --- 2_design_discussion/27_sku_support.md

[edk2] [Patch] Build Spec: Update Version to 1.28

Cc: Liming Gao Cc: Michael Kinney Cc: Kevin W Shaw Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yonghong Zhu --- README.md | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/README.md b/README.md index d9d2146..9fb7e93 100644 --- a/README.md

[edk2] [Patch] DEC spec: Update version to 1.27

Cc: Liming Gao Cc: Michael Kinney Cc: Kevin W Shaw Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yonghong Zhu --- 2_dec_file_overview/24_[defines]_usage.md| 4 ++-- 3_edk_ii_dec_file_format/34_[defines]_section.md | 16 README.md

[edk2] [Patch] DSC spec: Update version to 1.28

Cc: Liming Gao Cc: Michael Kinney Cc: Kevin W Shaw Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yonghong Zhu --- 2_dsc_overview/23_[defines]_section_processing.md | 6 +++--- 2_dsc_overview/README.md | 4 ++-- 3_edk_ii_dsc_file_format/35_

[edk2] [Patch] Expression spec: update version to 1.3

Cc: Liming Gao Cc: Michael Kinney Cc: Kevin W Shaw Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yonghong Zhu --- README.md | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/README.md b/README.md index 60086a9..ae725ca 100644 --- a/README.md

[edk2] [Patch] FDF spec: Update version to 1.28

Cc: Liming Gao Cc: Michael Kinney Cc: Kevin W Shaw Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yonghong Zhu --- 2_fdf_design_discussion/README.md| 4 ++-- 3_edk_ii_fdf_file_format/34_[defines]_section.md | 16 README.md

[edk2] [Patch] INF spec: Update version to 1.27

Cc: Liming Gao Cc: Michael Kinney Cc: Kevin W Shaw Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yonghong Zhu --- 2_inf_overview/24_[defines]_section.md | 8 3_edk_ii_inf_file_format/34_[defines]_section.md | 22 +++--- README.md

[edk2] [Patch] edk2-VfrSpecification: update version to 1.92

Cc: Liming Gao Cc: Michael Kinney Cc: Kevin W Shaw Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yonghong Zhu --- README.md | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/README.md b/README.md index 0a596eb..3c40cab 100644 --- a/README.md

[edk2] [PATCH] BaseTools: Parse PCD GUID name in FILE statement issue

modules. Cc: Liming Gao Cc: Yonghong Zhu Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yunhua Feng --- BaseTools/Source/Python/Workspace/WorkspaceDatabase.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/BaseTools/Source/Python/Workspace

  1   2   3   4   5   6   >