Re: [edk2] [PATCH 2/4] Platforms: Add ZX RTC driver for Sanchip SoC

2017-08-10 Thread Leif Lindholm
Just another thought. On Wed, Aug 09, 2017 at 10:12:37PM +0800, Jun Nie wrote: > diff --git > a/Silicon/Sanchip/Library/Zx6718RealTimeClockLib/Zx296718RealTimeClock.h > b/Silicon/Sanchip/Library/Zx6718RealTimeClockLib/Zx296718RealTimeClock.h > new file mode 100644 > index 000..3b5a4d4 > ---

Re: [edk2] [PATCH 2/4] Platforms: Add ZX RTC driver for Sanchip SoC

2017-08-10 Thread Leif Lindholm
On Wed, Aug 09, 2017 at 10:12:37PM +0800, Jun Nie wrote: > Runtime service is not supported yet. > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Jun Nie > --- > .../Zx6718RealTimeClockLib/Zx296718RealTimeClock.c | 376 > + >

Re: [edk2] [PATCH 4/4] Platforms/zx: Add platform build system files

2017-08-10 Thread Leif Lindholm
On Wed, Aug 09, 2017 at 10:12:39PM +0800, Jun Nie wrote: > Add platform build system files, including *.dsc *.fdf *.dec > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Jun Nie > --- > Platform/Sanchip/Zx296718Evb/Zx296718Evb.dec | 33 ++ >

Re: [edk2] Multiple Device ID support in EfiRom

