Re: [edk2] [PATCH] SecurityPkg TcgDxe: Simplify debug msg when "TPM not working properly"

Reviewed-by: Chao Zhang From: Yao, Jiewen Sent: Thursday, June 8, 2017 4:39 PM To: Zeng, Star ; edk2-devel@lists.01.org Cc: Yao, Jiewen ; Chan, Amy ; Chiu, Chasel ; Zhang, Chao B

[edk2] [Patch][edk2-platforms/devel-MinnowBoard3-UDK2017] Change PcdMinimalValidYear.

Change PcdMinimalValidYear to 2017. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: zwei4 --- Platform/BroxtonPlatformPkg/PlatformDsc/Components.dsc | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git

Re: [edk2] [PATCH] EmbeddedPkg/MmcDxe: Add alignment for ECSD data

2017-06-08 22:55 GMT+08:00 Andrew Fish : > > On Jun 8, 2017, at 1:18 AM, Jun Nie wrote: > > Add alignment for ECSD data for DMA access. Otherwise > the data is corrupted on Sanechips platform. > > Contributed-under: TianoCore Contribution Agreement 1.0 >

Re: [edk2] [PATCH] EmbeddedPkg/MmcDxe: Add non-DDR timing mode support

2017-06-09 0:55 GMT+08:00 Leif Lindholm : > On Thu, Jun 08, 2017 at 04:39:44PM +0800, Haojian Zhuang wrote: >> On 2017/6/8 16:12, Jun Nie wrote: >> >Only DDR mode is support for 8bit mode currently. Add >> >non-DDR case when configuring ECSD. >> > >> >Contributed-under:

Re: [edk2] [PATCH] ShellPkg/alias: Return status for alias deletion

All, I just sent out a patch "[edk2] [PATCH] Shell/alias: Print detailed error when deleting alias", could you please review it? Thanks/Ray > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Shah, Tapan > Sent: Friday, June 9, 2017 12:22 AM >

[edk2] [PATCH] Shell/alias: Print detailed error when deleting alias

STR_GEN_ERR_NOT_FOUND is added and currently is only used by alias command. This string template can be used by other commands as well. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Jaben Carsey Cc: Tapan Shah

Re: [edk2] [PATCH] MdeModulePkg/PciHostBridgeDxe: Make bitwise operands of the same size

> -Original Message- > From: Laszlo Ersek [mailto:ler...@redhat.com] > Sent: Friday, June 09, 2017 4:40 AM > To: Wu, Hao A; edk2-devel@lists.01.org > Cc: Yao, Jiewen > Subject: Re: [edk2] [PATCH] MdeModulePkg/PciHostBridgeDxe: Make bitwise > operands of the same size > > On 06/08/17

Re: [edk2] [PATCH v1 1/1] ShellPkg/ifconfig: Update help message

Reviewed-by: Wu Jiaxin > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > hegdenag > Sent: Thursday, June 8, 2017 2:41 PM > To: edk2-devel@lists.01.org > Cc: Carsey, Jaben ; Ni, Ruiyu

Re: [edk2] [PATCH v1 1/1] NetworkPkg/HttpBootDxe: Handle new #define in HttpBootDxe

Reviewed-by: Wu Jiaxin > -Original Message- > From: hegdenag [mailto:nagaraj-p.he...@hpe.com] > Sent: Thursday, June 8, 2017 2:41 PM > To: edk2-devel@lists.01.org > Cc: Wu, Jiaxin ; srira...@hpe.com; Fu, Siyuan > ; Ye, Ting

Re: [edk2] [PATCH v1 1/1] MdeModulePkg/DxeHttpLib: Handle new #define in HttpMappingToStatusCode

Reviewed-by: Wu Jiaxin > -Original Message- > From: hegdenag [mailto:nagaraj-p.he...@hpe.com] > Sent: Thursday, June 8, 2017 2:40 PM > To: edk2-devel@lists.01.org > Cc: Wu, Jiaxin ; srira...@hpe.com; Fu, Siyuan > ; Ye, Ting

