Re: [edk2] [Patch] BaseTools: PatchCheck to align with wiki and report error for EFI_D_*

2016-10-16 Thread Zhu, Yonghong
Thanks Jordan. I will separate the patches and send out the updated version. Besides can you help to update the wiki ? Best Regards, Zhu Yonghong -Original Message- From: Justen, Jordan L Sent: Sunday, October 16, 2016 4:14 AM To: Zhu, Yonghong ;

Re: [edk2] [PATCH 0/5] OvmfPkg/QemuVideoDxe: cleanups and tweaks

2016-10-16 Thread Laszlo Ersek
On 10/15/16 22:25, Jordan Justen wrote: > Series Reviewed-by: Jordan Justen Commit range 5b54c92a6537..4dd8787a20e2. Thank you! Laszlo > On 2016-10-12 03:09:00, Laszlo Ersek wrote: >> Reviewing (part of) Ray's recent FrameBufferBltLib changes gave me the >> idea for

Re: [edk2] [PATCH] GPT Shell Application/Library

2016-10-16 Thread Laszlo Ersek
On 10/16/16 07:23, Vladimir Olovyannikov wrote: > This allows managing (create, delete, modify, fat format) of GPT > partitions from within UEFI Shell. > Syntax: > gpt [device_mapped_name] [parameters...] > See usage examples in the .uni file > --- >

Re: [edk2] [PATCH 0/2] Fix typo 'Ihis' with 'This' in codes

2016-10-16 Thread Yao, Jiewen
Reviewed-by: jiewen@intel.com > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Hao Wu > Sent: Monday, October 17, 2016 9:33 AM > To: edk2-devel@lists.01.org > Cc: Wu, Hao A > Subject: [edk2] [PATCH 0/2] Fix typo

Re: [edk2] [PATCH 1/1] MdePkg/IoLib: Add support for big-endian MMIO

2016-10-16 Thread Gao, Liming
Bhupesh: In this patch, five class APIs are new added. They are MmioReadBe16(),MmioWriteBe16(),MmioClearSetBe16(),MmioSetBitsBe16(),MmioClearBitsBe16(). In fact, they can map to the existing MMIO APIs. Below is their mapping. And, I understand some hardware uses little-endian MMIO interfaces,

[edk2] [PATCH 2/2] SecurityPkg: Fix typo 'Ihis' with 'This' in codes

2016-10-16 Thread Hao Wu
Cc: Jiewen Yao Cc: Chao Zhang Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu --- SecurityPkg/Include/Library/HashLib.h | 4 ++--

[edk2] [PATCH 0/2] Fix typo 'Ihis' with 'This' in codes

2016-10-16 Thread Hao Wu
Hao Wu (2): MdePkg/TrEEProtocol.h: Fix typo 'Ihis' with 'This' in comment SecurityPkg: Fix typo 'Ihis' with 'This' in codes MdePkg/Include/Protocol/TrEEProtocol.h | 4 ++-- SecurityPkg/Include/Library/HashLib.h | 4 ++--

[edk2] [PATCH 1/2] MdePkg/TrEEProtocol.h: Fix typo 'Ihis' with 'This' in comment

2016-10-16 Thread Hao Wu
Cc: Jiewen Yao Cc: Liming Gao Cc: Michael Kinney Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu --- MdePkg/Include/Protocol/TrEEProtocol.h | 4 ++-- 1 file changed, 2

Re: [edk2] [PATCH 1/1] MdePkg/IoLib: Add support for big-endian MMIO

2016-10-16 Thread Kinney, Michael D
Bhupesh, It is also possible to add an ARM specific PCD to select endianness and update MdePkg/Library/BaseIoLibIntrinsic/IoLibArm.c to use that PCD in MmioRead/Write() APIs in that file to support both endian types. You can use the SwapBytesxx() functions from BaseLib(as Laszlo suggested)

Re: [edk2] [Patch] BaseTools: Update sign tool to make MonotonicCount *after* Payload

2016-10-16 Thread Gao, Liming
Reviewed-by: Liming Gao > -Original Message- > From: Yao, Jiewen > Sent: Friday, October 14, 2016 9:11 PM > To: Zhu, Yonghong ; edk2-devel@lists.01.org > Cc: Gao, Liming > Subject: RE: [edk2] [Patch] BaseTools: Update

[edk2] [PATCH] ArmPkg ArmVirtPkg: fix the GIC base address variables as 64-bit

2016-10-16 Thread Dennis Chen
Since ACPI spec defines the GIC base addresses (CPU interface, Distributor and Redistributor*GICv3 only*) as 64-bit, so we should define these corresponding base address variables as 64-bit instead of 32-bit. This patch redefines them according to the ACPI spec. Contributed-under: TianoCore

Re: [edk2] [Patch] BaseTools: PatchCheck to align with wiki and report error for EFI_D_*

2016-10-16 Thread Jordan Justen
On 2016-10-16 06:32:32, Zhu, Yonghong wrote: > Thanks Jordan. I will separate the patches and send out the updated version. > > Besides can you help to update the wiki ? > How does it look? https://github.com/tianocore/tianocore.github.io/wiki/Commit-Message-Format -Jordan > >

Re: [edk2] [Patch] BaseTools: PatchCheck to align with wiki and report error for EFI_D_*

2016-10-16 Thread Jordan Justen
On 2016-10-16 06:32:32, Zhu, Yonghong wrote: > Thanks Jordan. I will separate the patches and send out the updated version. > One other suggestion for the commit message on your patch. Can you add this at the bottom before the Contributed-under? Fixes: