[edk2] [PATCH 10/11] PcAtChipsetPkg: Remove KbcResetDxe

2016-08-09 Thread Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Feng Tian Cc: Amy Chan --- PcAtChipsetPkg/KbcResetDxe/KbcReset.uni | 27 --- PcAtChipsetPkg/KbcResetDxe/KbcResetExtra.uni | 25 ---

[edk2] [PATCH 01/11] MdePkg/UefiSpec.h: Align function header of ResetSystem to UEFI Spec

2016-08-09 Thread Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Liming Gao --- MdePkg/Include/Uefi/UefiSpec.h | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/MdePkg/Include/Uefi/UefiSpec.h

[edk2] [PATCH 08/11] Vlv2TbltDevicePkg/ResetSystemLib: Implement ResetPlatformSpecific

2016-08-09 Thread Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: David Wei --- .../Library/ResetSystemLib/ResetSystemLib.c| 23 -- 1 file changed, 21 insertions(+), 2 deletions(-) diff --git

[edk2] [PATCH 06/11] PcAtChipsetPkg/ResetSystemLib: Implement ResetPlatformSpecific

2016-08-09 Thread Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Feng Tian Cc: Amy Chan --- .../Library/ResetSystemLib/ResetSystemLib.c | 21 + 1 file changed, 21 insertions(+) diff

[edk2] [PATCH 04/11] CorebootPayloadPkg/ResetSystemLib: Implement ResetPlatformSpecific

2016-08-09 Thread Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Maurice Ma Cc: Prince Agyeman --- .../Library/ResetSystemLib/ResetSystemLib.c| 23 +- 1 file changed, 22

[edk2] [PATCH 03/11] MdeModulePkg/BaseResetSystemLibNull: Implement ResetPlatformSpecific

2016-08-09 Thread Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Feng Tian Cc: Amy Chan --- .../BaseResetSystemLibNull.c | 23 +- 1 file changed, 22 insertions(+), 1

[edk2] [PATCH 05/11] OvmfPkg/ResetSystemLib: Implement ResetPlatformSpecific

2016-08-09 Thread Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Jordan Justen Cc: Laszlo Ersek --- OvmfPkg/Library/ResetSystemLib/ResetSystemLib.c | 23 ++- 1 file changed, 22

[edk2] [PATCH 09/11] OvmfPkg: Use MdeModulePkg/ResetSystemRuntimeDxe

2016-08-09 Thread Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Jordan Justen Cc: Laszlo Ersek --- OvmfPkg/OvmfPkgIa32.dsc| 2 +- OvmfPkg/OvmfPkgIa32.fdf| 2 +- OvmfPkg/OvmfPkgIa32X64.dsc | 2 +-

[edk2] [PATCH 02/11] MdeModulePkg: Add API ResetPlatformSpecific() to ResetSystemLib.h

2016-08-09 Thread Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Feng Tian Cc: Amy Chan --- MdeModulePkg/Include/Library/ResetSystemLib.h | 22 -- 1 file changed, 20 insertions(+), 2

[edk2] [PATCH 11/11] MdeModulePkg/ResetSystemRuntimeDxe: Support EfiResetPlatformSpecific

2016-08-09 Thread Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Feng Tian Cc: Amy Chan --- MdeModulePkg/Universal/ResetSystemRuntimeDxe/ResetSystem.c | 14 +++--- 1 file changed, 11 insertions(+), 3

[edk2] [PATCH 00/11] Add ResetPlatformSpecific() to ResetSystemLib

2016-08-09 Thread Ruiyu Ni
The patches add ResetPlatformSpecific() to ResetSystemLib library class and all library instances. The KbcResetDxe driver is removed from PcAtChipsetPkg. Ruiyu Ni (11): MdePkg/UefiSpec.h: Align function header of ResetSystem to UEFI Spec MdeModulePkg: Add API ResetPlatformSpecific() to

[edk2] [PATCH 07/11] QuarkSocPkg/ResetSystemLib: Implement ResetPlatformSpecific

