Re: [edk2] [RFC V2] Proposal to organize packages into directories

2016-05-28 Thread Bhupesh Sharma
Hi Mike, This looks really nice. I have a couple of comments with regards to the "Driver" directory. This is the proposed structure: > Driver > EmbeddedPkg > FatPkg > NetworkPkg > OptionRomPkg > Vendor Are there any plans to make it eventually look something like (I unders

Re: [edk2] Shell version 2.2

2016-08-05 Thread Bhupesh Sharma
Unfortunately the 'ver' command on the shell also shows up a 2.1 shell version with the latest edk2/master. So, we think it is rather a bug. Regards, Bhupesh > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Tim Lewis > Sent: Saturday, Augu

[edk2] Persistent Variable Storage and PlatformBootTimeOut

2016-09-06 Thread Bhupesh Sharma
Hi EDK2 experts, We have recently added the support for storing UEFI run-time variables on underlying NOR Flash slave on our ARMV8 NXP board. We are successfully able to change the values of variables like boot-order using the latest MdeModulePkg BDS: MdeModulePkg/Universal/BdsDxe/BdsD

[edk2] [PATCH 1/1] AcpiTool: Fix minor casing and file path issues

2017-01-12 Thread Bhupesh Sharma
: - Compiled for AARCH64 architecture using GCC4.9 toolchain from Linaro. - Exercised the 'DumpACPI.efi' thus generated on Qemu for AARCH64 as an application. - Able to dump all relevant ACPI tables. This patch is also available here for easy access: https://github.com/bhupesh-sharma/Edki

Re: [edk2] [PATCH] ShellPkg: Add acpiview tool to dump ACPI tables

2017-01-13 Thread Bhupesh SHARMA
Hi All, On Fri, Dec 23, 2016 at 7:01 AM, Yao, Jiewen wrote: > HI Even > Thank you for the response. > > *Mike Kinney* is managing staging tree – he may help answer the question on > staging. ☺ > > I am glad to co-work with you on that. > > Merry Christmas! > > > Thank you > Yao Jiewen > > From:

[edk2] [PATCH 1/1] ArmVirtPkg/ArmVirtQemu: Install BGRT ACPI table

2017-01-16 Thread Bhupesh Sharma
While debugging OS for ACPI BGRT support (especially on VMs), it is very useful to have the EFI firmware to export the ACPI BGRT table. This patch tries to add this support in ArmVirtPkg. Cc: Ard Biesheuvel Cc: Laszlo Ersek Signed-off-by: Bhupesh Sharma Contributed-under: TianoCore

Re: [edk2] [PATCH 1/1] AcpiTool: Fix minor casing and file path issues

2017-01-16 Thread Bhupesh Sharma
Hi Andrew, On Sat, Jan 14, 2017 at 7:33 AM, Andrew Fish wrote: > >> On Jan 12, 2017, at 11:56 PM, Bhupesh Sharma wrote: >> >> This patch fixes some minor casing and file path issues > > Should we add a Linux builder with a case sensitive file system? > > macOS

Re: [edk2] [PATCH V2 1/1] ArmPlatformPkg/TZASC: Allow specifying subregions to be disabled

2017-01-18 Thread Bhupesh SHARMA
Hi Leif, Thanks for the review. On Mon, Jan 16, 2017 at 10:50 PM, Leif Lindholm wrote: > On Tue, Dec 27, 2016 at 02:45:42AM +0530, Bhupesh Sharma wrote: >> ARM TZASC-380 IP provides a mechanism to split memory regions being >> protected via it into eight equal-sized sub-regions

[edk2] [PATCH V3 1/1] ArmPlatformPkg/TZASC: Allow specifying subregions to be disabled

2017-01-20 Thread Bhupesh Sharma
help/topic/com.arm.doc.ddi0431c/CHDIGDCI.html Cc: Leif Lindholm Cc: Ard Biesheuvel Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Bhupesh Sharma [bhupesh.li...@gmail.com : Added my gmail ID as the NXP one is no longer valid] Signed-off-by: Bhupesh Sharma --- Changes from v2

Re: [edk2] [PATCH V3 1/1] ArmPlatformPkg/TZASC: Allow specifying subregions to be disabled

2017-01-29 Thread Bhupesh SHARMA
On Thu, Jan 26, 2017 at 8:06 PM, Leif Lindholm wrote: > On Fri, Jan 20, 2017 at 05:10:45PM +0530, Bhupesh Sharma wrote: >> ARM TZASC-380 IP provides a mechanism to split memory regions being >> protected via it into eight equal-sized sub-regions. A bit-setting >> all

[edk2] EDK2 master build failure

2016-11-17 Thread Bhupesh Sharma
Hi EDK2 experts, I have just rebased my code on the latest EDK2 master branch and now when I am trying to compile the same I am getting the following errors (for my ARMv8 build): # ./build.sh RELEASE clean; ./build.sh RELEASE .. Welcome to LS1043A UEFI Bui

Re: [edk2] EDK2 master build failure

2016-11-18 Thread Bhupesh Sharma
ded in windows OS and > shared to virtual linux machine, then do linux build will cause such > issue. To avoid it, we download edk2 source code in Linux machine and > build them pass. > > Thanks > Liming > > -Original Message- > > From: edk2-devel [mailto:edk2-d

[edk2] [PATCH 1/1] Add BGRT ACPI table inclusion support in OVMF

2016-12-25 Thread Bhupesh Sharma
ensured that the BGRT logo is properly prepared and can be viewed with user-space tools (like 'Gwenview' on KDE, for example): $ file /sys/firmware/acpi/bgrt/image /sys/firmware/acpi/bgrt/image: PC bitmap, Windows 3.x format, 193 x 58 x 24 Cc: Laszlo Ersek Signed-off-by: Bhup

[edk2] [PATCH V2 1/1] ArmPlatformPkg/TZASC: Allow specifying subregions to be disabled

2016-12-26 Thread Bhupesh Sharma
help/topic/com.arm.doc.ddi0431c/CHDIGDCI.html Cc: Leif Lindholm Cc: Ard Biesheuvel Signed-off-by: Bhupesh Sharma Contributed-under: TianoCore Contribution Agreement 1.0 --- .../Library/ArmVExpressSecLibCTA9x4/CTA9x4Sec.c| 14 +++--- ArmPlatformPkg/Drivers/ArmTrustZone/ArmT

Re: [edk2] [PATCH 1/1] Add BGRT ACPI table inclusion support in OVMF

2017-01-04 Thread Bhupesh SHARMA
Hi Laszlo, Many thanks for taking out the time to review this. On Tue, Jan 3, 2017 at 8:16 PM, Laszlo Ersek wrote: > On 12/26/16 06:25, Bhupesh Sharma wrote: >> During debugging Linux kernel for ACPI BGRT support >> (especially on VMs), it is sometimes very useful to have >

[edk2] [PATCH V2 1/1] OvmfPkg: Install BGRT ACPI table

2017-01-06 Thread Bhupesh Sharma
escription : == Cc: Jordan Justen Cc: Laszlo Ersek Signed-off-by: Bhupesh Sharma Contributed-under: TianoCore Contribution Agreement 1.0 --- Changes since v1: - Improved the commit log and cc list as per Laszlo's suggestions. - Also tested

[edk2] Using multiple SNP drivers on Shell

2016-03-31 Thread Bhupesh Sharma
Hi EDK2 experts, We are facing an issue when we enable two SNP (Simple Network) drivers in our EDK2 setup over a ARMv8 based SoC - NXP LS2080A. Here, I have two Ethernet interfaces - one is a MAC on SoC + Phy chip interface, while the other is a E1000 NIC card connected over a PCIe slot. A) Wh

Re: [edk2] Using multiple SNP drivers on Shell

2016-04-01 Thread Bhupesh Sharma
Hello, Any suggestions for this issue? Is this a known issue? Regards, Bhupesh > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Bhupesh Sharma > Sent: Thursday, March 31, 2016 5:18 PM > To: edk2-devel@lists.01.org > Sub

Re: [edk2] Using multiple SNP drivers on Shell

2016-04-04 Thread Bhupesh Sharma
> From: David Van Arnem > Sent: Friday, April 01, 2016 11:53 PM > On 04/01/2016 06:48 AM, Bhupesh Sharma wrote: > > Hello, > > > > Any suggestions for this issue? Is this a known issue? > > > > Hi Bhupesh, > > The ping command allows you to speci

Re: [edk2] Using multiple SNP drivers on Shell

2016-04-04 Thread Bhupesh Sharma
> From: Laszlo Ersek [mailto:ler...@redhat.com] > Sent: Monday, April 04, 2016 11:06 PM > > On 04/04/16 19:24, David Van Arnem wrote: > > On 04/04/2016 03:17 AM, Laszlo Ersek wrote: > > >> The most recent version (that I have) of the Shell Spec is 2.2. That > >> version lists both "ping" and "pin

Re: [edk2] Using multiple SNP drivers on Shell

2016-04-04 Thread Bhupesh Sharma
> -Original Message- > From: Carsey, Jaben [mailto:jaben.car...@intel.com] > Sent: Monday, April 04, 2016 11:28 PM > To: Laszlo Ersek; Bhupesh Sharma > Cc: edk2-devel@lists.01.org; David Van Arnem; Carsey, Jaben > Subject: RE: [edk2] Using multiple SN

Re: [edk2] Using multiple SNP drivers on Shell

2016-04-05 Thread Bhupesh Sharma
mpatible issue, the patch will keep '-_s' > command option unchanged, only add the new option '-s' and make the old > option '-_s' function same as new one. Thanks Jiaxin, that seems fine for me :) Regards, Bhupesh > > Thanks. > Jiaxin > > >