Re: [edk2] [PATCH v1 1/1] MdePkg/Http.h: Add #define for 308 redirect

Thanks the contribution. Reviewed-by: Wu Jiaxin > -Original Message- > From: hegdenag [mailto:nagaraj-p.he...@hpe.com] > Sent: Thursday, June 8, 2017 2:39 PM > To: edk2-devel@lists.01.org > Cc: Wu, Jiaxin ; srira...@hpe.com; Fu, Siyuan >

[edk2] DxeImageVerificationHandler

Can the DxeImageVerificationHandler function be called in shell to check the signature of a loaded image? Is this even a valid approach or is there a different way to check the signature? Initial attempt failed at the PeCoffLoaderGetImageInfo function (ReturnCode=Unsupported). Thanks

Re: [edk2] [PATCH] OvmfPkg/AcpiPlatformDxe: fix spurious uninitialized var warning

On 06/08/17 21:05, Ard Biesheuvel wrote: > Commit 4275f38507a4 ("OvmfPkg/AcpiPlatformDxe: alloc blobs from 64-bit > space unless restricted") introduced a variable which is [incorrectly] > identified by GCC as being potentially uninitialized. So let's just set > it to NULL before use. > >

Re: [edk2] [PATCH staging][BaseToolsOpt 0/4] Enable multiple driver combination

Hi Liming, (CC Jordan) On 06/08/17 08:55, Liming Gao wrote: > Combine more drivers into the single one can reduce the image size and > compile link time. This patch adds this support in BaseTools. > > Liming Gao (4): > BaseTools: Merge multiple drivers into one for size and link >

Re: [edk2] [PATCH] MdeModulePkg/PciHostBridgeDxe: Make bitwise operands of the same size

On 06/08/17 10:22, Hao Wu wrote: > Cc: Jiewen Yao > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Hao Wu > --- > MdeModulePkg/Bus/Pci/PciHostBridgeDxe/PciRootBridgeIo.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) >

[edk2] [PATCH v2 08/10] BaseTools: Tools change to support PI v1.5 Specification.

This patch registers SMM_STANDALONE and SMM_CORE_STANDALONE module type with python build tools. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jiewen Yao Signed-off-by: Supreeth Venkatesh ---

[edk2] [PATCH v2 09/10] BaseTools: Tools change to support PI v1.5 Specification.

This patch checks SUP_MODULE_SMM_CORE_STANDALONE and SUP_MODULE_SMM_STANDALONE module compatibility with PI specification version. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jiewen Yao Signed-off-by: Supreeth Venkatesh

[edk2] [PATCH v2 06/10] BaseTools: Tools change to support PI v1.5 Specification.

This patch registers SMM_STANDALONE and SMM_CORE_STANDALONE module types with CommonClass and PackageIncludePkgHeaderClass in CommonDataClass. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jiewen Yao Signed-off-by: Supreeth Venkatesh

[edk2] [PATCH v2 10/10] BaseTools: Tools change to support PI v1.5 Specification.

This patch registers SMM_STANDALONE with Ffs class. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jiewen Yao Signed-off-by: Supreeth Venkatesh --- BaseTools/Source/Python/Eot/FvImage.py | 1 + 1 file changed, 1

[edk2] [PATCH v2 07/10] BaseTools: Tools change to support PI v1.5 Specification.

This patch verifies SMM_CORE_STANDALONE module compatibility with PI specification version. Also, it registers SMM_STANDALONE/SMM_CORE_STANDALONE modules with FdfParser class and provides mapping between SMM_STANDALONE and SMM_CORE_STANDALONE module type in FDF with EFI_FV_FILETYPE_SMM_STANDALONE

[edk2] [PATCH v2 05/10] BaseTools: Tools change to support PI v1.5 Specification.