2016-08-09 Thread Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Michael D Kinney Cc: Kelly Steele --- .../Library/ResetSystemLib/ResetSystemLib.c| 22 +- 1 file changed, 21

Re: [edk2] SATA AHCI issue

2016-08-09 Thread Tian, Feng
PRD entries are organized like AHCI spec Figure 13 said. Thanks Feng -Original Message- From: Shaveta Leekha [mailto:shaveta.lee...@nxp.com] Sent: Tuesday, August 9, 2016 7:40 PM To: Tian, Feng ; Linaro UEFI Mailman List ;

Re: [edk2] [PATCH v5 2/4] BaseTools-Conf:Introduce CLANG38 new toolchain for x86

2016-08-09 Thread Gao, Liming
Reviewed-by: Liming Gao > -Original Message- > From: Shi, Steven > Sent: Wednesday, August 03, 2016 5:43 PM > To: edk2-devel@lists.01.org; Gao, Liming ; Carsey, > Jaben > Cc: af...@apple.com; Kinney, Michael D

[edk2] [patch] MdeModulePkg/Browser: Enhance the logic when getting value from AltResp

2016-08-09 Thread Dandan Bi
This patch is to enhance SetupBrowser to handle following two cases: 1. When searching BlockName in AltResp, the hex digits of related BlockName in AltResp may be in uppercase. 2. When converting the Value in AltResp to HiiValue, the length of value string is bigger than the length of

Re: [edk2] How do you debug Package errors upon Bootup ?

2016-08-09 Thread Andrew Fish
> On Aug 9, 2016, at 9:03 AM, Shubha Ramani wrote: > > Stuff like this: > ERROR: C4002:V03051002 I06D33944A-EC75-4855-A54D-809C75241F6C > These look like report status codes.

Re: [edk2] Is there something like "nm" or "obdump" in Linux ?

2016-08-09 Thread Andrew Fish
> On Aug 9, 2016, at 8:48 AM, Shubha Ramani wrote: > > The reason being, I'd like to see the actual compiled and linked addresses of > certain Boot Services functions (like Timer interrupt handling). In general with the edk2 you can only get offsets from the linked

[edk2] How do you debug Package errors upon Bootup ?

2016-08-09 Thread Shubha Ramani
Stuff like this: ERROR: C4002:V03051002 I06D33944A-EC75-4855-A54D-809C75241F6C ERROR: C4002:V03051002 I06D33944A-EC75-4855-A54D-809C75241F6C ERROR: C4002:V03051002 I06D33944A-EC75-4855-A54D-809C75241F6C ERROR: C4002:V03051002 I06D33944A-EC75-4855-A54D-809C75241F6C ERROR:

[edk2] Is there something like "nm" or "obdump" in Linux ?

2016-08-09 Thread Shubha Ramani
The reason being, I'd like to see the actual compiled and linked addresses of certain Boot Services functions (like Timer interrupt handling). Thanks, Shubha Shubha D. ramanishubharam...@gmail.com shubharam...@yahoo.com ___ edk2-devel mailing list

Re: [edk2] [PATCH] Update ReadMe.MD file.

2016-08-09 Thread Mudusuru, Giri P
Hi Mang, Please fix non ASCII characters "Run the Intel� FSP installer " to "Run the Intel(R) FSP installer" Reviewed-by: Giri P Mudusuru Thanks, -Giri > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Guo, >

Re: [edk2] [PATCH v2 1/6] ShellPkg: TAB logic incorrectly chops out fs0: when typing fs0:

2016-08-09 Thread Shah, Tapan
Series Reviewed By: Tapan Shah -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Ruiyu Ni Sent: Tuesday, August 09, 2016 12:44 AM To: edk2-devel@lists.01.org Cc: Jaben Carsey Subject: [edk2] [PATCH v2

Re: [edk2] [MdeModulePkg] SetVirtualAddressMap() crashed due to DxeReportStatusCodeLib assuming the state of the BootService Memory at runtime.

2016-08-09 Thread Andrew Fish
> On Aug 8, 2016, at 7:26 PM, Zeng, Star wrote: > > On 2016/8/9 10:07, Andrew Fish wrote: >> >>> On Aug 8, 2016, at 6:21 PM, Zeng, Star wrote: >>> >>> Andrew, >>> >>> Should MdeModulePkg/Library/RuntimeDxeReportStatusCodeLib be used for your >>>

Re: [edk2] [PATCH] MdeModulePkg/EbcDxe: add ARM support

2016-08-09 Thread Ard Biesheuvel
On 9 August 2016 at 16:19, Ard Biesheuvel wrote: > This is a port of the recently proposed AARCH64 implementation of > the EBC runtime to ARM. It is a proof of concept: it runs > HelloWorld.efi, but it is likely that there are issues with > function argument marshalling

Re: [edk2] Toolchain question

2016-08-09 Thread Ard Biesheuvel
On 9 August 2016 at 12:02, Leif Lindholm wrote: > On Tue, Aug 09, 2016 at 11:02:55AM +0200, Michael Zimmermann wrote: >> > I use both, and never notice any difference. The primary differences >> > are newlib vs glibc, and in some cases, whether symbols are decorated >> >

Re: [edk2] Toolchain question

2016-08-09 Thread Leif Lindholm
On Tue, Aug 09, 2016 at 11:02:55AM +0200, Michael Zimmermann wrote: > > I use both, and never notice any difference. The primary differences > > are newlib vs glibc, and in some cases, whether symbols are decorated > > with a leading _ > > The actual code generation is more dependent on the

Re: [edk2] [PATCH] ArmVirtPkg: Add NFIT report feature for ArmVirtPkg for ramdisks

2016-08-09 Thread Sajjan, Vikas C
Adding Maintainers. -Original Message- From: Sajjan, Vikas C Sent: Tuesday, August 09, 2016 1:28 PM To: edk2-devel@lists.01.org Cc: Sajjan, Vikas C Subject: [PATCH] ArmVirtPkg: Add NFIT report feature for ArmVirtPkg for ramdisks Adds NFIT report feature for

Re: [edk2] Toolchain question

2016-08-09 Thread Michael Zimmermann
> I use both, and never notice any difference. The primary differences > are newlib vs glibc, and in some cases, whether symbols are decorated > with a leading _ > The actual code generation is more dependent on the default target > (i.e., -march/-mthumb for ARM) than bare-metal/hosted. > I only

Re: [edk2] Toolchain question

2016-08-09 Thread Leif Lindholm
On Tue, Aug 09, 2016 at 06:07:36AM +0200, Michael Zimmermann wrote: > As for the kind of toolchain to use: there's always (at least) two types: > bare metal toolchains(arm-eabi,arm-none-eabi, aarch64-elf), and the ones > with a OS ABI(androideabi, linux-gnueabi,linux-gnueabihf, ...). While >

Re: [edk2] [patch] MdeModulePkg/AtaAtapiPassThru: update AtaStatusBlock after cmd exec

2016-08-09 Thread Zeng, Star
Reviewed-by: Star Zeng -Original Message- From: Tian, Feng Sent: Tuesday, August 9, 2016 3:42 PM To: Zeng, Star Cc: edk2-devel@lists.01.org Subject: [patch] MdeModulePkg/AtaAtapiPassThru: update AtaStatusBlock after cmd exec

Re: [edk2] [PATCH 1/2] StdLib/LibC: avoid LTO code for compiler intrinsics

2016-08-09 Thread Ard Biesheuvel
On 8 August 2016 at 16:56, Carsey, Jaben wrote: > Reviewed-by: Jaben Carsey > Pushed as 78d706e23512 StdLib/LibC: avoid LTO code for compiler intrinsics 1fbd0ca16a99 StdLib/LibC ARM AARCH64: do not redefine compiler intrinsics Thanks all >>

Re: [edk2] SATA AHCI issue

2016-08-09 Thread Tian, Feng
Hi, Shaveta I saw you are using TFTP cmd to do the test? Did you ever test copying file from usb stick to the SATA HDD? I just copied a 70M file from usb to SATA and from SATA to SATA. It works well. Does this issue happen at all of your HDDs or only one specific brand model? Do you have

[edk2] [PATCH] ArmVirtPkg: Add NFIT report feature for ArmVirtPkg for ramdisks

2016-08-09 Thread Vikas C Sajjan
Adds NFIT report feature for ArmVirtPkg for ramdisks of reserved memory type. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Vikas C Sajjan --- ArmVirtPkg/ArmVirtQemu.dsc | 4 ArmVirtPkg/ArmVirtQemu.fdf | 2 ++

Re: [edk2] SATA AHCI issue

2016-08-09 Thread Shaveta Leekha
Hi Feng, Complete stack is: MdeModulePkg/Bus/Ata/AtaAtapiPassThru/AtaAtapiPassThru.inf MdeModulePkg/Bus/Ata/AtaBusDxe/AtaBusDxe.inf MdeModulePkg/Universal/Disk/DiskIoDxe/DiskIoDxe.inf MdeModulePkg/Universal/Disk/PartitionDxe/PartitionDxe.inf FatPkg/FatPei/FatPei.inf

Re: [edk2] [PATCH 0/2] BaseTools ARM/AARCH64: improve XIP support

2016-08-09 Thread Ard Biesheuvel
On 9 August 2016 at 06:01, Gao, Liming wrote: > Reviewed-by: Liming Gao > Thanks Pushed as 3cdbd7528bd4 BaseTools CLANG35: add missing XIP flags for AARCH64 0f73cca02b64 BaseTools ARM: impose strict alignment only for XIP modules >> -Original

[edk2] [patch] MdeModulePkg/AtaAtapiPassThru: update AtaStatusBlock after cmd exec

2016-08-09 Thread Feng Tian
AhciDumpPortStatus doesn't fully populate all the fields of AtaStatusBlock after completing command execution, which may bring issue if someone depends on the return status. Cc: Star Zeng Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Feng Tian

Re: [edk2] SATA AHCI issue

2016-08-09 Thread Shaveta Leekha
Hi Feng, Yes for 30MB file, multiple PRD entries would have been created. But I able to write file only upto 4MB size. The protocol stack I am using is: Generic AtaBusDxe driver use EfiAtaPassThruProtocol Generic AtaAtapiPassThru driver consumes "EfiIdeControllerInitProtocol" and produces

[edk2] [PATCH] Update ReadMe.MD file.

2016-08-09 Thread Guo, Mang
Update ReadMe.MD file because code structure of BraswellPlatformPkg was changed. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Guo Mang --- ReadMe.MD | 30 -- 1 file changed, 24 insertions(+), 6 deletions(-) diff --git

Re: [edk2] [PATCH] Update ReadMe.MD file.

2016-08-09 Thread Wei, David
Hi Mang, Below two sentences are not much clear. Could you rewrite them? + BraswellPlatformPkg hash is f71cf3bf29aa216bc72f703815a7f96fb2ca0592(2016-08-05 release): + https://firmware.intel.com/sites/default/files/BraswellPlatformPkg_2016-08-05.zip + BraswellPlatformPkg hash is

Re: [edk2] Toolchain question

2016-08-09 Thread Ard Biesheuvel
On 9 August 2016 at 06:07, Michael Zimmermann wrote: > Hi, > > I assume that you are using linux because you're talking about ELF files. > MinGW may be able to compile PE images in Linux but tbh there's no reason > for that and the "freaking" translation is actually a

[edk2] [PATCH] Update ReadMe.MD file.

2016-08-09 Thread Guo, Mang
Update ReadMe.MD file because code structure of BraswellPlatformPkg was changed. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Guo Mang --- ReadMe.MD | 28 ++-- 1 file changed, 22 insertions(+), 6 deletions(-) diff --git