[edk2] [Patch] BaseTools build_rule.template: Update aslc rule for XCODE tool chain

2018-11-02 Thread Liming Gao
Update aslc rule to rename the temp output file from .efi to .pecoff. This change can avoid the conflict .efi file name in output directory. One is the driver image, another is aslc temp output file. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Yonghong

Re: [edk2] [PATCH v2 3/5] Vlv2TbltDevicePkg: Remove EdkShellBinPkg with ShellPkg

2018-11-02 Thread Wei, David
Reviewed-by: David Wei Thanks, David Wei Intel SSG/STO/UEFI BIOS -Original Message- From: Zhang, Shenglei Sent: Thursday, November 1, 2018 2:27 PM To: edk2-devel@lists.01.org Cc: Wei, David ; Sun, Zailiang Subject: [PATCH v2 3/5] Vlv2TbltDevicePkg:

Re: [edk2] [PATCH v2 3/4] MdeModulePkg/SdMmcPciHcDxe: Add SwitchClockFreqPost to SdMmcOverride

2018-11-02 Thread Wu, Hao A
> -Original Message- > From: Marcin Wojtas [mailto:m...@semihalf.com] > Sent: Friday, November 02, 2018 5:39 PM > To: Wu, Hao A > Cc: edk2-devel-01; Tomasz Michalec; nad...@marvell.com; Gao, Liming; > Kinney, Michael D > Subject: Re: [edk2] [PATCH v2 3/4] MdeModulePkg/SdMmcPciHcDxe: Add >

Re: [edk2] [PATCH v2 2/4] MdeModulePkg/SdMmcPciHcDxe: Add UhsSignaling to SdMmcOverride protocol

