[edk2-devel] [PATCH] BaseTools: Add FMMT Return Code for Error checking

2024-04-11 Thread Yuwei Chen
Cc: Rebecca Cran Cc: Liming Gao Cc: Bob Feng Signed-off-by: Yuwei Chen --- BaseTools/Source/Python/FMMT/FMMT.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/BaseTools/Source/Python/FMMT/FMMT.py b/BaseTools/Source/Python/FMMT/FMMT.py index 26fc4c5792..d9f1c265a7 100644

Re: [edk2-devel] [Patch V2] [edk2-staging] BaseTools: Update ReadMe in Python VfrCompiler Tool

2024-03-27 Thread Yuwei Chen
Reviewed-by: Yuwei Chen > -Original Message- > From: Yang, Yuting2 > Sent: Wednesday, March 27, 2024 4:43 PM > To: devel@edk2.groups.io > Cc: Rebecca Cran ; Liming Gao > ; Feng, Bob C ; Chen, > Christine > Subject: [Patch V2] [edk2-staging] BaseTools:

Re: [edk2-devel] [PATCH] BaseTools: FMMT Skip empty Lines while parsing FMMTConfig.ini

2024-03-11 Thread Yuwei Chen
Ali, please also create PR in BaseTools repo. Reviewed-by: Yuwei Chen > -Original Message- > From: S, Ashraf Ali > Sent: Wednesday, February 28, 2024 5:57 PM > To: devel@edk2.groups.io > Cc: S, Ashraf Ali ; Rebecca Cran > ; Liming Gao ; Feng, Bob > C ; Chen, C

Re: [edk2-devel] Fixing edk2-basetools CI

2024-02-22 Thread Yuwei Chen
Thanks a lot Rebecca~~ Thanks, Christine > -Original Message- > From: devel@edk2.groups.io On Behalf Of Michael > D Kinney > Sent: Saturday, February 17, 2024 1:24 AM > To: Rebecca Cran ; Joey Vagedes > ; Rebecca Cran > ; devel@edk2.groups.io; Sean > ; Michael Kubacki > > Cc: Kinney,

Re: [edk2-devel] [edk2-stable202311][PATCH] BaseTools: Python VfrCompiler implementation

2024-01-09 Thread Yuwei Chen
> > C ; Yang, Yuting2 ; > > Hartung, Stephen > > Subject: Re: [edk2-devel] [edk2-stable202311][PATCH] BaseTools: Python > > VfrCompiler implementation > > > > On Thu, Dec 7, 2023 at 9:08 AM Yuwei Chen > wrote: > > > > > > Hi Liming,

Re: [edk2-devel] [PATCH v1 1/1] BaseTools: Fix raw strings containing valid escape characters

2024-01-09 Thread Yuwei Chen
ssion issue? Or does it resolve new usage? I am curious why it is not reported before. Thanks Liming > -邮件原件- > 发件人: devel@edk2.groups.io<mailto:devel@edk2.groups.io> > mailto:devel@edk2.groups.io>> 代表 Yuwei Chen > 发送时间: 2024年1月8日 13:37 > 收件人: devel@edk2.

Re: [edk2-devel] [PATCH v1 1/1] BaseTools: Fix raw strings containing valid escape characters

2024-01-07 Thread Yuwei Chen
Hi Liming and Cran, We should merge this fixing patch asap as it influences BaseTool's usage. Thanks, Christine > -Original Message- > From: devel@edk2.groups.io On Behalf Of Yuwei > Chen > Sent: Monday, January 8, 2024 9:16 AM > To: Joey Vagedes ; devel@edk2.groups.i

Re: [edk2-devel] [PATCH v1 1/1] BaseTools: Fix raw strings containing valid escape characters

