Re: [edk2] [PATCH] MdeModulePkg: Fixed compiler warning 'unreachable code'

2012-09-27 Thread Tian, Feng
Thanks for your effort. I will check it in after review pass. From: Olivier Martin [mailto:olivier.mar...@arm.com] Sent: Thursday, September 27, 2012 21:51 To: Tian, Feng Cc: edk2-devel@lists.sourceforge.net Subject: [PATCH] MdeModulePkg: Fixed compiler warning 'unreachable code' Dear MdeModuleP

[edk2] UEFI 2.3.1 / Windows 8

2012-09-27 Thread Ian Noone
Sorry in advance if this is not the correct forum. Does Windows 8 REQUIRE UEFI 2.3.1 to get the logo? I understand it will support EFI but is passing SCT 2.3.1 a requirement? I cannot seem to find tidbit anywhere. TIA!

Re: [edk2] Questions on EDKII Network stack

2012-09-27 Thread Tao Yuan
Hi Ting, Thanks a lot! Best regards, Tao On Wed, Sep 26, 2012 at 11:04 PM, Ye, Ting wrote: > See below. > > ** ** > > *From:* Tao Yuan [mailto:yuanta...@gmail.com] > *Sent:* Thursday, September 27, 2012 10:07 AM > *To:* edk2-devel@lists.sourceforge.net > *Subject:* Re: [edk2] Questions

[edk2] FW: [PATCH] Add ARM Binaries to the EDK2 repository

2012-09-27 Thread Olivier Martin
Dear EDK2 development community, I realized a long time after sending the email below to the FatBinPkg/ShellBinPkg and EdkBinPkg packages I forgot to 'cc' the mailing list. Ray and Jaben have accepted the ARM Binaries in the packages they maintain. For this first drop, I have picked the latest ver

[edk2] [PATCH 0/4] OvmfPkg: virtio-blk driver

2012-09-27 Thread Laszlo Ersek
The first patch adds the driver. The rest enables QemuBootOrder to recognize virtio-blk devices. Tested as follows: (1) Booted the Fedora 18 Alpha Live CD (XFCE flavor) with OVMF under RHEL-6.3 qemu-kvm. The ISO image was accessed as an IDE CD-ROM. The target disk was virtio-blk, but in t

[edk2] [PATCH 2/4] OvmfPkg: QemuBootOrder: whitespace fix

2012-09-27 Thread Laszlo Ersek
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Laszlo Ersek --- OvmfPkg/Library/PlatformBdsLib/QemuBootOrder.c |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/OvmfPkg/Library/PlatformBdsLib/QemuBootOrder.c b/OvmfPkg/Library/PlatformBdsLib/Qe

[edk2] [PATCH 4/4] OvmfPkg: QemuBootOrder: recognize virtio-blk OFW device paths

2012-09-27 Thread Laszlo Ersek
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Laszlo Ersek --- OvmfPkg/Library/PlatformBdsLib/QemuBootOrder.c | 26 1 files changed, 26 insertions(+), 0 deletions(-) diff --git a/OvmfPkg/Library/PlatformBdsLib/QemuBootOrder.c b/OvmfPkg/Librar

[edk2] [PATCH 3/4] OvmfPkg: QemuBootOrder: lower the required number of OFW nodes to 2

2012-09-27 Thread Laszlo Ersek
2 nodes in an OpenFirmware device path are sufficient for the generic check at the beginning of TranslateOfwNodes(). The driver specific branches check for the necessary nodes individually. The number of nodes saved for examination is unchanged. Contributed-under: TianoCore Contribution Agreement

Re: [edk2] [PATCH] MdeModulePkg: Fixed compiler warning 'unreachable code'

2012-09-27 Thread Gao, Liming
The patch is good. Reviewed-by: Liming Gao From: Olivier Martin [mailto:olivier.mar...@arm.com] Sent: Thursday, September 27, 2012 9:51 PM To: Tian, Feng Cc: edk2-devel@lists.sourceforge.net Subject: [edk2] [PATCH] MdeModulePkg: Fixed compiler warning 'unreachable code' Dear MdeModulePkg maintai

[edk2] [PATCH] MdeModulePkg: Fixed compiler warning 'unreachable code'

2012-09-27 Thread Olivier Martin
Dear MdeModulePkg maintainer, Please find the attached patch that removes some trivial 'unreachable code' from MdeModulePkg. Some compilers complain when an unreachable code is detected in the code. This patch removes the unreachable 'break' instruction after a 'return' instruction in the switch(

Re: [edk2] Mtftp6->Configure() might cause dangling Udp6Io.

2012-09-27 Thread Ye, Ting
Hi Mars, We already noticed this issue and now working on patch check-in process. Thanks for reporting. Best regards, Ting From: Mars Lin [mailto:mars_...@phoenix.com] Sent: Thursday, September 27, 2012 4:03 PM To: edk2-devel@lists.sourceforge.net Subject: [edk2] Mtftp6->Configure() might cause

[edk2] Mtftp6->Configure() might cause dangling Udp6Io.

2012-09-27 Thread Mars Lin
Hi Edk2 network developers, Would like to bring up a potential issue here for current Mtftp6 implement when its comsumer, PxeBc for example, use configure() for different operations. In case of Mtftp6->Configure (Mtftp6, Config) and Mtftp6->Configure (Mtftp6, NULL) been used in pair for specific