Re: [edk2] [PATCH] ArmPlatformPkg: fixups for 64-bit mailbox pointers

2016-04-18 Thread Bhupesh Sharma
We also should consider ARM SOCs which have Internal ROM code running even before the ATF starts running in EL3. Such a internal ROM code might be configured to make sure that only the primary core runs the ATF and UEFI code and the secondaries enter the ATF code first time only when Linux does

Re: [edk2] [Patch] ShellPkg: Enhance ping to select the interface automatically

2016-04-21 Thread Bhupesh Sharma
Hello Jiaxin, Sorry for the delay in reply - I was on holidays. I will check this patch at my end and will soon get back with the results. Regards, Bhupesh > -Original Message- > From: Wu, Jiaxin [mailto:jiaxin...@intel.com] > Sent: Thursday, April 21, 2016 7:35 AM > To: Bh

Re: [edk2] [PATCH 1/1] ArmPkg/OpteeLib: Add APIs to communicate with OP-TEE

2018-08-28 Thread Bhupesh Sharma
Hi Sumit, On Tue, Aug 28, 2018 at 10:04 PM, Sumit Garg wrote: > Hi Achin, > > On Tue, 28 Aug 2018 at 18:38, Achin Gupta wrote: >> >> Hi Sumit, >> >> Apologies for not replying sooner. Some questions and thoughts inline. >> >> On Mon, Aug 27, 2018 at 03:28:52PM +0530, Sumit Garg wrote: >> > On Fr