2024-01-07 Thread Yuwei Chen
Hi Joey, Thanks for fixing. Reviewed-by: Yuwei Chen > -Original Message- > From: Joey Vagedes > Sent: Friday, December 29, 2023 12:48 AM > To: devel@edk2.groups.io > Cc: Rebecca Cran ; Gao, Liming > ; Feng, Bob C ; Chen, > Christine > Subject: [PATCH v1

Re: [edk2-devel] [PATCH] BaseTools: Remove Duplicate sets of SkuName and SkuId from allskuset

2024-01-07 Thread Yuwei Chen
Reviewed-by: Yuwei Chen > -Original Message- > From: S, Ashraf Ali > Sent: Monday, December 25, 2023 11:28 PM > To: devel@edk2.groups.io > Cc: S, Ashraf Ali ; Chen, Christine > ; Rebecca Cran ; Gao, Liming > ; Feng, Bob C ; Chan, > Amy ; Chaganty, Rangasa

Re: [edk2-devel] [PATCH V2 1/1] Silicon/Intel/FitGen:FIT change for FBM entry.

2023-12-17 Thread Yuwei Chen
Reviewed-by: Yuwei Chen > -Original Message- > From: Liu, Liqi > Sent: Tuesday, December 12, 2023 4:42 PM > To: devel@edk2.groups.io > Cc: Liu, Liqi ; Chen, Christine ; > Feng, Bob C ; Chaganty, Rangasai V > ; Gao, Liming > Subject: [PATCH V2 1/1] Silicon

Re: [edk2-devel] [PATCH] BaseTools: FMMT GuidTool Auto Select Config file Enabling

2023-12-15 Thread Yuwei Chen
Edk2-basetools repo PR link: https://github.com/tianocore/edk2-basetools/pull/100 Thanks, Christine > -Original Message- > From: devel@edk2.groups.io On Behalf Of Yuwei > Chen > Sent: Friday, December 15, 2023 5:12 PM > To: devel@edk2.groups.io > Cc: Rebecca Cran ; Ga

[edk2-devel] [PATCH] BaseTools: FMMT GuidTool Auto Select Config file Enabling

2023-12-15 Thread Yuwei Chen
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4624 Currently, Python FMMT tool does not support automatically select FMMTConf.ini file which saves GuidTool settings. This patch supports this features. Cc: Rebecca Cran Cc: Liming Gao Cc: Bob Feng Signed-off-by: Yuwei Chen

Re: [edk2-devel] [Patch V2] BaseTools: VfrCompiler Adds DefaultValueError Feature

2023-12-12 Thread Yuwei Chen
+Cc Yang, Yuting2 > -Original Message- > From: Yang, Yuting2 > Sent: Tuesday, December 12, 2023 5:01 PM > To: devel@edk2.groups.io > Cc: Rebecca Cran ; Gao, Liming > ; Feng, Bob C ; Chen, > Christine > Subject: [Patch V2] BaseTools: VfrCompiler Adds DefaultValueError Feature > > Add

Re: [edk2-devel] [Patch V2 1/4] BaseTools: fixing FMMT ShrinkFv issue

2023-12-11 Thread Yuwei Chen
gt; Feng, Bob C > > > Cc: 'Rebecca Cran' ; Guo, Gua > > > > Subject: 回复: [edk2-devel] [Patch V2 1/4] BaseTools: fixing FMMT > > ShrinkFv > > > issue > > > > > > Yuwei: > > > Please update commit message with reviewed-by info,

[edk2-devel] [PATCH] [edk2-staging]BaseTools: Python VfrCompiler Tool Update

2023-12-07 Thread Yuwei Chen
-=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#112216): https://edk2.groups.io/g/devel/message/112216 Mute This Topic: https://groups.io/mt/103051162/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe:

Re: [edk2-devel] [edk2-stable202311][PATCH] BaseTools: Python VfrCompiler implementation

2023-12-07 Thread Yuwei Chen
Hi Liming, Is this feature been tested and reviewed these two weeks?  Thanks, Christine From: devel@edk2.groups.io On Behalf Of Yuwei Chen Sent: Monday, November 13, 2023 8:32 AM To: Gao, Liming ; 'Rebecca Cran' ; Gao, Liming ; Zimmer, Vincent ; Kinney, Michael D ; 'Leif Lindholm' ; 'Andrew

Re: [edk2-devel] [Patch V2 1/4] BaseTools: fixing FMMT ShrinkFv issue

2023-12-03 Thread Yuwei Chen
d push > label for this patch set. > > Thanks > Liming > > -----邮件原件- > > 发件人: devel@edk2.groups.io 代表 Yuwei Chen > > 发送时间: 2023年11月29日 16:26 > > 收件人: Gao, Liming ; Feng, Bob C > > > > 抄送: 'Rebecca Cran' ; devel@edk2.groups.io; Chen, > > Christ

Re: [edk2-devel] [Patch V2 1/4] BaseTools: fixing FMMT ShrinkFv issue

2023-11-29 Thread Yuwei Chen
Hi Liming and Cran, I have created the Edk2 PR for this one: https://github.com/tianocore/edk2/pull/5009 As it has been reviewed, please help merge that. Thanks~ Thanks, Yuwei (Christine) > -Original Message- > From: devel@edk2.groups.io On Behalf Of Yuwei > Chen >

Re: [edk2-devel] [PATCH v1 1/1] BaseTools/Conf/target.template: Use VS2019 as default tool chain

2023-11-28 Thread Yuwei Chen
Reviewed-by: Yuwei Chen > -Original Message- > From: mikub...@linux.microsoft.com > Sent: Wednesday, November 29, 2023 3:49 AM > To: devel@edk2.groups.io > Cc: Feng, Bob C ; Gao, Liming > ; Rebecca Cran ; Chen, > Christine > Subject: [PATCH v1 1/1] BaseTools/C

Re: [edk2-devel] [PATCH 1/1] BaseTools/Scripts/PatchCheck.py: Check for Change-id

2023-11-28 Thread Yuwei Chen
i > Mujawar ; yeoreum@arm.com > Subject: RE: [edk2-devel] [PATCH 1/1] BaseTools/Scripts/PatchCheck.py: Check > for Change-id > > It's good. But I am curious why --ignore-change-id is needed? > > Thanks, > Ray > > -Original Message- > > From: d

Re: [edk2-devel] [PATCH 1/1] BaseTools/Scripts/PatchCheck.py: Check for Change-id

2023-11-28 Thread Yuwei Chen
The patch is good for me. Reviewed-by: Yuwei Chen > -Original Message- > From: Pierre Gondois > Sent: Wednesday, November 22, 2023 9:15 PM > To: devel@edk2.groups.io > Cc: Rebecca Cran ; Gao, Liming > ; Feng, Bob C ; Chen, > Christine ; Sami Mujawar > ; yeore

Re: [edk2-devel] [edk2-stable202311][PATCH] BaseTools: Python VfrCompiler implementation

2023-11-12 Thread Yuwei Chen
Hi Liming, I know your point. Would you like to start reviewing it and give us more feedbacks? We really hope it can be merged as soon as possible. We can do the merging after the stable tag, but just like you said, maybe we need to review it firstly. Looking forward to your feedbacks, and I

Re: [edk2-devel] [PATCH v3 2/7] BaseTools/Plugin/CodeQL: Add CodeQL build plugin

2023-10-24 Thread Yuwei Chen
Reviewed-by: Yuwei Chen > -Original Message- > From: mikub...@linux.microsoft.com > Sent: Wednesday, October 18, 2023 9:05 AM > To: devel@edk2.groups.io > Cc: Feng, Bob C ; Gao, Liming > ; Kinney, Michael D > ; Rebecca Cran ; Sean > Brogan ; Chen, Christine >

[edk2-devel] [PATCH 1/1] BaseTools: trim warning to error

2023-09-22 Thread Yuwei Chen
As the error is changed to warning, Trim.py will skip the build error when the source code have exactly issue. This patch change warning to error to opens the checking. Cc: Rebecca Cran Cc: Liming Gao Cc: Bob Feng Signed-off-by: Yuwei Chen --- BaseTools/Source/Python/Trim/Trim.py | 4 ++-- 1

Re: [edk2-devel] [PATCH] Platform/Intel: Enhance the BOARD_ID without PCH_TYPE support

2023-09-15 Thread Yuwei Chen
Reviewed-by: Yuwei Chen > -Original Message- > From: Srinivasan, ManickamX > Sent: Wednesday, September 13, 2023 4:56 PM > To: devel@edk2.groups.io > Cc: Srinivasan, ManickamX ; Feng, Bob C > ; Gao, Liming ; Chen, > Christine ; T V, Krishnamoorthy > > Subj

Re: [edk2-devel] [Patch V2 1/4] BaseTools: fixing FMMT ShrinkFv issue

2023-09-10 Thread Yuwei Chen
n, Christine > > Sent: Thursday, June 29, 2023 11:34 AM > > To: devel@edk2.groups.io > > Cc: Rebecca Cran ; Gao, Liming > > ; Feng, Bob C > > Subject: [Patch V2 1/4] BaseTools: fixing FMMT ShrinkFv issue > > > > 1. FvLength not change issue; > > 2. FileS

Re: [edk2-devel] [PATCH] Platform/Intel: Enhance the BOARD_ID without PCH_TYPE support

2023-08-29 Thread Yuwei Chen
Reviewed-by: Yuwei Chen > -Original Message- > From: Srinivasan, ManickamX > Sent: Wednesday, August 23, 2023 3:29 PM > To: devel@edk2.groups.io > Cc: Srinivasan, ManickamX ; Feng, Bob C > ; Gao, Liming ; Chen, > Christine ; T V, Krishnamoorthy > > Subj

[edk2-devel] [PATCH] BaseTools: Add FMMT BinPipWrappers

2023-07-26 Thread Yuwei Chen
Cc: Rebecca Cran Cc: Liming Gao Cc: Bob Feng Signed-off-by: Yuwei Chen --- BaseTools/BinPipWrappers/PosixLike/FMMT | 12 BaseTools/BinPipWrappers/WindowsLike/FMMT.bat | 3 +++ 2 files changed, 15 insertions(+) create mode 100755 BaseTools/BinPipWrappers/PosixLike/FMMT

Re: [edk2-devel] [Patch V3] [edk2-staging] BaseTools/FMMT: C FMMT match new version machine defines

2023-07-09 Thread Yuwei Chen
-devel] [Patch V3] [edk2-staging] BaseTools/FMMT: C > FMMT match new version machine defines > > Christine: > I just push this patch @ bab1724cdf725b7d22869c79fa2b86ed47e01110 in > edk2-staging. > > Thanks > Liming > > -邮件原件- > > 发件人: devel@edk2.gro