This patch adds support for FdfParser tool to parse SMM_STANDALONE and SMM_CORE_STANDALONE modules. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jiewen Yao Signed-off-by: Supreeth Venkatesh ---

[edk2] [PATCH v2 03/10] BaseTools: Tools change to support PI v1.5 Specification.

This patch adds changes to auto generate SMM_CORE_STANDALONE and SMM_STANDALONE Entry Point templates. Also, it adds changes to help auto generate dependency expressions for SMM_STANDALONE modules. PI Specification v1.5 specifies Management Mode System Table (MMST) which is a collection of

[edk2] [PATCH v2 01/10] BaseTools: Tools change to support PI v1.5 Specification.

PI specification v1.5 defines new firmware volume file types. This patch adds the new file type EFI_FV_FILETYPE_SMM_STANDALONE in GenFfs tool. *_SMM_* regex instead of *_MM_* is used to preserve backwards compatibility. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by:

[edk2] [PATCH v2 02/10] BaseTools: Tools change to support PI v1.5 Specification.

PI v1.5 Specification Volume 4 defines Management Mode Core Interface. In order to support Management Mode Core Interface, Module Types MM_STANDALONE, MM_CORE_STANDALONE are needed. This patch ensures that SMM_STANDALONE, SMM_CORE_STANDALONE Modules are treated as EFI Boot Service Driver in GenFw

[edk2] [PATCH v2 00/10] *** BaseTools: Tools change to support PI v1.5 Specification. ***

*** PI v1.5 Specification Volume 4 defines Management Mode Core Interface. In order to support Management Mode Core Interface, Module Types MM_STANDALONE, MM_CORE_STANDALONE are needed. PI specification v1.5 defines the following new file types: #define EFI_FV_FILETYPE_MM 0x0A #define

Re: [edk2] [PATCH 1/1] OvmfPkg/AcpiPlatformDxe: alloc blobs from 64-bit space unless restricted

On 8 June 2017 at 18:40, Laszlo Ersek wrote: > On 06/08/17 12:11, Ard Biesheuvel wrote: >> On 7 June 2017 at 23:10, Laszlo Ersek wrote: >>> On 06/06/17 20:16, Laszlo Ersek wrote: On 06/05/17 19:47, Jordan Justen wrote: > On 2017-06-03 08:42:03,

[edk2] [PATCH] OvmfPkg/AcpiPlatformDxe: fix spurious uninitialized var warning

Commit 4275f38507a4 ("OvmfPkg/AcpiPlatformDxe: alloc blobs from 64-bit space unless restricted") introduced a variable which is [incorrectly] identified by GCC as being potentially uninitialized. So let's just set it to NULL before use. Contributed-under: TianoCore Contribution Agreement 1.0

Re: [edk2] [PATCH 1/1] OvmfPkg/AcpiPlatformDxe: alloc blobs from 64-bit space unless restricted

On 06/08/17 12:11, Ard Biesheuvel wrote: > On 7 June 2017 at 23:10, Laszlo Ersek wrote: >> On 06/06/17 20:16, Laszlo Ersek wrote: >>> On 06/05/17 19:47, Jordan Justen wrote: On 2017-06-03 08:42:03, Laszlo Ersek wrote: > ... by narrower than 8-byte ADD_POINTER

[edk2] [PATCH 1/5] OvmfPkg: introduce Q35TsegSizeLib (class header and sole lib instance)

OvmfPkg contains three modules that work with the TSEG (SMRAM) size: PlatformPei (PEIM), SmmAccessPei (PEIM), and SmmAccess2Dxe (DXE_DRIVER). These modules open-code the interpretation of the ESMRAMC register's TSEG_SZ bit-field. That is OK as long as we stick with the Q35 hardware spec and

[edk2] [PATCH 0/5] OvmfPkg: recognize an extended TSEG when QEMU offers it