2017-08-10 Thread Gao, Liming
This patch is not integrated into edk2 trunk. Could you file this issue in bugzillar (https://bugzilla.tianocore.org/) and attach this patch? We will add it into edk2 trunk. > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Tomas > Pilar

Re: [edk2] [PATCH 1/1] BaseTools/tools_def.template: revert to large code model for X64/GCC5/LTO

2017-08-10 Thread Shi, Steven
Hi Laszlo, I'm trying to reproduce your boot failure with OVMF in my Ubuntu system, but not succeed. My GCC was built from GCC main trunk code in 20170601, and my ld linker is version 2.28. Could you try the ld 2.28 with your gcc-7.1 and check whether it works in your side? Or, do you know

Re: [edk2] [Patch] BaseTools: Fix Segmentation fault: 11 when build AppPkg with XCODE5

2017-08-10 Thread Gao, Liming
Andrew: I get your point. I agree edk2 firmware to support old EFI image. And, I review the change in BasePeCoffLib. But, I think it is unnecessary. Below is the current logic. On the first loop, DebugEntry.Type is EFI_IMAGE_DEBUG_TYPE_CODEVIEW, then will return. It doesn't enter into second

Re: [edk2] [RFC Patch 3/3] BaseTools/Scripts: Add sample makefile for use with RunMakefile.py

2017-08-10 Thread Zhu, Yonghong
Hi Mike, The patch 3 has some "Tab character used". Please fix it when you commit. Others are good to me. Reviewed-by: Yonghong Zhu Best Regards, Zhu Yonghong -Original Message- From: Kinney, Michael D Sent: Friday, August 04, 2017 1:30 AM To:

[edk2] [PATCH] IntelSiliconPkg: Fix VS2015 NOOPT IA32 build failure in IntelVTdDxe

2017-08-10 Thread Star Zeng
There are VS2015 NOOPT IA32 build failure like below in IntelVTdDxe. XXX.lib(XXX.obj) : error LNK2001: unresolved external symbol __allshl XXX.lib(XXX.obj) : error LNK2001: unresolved external symbol __aullshr This patch is to update Vtd.h to use UINT32 instead of UINT64 for bitfields in

Re: [edk2] [PATCH 1/2] ShellPkg/driver: Show Image Name in non-SFO mode

2017-08-10 Thread Ard Biesheuvel
This patch is breaking the GCC build: ShellPkg/Library/UefiShellDriver1CommandsLib/Drivers.c: In function 'GetImageNameFromHandle': 220:25: error: passing argument 3 of 'gBS->HandleProtocol' from incompatible pointer type [-Werror] 220:25: note: expected 'void **' but argument is of type 'struct

Re: [edk2] [Patch 0/3] Support Ip4Config2/Ip6Config.SetData interface to clear specific configuration

2017-08-10 Thread Ye, Ting
Series Reviewed-by: Ye Ting -Original Message- From: Wu, Jiaxin Sent: Wednesday, July 26, 2017 2:28 PM To: edk2-devel@lists.01.org Cc: Ye, Ting ; Fu, Siyuan ; Wu, Jiaxin Subject: [Patch 0/3] Support

Re: [edk2] [Patch 2/2] QuarkPlatformPkg/Readme.md: edk2-non-osi directory layout

2017-08-10 Thread Leif Lindholm
On Wed, Aug 09, 2017 at 12:39:57PM -0700, Michael D Kinney wrote: > The following commit moved the QuarkSocBinPkg from the root > directory to the the Silicon/Intel directory. > > https://github.com/tianocore/edk2-non-osi/commit/182e85d04566800fe188de4b1c30a50533dd74b7 > > The following updates

[edk2] [Patch] BaseTools: Don't need to add extra quotes when UI string from file

2017-08-10 Thread Yonghong Zhu
From: Bin Wang when the UI string is read from files, we don't need to add the extra quotes. Otherwise, it will cause UI name has this extra quotes. Cc: Liming Gao Cc: Yonghong Zhu Contributed-under: TianoCore Contribution

Re: [edk2] [PATCH v1 00/14] OvmfPkg/Virtio: Add APIs to map system physical to device address

2017-08-10 Thread Laszlo Ersek
On 08/10/17 00:44, Brijesh Singh wrote: > > > On 08/09/2017 05:38 PM, Laszlo Ersek wrote: >> On 08/07/17 13:58, Brijesh Singh wrote: >>> Currently, virtio drivers provides the system physical address to the >>> device. >>> However, some systems may feature an IOMMU that requires the drivers >>>

[edk2] [patch] BaseTools/UPT: Support Multiple Installation

2017-08-10 Thread hesschen
Add a new feature to UPT to support installing multiple DIST packages in one time. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: hesschen --- .../Source/Python/UPT/Core/DependencyRules.py | 18 -

Re: [edk2] [PATCH 4/4] OvmfPkg: Introduce UDF_ENABLE build flag

2017-08-10 Thread Laszlo Ersek
On 08/09/17 19:51, Andrew Fish wrote: > >> On Aug 9, 2017, at 10:33 AM, Laszlo Ersek wrote: >> >> On 08/09/17 17:45, Andrew Fish wrote: >>> On Aug 9, 2017, at 2:44 AM, Laszlo Ersek wrote: CC Ard and Andrew On 08/08/17 21:31, Paulo

Re: [edk2] [PATCH 1/4] Platforms: Add Sanchip Zx296718 basic library

2017-08-10 Thread Laszlo Ersek
Thanks for the CC On 08/10/17 15:04, Leif Lindholm wrote: > You've reworked this to be targeted for edk2-platforms, which is > excellent! However: > - Please update subject line to match. > - Please also cc edk2-devel@lists.01.org (since you are with Linaro, > it makes sense to still cc

Re: [edk2] [PATCH 3/4] Platforms/zx: Add boot manager lib and entries

2017-08-10 Thread Leif Lindholm
On Wed, Aug 09, 2017 at 10:12:38PM +0800, Jun Nie wrote: > Add boot manager lib and entries, including Android and Grub. > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Jun Nie > --- > .../Drivers/Zx296718EvbDxe/Zx296718EvbDxe.c| 105

Re: [edk2] [Patch 0/2] Update Quark for edk2-non-osi changes

2017-08-10 Thread Steele, Kelly
Reviewed by: Kelly Steele -Original Message- From: Kinney, Michael D Sent: August 09, 2017 12:40 To: edk2-devel@lists.01.org Cc: Leif Lindholm ; Steele, Kelly ; Gao, Liming Subject: [Patch

Re: [edk2] [Patch 2/2] QuarkPlatformPkg/Readme.md: edk2-non-osi directory layout

2017-08-10 Thread Steele, Kelly
Reviewed by: Kelly Steele -Original Message- From: Kinney, Michael D Sent: August 09, 2017 12:40 To: edk2-devel@lists.01.org Cc: Leif Lindholm ; Steele, Kelly Subject: [Patch 2/2] QuarkPlatformPkg/Readme.md:

Re: [edk2] [Patch 1/2] QuarkSocPkg/MemoryInit: Remove use of memset()/memcpy()

2017-08-10 Thread Steele, Kelly
Reviewed by: Kelly Steele -Original Message- From: Kinney, Michael D Sent: August 09, 2017 12:40 To: edk2-devel@lists.01.org Cc: Steele, Kelly ; Gao, Liming Subject: [Patch 1/2] QuarkSocPkg/MemoryInit: Remove use of

Re: [edk2] Does a double Page free report "ConvertPages: Incompatible memory types", maybe we could do better.

2017-08-10 Thread Andrew Fish
Also I forgot to mention the double page free was in the DXE Core LoadImage() on an error path. CoreLoadPeImage() ... return EFI_SUCCESS; Done: // // Free memory. // if (DstBufAlocated) { CoreFreePages (Image->ImageContext.ImageAddress, Image->NumberOfPages); } ...

[edk2] [PATCH 1/3] Maintainers.txt: List Tianocore Stewards

2017-08-10 Thread Jordan Justen
Cc: Andrew Fish Cc: Leif Lindholm Cc: Michael D Kinney Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Jordan Justen --- Maintainers.txt | 6 ++ 1 file changed, 6

[edk2] [PATCH 3/3] Maintainers.txt: Add Package Reviewer role

2017-08-10 Thread Jordan Justen
Although everyone is encouraged to review patches and add their Reviewed-by reply for a patch, with the Package Reviewer role we identify additional community members that will be Cc'd for patches made to a package. A distinction between a Package Maintainer and Reviewer is that Maintainers will

[edk2] [PATCH 2/3] Maintainers.txt: Update Package Maintainer description

2017-08-10 Thread Jordan Justen
Add note that the Package Maintainer is responsible for reviewing and pushing package changes to source control. Cc: Andrew Fish Cc: Leif Lindholm Cc: Michael D Kinney Contributed-under: TianoCore Contribution Agreement 1.1

[edk2] [PATCH 1/1] StdLib/EfiSocketLib: Fix ABI mismatch for 2 event functions

2017-08-10 Thread Thomas Palmer
The gBS->CreateEvent expects a EFI_EVENT_NOTIFY function as the third argument. The EFIAPI token is an important component of that prototype. Its absence can cause unexpected issues on DEBUG systems built with GCC due to ABI mismatches. Both EslTcp4ConnectComplete and EslTcp6ConnectComplete did

Re: [edk2] [PATCH 1/1] StdLib/EfiSocketLib: Fix ABI mismatch for 2 event functions

2017-08-10 Thread Carsey, Jaben
Looks good to me. Daryl? > -Original Message- > From: Thomas Palmer [mailto:thomas.pal...@hpe.com] > Sent: Thursday, August 10, 2017 3:35 PM > To: edk2-devel@lists.01.org > Cc: edk2-li...@mc2research.org; Carsey, Jaben ; > joseph.shiffl...@hpe.com; Thomas Palmer

Re: [edk2] [PATCH 1/3] Maintainers.txt: List Tianocore Stewards

2017-08-10 Thread Leif Lindholm
I don't actually have a strong opinion that the number of maintainers per package need to be kept at 2, and I'm not sure we have actually codified that anywhere. But I also don't have any issues with any of the changes suggested in this series. So, in order to keep the noise down, for the series:

Re: [edk2] [PATCH 1/3] Maintainers.txt: List Tianocore Stewards

2017-08-10 Thread Kinney, Michael D
Series Reviewed-by: Michael D Kinney > -Original Message- > From: Justen, Jordan L > Sent: Thursday, August 10, 2017 3:11 PM > To: edk2-devel@lists.01.org > Cc: Justen, Jordan L ; Andrew Fish > ; Leif Lindholm

Re: [edk2] [PATCH 1/3] Maintainers.txt: List Tianocore Stewards

2017-08-10 Thread Andrew Fish
Series Reviewed-by: Andrew Fish > > On Aug 10, 2017, at 4:13 PM, Kinney, Michael D > wrote: > > Series > > Reviewed-by: Michael D Kinney > >> -Original Message- >> From: Justen, Jordan

Re: [edk2] [PATCH 0/4] read-only UDF file system support

2017-08-10 Thread Paulo Alcantara
Hi, (sorry for the late reply) On 09/08/2017 22:11, Ni, Ruiyu wrote: Regards, Ray -Original Message- From: Paulo Alcantara [mailto:pca...@zytor.com] Sent: Wednesday, August 9, 2017 10:01 PM To: Ni, Ruiyu ; Zeng, Star ;

Re: [edk2] [Patch] BaseTools: Fix Segmentation fault: 11 when build AppPkg with XCODE5

2017-08-10 Thread Andrew Fish
> On Aug 10, 2017, at 3:38 AM, Gao, Liming wrote: > > Andrew: > If this is a mtoc bug, I suggest to update GenFw to always correct it in the > generated EFI image. If so, the EFI image is always correct. There is no > change requirement in PeCoff library in MdePkg. >

Re: [edk2] [PATCH v2 1/1] OvmfPkg : QemuFwCfgLib: Use BusMasterCommandBuffer to map FW_CFG_DMA_ACCESS

2017-08-10 Thread Jordan Justen
On 2017-08-04 13:25:09, Laszlo Ersek wrote: > > > > I have another reason why going beyond 80 is not a good idea for code > > that doesn't apply to normal reading. If you need ~120 columns visible > > to view some lines, then most lines will end up having a lot of wasted > > horizontal whitespace

Re: [edk2] [PATCH v1 03/14] OvmfPkg/VirtioPciDeviceDxe: Implement new member functions

2017-08-10 Thread Brijesh Singh
Hi Laszlo, On 08/09/2017 12:09 PM, Laszlo Ersek wrote: [Snip] + +EFI_STATUS +EFIAPI +VirtioPciUnmapSharedBuffer ( + VIRTIO_DEVICE_PROTOCOL*This, + VOID *Mapping + ) +{ + return EFI_SUCCESS; +} (9) Please refresh the function signatures in both

Re: [edk2] [Patch V2 1/4] BaseTools: Fix Xcode 9 Beta treating 32-bit left shift as undefined

2017-08-10 Thread Gao, Liming
Reviewed-by: Liming Gao > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Yonghong Zhu > Sent: Thursday, August 3, 2017 5:00 PM > To: edk2-devel@lists.01.org > Cc: Kinney, Michael D ; Andrew

Re: [edk2] [Patch] BaseTools: Fix Segmentation fault: 11 when build AppPkg with XCODE5

2017-08-10 Thread Gao, Liming
Andrew: If this is a mtoc bug, I suggest to update GenFw to always correct it in the generated EFI image. If so, the EFI image is always correct. There is no change requirement in PeCoff library in MdePkg. Thanks Liming From: af...@apple.com [mailto:af...@apple.com] Sent: Tuesday, August 8,

Re: [edk2] [Patch V2 3/4] IntelFrameworkModulePkg: Fix Xcode 9 Beta treating 32-bit left shift as undefined

2017-08-10 Thread Gao, Liming
Reviewed-by: Liming Gao > -Original Message- > From: Zhu, Yonghong > Sent: Thursday, August 3, 2017 5:00 PM > To: edk2-devel@lists.01.org > Cc: Gao, Liming ; Kinney, Michael D > ; Fan, Jeff ;

[edk2] [Patch] BaseTools: Support TabSpace between section tag in DEC file

2017-08-10 Thread Yonghong Zhu
From: Yanyan Zhang Per DEC spec, multiple section tag use to separate, and it can support Tab, so this patch fix the bug to use Tab. ::= {} {} ::= * Cc: Liming Gao Cc: Yonghong Zhu Contributed-under: TianoCore

[edk2] [PATCH 0/2] DxeCore: Fix double free pages on LoadImage failure path

2017-08-10 Thread Star Zeng
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=667 REF: https://lists.01.org/pipermail/edk2-devel/2017-August/013112.html Star Zeng (2): MdeModulePkg DxeCore: Fix double free pages on LoadImage failure path MdeModulePkg DxeCore: Enhance "ConvertPages: Incompatible memory types"

[edk2] [PATCH 2/2] MdeModulePkg DxeCore: Enhance "ConvertPages: Incompatible memory types"

2017-08-10 Thread Star Zeng
When double free pages by FreePages() or allocate allocated pages by AllocatePages() with AllocateAddress type, the code will print debug message "ConvertPages: Incompatible memory types", but the debug message is not very obvious for the error paths by FreePages() or AllocatePages(). Refer

Re: [edk2] [PATCH v2 1/1] OvmfPkg : QemuFwCfgLib: Use BusMasterCommandBuffer to map FW_CFG_DMA_ACCESS

2017-08-10 Thread Laszlo Ersek
On 08/10/17 20:36, Jordan Justen wrote: > On 2017-08-04 13:25:09, Laszlo Ersek wrote: >>> >>> I have another reason why going beyond 80 is not a good idea for code >>> that doesn't apply to normal reading. If you need ~120 columns visible >>> to view some lines, then most lines will end up having

Re: [edk2] [PATCH v1 03/14] OvmfPkg/VirtioPciDeviceDxe: Implement new member functions

2017-08-10 Thread Laszlo Ersek
On 08/10/17 20:41, Brijesh Singh wrote: > Hi Laszlo, > > On 08/09/2017 12:09 PM, Laszlo Ersek wrote: > [Snip] > >> >>> + >>> +EFI_STATUS >>> +EFIAPI >>> +VirtioPciUnmapSharedBuffer ( >>> + VIRTIO_DEVICE_PROTOCOL*This, >>> + VOID *Mapping >>> + ) >>> +{ >>> + return

Re: [edk2] [PATCH v2] MdeModulePkg/EmmcDxe: Make sure no extra data is erased by EraseBlocks

2017-08-10 Thread Zeng, Star
Two minor comments, others are good to me. 1. Could the code use (LastLba + 1 - EndGroupLba) instead of (LastLba - EndGroupLba + 1)? 2. Could the code have the debug message "DEBUG ((EFI_D_ERROR, "EmmcEraseBlocks(): Lba 0x%x BlkNo 0x%x Event %p with %r\n", Lba, BlockNum, Token->Event,

[edk2] OVMF compile error

2017-08-10 Thread Chen, Farrah
Hi, When I build ovmf with commit: 76c6f69ccadc7835c9616b077d9ff1b8e46fe49e, the following error occurred: git clone https://github.com/tianocore/edk2.git cd edk2 OvmfPkg/build.sh -a X64

Re: [edk2] [PATCH] IntelSiliconPkg: Fix VS2015 NOOPT IA32 build failure in IntelVTdDxe

2017-08-10 Thread Yao, Jiewen
Reviewed-by: jiewen@intel.com > -Original Message- > From: Zeng, Star > Sent: Thursday, August 10, 2017 3:18 PM > To: edk2-devel@lists.01.org > Cc: Zeng, Star ; Yao, Jiewen > Subject: [PATCH] IntelSiliconPkg: Fix VS2015 NOOPT IA32 build

[edk2] [Patch v2] NetworkPkg/HttpDxe: Handle the HttpVersionUnsupported in the HttpConfigData

2017-08-10 Thread Jiaxin Wu
v2: * Refine the patch by changing the '==' to '>='. Cc: Ye Ting Cc: Jin Eric Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Wu Jiaxin --- NetworkPkg/HttpDxe/HttpImpl.c | 4 1 file changed, 4

[edk2] [PATCH V2] Maintainers.txt: Change maintainer for Intel*Pkg.

2017-08-10 Thread Jiewen Yao
Since Giri left Intel, we change 3 Intel*Pkg maintainer. Cc: Chasel Chiu Cc: Amy Chan Cc: Rangasai V Chaganty Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jiewen Yao ---

[edk2] [PATCH 0/1] BaseTools/tools_def.template: revert to large code model for X64/GCC5/LTO

2017-08-10 Thread Laszlo Ersek
Patch #1 explains it all (grab a coffee first). For testing, it's likely most robust to run git clean -fdx exit in your terminal, before fetching & checking out my branch, and rebuilding OVMF. Repo: https://github.com/lersek/edk2.git Branch: gcc7_lto Cc: Alex Williamson

Re: [edk2] [Patch] BaseTools: Don't need to add extra quotes when UI string from file

2017-08-10 Thread Zhu, Yonghong
Reviewed-by: Yonghong Zhu Best Regards, Zhu Yonghong -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Yonghong Zhu Sent: Thursday, August 10, 2017 4:58 PM To: edk2-devel@lists.01.org Cc: Wang, BinX A

Re: [edk2] [Patch] NetworkPkg/HttpBootDxe: Refine the coding style.

2017-08-10 Thread Ye, Ting
Reviewed-by: Ye Ting -Original Message- From: Wu, Jiaxin Sent: Tuesday, August 08, 2017 3:25 PM To: edk2-devel@lists.01.org Cc: Ye, Ting ; Bi, Dandan ; Wu, Jiaxin Subject: [Patch] NetworkPkg/HttpBootDxe:

Re: [edk2] [PATCH v6 2/2] EmbeddedPkg/AndroidBoot: boot android kernel from storage

2017-08-10 Thread Leif Lindholm
On Wed, Aug 09, 2017 at 06:39:23PM +0800, Jun Nie wrote: > Add an android kernel loader that could load kernel from storage > device. > This android boot image BDS add addtitional cmdline/dtb/ramfs > support besides kernel that is introduced by Android boot header. > > This patch is derived from

Re: [edk2] [PATCH 1/2] ShellPkg/driver: Show Image Name in non-SFO mode

2017-08-10 Thread Leif Lindholm
At a quick glance, the fix should probably be diff --git a/ShellPkg/Library/UefiShellDriver1CommandsLib/Drivers.c b/ShellPkg/Library/UefiShellDriver1CommandsLib/Drivers.c index 4d876bb108..26b785c563 100644 --- a/ShellPkg/Library/UefiShellDriver1CommandsLib/Drivers.c +++

Re: [edk2] Does a double Page free report "ConvertPages: Incompatible memory types", maybe we could do better.

2017-08-10 Thread Laszlo Ersek
On 08/10/17 03:03, Andrew Fish wrote: > It looks to me like if you Free pages, after you free pages you hit this > DEBUG message. > > > https://github.com/tianocore/edk2/blob/master/MdeModulePkg/Core/Dxe/Mem/Page.c#L790 > > if (!(NewType == EfiConventionalMemory ? 1 : 0) ^ (Entry->Type

Re: [edk2] [Patch V2 4/4] MdePkg: Fix Xcode 9 Beta treating 32-bit left shift as undefined

2017-08-10 Thread Gao, Liming
Reviewed-by: Liming Gao > -Original Message- > From: Zhu, Yonghong > Sent: Thursday, August 3, 2017 5:00 PM > To: edk2-devel@lists.01.org > Cc: Gao, Liming ; Kinney, Michael D > ; Andrew Fish >

Re: [edk2] [PATCH v3 1/2] MdePkg/BaseLib: Add IsNodeInList() function.

2017-08-10 Thread Gao, Liming
Marvin: I have no other comments. Reviewed-by: Liming Gao Thanks Liming > -Original Message- > From: Marvin Häuser [mailto:marvin.haeu...@outlook.com] > Sent: Friday, August 4, 2017 3:52 AM > To: edk2-devel@lists.01.org > Cc: Kinney, Michael D

Re: [edk2] [PATCH 1/4] Platforms: Add Sanchip Zx296718 basic library

2017-08-10 Thread Leif Lindholm
You've reworked this to be targeted for edk2-platforms, which is excellent! However: - Please update subject line to match. - Please also cc edk2-devel@lists.01.org (since you are with Linaro, it makes sense to still cc linaro-uefi) To get the appropriate subject line for edk2-devel, as

Re: [edk2] [PATCH v2] MdeModulePkg/EmmcDxe: Make sure no extra data is erased by EraseBlocks

2017-08-10 Thread Wu, Hao A
> -Original Message- > From: Zeng, Star > Sent: Friday, August 11, 2017 10:37 AM > To: Wu, Hao A; edk2-devel@lists.01.org > Cc: Wu, Hao A; Ni, Ruiyu; Zeng, Star > Subject: RE: [edk2] [PATCH v2] MdeModulePkg/EmmcDxe: Make sure no extra > data is erased by EraseBlocks > > Two minor

Re: [edk2] [Patch] BaseTools: Fix Segmentation fault: 11 when build AppPkg with XCODE5

2017-08-10 Thread Gao, Liming
Andrew: Edk2 Build system calls GenFw to generate EFI image in build phase. Even if this image is not built into BIOS image, its EFI image will be generated by GenFw. So, only if this EFI image is built from EDK2 project, it can be updated by GenFw tool. You can see this step in

Re: [edk2] [PATCH v3] MdeModulePkg/EmmcDxe: Make sure no extra data is erased by EraseBlocks

2017-08-10 Thread Zeng, Star
Reviewed-by: Star Zeng -Original Message- From: Wu, Hao A Sent: Friday, August 11, 2017 12:38 PM To: edk2-devel@lists.01.org Cc: Wu, Hao A ; Zeng, Star ; Ni, Ruiyu Subject: [PATCH v3]

Re: [edk2] [Patch] BaseTools: Fix Segmentation fault: 11 when build AppPkg with XCODE5

2017-08-10 Thread Andrew Fish
> On Aug 10, 2017, at 9:48 PM, Gao, Liming wrote: > > Andrew: > Edk2 Build system calls GenFw to generate EFI image in build phase. Even if > this image is not built into BIOS image, its EFI image will be generated by > GenFw. So, only if this EFI image is built from