Re: [edk2-devel] [Patch V3] [edk2-staging] BaseTools/FMMT: C FMMT match new version machine defines

2023-07-04 Thread Yuwei Chen
> > Subject: 回复: [Patch V3] [edk2-staging] BaseTools/FMMT: C FMMT match > new version machine defines > > Reviewed-by: Liming Gao > > > -邮件原件- > > 发件人: Yuwei Chen > > 发送时间: 2023年6月27日 10:54 > > 收件人: devel@edk2.groups.io > > 抄送: Rebecca Cra

[edk2-devel] [Patch V2 4/4] BaseTools: FMMT replace new free space fixing in replace

2023-06-28 Thread Yuwei Chen
In FMMT replace function, when newffs size <= targetffs size, the new free space is calculated wrong as loss the pad data delta size. That will cause invalid binary generated. This patch fixes this issue. Cc: Rebecca Cran Cc: Liming Gao Cc: Bob Feng Signed-off-by: Yuwei Chen --- BaseTo

[edk2-devel] [Patch V2 3/4] BaseTools: FMMT support ELF UPLD parser

2023-06-28 Thread Yuwei Chen
''' Cc: Rebecca Cran Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- BaseTools/Source/Python/FMMT/FMMT.py | 2 ++ BaseTools/Source/Python/FMMT/core/BinaryFactoryProduct.py | 36 +++- BaseTools/Source/Python/FMMT/core/BiosTree.py

[edk2-devel] [Patch V2 2/4] BaseTools: FMMT replace output file is not generated successfully

2023-06-28 Thread Yuwei Chen
For replace function, when target Ffs and new ffs are with same size, the output file can not be generated successfully. This patch fixes this issue. Cc: Rebecca Cran Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- BaseTools/Source/Python/FMMT/core/BiosTree.py | 4 ++-- BaseTools

[edk2-devel] [Patch V2 1/4] BaseTools: fixing FMMT ShrinkFv issue

2023-06-28 Thread Yuwei Chen
1. FvLength not change issue; 2. FileSystemGuid align with File Size; Cc: Rebecca Cran Cc: Liming Gao Cc: Bob Feng Signed-off-by: Yuwei Chen --- BaseTools/Source/Python/FMMT/core/FMMTOperation.py | 2 +- BaseTools/Source/Python/FMMT/core/FvHandler.py | 10 +++--- 2 files changed, 8

[edk2-devel] [Patch V3] [edk2-staging] BaseTools/FMMT: C FMMT match new version machine defines

2023-06-26 Thread Yuwei Chen
BaseTools have switch from EFI_IMAGE_MACHINE_* to IMAGE_FILE_MACHINE_*. Thus, match the FCE & FMMT tool in edk2-staging repo with the new version define. Cc: Rebecca Cran Cc: Liming Gao Cc: Bob Feng Signed-off-by: Yuwei Chen --- V3 modify the patch with the correct define BaseTools/Sour

[edk2-devel] [Patch V2] [edk2-staging] BaseTools/FMMT: C FMMT match new version machine defines

2023-06-21 Thread Yuwei Chen
BaseTools have switch from EFI_IMAGE_MACHINE_* to IMAGE_FILE_MACHINE_*. Thus, match the FCE & FMMT tool in edk2-staging repo with the new version define. Cc: Rebecca Cran Cc: Liming Gao Cc: Bob Feng Signed-off-by: Yuwei Chen --- BaseTools/Source/C/FMMT/Rebase.c | 12 ++-- 1

Re: [edk2-devel] [PATCH 1/1] BaseTools: FMMT GuidTool Auto Select Config file

2023-06-21 Thread Yuwei Chen
ming ; Feng, Bob C > > Subject: Re: [edk2-devel] [PATCH 1/1] BaseTools: FMMT GuidTool Auto Select > Config file > > That won't work on Unix, where the separator is ':', not ';'. > > > -- > > Rebecca Cran > > > On 6/5/23 02:12, Yuwei Chen wrote: > > Cu

[edk2-devel] [PATCH] [edk2-staging]BaseTools/FMMT: C FMMT match new version machine defines

2023-06-15 Thread Yuwei Chen
BaseTools have switch from EFI_IMAGE_MACHINE_* to IMAGE_FILE_MACHINE_*. Thus, match the FCE & FMMT tool in edk2-staging repo with the new version define. Cc: Rebecca Cran Cc: Liming Gao Cc: Bob Feng Signed-off-by: Yuwei Chen --- BaseTools/Source/C/FMMT/Rebase.c | 12 ++-- 1

[edk2-devel] [PATCH 1/1] BaseTools: FMMT GuidTool Auto Select Config file

2023-06-05 Thread Yuwei Chen
Currently, Python FMMT tool does not support automatically select FMMTConf.ini file which saves GuidTool settings. This patch support this features. Cc: Rebecca Cran Cc: Liming Gao Cc: Bob Feng Signed-off-by: Yuwei Chen --- BaseTools/Source/Python/FMMT/core/GuidTools.py | 2 +- 1 file

Re: [edk2-devel] [Patch V1 1/3] BaseTools: fixing FMMT ShrinkFv issue

2023-04-18 Thread Yuwei Chen
For edk2-basetools repo, have made a PR for these three patches: https://github.com/tianocore/edk2-basetools/pull/95 Thanks, Christine (Yuwei) > -Original Message- > From: devel@edk2.groups.io On Behalf Of Yuwei > Chen > Sent: Tuesday, April 18, 2023 5:53 PM > To: devel

[edk2-devel] [Patch V1 3/3] BaseTools: FMMT support ELF UPLD parser

2023-04-18 Thread Yuwei Chen
''' Cc: Rebecca Cran Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- BaseTools/Source/Python/FMMT/FMMT.py | 2 ++ BaseTools/Source/Python/FMMT/core/BinaryFactoryProduct.py | 36 +++- BaseTools/Source/Python/FMMT/core/BiosTree.py