Re: [edk2] [RFC PATCH] MdePkg/BaseIoLibIntrinsic ARM AARCH64: avoid C code for MMIO access

2018-06-05 Thread Bhupesh Sharma
On Tue, Jun 5, 2018 at 1:00 PM, Ard Biesheuvel wrote: > Even though MMIO shares the address space with ordinary memory, the > accesses involved are *not* ordinary memory accesses, and so it is > a bad idea to let the compiler generate them using pointer dereferences. > > Instead, introduce a set o

Re: [edk2] Building EDK2-UEFI for AARCH64-BE

2016-01-07 Thread Bhupesh Sharma
Hi Ard, > From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] > Sent: Thursday, January 07, 2016 1:46 PM > > On 7 January 2016 at 08:05, Bhupesh Sharma > wrote: > > Hi Ard, Leif > > > > I am trying to experiment with compile the EDK2 UEFI using the AARCH

[edk2] UEFI Shell: Backspace doesn't work

2016-02-02 Thread Bhupesh Sharma
Hi, I think some of the edk2 shell users might have already noticed this, that the Backspace input doesn't work on the UEFI shell (I am using the latest edk2/master branch). On Qemu-AARCH64 2.5 build, I can get 'Ctrl + H' to work as the Backspace key on the shell, but on my AARCH64 platform eve

Re: [edk2] UEFI Shell: Backspace doesn't work

2016-02-02 Thread Bhupesh Sharma
> From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] > Sent: Tuesday, February 02, 2016 1:44 PM > On 2 February 2016 at 09:05, Bhupesh Sharma > wrote: > > Hi, > > > > I think some of the edk2 shell users might have already noticed this, > > that the Back

Re: [edk2] UEFI Shell: Backspace doesn't work