2018-11-02 Thread Wu, Hao A
> -Original Message- > From: Marcin Wojtas [mailto:m...@semihalf.com] > Sent: Friday, November 02, 2018 8:17 PM > To: Wu, Hao A > Cc: edk2-devel-01; Kinney, Michael D; Gao, Liming; Leif Lindholm; Ard > Biesheuvel; nad...@marvell.com; j...@semihalf.com; Tomasz Michalec > Subject: Re: [PATCH

Re: [edk2] [PATCH edk2-platforms 01/27] Platform/Microsoft: Add OpteeClientPkg dec

2018-11-02 Thread Chris Co via edk2-devel
Hi Sumit, > -Original Message- > From: Sumit Garg > Sent: Thursday, November 1, 2018 10:24 PM > To: Chris Co > Cc: Leif Lindholm ; edk2-devel@lists.01.org; Ard > Biesheuvel ; Michael D Kinney > ; tee-...@lists.linaro.org > Subject: Re: [PATCH edk2-platforms 01/27] Platform/Microsoft:

Re: [edk2] [PATCH 1/1] MdePkg/Base.h: Implement BASE_CR() via OFFSET_OF().

2018-11-02 Thread Andrew Fish
+1 Reviewed-by: Andrew Fish > On Nov 2, 2018, at 10:28 AM, Kinney, Michael D > wrote: > > Marvin, > > Thanks. This is a good improvement. > > Reviewed-by: Michael D Kinney > > Mike > >> -Original Message- >> From: Gao, Liming >> Sent: Thursday, November 1, 2018 6:33 PM >> To:

Re: [edk2] [PATCH 1/1] MdePkg/Base.h: Implement BASE_CR() via OFFSET_OF().

2018-11-02 Thread Kinney, Michael D
Marvin, Thanks. This is a good improvement. Reviewed-by: Michael D Kinney Mike > -Original Message- > From: Gao, Liming > Sent: Thursday, November 1, 2018 6:33 PM > To: marvin.haeu...@outlook.com; edk2-devel@lists.01.org > Cc: Kinney, Michael D > Subject: RE: [PATCH 1/1]

[edk2] [PATCH edk2-platforms 5/5] Platform/ARM/Sgi: Add platform support for SGI-Clark.Helios

2018-11-02 Thread Chandni Cherukuri
Platform support for SGI-Clark.Helios platform has been added. Change-Id: Iaf7962334b6366d35a195e4c94341d1972783a81 Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Chandni Cherukuri --- Platform/ARM/SgiPkg/Drivers/PlatformDxe/PlatformDxe.c | 3 +++

[edk2] [PATCH edk2-platforms 4/5] Platform/ARM/Sgi: Add ACPI tables for SGI-Clark.Helios platform

2018-11-02 Thread Chandni Cherukuri
Added ACPI tables and GUID name 'gSgiClarkHeliosAcpiTablesFileGuid' for SGI-Clark.Helios platform. Change-Id: Ib212c0b27ad1f9f709385ac92e5e06b1bd33bfc0 Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Chandni Cherukuri ---

[edk2] [PATCH edk2-platforms 3/5] Platform/ARM/Sgi: Add support for SGI-Clark.Ares platform

2018-11-02 Thread Chandni Cherukuri
Platform support for SGI-Clark.Ares platform has been added Change-Id: I3bb15950963f1ccbf3c099894e90ef730dfc3f8e Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Chandni Cherukuri --- Platform/ARM/SgiPkg/Drivers/PlatformDxe/PlatformDxe.c | 16 +---

[edk2] [PATCH edk2-platforms 2/5] Platform/ARM/Sgi: Add ACPI tables for SGI-Clark.Ares platform

2018-11-02 Thread Chandni Cherukuri
Added ACPI tables and GUID name 'gSgiClarkAresAcpiTablesFileGuid' for SGI-Clark.Ares platform. Change-Id: Ic58fdc4cdd5919b59ca7752ab3468b41cb79d0f0 Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Chandni Cherukuri ---

[edk2] [PATCH edk2-platforms 1/5] Platform/ARM/Sgi: Adapt to changes in system-id DT node.

2018-11-02 Thread Chandni Cherukuri
The 'system-id' node of HW_CONFIG device tree has been updated to have a new property 'config-id' to hold the platform configuration value. Prior to this, configuration ID value was represented by the the upper four bits of the 'platform ID' property value but it now has a seperate property of its

[edk2] [PATCH edk2-platforms 0/5] Add platform support for SGI-Clark platforms

2018-11-02 Thread Chandni Cherukuri
Rest of the patches add support for SGI-Clark.Ares and SGI-Clark.Helios platforms. ___ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel

Re: [edk2] [PATCH v2 1/5] CorebootPayloadPkg: Remove EdkShellBinPkg in FDF

2018-11-02 Thread Ma, Maurice
Hi, Shenglei, Thank you for the patch. I saw many other references in CorebootPayloadPkg DSC files for FULL_BIN. I think we should remove all of them from both DSC and FDF at the same time. The current SHELL_TYPE defined in DSC file is FULL_BIN. It needs to be changed to UEFI_BIN.

Re: [edk2] Reminder for deleting 3 network drivers in MdeModulePkg

2018-11-02 Thread Andrew Fish
> On Nov 1, 2018, at 2:01 AM, Leif Lindholm wrote: > > On Thu, Nov 01, 2018 at 06:01:31AM +, Gao, Liming wrote: >> Leif: >> If all Stewards agree this change to remove MdeModulePkg >> Tcp4Dxe/UefiPxeBcDxe/IScsiDxe, we will try to include it in >> edk2-stable201811 stable tag. >> >>

Re: [edk2] [Patch V3] BaseTools: Use common cc flag for building PcdValueInit.

2018-11-02 Thread Gao, Liming
Reviewed-by: Liming Gao > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of BobCF > Sent: Friday, November 2, 2018 1:57 PM > To: edk2-devel@lists.01.org > Cc: Gao, Liming > Subject: [edk2] [Patch V3] BaseTools: Use common cc flag for building >

Re: [edk2] Plan to move all network components to NetworkPkg

2018-11-02 Thread Gao, Liming
Siyuan: Could you list the mapping between current location and new location of Network IPv4 stack? It will be easy to be understood. And, I prefer to change the platform DSC/FDF per each BZ. That means I expect two commits in DSC/FDF. One is for BZ 1278, another is for 1293. The different

Re: [edk2] [PATCH] UnixPkg: Remove UnixPkg and update Maintainers.txt

2018-11-02 Thread Gao, Liming
Reviewed-by: Liming Gao > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Leif > Lindholm > Sent: Friday, November 2, 2018 7:20 PM > To: Zhang, Shenglei > Cc: Kinney, Michael D ; edk2-devel@lists.01.org; > Laszlo Ersek > Subject: Re: [edk2]

Re: [edk2] Reminder for deleting 3 network drivers in MdeModulePkg

2018-11-02 Thread Gao, Liming
This plan is OK to me. Thanks Liming > -Original Message- > From: Leif Lindholm [mailto:leif.lindh...@linaro.org] > Sent: Thursday, November 1, 2018 5:02 PM > To: Gao, Liming > Cc: Fu, Siyuan ; af...@apple.com; Kinney, Michael D > ; edk2-devel@lists.01.org > Subject: Re: [edk2]

Re: [edk2] [PATCH 2/6] IntelFrameworkModulePkg: fix build for AARCH64/ARM

2018-11-02 Thread Gao, Liming
That's good. Thanks! > -Original Message- > From: Leif Lindholm [mailto:leif.lindh...@linaro.org] > Sent: Friday, November 2, 2018 6:15 PM > To: Gao, Liming > Cc: edk2-devel@lists.01.org > Subject: Re: [PATCH 2/6] IntelFrameworkModulePkg: fix build for AARCH64/ARM > > On Fri, Nov 02,

Re: [edk2] [PATCH v2 2/4] MdeModulePkg/SdMmcPciHcDxe: Add UhsSignaling to SdMmcOverride protocol

2018-11-02 Thread Marcin Wojtas
pt., 2 lis 2018 o 09:21 Marcin Wojtas napisał(a): > > Hi Hao, > > czw., 1 lis 2018 o 08:06 Wu, Hao A napisał(a): > > > > Hi Marcin, > > > > > -Original Message- > > > From: Marcin Wojtas [mailto:m...@semihalf.com] > > > Sent: Friday, October 05, 2018 9:25 PM > > > To:

Re: [edk2] [PATCH] UnixPkg: Remove UnixPkg and update Maintainers.txt

2018-11-02 Thread Leif Lindholm
On Fri, Nov 02, 2018 at 10:46:30AM +0800, Shenglei Zhang wrote: > UnixPkg is deprecated. > The same functionality is supported using the EmulatorPkg. > Please see EmulatorPkg/README for more information about > EmulatorPkg. > https://bugzilla.tianocore.org/show_bug.cgi?id=1291 > > Cc: Andrew Fish

[edk2] [PATCH v2] AppPkg: fix webserver build for !Ia32/X64

2018-11-02 Thread Leif Lindholm
The WebServer application is not meant to be Ia32/X64 specific, and would build for other architectures, if it wasn't for the #include in WebServer.h. Move that statement to Mtrr.c instead, which is the only consumer, and is already being filtered out for other architectures. Cc: Daryl

Re: [edk2] Capsules and versions

2018-11-02 Thread Tomas Pilar (tpilar)
Hi Sean, This solves my problem, thank you very much! Cheers, Tom On 01/11/2018 16:43, Sean Brogan wrote: > Tom, > > Short answer: It is tool dependent. > > On Windows where the ESRT and FMP have been used for years there is another > file that contains that information (Capsules are just

Re: [edk2] [PATCH v2 0/4] SdMmcOverride extension

2018-11-02 Thread Marcin Wojtas
Hao, czw., 1 lis 2018 o 08:11 Wu, Hao A napisał(a): > > Hi Marcin and Ard, > > > -Original Message- > > From: Marcin Wojtas [mailto:m...@semihalf.com] > > Sent: Friday, October 05, 2018 9:25 PM > > To: edk2-devel@lists.01.org > > Cc: Tian, Feng; Kinney, Michael D; Gao, Liming;

Re: [edk2] [PATCH 2/6] IntelFrameworkModulePkg: fix build for AARCH64/ARM

2018-11-02 Thread Leif Lindholm
On Fri, Nov 02, 2018 at 01:39:24AM +, Gao, Liming wrote: > Leif: > There is one mistake in comments >+ #^M. Could you clean it? > With this change, Reviewed-by: Liming Gao . Whoops, thanks for spotting. Will do. > Besides, have you verified AARCH64 GCC build? AARCH64 and ARM, GCC5

Re: [edk2] [PATCH v2 4/4] MdeModulePkg/SdMmcPciHcDxe: Allow overriding base clock frequency

2018-11-02 Thread Marcin Wojtas
Hi Hao, czw., 1 lis 2018 o 08:11 Wu, Hao A napisał(a): > > Hi Marcin, > > > -Original Message- > > From: Marcin Wojtas [mailto:m...@semihalf.com] > > Sent: Friday, October 05, 2018 9:25 PM > > To: edk2-devel@lists.01.org > > Cc: Tian, Feng; Kinney, Michael D; Gao, Liming;

Re: [edk2] [PATCH v2 3/4] MdeModulePkg/SdMmcPciHcDxe: Add SwitchClockFreqPost to SdMmcOverride

2018-11-02 Thread Marcin Wojtas
Hi Hao, czw., 1 lis 2018 o 08:06 Wu, Hao A napisał(a): > > > -Original Message- > > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > > Marcin Wojtas > > Sent: Friday, October 05, 2018 9:25 PM > > To: edk2-devel@lists.01.org > > Cc: Tian, Feng; t...@semihalf.com;

Re: [edk2] [PATCH] Maintainers.txt: Change package maintainer of IntelSiliconPkg

2018-11-02 Thread Yao, Jiewen
reviewed by:jiewen@intel.com thank you! Yao, Jiewen > 在 2018年11月2日,下午5:19,Ni, Ruiyu 写道: > > Contributed-under: TianoCore Contribution Agreement 1.1 > Signed-off-by: Ruiyu Ni > Cc: Jiewen Yao > --- > Maintainers.txt | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git

[edk2] [PATCH] Maintainers.txt: Change package maintainer of UefiCpuPkg

2018-11-02 Thread Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Ruiyu Ni Cc: Eric Dong --- Maintainers.txt | 1 + 1 file changed, 1 insertion(+) diff --git a/Maintainers.txt b/Maintainers.txt index 937aedc664..29d1e10c73 100644 --- a/Maintainers.txt +++ b/Maintainers.txt @@ -259,6

[edk2] [PATCH] Maintainers.txt: Change package maintainer of IntelSiliconPkg

2018-11-02 Thread Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Ruiyu Ni Cc: Jiewen Yao --- Maintainers.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Maintainers.txt b/Maintainers.txt index 36c6689afd..937aedc664 100644 --- a/Maintainers.txt +++

Re: [edk2] [PATCH] BaseTools: Check the max size for string PCD.

2018-11-02 Thread Feng, Bob C
Reviewed-by: Bob Feng -Original Message- From: Zhao, ZhiqiangX Sent: Wednesday, October 31, 2018 6:36 PM To: edk2-devel@lists.01.org Cc: Zhao, ZhiqiangX ; Gao, Liming ; Zhu, Yonghong ; Feng, Bob C Subject: [PATCH] BaseTools: Check the max size for string PCD. According to

[edk2] [Patch] UefiCpuPkg/MpInitLib: Rollback old change 2a5997f8.

2018-11-02 Thread Eric Dong
In some special cases, after BSP send Init-sipi-sipi signal AP need more time to start the Ap procedure. In this case BSP may think AP has finish its task but truly AP not begin yet. Rollback former change to keep the status which only used when AP truly finished task. Cc: Ruiyu Ni

Re: [edk2] [PATCH v2 2/4] MdeModulePkg/SdMmcPciHcDxe: Add UhsSignaling to SdMmcOverride protocol

2018-11-02 Thread Marcin Wojtas
Hi Hao, czw., 1 lis 2018 o 08:06 Wu, Hao A napisał(a): > > Hi Marcin, > > > -Original Message- > > From: Marcin Wojtas [mailto:m...@semihalf.com] > > Sent: Friday, October 05, 2018 9:25 PM > > To: edk2-devel@lists.01.org > > Cc: Tian, Feng; Kinney, Michael D; Gao, Liming;

[edk2] Plan to move all network components to NetworkPkg

2018-11-02 Thread Fu, Siyuan
Hi, All The edk2 Bugzilla #1278 "Delete IPv4 only TCP/iSCSI/PXE drivers in MdeModulePkg" has been confirmed to be included in edk2-stable201903 tag, so platform DSC/FDF need to be updated to adopt the change by that time. I think might be an opportunity moment to make a little more - move all

[edk2] [edk2-test][Patch] uefi-sct/SctPkg:The Lun display order issue in iSCSI device path text

2018-11-02 Thread Eric Jin
The LUN is an 8 byte array that is displayed in hexadecimal format with byte 0 first (i.e., on the left) and byte 7 last (i.e, on the right). Cc: Supreeth Venkatesh Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Eric Jin ---

Re: [edk2] [PATCH 2/2] MdeModulePkg/BaseSortLib: Enable for all module types

2018-11-02 Thread Gao, Liming
Yes. I will wait for another patch in this patch set to be reviewed. >-Original Message- >From: Zeng, Star >Sent: Friday, November 02, 2018 1:00 PM >To: Gao, Liming ; Jeff Brasen ; >Ni, Ruiyu ; edk2-devel@lists.01.org >Cc: Kinney, Michael D ; Gao, Liming >; Zeng, Star >Subject: RE:

Re: [edk2] [PATCH] uefi-sct/SctPkg:The Lun display order issue in iSCSI device path text

2018-11-02 Thread Jin, Eric
Hi Supreeth, The intention of the patch is to fix the LUN display issue in the iSCSI device path text. In the UEFI spec, the according definition is clarified as " The LUN is an 8 byte array that is displayed in hexadecimal format with byte 0 first (i.e., on the left) and byte 7 last (i.e,

Re: [edk2] [Patch 1/1] BaseTools tools_def.template: Add GCC link script in X86 ASLDLINK_FLAGS

2018-11-02 Thread Zhu, Yonghong
Reviewed-by: Yonghong Zhu Best Regards, Zhu Yonghong -Original Message- From: Gao, Liming Sent: Thursday, November 01, 2018 10:47 PM To: edk2-devel@lists.01.org Cc: Zhu, Yonghong Subject: [Patch 1/1] BaseTools tools_def.template: Add GCC link script in X86 ASLDLINK_FLAGS This fix