In we found that the SMM_REQUIRE build of OVMF cannot boot with as many VCPUs as we'd like, due to SMRAM exhaustion (even with the largest TSEG, 8MB). Related thread on edk2-devel:

[edk2] [PATCH 5/5] OvmfPkg/Q35TsegSizeLib: recognize an extended TSEG when QEMU offers it

This patch interfaces with the QEMU feature introduced in QEMU patch q35/mch: implement extended TSEG sizes Excerpt: > The q35 machine type currently lets the guest firmware select a 1MB, 2MB > or 8MB TSEG (basically, SMRAM) size. In edk2/OVMF, we use 8MB, but even > that is not enough when a

[edk2] [PATCH 4/5] OvmfPkg/SmmAccess: rebase shared PEIM/DXE code to Q35TsegSizeLib

SmmAccessPei and SmmAccess2Dxe share the internals between their PEI_SMM_ACCESS_PPI and EFI_SMM_ACCESS2_PROTOCOL implementations, respectively, in "SmramInternal.c". In this patch, convert "SmramInternal.c" to Q35TsegSizeLib client code. Replace any mapping, from MCH_ESMRAMC_TSEG_xMB bitmask

[edk2] [PATCH 3/5] OvmfPkg/SmmAccess: rebase code unique to SmmAccessPei to Q35TsegSizeLib

SmmAccessPei and SmmAccess2Dxe share the internals between their PEI_SMM_ACCESS_PPI and EFI_SMM_ACCESS2_PROTOCOL implementations, respectively, in "SmramInternal.c". In this patch, convert the code that is unique to SmmAccessPei -- namely SmmAccessPeiEntryPoint() -- to Q35TsegSizeLib client code,

Re: [edk2] [PATCH] EmbeddedPkg/MmcDxe: Add non-DDR timing mode support

On Thu, Jun 08, 2017 at 04:39:44PM +0800, Haojian Zhuang wrote: > On 2017/6/8 16:12, Jun Nie wrote: > >Only DDR mode is support for 8bit mode currently. Add > >non-DDR case when configuring ECSD. > > > >Contributed-under: TianoCore Contribution Agreement 1.0 > >Signed-off-by: Jun Nie

Re: [edk2] [PATCH] EmbeddedPkg/MmcDxe: Add non-DDR timing mode support

Hi Jun, Don't forget to cc the package maintainers on your patch submission, as found in top-level Maintainers.txt. On Thu, Jun 08, 2017 at 04:12:27PM +0800, Jun Nie wrote: > Only DDR mode is support for 8bit mode currently. Add > non-DDR case when configuring ECSD. > > Contributed-under:

Re: [edk2] [PATCH] EmbeddedPkg/MmcDxe: Add alignment for ECSD data

On Thu, Jun 08, 2017 at 07:55:11AM -0700, Andrew Fish wrote: > > > On Jun 8, 2017, at 1:18 AM, Jun Nie wrote: > > > > Add alignment for ECSD data for DMA access. Otherwise > > the data is corrupted on Sanechips platform. > > > > Contributed-under: TianoCore Contribution

Re: [edk2] [PATCH] ShellPkg/alias: Return status for alias deletion

I would suggest to keep FILE/DIR tokens as-is to avoid more code changes. New token for alias... to match the wording it can be "%s: not found." -Original Message- From: Carsey, Jaben [mailto:jaben.car...@intel.com] Sent: Thursday, June 08, 2017 11:16 AM To: Shah, Tapan

Re: [edk2] [PATCH] ShellPkg/alias: Return status for alias deletion

Wonder if we should refactor FILE_NOT_FOUND, DIR_NOT_FOUND, and this into "%s '%s' not found"? > -Original Message- > From: Shah, Tapan [mailto:tapands...@hpe.com] > Sent: Thursday, June 08, 2017 9:14 AM > To: Carsey, Jaben ; Ni, Ruiyu > Cc:

Re: [edk2] [PATCH] ShellPkg/alias: Return status for alias deletion

Correct. STR_GEN_ERR_NOT_FOUND should be "alias: '' does not exist." -Original Message- From: Carsey, Jaben [mailto:jaben.car...@intel.com] Sent: Thursday, June 08, 2017 11:10 AM To: Shah, Tapan ; Ni, Ruiyu Cc: edk2-devel@lists.01.org Subject:

Re: [edk2] [PATCH] ShellPkg/alias: Return status for alias deletion

Tapa, Is your thinking like this: if (EFI_ERROR(Status)) { if (Status == EFI_ACCESS_DENIED) { ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_ERR_AD), gShellLevel3HiiHandle, L"alias"); ShellStatus = SHELL_ACCESS_DENIED; } else if (Status == EFI_NOT_FOUND) {

Re: [edk2] [PATCH] ShellPkg/alias: Return status for alias deletion

EFI_NOT_FOUND should be handled with the token I mentioned for a clear message. All other errors should be printed using STR_GEN_ERR_UK. -Original Message- From: Ni, Ruiyu [mailto:ruiyu...@intel.com] Sent: Wednesday, June 07, 2017 5:14 PM To: Shah, Tapan Cc: Carsey,

Re: [edk2] [PATCH v1 1/1] ShellPkg/ifconfig: Update help message

Seems quite straight forward to me. Reviewed-by: Jaben Carsey > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Shah, Tapan > Sent: Thursday, June 08, 2017 8:55 AM > To: Hegde, Nagaraj P ;

Re: [edk2] [PATCH v1 1/1] ShellPkg/ifconfig: Update help message

Reviewed-by: Tapan Shah -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Hegde, Nagaraj P Sent: Thursday, June 08, 2017 1:41 AM To: edk2-devel@lists.01.org Cc: jaben.car...@intel.com; ruiyu...@intel.com; siyuan...@intel.com;

Re: [edk2] [PATCH] EmbeddedPkg/MmcDxe: Add alignment for ECSD data

> On Jun 8, 2017, at 1:18 AM, Jun Nie wrote: > > Add alignment for ECSD data for DMA access. Otherwise > the data is corrupted on Sanechips platform. > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Jun Nie

Re: [edk2] [PATCH 00/10] *** BaseTools: Tools change to support PI v1.5 Specification. ***

On Thu, 2017-06-08 at 03:32 +, Gao, Liming wrote: > Supreeth: >   Thanks for your contribution. I review this patch set, and have two > comments.  Thanks Liming for reviewing the Patch set. > 1) This is added in PI1.5. So, its version check should be with > 0x00010032 (1 and 50), not

Re: [edk2] EDK2 build using Visual Studio