2016-02-02 Thread Bhupesh Sharma
> From: Ryan Harkin [mailto:ryan.har...@linaro.org] > Sent: Tuesday, February 02, 2016 4:57 PM > > On 2 February 2016 at 10:18, Laszlo Ersek wrote: > > On 02/02/16 10:09, Bhupesh Sharma wrote: > >>> From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] > >&

Re: [edk2] UEFI Shell: Backspace doesn't work

2016-02-04 Thread Bhupesh Sharma
> From: Ryan Harkin [mailto:ryan.har...@linaro.org] > Sent: Tuesday, February 02, 2016 5:35 PM > > On 2 February 2016 at 11:59, Laszlo Ersek wrote: > > On 02/02/16 12:53, Bhupesh Sharma wrote: > >>> From: Ryan Harkin [mailto:ryan.har...@linaro.org] > >>>

Re: [edk2] SATA 3.0 AHCI host controller codebase

2016-02-06 Thread Bhupesh Sharma
> From: Laszlo Ersek > Sent: Friday, February 05, 2016 2:27 AM > > On 02/03/16 22:39, Leif Lindholm wrote: > > Hi Shaveta, > > > > On Tue, Feb 02, 2016 at 07:05:03AM +, Shaveta Leekha wrote: > >> Is there some SATA 3.0 AHCI driver implementation in UEFI / EDK code? > >> The one I need to write

Re: [edk2] SATA 3.0 AHCI host controller codebase

2016-02-09 Thread Bhupesh Sharma
> From: Laszlo Ersek [mailto:ler...@redhat.com] > Sent: Monday, February 08, 2016 1:20 PM > > On 02/06/16 09:38, Bhupesh Sharma wrote: > >> From: Laszlo Ersek > >> Sent: Friday, February 05, 2016 2:27 AM > >> > >> On 02/03/16 22:39, Leif Lindholm

[edk2] Memory sharing between PEI and DXE phases

2016-02-15 Thread Bhupesh Sharma
Hi Experts, I have a doubt regarding memory sharing between PEI and DXE phases. Let's say I have a PEI library 'NorLib.c' and a DXE driver 'NorDxeDriver.c', where the DXE driver uses some APIs of the PEI Library to obtain information and provide functionalities to the upper layers. Now, if I a

[edk2] Allocating Aligned Pages

2016-02-16 Thread Bhupesh Sharma
Hi Experts, We are using the 'AllocateAlignedPages' function (inside 'MdePkg/Library/UefiMemoryAllocationLib/MemoryAllocationLib.c') to allocate 512MB aligned memory chunk in a DXE driver. However the memory chunk which gets allocated by this API is around 1GB. I understand that this might be

Re: [edk2] Allocating Aligned Pages