[edk2-devel] [Patch V1 3/3] BaseTools: FMMT support ELF UPLD parser

2023-04-18 Thread Yuwei Chen
-=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#103156): https://edk2.groups.io/g/devel/message/103156 Mute This Topic: https://groups.io/mt/98340352/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe:

[edk2-devel] [Patch V1 2/3] BaseTools: FMMT replace output file is not generated successfully

2023-04-18 Thread Yuwei Chen
For replace function, when target Ffs and new ffs are with same size, the output file can not be generated successfully. This patch fixes this issue. Cc: Rebecca Cran Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- BaseTools/Source/Python/FMMT/core/BiosTree.py | 4 ++-- BaseTools

[edk2-devel] [Patch V1 1/3] BaseTools: fixing FMMT ShrinkFv issue

2023-04-18 Thread Yuwei Chen
1. FvLength not change issue; 2. FileSystemGuid align with File Size; Cc: Rebecca Cran Cc: Liming Gao Cc: Bob Feng Signed-off-by: Yuwei Chen --- BaseTools/Source/Python/FMMT/core/FMMTOperation.py | 2 +- BaseTools/Source/Python/FMMT/core/FvHandler.py | 10 +++--- 2 files changed, 8

[edk2-devel] [PATCH] Maintainers: Add 'Yuwei Chen' as BaseTools maintainer.

2023-03-08 Thread Yuwei Chen
Add myself as maintainer for the BaseTools Pkg. Cc: Michael D Kinney Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- Maintainers.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Maintainers.txt b/Maintainers.txt index d4607a..463d9567e1 100644

Re: [edk2-devel][PATCH V1 1/1] BaseTools: Generate compile information in build report

2023-02-14 Thread Yuwei Chen
een reviewed for edk2-basetools repo? > > Mike > > > -Original Message- > > From: devel@edk2.groups.io On Behalf Of Yuwei > > Chen > > Sent: Tuesday, February 14, 2023 6:44 PM > > To: Palomino Sosa, Guillermo A ; > > devel@edk2.groups.io > > C

Re: [edk2-devel][PATCH V1 1/1] BaseTools: Generate compile information in build report

2023-02-14 Thread Yuwei Chen
Reviewed-by: Yuwei Chen > -Original Message- > From: Palomino Sosa, Guillermo A > Sent: Tuesday, February 7, 2023 11:07 AM > To: devel@edk2.groups.io > Cc: Feng, Bob C ; Gao, Liming > ; Chen, Christine > Subject: [edk2-devel][PATCH V1 1/1] BaseTools: Generate

[edk2-devel] [Patch V1 2/2] BaseTools: FMMT support ELF UPLD parser