> On Jun 8, 2017, at 3:29 AM, GN Keshava wrote: > > Hi Liming, > > Sorry, I mixed up the things :( > > If I try VS2015x86 , I'm getting this error: > > Build environment: Windows-post2008Server-6.2.9200 > Build start time: 15:54:28, Jun.08 2017 > > build: : warning:

[edk2] [Patch] BaseTools: Enhance the report to not show the empty section

Enhance the report to not show the empty section, eg: Module Library Sub-section, if there is nothing in this section, we will not show it in the report. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu

[edk2] [Patch] BaseTools: support building the same INF more than once with -m option

Currently DSC file [Components] Section can support building the same INF more than once for the same arch, this patch support build with -m option to generate multiple instances. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong

Re: [edk2] [PATCH] EmbeddedPkg/MmcDxe: Add alignment for ECSD data

On 2017/6/8 16:18, Jun Nie wrote: Add alignment for ECSD data for DMA access. Otherwise the data is corrupted on Sanechips platform. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jun Nie --- EmbeddedPkg/Universal/MmcDxe/Mmc.h | 2 +- 1 file

Re: [edk2] EDK2 build using Visual Studio

Hi Liming, Sorry, I mixed up the things :( If I try VS2015x86 , I'm getting this error: Build environment: Windows-post2008Server-6.2.9200 Build start time: 15:54:28, Jun.08 2017 build: : warning: Tool chain [VS2015x86] is not defined build... : error 4000: Not available [VS2015x86]

Re: [edk2] [PATCH 1/1] OvmfPkg/AcpiPlatformDxe: alloc blobs from 64-bit space unless restricted

On 7 June 2017 at 23:10, Laszlo Ersek wrote: > On 06/06/17 20:16, Laszlo Ersek wrote: >> On 06/05/17 19:47, Jordan Justen wrote: >>> On 2017-06-03 08:42:03, Laszlo Ersek wrote: ... by narrower than 8-byte ADD_POINTER references. Introduce the

Re: [edk2] [PATCH] EmbeddedPkg/MmcDxe: Add non-DDR timing mode support

On 2017/6/8 16:12, Jun Nie wrote: Only DDR mode is support for 8bit mode currently. Add non-DDR case when configuring ECSD. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jun Nie --- EmbeddedPkg/Universal/MmcDxe/MmcIdentification.c | 5 - 1

Re: [edk2] [PATCH] SecurityPkg TcgDxe: Simplify debug msg when "TPM not working properly"

Reviewed-by: jiewen@intel.com -Original Message- From: Zeng, Star Sent: Thursday, June 8, 2017 10:33 AM To: edk2-devel@lists.01.org Cc: Zeng, Star >; Yao, Jiewen

Re: [edk2] [PATCH] MdeModulePkg/PciHostBridgeDxe: Make bitwise operands of the same size

Reviewed-by: jiewen@intel.com > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Hao > Wu > Sent: Thursday, June 8, 2017 4:23 PM > To: edk2-devel@lists.01.org > Cc: Wu, Hao A ; Yao, Jiewen > Subject:

[edk2] [PATCH] MdeModulePkg/PciHostBridgeDxe: Make bitwise operands of the same size

Cc: Jiewen Yao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu --- MdeModulePkg/Bus/Pci/PciHostBridgeDxe/PciRootBridgeIo.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[edk2] [PATCH] EmbeddedPkg/MmcDxe: Add alignment for ECSD data

Add alignment for ECSD data for DMA access. Otherwise the data is corrupted on Sanechips platform. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jun Nie --- EmbeddedPkg/Universal/MmcDxe/Mmc.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

[edk2] [PATCH] EmbeddedPkg/MmcDxe: Add non-DDR timing mode support

Only DDR mode is support for 8bit mode currently. Add non-DDR case when configuring ECSD. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jun Nie --- EmbeddedPkg/Universal/MmcDxe/MmcIdentification.c | 5 - 1 file changed, 4 insertions(+), 1

Re: [edk2] [Patch][edk2-platforms/devel-MinnowBoard3-UDK2017] Build script.

Reviewed-by: zwei4 Thanks, David Wei -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of lushifex Sent: Thursday, June 08, 2017 3:58 PM To: edk2-devel@lists.01.org Cc: Wei, David

[edk2] [Patch][edk2-platforms/devel-MinnowBoard3-UDK2017] Build script.

Change build script to differentiate FAB A and FAB B Board. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: lushifex --- BuildBIOS.bat | 14 +- BuildBIOS.sh | 31

[edk2] [PATCH 1/2] BaseTools/PatchCheck.py: Fix misreport for binary changes in patch

For a patch file that: 1. Contains a binary change 2. Contains any other changes after the binary change PatchCheck.py will complains with the following error: * Patch format error: diff found after end of patch Line: literal This commit resolves this misreport. Cc: Liming Gao

[edk2] [PATCH 0/2] Some refines to BaseTools/PatchCheck.py

The following two refinements are made to PatchCheck.py: a. Resolves misreport of 'Patch format error: diff found after end of patch', when a patch file contains a binary file change and any other changes after the binary change. b. Adds detection of adding new binary files in a patch file

[edk2] [PATCH 2/2] BaseTools/PatchCheck.py: Add warning info for new binary files

The commit adds the detection of adding new binary files in a patch file or in a commit. The following warning messages will be appended at the end of the script output: WARNING - The following binary files will be added into the repository: ... Cc: Liming Gao Cc:

[edk2] PXE Device path has just only MAC

Hello all, I've built and run the Nt32Pkg source from here. I've enabled PXE boot support and I could see only MAC device path as (005056C2, 0x1) for PXEv4 boot options (UEFI PXEv4 MAC(005056C000XX)) and for some other PXE boot options as

[edk2] [PATCH staging][BaseToolsOpt 0/4] Enable multiple driver combination

Combine more drivers into the single one can reduce the image size and compile link time. This patch adds this support in BaseTools. Liming Gao (4): BaseTools: Merge multiple drivers into one for size and link performance OvmfPkg: Update QemuVideo and VirtioGpuDxe to use NULL as

[edk2] [PATCH staging][BaseToolsOpt 4/4] Update Readme.MD to include multiple driver combination.

Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao --- Readme.MD | 1 + 1 file changed, 1 insertion(+) diff --git a/Readme.MD b/Readme.MD index a436dd5..b5646be 100644 --- a/Readme.MD +++ b/Readme.MD @@ -15,6 +15,7 @@ identified to be

[edk2] [PATCH staging][BaseToolsOpt 2/4] OvmfPkg: Update QemuVideo and VirtioGpuDxe to use NULL as DriverBindingHandle

To combine two drivers into one, they can't share ImageHandle as DriverBindingHandle. So, update their code to use NULL as DriverBindingHandle. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao --- OvmfPkg/QemuVideoDxe/Driver.c| 2 +-

[edk2] [PATCH staging][BaseToolsOpt 3/4] OvmfPkg: Combine QemuVideoDxe and VirtioGpuDxe to one driver

This is an example to show the driver combination in Platform.dsc. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao --- OvmfPkg/OvmfPkgIa32X64.dsc | 6 -- OvmfPkg/OvmfPkgIa32X64.fdf | 2 +- 2 files changed, 5 insertions(+), 3

[edk2] [PATCH staging][BaseToolsOpt 1/4] BaseTools: Merge multiple drivers into one for size and link performance

Update BaseTools to support the multiple driver combination. The merge style reuses the library instance syntax in package.dsc file. The below example is to combine VirtioGpu and QemuVideoDxe driver into one driver. VirtioGpu driver entry point will be executed first, QemuVideoDxe entry point will

[edk2] [PATCH v1 1/1] ShellPkg/ifconfig: Update help message

Couple of instances had IP4 mentioned, instead of IPv4. Changing all to IPv4 to maintain consistency. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hegde Nagaraj P ---

[edk2] [PATCH v1 1/1] NetworkPkg/HttpBootDxe: Handle new #define in HttpBootDxe

Add a error message in HttpBootPrintErrorMessage for HTTP 308 Redirect. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hegde Nagaraj P --- NetworkPkg/HttpBootDxe/HttpBootSupport.c | 4 1 file changed, 4 insertions(+) diff --git

[edk2] [PATCH v1 1/1] MdeModulePkg/DxeHttpLib: Handle new #define in HttpMappingToStatusCode

Related to the ECR for support of HTTP Redirect (308) Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hegde Nagaraj P --- MdeModulePkg/Library/DxeHttpLib/DxeHttpLib.c | 2 ++ 1 file changed, 2 insertions(+) diff --git

[edk2] [PATCH v1 1/1] MdePkg/Http.h: Add #define for 308 redirect

Adding Support for the ECR for HTTP 308 redirect. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hegde Nagaraj P --- MdePkg/Include/Protocol/Http.h | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git