2016-02-17 Thread Bhupesh Sharma
> From: af...@apple.com [mailto:af...@apple.com] > Sent: Wednesday, February 17, 2016 1:34 PM > > On Feb 16, 2016, at 11:52 PM, Bhupesh Sharma > wrote: > > > > Hi Experts, > > > > We are using the 'AllocateAlignedPages' function (i

Re: [edk2] Exporting discontiguous System Memory to EFI STUB

2016-09-21 Thread Bhupesh Sharma
Thanks Ard, Laszlo. See replies in-line: > From: Ard Biesheuvel > Sent: Thursday, September 15, 2016 5:01 PM > > On 15 September 2016 at 10:01, Laszlo Ersek wrote: > > On 09/15/16 10:45, Sakar Arora wrote: > >> Hi > >> > >> This is in aarch64 UEFI context. > >> > >> The efi stub code ignores an

Re: [edk2] Exporting discontiguous System Memory to EFI STUB

2016-09-21 Thread Bhupesh Sharma
Hi Ard, > From: Ard Biesheuvel > Sent: Wednesday, September 21, 2016 4:50 PM > > On 21 September 2016 at 12:17, Laszlo Ersek wrote: > > On 09/21/16 11:10, Bhupesh Sharma wrote: > >> Thanks Ard, Laszlo. > >> > >> See replies in-line: > &g

Re: [edk2] [RFC V2] EDK2 Platform Proposal

2016-10-04 Thread Bhupesh Sharma
> From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Kinney, Michael D > Sent: Friday, September 23, 2016 2:25 AM > Hello, > > > > Here is the V2 version of the proposal for the edk2-platforms repo. > > > > Changes from V1: > > > > * edk2-platform is

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

2016-10-14 Thread Bhupesh Sharma
Signed-off-by: Meenakshi Aggarwal Signed-off-by: Bhupesh Sharma --- MdePkg/Include/Library/IoLib.h | 364 MdePkg/Library/BaseIoLibIntrinsic/IoLibArm.c | 479 +++ 2 files changed, 843 insertions(+) diff --git a/MdePkg/Include/Library/IoLib.h b

[edk2] [PATCH 1/1] ArmPlatformPkg/ArmTrustZone: Add support for specifying Subregions to be disabled

2016-10-14 Thread Bhupesh Sharma
c.ddi0431c/CJABCFHB.html Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Bhupesh Sharma Cc: Ard Biesheuvel --- .../Library/ArmVExpressSecLibCTA9x4/CTA9x4Sec.c | 21 ++--- ArmPlatformPkg/Drivers/ArmTrustZone/ArmTrustZone.c | 5 +++-- 2 files changed, 17

Re: [edk2] [PATCH 1/1] ArmPlatformPkg/ArmTrustZone: Add support for specifying Subregions to be disabled

2016-10-14 Thread Bhupesh Sharma
This patch is missing a .h file. Please discard. I will RESEND the patch with the fix. Apologies for the inconvenience. Regards, Bhupesh > -Original Message- > From: Bhupesh Sharma [mailto:bhupesh.sha...@nxp.com] > Sent: Friday, October 14, 2016 4:22 PM > To: edk2-de...

[edk2] [RESEND PATCH 1/1] ArmPlatformPkg/ArmTrustZone: Add support for specifying Subregions to be disabled

2016-10-14 Thread Bhupesh Sharma
c.ddi0431c/CJABCFHB.html Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Bhupesh Sharma Cc: Ard Biesheuvel --- .../Library/ArmVExpressSecLibCTA9x4/CTA9x4Sec.c | 21 ++--- ArmPlatformPkg/Drivers/ArmTrustZone/ArmTrustZone.c | 5 +++-- ArmPlatformPkg/Inclu

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

2016-10-14 Thread Bhupesh Sharma
Hi Laszlo, > From: Laszlo Ersek [mailto:ler...@redhat.com] > Sent: Friday, October 14, 2016 5:37 PM > > On 10/14/16 11:33, Bhupesh Sharma wrote: > > Various IPs on NXP/FSL SoCs having ARM64 cores have big-endian MMIO > > interfaces. > > > > This implies tha

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

2016-10-17 Thread Bhupesh Sharma
Hi Ard, > -Original Message- > From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] > Sent: Monday, October 17, 2016 1:12 PM > To: Kinney, Michael D > Cc: Gao, Liming ; Bhupesh Sharma > ; edk2-de...@ml01.01.org > Subject: Re: [edk2] [PATCH 1/1] MdePkg/IoLib

Re: [edk2] [RESEND PATCH 1/1] ArmPlatformPkg/ArmTrustZone: Add support for specifying Subregions to be disabled

2016-10-17 Thread Bhupesh Sharma
Hi Ard, Leif, Any comments on this patch ? > From: Bhupesh Sharma [mailto:bhupesh.sha...@nxp.com] > Sent: Friday, October 14, 2016 4:40 PM > > ARM TZASC-380 IP provides a mechanism to split memory regions being > protected via it into eight equal-sized sub-regions, with a bit set

Re: [edk2] [RESEND PATCH 1/1] ArmPlatformPkg/ArmTrustZone: Add support for specifying Subregions to be disabled

2016-10-26 Thread Bhupesh Sharma
Hi Leif, Thanks for the review. Please see my replies in-line. > From: Leif Lindholm [mailto:leif.lindh...@linaro.org] > Sent: Monday, October 17, 2016 6:55 PM > > On Mon, Oct 17, 2016 at 10:18:01AM +0000, Bhupesh Sharma wrote: > > Hi Ard, Leif, > > > > Any com

Re: [edk2] [RESEND PATCH 1/1] ArmPlatformPkg/ArmTrustZone: Add support for specifying Subregions to be disabled

2016-10-26 Thread Bhupesh Sharma
Hi Ard, > From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] > Sent: Monday, October 17, 2016 7:46 PM > > On 17 October 2016 at 14:25, Leif Lindholm > wrote: > > On Mon, Oct 17, 2016 at 10:18:01AM +, Bhupesh Sharma wrote: > >> Hi Ard, Leif, > &