2023-01-30 Thread Yuwei Chen
''' Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- edk2basetools/FMMT/FMMT.py | 2 ++ edk2basetools/FMMT/core/BinaryFactoryProduct.py | 36 +++- edk2basetools/FMMT/core/BiosTree.py | 49

[edk2-devel] [Patch V1 1/2] BaseTools: Fix FMMT FvHandler replace status bug

2023-01-30 Thread Yuwei Chen
For replace function, when target Ffs and new ffs are with same size, the output file can not be generated successfully. This patch fixes this issue. Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- edk2basetools/FMMT/core/BiosTree.py | 4 ++-- edk2basetools/FMMT/core/FvHandler.py

[edk2-devel] [Patch V2 3/3] [edk2-staging]BaseTools: Add new build option for Variable default value generation

2023-01-29 Thread Yuwei Chen
Add new build option "--gen-default-variable-bin" for Variable default value generation from Python VFR tool's extended json output file. Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- BaseTools/Source/Python/AutoGen/DataPipe.py | 2 ++ BaseTools/Source/Pyth

[edk2-devel] [Patch V2 2/3] [edk2-staging]BaseTools: Add new build option for PyVfrCompiler yaml generation

2023-01-29 Thread Yuwei Chen
Add new build option "--vfr-yaml-enable" for Python VfrCompiler extended output yaml file generation. Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- BaseTools/Source/Python/AutoGen/DataPipe.py | 2 ++ BaseTools/Source/Python/AutoGen/ModuleAutoGen.py | 8

[edk2-devel] [Patch V1 3/3] BaseTools: Add new build option for Variable default value generation

2023-01-29 Thread Yuwei Chen
Add new build option "--gen-default-variable-bin" for Variable default value generation from Python VFR tool's extended json output file. Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- edk2basetools/AutoGen/DataPipe.py | 2 ++ edk2basetools/AutoGen/GenDef

[edk2-devel] [Patch V1 2/3] BaseTools: Add new build option for PyVfrCompiler yaml generation

2023-01-29 Thread Yuwei Chen
Add new build option "--vfr-yaml-enable" for Python VfrCompiler extended output yaml file generation. Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- edk2basetools/AutoGen/DataPipe.py | 2 ++ edk2basetools/AutoGen/ModuleAutoGen.py | 8 edk2baseto

[edk2-devel] [Patch V1 1/3] BaseTools: Add Python VfrCompiler tool

2023-01-29 Thread Yuwei Chen
-=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#99275): https://edk2.groups.io/g/devel/message/99275 Mute This Topic: https://groups.io/mt/96622973/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe:

[edk2-devel] [Patch V1 3/3] BaseTools: Add new build option for Variable default value generation

2022-11-15 Thread Yuwei Chen
Add new build option "--gen-default-variable-bin" for Variable default value generation from Python VFR tool's extended json output file. Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- edk2basetools/AutoGen/DataPipe.py | 2 ++ edk2basetools/AutoGen/GenDef

[edk2-devel] [Patch V1 2/3] BaseTools: Add new build option for PyVfrCompiler yaml generation

2022-11-15 Thread Yuwei Chen
Add new build option "--vfr-yaml-enable" for Python VfrCompiler extended output yaml file generation. Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- edk2basetools/AutoGen/DataPipe.py | 2 ++ edk2basetools/AutoGen/ModuleAutoGen.py | 8 edk2baseto

[edk2-devel] [Patch V2 3/3] [edk2-staging]BaseTools: Add new build option for Variable default value generation

2022-11-15 Thread Yuwei Chen
Add new build option "--gen-default-variable-bin" for Variable default value generation from Python VFR tool's extended json output file. Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- BaseTools/Source/Python/AutoGen/DataPipe.py | 2 ++ BaseTools/Source/Pyth

[edk2-devel] [Patch V2 2/3] [edk2-staging]BaseTools: Add new build option for PyVfrCompiler yaml generation

2022-11-15 Thread Yuwei Chen
Add new build option "--vfr-yaml-enable" for Python VfrCompiler extended output yaml file generation. Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- BaseTools/Source/Python/AutoGen/DataPipe.py | 2 ++ BaseTools/Source/Python/AutoGen/ModuleAutoGen.py | 8

[edk2-devel] [Patch V1 3/3] [edk2-staging]BaseTools: Add new build option for Variable default value generation

2022-11-14 Thread Yuwei Chen
Add new build option "--gen-default-variable-bin" for Variable default value generation from Python VFR tool's extended json output file. Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- BaseTools/Source/Python/AutoGen/DataPipe.py | 2 ++ BaseTools/Source/Pyth

[edk2-devel] [Patch V1 2/3] [edk2-staging]BaseTools: Add new build option for PyVfrCompiler yaml generation

2022-11-14 Thread Yuwei Chen
Add new build option "--vfr-yaml-enable" for Python VfrCompiler extended output yaml file generation. Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- BaseTools/Source/Python/AutoGen/DataPipe.py | 2 ++ BaseTools/Source/Python/AutoGen/ModuleAutoGen.py | 8

Re: [edk2-devel] [PATCH] BaseTools: Fix build option overrides Pcd Feature Flag issue

2022-10-28 Thread Yuwei Chen
Cc Mike. Thanks, Christine > -Original Message- > From: devel@edk2.groups.io On Behalf Of Yuwei > Chen > Sent: Tuesday, October 25, 2022 4:51 PM > To: devel@edk2.groups.io > Cc: Feng, Bob C ; Gao, Liming > > Subject: [edk2-devel] [PATCH] BaseTools: Fix bu

[edk2-devel] [PATCH] BaseTools: Fix build option overrides Pcd Feature Flag issue

2022-10-25 Thread Yuwei Chen
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4081 INF [Sources] section Feature Flag Expressions do not use override values from build --pcd option currently. This patch fix this issue. Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- edk2basetools/Workspace/InfBuildData.py

Re: [edk2-devel] [PATCH 1/1] pip-requirement: Upgrade the edk2-basetools version from 0.1.29 to 0.1.39

2022-10-18 Thread Yuwei Chen
Reviewed-by: Yuwei Chen > -Original Message- > From: Feng, Bob C > Sent: Sunday, October 16, 2022 8:11 PM > To: devel@edk2.groups.io > Cc: Feng, Bob C ; Gao, Liming > ; Chen, Christine > Subject: [PATCH 1/1] pip-requirement: Upgrade the edk2-basetools version &g

[edk2-devel] [PATCH] BaseTools: FMMT replace output file is not generated successfully

2022-10-18 Thread Yuwei Chen
For replace function, when target Ffs and new ffs are with same size, the output file can not be generated successfully. This patch fixes this issue. Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- edk2basetools/FMMT/core/BiosTree.py | 4 ++-- edk2basetools/FMMT/core/FvHandler.py

[edk2-devel] [PATCH 2/2] BaseTools/FMMT: Add Shrink Fv function

2022-09-15 Thread Yuwei Chen
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3938 This function is used to remove the useless FV free space. Usage: FMMT -s Inputfile Outputfile Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- BaseTools/Source/Python/FMMT/FMMT.py | 7 + .../Source/Python/FMMT

[edk2-devel] [PATCH 1/2] BaseTools/FMMT: Add Extract FV function

2022-09-15 Thread Yuwei Chen
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3938 With this patch "-e" parameter supports extract FV function. Usage: FMMT -e Inputfile TargetFv Outputfile Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- BaseTools/Source/Python/FMMT/FMMT.py | 3 +-

Re: [edk2-devel] [PATCH 1/1] pip-requirement: Upgrade the edk2-basetools version from 0.1.24 to 0.1.28

2022-09-05 Thread Yuwei Chen
Reviewed-by: Yuwei Chen > -Original Message- > From: Feng, Bob C > Sent: Friday, September 2, 2022 2:18 PM > To: devel@edk2.groups.io > Cc: Feng, Bob C ; Gao, Liming > ; Chen, Christine ; > Kinney, Michael D > Subject: [PATCH 1/1] pip-requirement: Upgrade th

Re: [edk2-devel] [PATCH] BaseTools/VolInfo: Correct buffer for GenCrc32 tool

2022-07-28 Thread Yuwei Chen
Hi Konstantin, this is really an issue need to be solved. When GenSec tool uses the guidtools to encode the section, it differentiates how GenCrc32 is used from other tools.. GenSec defines a CRC32_SECTION_HEADER structure for GenCrc32, and calculate the DataOffset of the Section with this

Re: [edk2-devel] [PATCH] BaseTools/VolInfo: Increase define for highest section value

2022-07-22 Thread Yuwei Chen
Reviewed-by: Yuwei Chen > -Original Message- > From: Konstantin Aladyshev > Sent: Monday, July 18, 2022 8:56 PM > To: devel@edk2.groups.io > Cc: Feng, Bob C ; Gao, Liming > ; Chen, Christine ; > Konstantin Aladyshev > Subject: [PATCH] BaseTools/VolInfo: Inc

Re: [edk2-devel] [PATCH] BaseTools/VolInfo: Correct alignment attributes display

2022-07-22 Thread Yuwei Chen
Reviewed-by: Yuwei Chen > -Original Message- > From: devel@edk2.groups.io On Behalf Of > Konstantin Aladyshev > Sent: Wednesday, July 13, 2022 9:11 PM > To: devel@edk2.groups.io > Cc: Feng, Bob C ; Gao, Liming > ; Chen, Christine ; > Konstantin Aladyshev > S

Re: [edk2-devel] [PATCH 1/1] BaseTools: INF should use latest Pcd value instead of default value

2022-07-21 Thread Yuwei Chen
Hi Yi, Could you describe the bug with an example? In which scenario the dsc defined pcd will not override the dec definition? Thanks, Christine > -Original Message- > From: devel@edk2.groups.io On Behalf Of yi1 li > Sent: Monday, July 11, 2022 1:48 PM > To: devel@edk2.groups.io > Cc:

[edk2-devel] [Patch V3 1/1] BaseTools: remove directly calling for gcc command

2022-07-21 Thread Yuwei Chen
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4003 In some environment, the directly calling for gcc is not supported. This patch fixes the issue. Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- BaseTools/Source/C/DevicePath/GNUmakefile | 13 - 1 file changed, 8

[edk2-devel] [Patch V2 1/1] BaseTools: remove directly calling for gcc command

2022-07-19 Thread Yuwei Chen
In some environment, the directly calling for gcc is not supported. This patch fixes the issue. Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- BaseTools/Source/C/DevicePath/GNUmakefile | 13 - 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/BaseTools

[edk2-devel] [PATCH 1/1] BaseTools: add '-p' for Linux 'cp' command.

2022-07-08 Thread Yuwei Chen
this issue. Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- BaseTools/Source/Python/AutoGen/GenMake.py| 2 +- BaseTools/Source/Python/Workspace/DscBuildData.py | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/BaseTools/Source/Python/AutoGen/GenMake.py b

[edk2-devel] [PATCH 1/1] BaseTools: remove directly calling for gcc command

2022-07-07 Thread Yuwei Chen
In some environment, the directly calling for gcc is not supported. Thus replace gcc to $(CC). Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- BaseTools/Source/C/DevicePath/GNUmakefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/BaseTools/Source/C/DevicePath

Re: [edk2-devel] [PATCH v3 3/3] [edk2-platforms] Silicon/Intel/FitGen: Support Startup ACM entries (Type 2) 0x200 Version

2022-07-03 Thread Yuwei Chen
Reviewed-by: Yuwei Chen > -Original Message- > From: Lin, Jason1 > Sent: Friday, July 1, 2022 11:10 PM > To: devel@edk2.groups.io > Cc: Lin, Jason1 ; Feng, Bob C ; > Gao, Liming ; Chen, Christine > ; Oram, Isaac W ; > Chaganty, Rangasai V ; Chiang, Dakota >

Re: [edk2-devel] [PATCH v3 2/3] [edk2-platforms] Silicon/Intel/FitGen: Reduce the typecasting and pointer usage

2022-07-03 Thread Yuwei Chen
Reviewed-by: Yuwei Chen > -Original Message- > From: Lin, Jason1 > Sent: Friday, July 1, 2022 11:10 PM > To: devel@edk2.groups.io > Cc: Lin, Jason1 ; Feng, Bob C ; > Gao, Liming ; Chen, Christine > ; Oram, Isaac W ; > Chaganty, Rangasai V ; Chiang, Dakota >

Re: [edk2-devel] [PATCH v3 1/3] [edk2-platforms] Silicon/Intel/FitGen: Support multiple Startup ACM Type 2 entries in FitGen tool

2022-07-03 Thread Yuwei Chen
Reviewed-by: Yuwei Chen > -Original Message- > From: devel@edk2.groups.io On Behalf Of Lin, > Jason1 > Sent: Friday, July 1, 2022 11:10 PM > To: devel@edk2.groups.io > Cc: Lin, Jason1 ; Feng, Bob C ; > Gao, Liming ; Chen, Christine > ; Oram, Isaac W ; > Ch

Re: [edk2-devel] [Patch V2] pip-requirements.txt: Update basetools version to 0.1.24

2022-07-03 Thread Yuwei Chen
Reviewed-by: Yuwei Chen > -Original Message- > From: devel@edk2.groups.io On Behalf Of Bob > Feng > Sent: Thursday, June 30, 2022 12:11 PM > To: devel@edk2.groups.io > Cc: Kinney, Michael D ; Kubacki, Michael > > Subject: [edk2-devel] [Patch V2] pip-requiremen

[edk2-devel] [PATCH 1/1] BaseTools: Fix DSC LibraryClass precedence rule

2022-06-30 Thread Yuwei Chen
RCH)] > [LibraryClasses.Common.$(MODULE_TYPE)] This patch fixes the issue. Cc: Bob Feng Cc: Liming Gao Cc: Long1 Huang Cc: Ray Ni Signed-off-by: Yuwei Chen --- BaseTools/Source/Python/Workspace/DscBuildData.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/BaseTools/Source/Python/Wor

Re: [edk2-devel] [Patch] BaseTools: Fix the GenMake bug for .cpp source file

2022-06-26 Thread Yuwei Chen
Reviewed-by: Yuwei Chen > -Original Message- > From: Feng, Bob C > Sent: Saturday, June 25, 2022 1:12 PM > To: devel@edk2.groups.io > Cc: Feng, Bob C ; Gao, Liming > ; Chen, Christine > Subject: [Patch] BaseTools: Fix the GenMake bug for .cpp source file &g

Re: [edk2-devel] BaseTools: Current status of BaseTools/Source/Python

2022-06-23 Thread Yuwei Chen
Hi Ayush, The migration is still working in progress. In order to ensure the normal use of users, the current bug fixes are still based on BaseTools in edk2. After all the migrations are completed, an official notification will be sent, and the developments and bug fixing works will be

Re: [edk2-devel] [PATCH 1/1] BaseTools/Ecc: Fix grammar in Ecc error message

2022-06-23 Thread Yuwei Chen
Reviewed-by: Yuwei Chen > -Original Message- > From: devel@edk2.groups.io On Behalf Of Rebecca > Cran > Sent: Monday, June 20, 2022 6:07 AM > To: devel@edk2.groups.io; Feng, Bob C ; Gao, Liming > ; Chen, Christine > Cc: Rebecca Cran > Subject: [edk2-devel] [

[edk2-devel] [PATCH] [edk2-staging] BaseTools:Remove useless FMMT entry file

2022-05-08 Thread Yuwei Chen
FMMT can be directly used as its compiled binary path saved in PATH, the FMMT in BaseTools/BinWrappers/PosixLike is useless now. This patch remove the useless file. Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- BaseTools/BinWrappers/PosixLike/FMMT | 29

[edk2-devel] [PATCH] [edk2-staging] BaseTools: Modify FMMT GNUmakefile

2022-05-08 Thread Yuwei Chen
Currently the path of compiled FMMT binary is not saved in PATH, this patch modified FMMT binary path for directly using. Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- BaseTools/Source/C/FMMT/GNUmakefile | 2 +- BaseTools/Source/C/FMMT/app.makefile | 25

[edk2-devel] [PATCH] Remove useless FMMT entry file

2022-05-08 Thread Yuwei Chen
FMMT can be directly used as its compiled binary path saved in PATH, the FMMT in BaseTools/BinWrappers/PosixLike is useless now. This patch remove the useless file. Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- BaseTools/BinWrappers/PosixLike/FMMT | 29

[edk2-devel] [PATCH] BaseTools: Modify FMMT GNUmakefile

2022-05-08 Thread Yuwei Chen
Currently the path of compiled FMMT binary is not saved in PATH, this patch modified FMMT binary path for directly using. Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- BaseTools/Source/C/FMMT/GNUmakefile | 2 +- BaseTools/Source/C/FMMT/app.makefile | 25

[edk2-devel] Proposal to merge python FMMT tool from edk2-staging to edk2 repo

2022-04-28 Thread Yuwei Chen
Hi all, Currently python FMMT tool has finished final tests on several scenarios and made some functions' optimization. All the changes have already been merged into edk2-staging PyFMMT branch. Since it have been under review for a long time Liming has ever given the agreement, I propose to

Re: [edk2-devel] [Patch V2] BaseTools: Fix DevicePath tool build failure issue

2022-04-10 Thread Yuwei Chen
This patch looks good to me. Reviewed-by: Yuwei Chen > -Original Message- > From: Feng, Bob C > Sent: Monday, April 4, 2022 11:46 AM > To: devel@edk2.groups.io > Cc: Feng, Bob C ; Rebecca Cran > ; Chen, Christine ; Gerd > Hoffmann ; Gao, Liming > Subject: [

Re: [edk2-devel] [PATCH v2 1/1] Basetools: fix gcc workaround

2022-04-01 Thread Yuwei Chen
hange from GCC5.4, I'd also > > suggest > we > > could create a new TOOL CHAIN for gcc12. > > The error reports I collected show in gcc5.4 , > "-Wno-error=stringop-overflow" > > is an unrecognized command line option which is available in gcc12. > > > >

Re: [edk2-devel] [PATCH v2 1/1] Basetools: fix gcc workaround

2022-03-30 Thread Yuwei Chen
ine (Yuwei) > -Original Message- > From: devel@edk2.groups.io On Behalf Of Yuwei > Chen > Sent: Thursday, March 31, 2022 8:51 AM > To: Gerd Hoffmann ; devel@edk2.groups.io > Cc: Feng, Bob C ; Rebecca Cran > ; Pawel Polawski ; Oliver > Steffen ; Gao, Liming > Su

Re: [edk2-devel] [PATCH v2 1/1] Basetools: fix gcc workaround

2022-03-30 Thread Yuwei Chen
Hi Hoffmann, I received some reports that some platforms are using gcc5.4. With this patch, these platform builds are still broken. Would you like to revert the original commits? Thanks, Christine (Yuwei) > -Original Message- > From: Gerd Hoffmann > Sent: Tuesday, March 29, 2022 4:38

Re: [edk2-devel] [Patch] BaseTools: Remove the redundant __FLEXIBLE_SIZE from PcdValueInit.c

2022-03-20 Thread Yuwei Chen
This patch reduces compilation time , it looks good to me. Reviewed-by: Yuwei Chen > -Original Message- > From: Feng, Bob C > Sent: Tuesday, March 15, 2022 8:17 PM > To: devel@edk2.groups.io > Cc: Gao, Liming ; Chen, Christine > > Subject: [Patch] BaseTools:

[edk2-devel] [PATCH] BaseTools: Catch the exception in build.py.

2022-03-01 Thread Yuwei Chen
-defined exception class should be imported. Signed-off-by: Mingyue Liang Cc: Bob Feng Cc: Liming Gao Cc: Yuwei Chen --- BaseTools/Source/Python/build/build.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/BaseTools/Source/Python/build/build.py b/BaseTools/Source/Python/build/build.py

[edk2-devel] [PATCH] BaseTools: Add BaseTools/Scripts to path.

2022-02-27 Thread Yuwei Chen
and Linux systems. Signed-off-by: Mingyue Liang Cc: Bob Feng Cc: Liming Gao Cc: Yuwei Chen --- BaseTools/BuildEnv | 24 +--- BaseTools/toolsetup.bat | 6 ++ 2 files changed, 27 insertions(+), 3 deletions(-) diff --git a/BaseTools/BuildEnv b/BaseTools/BuildEnv index

Re: [edk2-devel] [PATCH v2 6/6] BaseTools: Upgrade the version of NASM tool

2022-01-16 Thread Yuwei Chen
This patch looks good to me. Reviewed-by: Yuwei Chen > -Original Message- > From: Lou, Yun > Sent: Monday, January 10, 2022 11:13 PM > To: devel@edk2.groups.io > Cc: Lou, Yun ; Feng, Bob C ; > Gao, Liming ; Chen, Christine > > Subject: [PATCH v2 6/6] BaseTo

Re: [edk2-devel] [Patch] BaseTools: Fix the bug of --cmd-len build option

2021-12-27 Thread Yuwei Chen
This patch looks good to me. Reviewed-by: Yuwei Chen > -Original Message- > From: Feng, Bob C > Sent: Friday, December 24, 2021 10:50 PM > To: devel@edk2.groups.io > Cc: Feng, Bob C ; Gao, Liming > ; Chen, Christine > Subject: [Patch] BaseTools: Fix the bug of -

Re: [edk2-devel] [PATCH 1/1] BaseTools: Fix StructurePcd offset error.

2021-10-28 Thread Yuwei Chen
oups.io > Cc: Feng, Bob C > Subject: 回复: [PATCH 1/1] BaseTools: Fix StructurePcd offset error. > > Yuwei: > > > -邮件原件- > > 发件人: Yuwei Chen > > 发送时间: 2021年10月27日 18:00 > > 收件人: devel@edk2.groups.io > > 抄送: Bob Feng ; Liming Gao > > >

[edk2-devel] RFC: Replace FMMT Tool from C to Python Version

2021-10-28 Thread Yuwei Chen
Hi everyone, I would like to propose the new FMMT Python tool, which is used to replace the origin C version tool. Currently the FMMT C tool is saved in edk2-staging repo, but its quality and coding style can't meet the Edk2 quality, which is hard to maintain (Hard/Duplicate Code; Regression

[edk2-devel] [PATCH 1/1] BaseTools: Fix StructurePcd offset error.

2021-10-27 Thread Yuwei Chen
Currently when parse VFR files to get the struct info, BOOLEAN type struct is ignored, which caused offset errors. This patch fixed these bugs. Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- BaseTools/Scripts/ConvertFceToStructurePcd.py | 4 ++-- 1 file changed, 2 insertions(+), 2

  1   2   >