Re: [edk2] [PATCH V2 09/10] ArmPlatformPkg/Bds: Added support for booting legacy kernel from BDS

2015-07-14 Thread Ryan Harkin
Sorry, pressed send too soon... On 14 July 2015 at 16:38, Ryan Harkin wrote: > > > On 14 July 2015 at 11:12, Olivier Martin wrote: > >> When PcdBdsLinuxSupport is enabled, users can create boot >> entries for the legacy EFI Linux loader. >> >> The ARM BDS

Re: [edk2] [PATCH V2 09/10] ArmPlatformPkg/Bds: Added support for booting legacy kernel from BDS

2015-07-14 Thread Ryan Harkin
On 14 July 2015 at 11:12, Olivier Martin wrote: > When PcdBdsLinuxSupport is enabled, users can create boot > entries for the legacy EFI Linux loader. > > The ARM BDS detects if the image is a EFI image if not > then it assumes it is a legacy Linux kernel (a kernel > without EFI Stub). > If the B

Re: [edk2] [PATCH 6/8] ArmPlatformPkg/Bds: Added support for booting legacy kernel from BDS

2015-07-13 Thread Ryan Harkin
On 13 July 2015 at 17:36, Olivier Martin wrote: > When PcdBdsLinuxSupport is enabled, users can create boot > entries for the legacy EFI Linux loader if it exists into > the firmware. > Surely this is only true if the kernel doesn't have an EFI header? If it has an EFI header, it will be forced

Re: [edk2] [PATCH V3 4/5] Add PCD for selecting terminal type at build time

2015-07-08 Thread Ryan Harkin
Hi Laszlo, On 8 July 2015 at 10:46, Laszlo Ersek wrote: > On 07/08/15 04:46, Heyi Guo wrote: > > Hi Roy, > > > > I have one question; please see my comments below. > > Indeed: > > > > > On 07/08/2015 08:44 AM, Roy Franz wrote: > >> From: Laszlo Ersek > >> > >> Add a fixed pointer PCD to allow b

Re: [edk2] [PATCH v2 0/7] Remove Ip4ConfigDxe module and related guid definitions from related packages.

2015-07-07 Thread Ryan Harkin
Thanks for updating, but perhaps I am reading the wrong thing. I don't see a commit message in any of the v2 patches describing what you are doing, like in your previous message on this thread. Also, you've added "CC: " to the commit message, but you haven't actually CC'd the maintainers in quest

Re: [edk2] [Patch 0/7] Remove Ip4ConfigDxe module and related guid definitions from related packages.

2015-07-07 Thread Ryan Harkin
On 7 July 2015 at 13:48, Ard Biesheuvel wrote: > On 7 July 2015 at 14:46, Ard Biesheuvel wrote: > > On 7 July 2015 at 03:27, jiaxinwu wrote: > >> Those patches are used to remove Ip4ConfigDxe module and related guid > definitions from related packages. > >> > > > > Could you please explain *why

Re: [edk2] [PATCH 0/3] ArmVExpressPkg: avoid PSCI reboot/poweroff on 32-bit platforms

2015-07-07 Thread Ryan Harkin
Hi Ard, On 7 July 2015 at 13:22, Ard Biesheuvel wrote: > On 6 July 2015 at 20:26, Ard Biesheuvel wrote: > > This fixes an issue reported by Ryan Harkin where 32-bit ARM platforms > without > > PSCI compliant firmware cannot be rebooted or powered off successfully at > >

Re: [edk2] ArmPlatformPkg: TC2 reset/shutdown broken in Tianocore EDK2 tree

2015-07-06 Thread Ryan Harkin
On 6 July 2015 at 17:32, Ard Biesheuvel wrote: > On 6 July 2015 at 18:11, Ryan Harkin wrote: > > The following 3 patches break reboot/shutdown from within UEFI on TC2 and > > probably all 32-bit ARM Ltd platforms: > > > > ee171cc 2015-05-08 ArmVExpressPkg: restrict

[edk2] ArmPlatformPkg: TC2 reset/shutdown broken in Tianocore EDK2 tree

2015-07-06 Thread Ryan Harkin
The following 3 patches break reboot/shutdown from within UEFI on TC2 and probably all 32-bit ARM Ltd platforms: ee171cc 2015-05-08 ArmVExpressPkg: restrict ArmVExpressSysConfigLib to SEC and DXE_DRIVER [Ard Biesheuvel] c889bf2 2015-05-08 ArmVExpressPkg: avoid the use of ArmVExpressSysConf

Re: [edk2] edk2[16933] EmbeddedPkg : Add FdtPlatformDxe driver

2015-06-18 Thread Ryan Harkin
> --- trunk/edk2/EmbeddedPkg/EmbeddedPkg.dec 2015-02-25 19:15:39 UTC > (rev 16932) > +++ trunk/edk2/EmbeddedPkg/EmbeddedPkg.dec 2015-02-25 19:20:57 UTC > (rev 16933) > @@ -52,6 +52,7 @@ >## FDT Configuration Table ># Include/Guid/Fdt.h >gFdtTableGuid = {

Re: [edk2] SF.net SVN: edk2:[15903] trunk/edk2/ArmPkg/Library/ArmArchTimerLib/ ArmArchTimerLib.inf

2015-06-18 Thread Ryan Harkin
[+Olivier at his proper email address] [+ linaro-uefi mailing list] On 18 June 2015 at 18:46, Ryan Harkin wrote: > TC2 no longer boots for me with this patch. If I revert it, it instead > crashes like this: > > add-symbol-file > /linaro/working/edk2.git/Build/ArmVExpress-CTA15

Re: [edk2] SF.net SVN: edk2:[15903] trunk/edk2/ArmPkg/Library/ArmArchTimerLib/ ArmArchTimerLib.inf

2015-06-18 Thread Ryan Harkin
TC2 no longer boots for me with this patch. If I revert it, it instead crashes like this: add-symbol-file /linaro/working/edk2.git/Build/ArmVExpress-CTA15-A7/DEBUG_GCC49/ARM/EmbeddedPkg/Drivers/FdtPlatformDxe/FdtPlatformDxe/DEBUG/FdtPlatformDxe.dll 0xBA835240 Loading driver at 0x000BA835000 Entry

Re: [edk2] [PATCH] Fix: MdeModulePkg: Implement UEFI25 HII Config keyword handler protocol.

2015-05-14 Thread Ryan Harkin
OK, so it seems Ard beat me to it and posted a patch already. On 14 May 2015 at 10:58, Ryan Harkin wrote: > This patch removes the unused DevicePathSize variable. > > The following commit introduces the DevicePathSize variable in function > GetStringIdFromDatabase: > &

[edk2] [PATCH] Fix: MdeModulePkg: Implement UEFI25 HII Config keyword handler protocol.

2015-05-14 Thread Ryan Harkin
le] UINTN DevicePathSize; Signed-off-by: Ryan Harkin --- MdeModulePkg/Universal/HiiDatabaseDxe/ConfigKeywordHandler.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/MdeModulePkg/Universal/HiiDatabaseDxe/ConfigKeywordHandler.c b/MdeModulePkg/Universal/HiiDatabaseDxe/ConfigKeywordHan

Re: [edk2] [Linaro-uefi] [PATCH 0/5] ArmPlatformPkg: Add Lan91x Dxe driver, and networking support for A15/A15 MPcore/FVP AArch64

2015-01-28 Thread Ryan Harkin
On 27 January 2015 at 10:56, Leif Lindholm wrote: > On Tue, Jan 27, 2015 at 10:36:23AM -, Olivier Martin wrote: > > Any reason to make the Network stack conditional? > > One reason to make the SMSC Lan91x Ethernet controller condition on some > > RTSM models is to use the virtio network drive

Re: [edk2] ARM64 UEFI Shell TFTP command

2015-01-10 Thread Ryan Harkin
Many thanks, Andrew. That looks very promising. I'll try it out next week. On 9 Jan 2015 17:21, "Andrew Fish" wrote: > > > On Jan 9, 2015, at 7:09 AM, Ryan Harkin wrote: > > > > I can't find a TFTP command in the shell on my ARM64 Juno platform. &g

[edk2] ARM64 UEFI Shell TFTP command

2015-01-09 Thread Ryan Harkin
I can't find a TFTP command in the shell on my ARM64 Juno platform. Did I miss something? (I think not) Considering TFTP is already supported via the ARM BDS menus, I'm thinking that a command can't be that hard to implement. Although maybe I'm being overly optimistic here! Google shows that m

Re: [edk2] PXE Boot file size

2014-05-14 Thread Ryan Harkin
OK, so it seems I was just a bit too quick off the mark, for Olivier/Brendan have just fixed it in revision 15527: http://sourceforge.net/p/edk2/code/15527 It hasn't hit the git mirror just yet, but I hacked it by hand and it works. Thanks to you all! On 14 May 2014 17:36, Ryan Harkin

Re: [edk2] PXE Boot file size

2014-05-14 Thread Ryan Harkin
Thanks for the pointers! Very valuable. On 14 May 2014 17:33, Laszlo Ersek wrote: > On 05/14/14 18:25, Laszlo Ersek wrote: > > On 05/14/14 16:57, Ryan Harkin wrote: > >> I have a platform set up to boot using PXE and I'm getting the following > >> error: > &

[edk2] PXE Boot file size

2014-05-14 Thread Ryan Harkin
I have a platform set up to boot using PXE and I'm getting the following error: PXE-E05: Download buffer is smaller than requested file. I'm using an Ubuntu tftpd-hpa server, so it supports the tsize option. The file I'm downloading is quite large at approx 8MB. I'm using an aarch64 / arm64

Re: [edk2] SF.net SVN: edk2:[15430] trunk/edk2/ArmPlatformPkg/ArmVExpressPkg/ ArmVExpress-FVP-AArch64.dsc

2014-04-07 Thread Ryan Harkin
I always build a single UEFI binary for Foundation and FVP Base models by defining ARM_FOUNDATION_FVP, so I guess I am now going to have problems when I run on an AEMv8 4+4 model? What would setting cores to 8 and clusters to 2 do to the Foundation model? Would it work OK? On 3 April 2014 21:04

Re: [edk2] [PATCH 0/1] ArmPlatformPkg/Bds: stop inputting more characters when

2014-03-12 Thread Ryan Harkin
On 12 March 2014 17:26, Olivier Martin wrote: > Thanks Ryan for the contribution. > I tested your patch and it seems doing the right things :-) > I pushed your change into SVN 15325. > Excellent. Thank you :-) > > > -Original Message- > > From: R

[edk2] [PATCH 1/1] ArmPlatformPkg/Bds: stop inputting more characters when string is full

2014-02-25 Thread Ryan Harkin
Agreement 1.0 Signed-off-by: Ryan Harkin --- ArmPlatformPkg/Bds/BdsHelper.c |8 +++- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/ArmPlatformPkg/Bds/BdsHelper.c b/ArmPlatformPkg/Bds/BdsHelper.c index 3142d85..7f0ef42 100644 --- a/ArmPlatformPkg/Bds/BdsHelper.c +++ b

[edk2] [PATCH 0/1] ArmPlatformPkg/Bds: stop inputting more characters when

2014-02-25 Thread Ryan Harkin
Hi Olivier, Here's the patch we discussed to stop the use inputting more characters than allowed. It also reverts the Boolean input back to 2 chars, as suggested in our email thread. Regards, Ryan. -- Flow-based real

Re: [edk2] SF.net SVN: edk2:[15236] trunk/edk2/ArmPlatformPkg/Bds/BdsHelper.c

2014-02-24 Thread Ryan Harkin
ways busy, but I'll be able to squeeze it in tomorrow/Wed.) > I will test your patch and merge it afterward. > Thanks! > Thanks, > Olivier > > > -Original Message- > > From: Ryan Harkin [mailto:ryan.har...@linaro.org] > > Sent: 17 February 2014 19

Re: [edk2] Upstream EDK2 broken with latest Linaro toolchain (13.12 onwards)

2014-02-21 Thread Ryan Harkin
On 21 February 2014 18:44, Andrew Fish wrote: > > On Feb 21, 2014, at 10:17 AM, Laszlo Ersek wrote: > > On 02/21/14 16:12, Ryan Harkin wrote: > > Hi Olivier, > > I've just noticed that the upstream EDK2 repository for the FVP AEMv8 > model is broken when bui

[edk2] Upstream EDK2 broken with latest Linaro toolchain (13.12 onwards)

2014-02-21 Thread Ryan Harkin
Hi Olivier, I've just noticed that the upstream EDK2 repository for the FVP AEMv8 model is broken when built with Linaro GCC 13.12 onwards. The error I see is: UEFI firmware (version built at 14:54:24 on Feb 21 2014) add-symbol-file /linaro/uefi/master/upstream/edk2.git/Build/ArmVExpress-FVP-AA

Re: [edk2] SF.net SVN: edk2:[15236] trunk/edk2/ArmPlatformPkg/Bds/BdsHelper.c

2014-02-17 Thread Ryan Harkin
On 12 February 2014 15:09, wrote: > Revision: 15236 > http://sourceforge.net/p/edk2/code/15236 > Author: oliviermartin > Date: 2014-02-12 15:09:58 + (Wed, 12 Feb 2014) > Log Message: > --- > ArmPlatformPkg/BdsLib: Let the user press enter when inputting booleans > > Co

Re: [edk2] [PATCH 0/3]

2014-01-15 Thread Ryan Harkin
On 15 January 2014 16:03, Olivier Martin wrote: > Hi Ryan, > > I forgot to mention you but I pushed your patches in SVN earlier today. > Ah-ha! Thanks Olivier, yes, I see them in the commit log now :-) > > Thanks, > Olivier > > > -Original Mess

[edk2] [PATCH 3/3] ArmPlatformPkg/ArmVExpressPkg: add virtio to RTSM A9 BSP

2014-01-14 Thread Ryan Harkin
Signed-off-by: Ryan Harkin --- .../ArmVExpressPkg/ArmVExpress-RTSM-A9x4.dsc | 11 +++ .../ArmVExpressPkg/ArmVExpress-RTSM-A9x4.fdf |8 +++- 2 files changed, 18 insertions(+), 1 deletion(-) diff --git a/ArmPlatformPkg/ArmVExpressPkg/ArmVExpress-RTSM-A9x4.dsc b

[edk2] [PATCH 2/3] ArmPlatformPkg/ArmVExpressPkg: add virtio to RTSM A15 BSP

2014-01-14 Thread Ryan Harkin
Signed-off-by: Ryan Harkin --- .../ArmVExpressPkg/ArmVExpress-RTSM-A15.dsc| 10 ++ .../ArmVExpressPkg/ArmVExpress-RTSM-A15.fdf|6 ++ 2 files changed, 16 insertions(+) diff --git a/ArmPlatformPkg/ArmVExpressPkg/ArmVExpress-RTSM-A15.dsc b/ArmPlatformPkg

[edk2] [PATCH 0/3]

2014-01-14 Thread Ryan Harkin
Hi Steven, In the following emails are the three patches that I'd like applied to the Linaro EDK2 tree for the 14.01 release: [PATCH 1/3] ArmPlatformPkg/ArmVExpressPkg: add virtio to RTSM A15 [PATCH 2/3] ArmPlatformPkg/ArmVExpressPkg: add virtio to RTSM A15 [PATCH 3/3] ArmPlatformPkg/ArmVExpress

[edk2] [PATCH 1/3] ArmPlatformPkg/ArmVExpressPkg: add virtio to RTSM A15 MPCore BSP

2014-01-14 Thread Ryan Harkin
Signed-off-by: Ryan Harkin --- .../ArmVExpressPkg/ArmVExpress-RTSM-A15_MPCore.dsc | 10 ++ .../ArmVExpressPkg/ArmVExpress-RTSM-A15_MPCore.fdf |6 ++ 2 files changed, 16 insertions(+) diff --git a/ArmPlatformPkg/ArmVExpressPkg/ArmVExpress-RTSM-A15_MPCore.dsc b/ArmPlatformPkg

Re: [edk2] [PATCH V2] Move RTSM VExpress variable storage to 256k flash blocks

2013-12-19 Thread Ryan Harkin
I just wanted to confirm that this works in the 13.12 release and in my own builds from the latest tree. Thanks Roy! On 12 December 2013 22:08, Roy Franz wrote: > Change the addresses/sizes of the variable storage areas to use 256k > blocks so UEFI is compatible with both the RTSM models and Q

Re: [edk2] [PATCH] Move RTSM VExpress variable storage to 256k flash blocks

2013-12-09 Thread Ryan Harkin
Hi Roy, This patch is in 2013.12-rc1 now and I am no longer able to save my config in the models. Can you confirm if it works for you and what setup you are testing? I see this on each boot: Loading driver at 0x000BF54D000 EntryPoint=0x000BF54D26D ArmVeNorFlashDxe.efi ValidateFvHeader: No Firmw

Re: [edk2] ARM/BDS: skip initrd if not found

2013-11-26 Thread Ryan Harkin
reference flash images in your CI (using the parameter > motherboard.flashloader1.fname="flash.dat”). Doing that you will not need > to create new boot entries everytime you want to test a new UEFI image. > That's a nice idea. Unfortunately, it doesn't fix the problem when r

Re: [edk2] ARM/BDS: skip initrd if not found

2013-11-26 Thread Ryan Harkin
that LEG provide something to *replace* ARM/BDS, not something that uses it. So I guess my patch is only relevant for those who use ARM/BDS. > > > Thanks, > > Olivier > > > > > > From: Ryan Harkin [mailto:ryan.har...@linaro.org] > Sent: 19 November 2

Re: [edk2] ARM/BDS: skip initrd if not found

2013-11-19 Thread Ryan Harkin
On 19 November 2013 10:34, Leif Lindholm wrote: > Hi Ryan, > > On Tue, Nov 19, 2013 at 08:30:25AM +0000, Ryan Harkin wrote: > > Hi Olivier/Steve/Leif/whoever is interested, > > > > I have a problem I'm trying to solve and I don't think there is a proper &

[edk2] ARM/BDS: skip initrd if not found

2013-11-19 Thread Ryan Harkin
e fine with me too. Opinions on a way forward and/or this patch? Regards, Ryan. note: the patch looks a little larger than it should because I had to indent the section beginning with the comment "// Check if the initrd is a uInitrd", although that section is unmodified otherwise. >F

Re: [edk2] [PATCH] MdeModulePkg/ArpDxe: Retrieved SnpMode only after configuring Snp

2013-10-16 Thread Ryan Harkin
On 9 October 2013 16:31, Ryan Harkin wrote: > I would like to revive discussion on this patch. It seems it has not been > accepted, perhaps we could have a comment here from the maintainer about > what is happening? > I think waiting a week is long enough to not make me seem imp

Re: [edk2] [PATCH] MdeModulePkg/MnpDxe: Checked returned value of Snp->GetStatus()

2013-10-09 Thread Ryan Harkin
I would like to revive discussion on this patch. It seems it has not been accepted, perhaps we could have a comment here from the maintainer about what is happening? On 19 July 2013 20:11, Olivier Martin wrote: > Thanks Ting for the feedback, > what about this new version (see attached patch)

Re: [edk2] [PATCH 00/10] Add network support for ARM VE and RTSM

2013-10-09 Thread Ryan Harkin
e LAN91x driver to the > multi-core A15 RTSM build > ArmPlatformPkg/ArmVExpressPkg: Add the LAN91x driver to the > single-core A15 RTSM build > ArmPlatformPkg/ArmVExpressPkg: Add the LAN91x driver to the A9 RTSM > build > ArmPlatformPkg/ArmVExpressPkg: Add the LAN91

Re: [edk2] [PATCH] MdeModulePkg/ArpDxe: Retrieved SnpMode only after configuring Snp

2013-10-09 Thread Ryan Harkin
I would like to revive discussion on this patch. It seems it has not been accepted, perhaps we could have a comment here from the maintainer about what is happening? On 19 July 2013 20:05, Olivier Martin wrote: > Sorry, I sent the old patch. Here is the new one. > _

Re: [edk2] [PATCH 02/10] MdeModulePkg//ArpDxe: Retrieved SnpMode only after configuring Snp

2013-10-09 Thread Ryan Harkin
On 9 October 2013 02:53, Fu, Siyuan wrote: > Hi, Pollack > > I don't agree with the opinion that the Snp.Mode can't be accessed before > Snp->Initialize() is called. According to UEFI spec, all the fields in > Snp.Mode structure must be discovered during driver initialization, per my > understand

Re: [edk2] [PATCH 01/10] MdeModulePkg/MnpDxe: Checked returned value of Snp->GetStatus()

2013-10-09 Thread Ryan Harkin
On 8 October 2013 09:51, Fu, Siyuan wrote: > Hi, Pollack and Martin > > Here calling Snp->GetStatus() without state code check is intend to update > the MediaPresent field of EFI_SIMPLE_NETWORK_MODE to reflect the latest > media status. Even though this call meet some errors we still need to > re

Re: [edk2] ShellPkg patch: Handle pool allocation failure

2013-09-20 Thread Ryan Harkin
This is exactly my use-case for wanting to change FreePool() to allow it to take a NULL pointer without throwing an ASSERT. Just saying... On 19 September 2013 18:21, Carsey, Jaben wrote: > Reviewed-by: Jaben Carsey > > I will commit this. > > -Jaben > > -Original Message- > From: Ser

Re: [edk2] [PATCH 5/8] OvmfPkg: Make the VirtIo devices use the new VIRTIO_DEVICE_PROTOCOL

2013-09-19 Thread Ryan Harkin
On 19 September 2013 12:30, Laszlo Ersek wrote: > On 09/17/13 18:54, Olivier Martin wrote: > > > OvmfPkg/Include/IndustryStandard/VirtioNet.h | 13 +- > > > diff --git a/OvmfPkg/Include/IndustryStandard/VirtioNet.h > b/OvmfPkg/Include/IndustryStandard/VirtioNet.h > > index 9f5291a..6bcb57

Re: [edk2] FreePool crashes the system when called with a NULL pointer

2013-09-17 Thread Ryan Harkin
On 17 September 2013 16:52, Andrew Fish wrote: > > On Sep 16, 2013, at 11:36 PM, Ryan Harkin wrote: > > Hi, > > Tracking down a crash I am seeing shows that I am effectively calling > FreePool() with a NULL pointer. The system then ASSERTS and hangs. > > I've n

[edk2] FreePool crashes the system when called with a NULL pointer

2013-09-16 Thread Ryan Harkin
Hi, Tracking down a crash I am seeing shows that I am effectively calling FreePool() with a NULL pointer. The system then ASSERTS and hangs. I've noticed various seemingly "random" asserts like this before and suspect it may be related. Of course, the assert output never helps track down the cu

Re: [edk2] Platform Specific Contribution

2013-07-29 Thread Ryan Harkin
On 27 July 2013 21:50, Grant Likely wrote: > [cc'ing ryan; he already has to deal with keeping multiple platforms > in sync with mainline] > > On Fri, Jul 26, 2013 at 4:06 PM, Andrew Fish wrote: >> On Jul 25, 2013, at 3:07 AM, Olivier Martin wrote: >> >> > Grant did some investigation a while ag

Re: [edk2] [PATCH] MdeModulePkg: fix compilation errors

2013-06-24 Thread Ryan Harkin
On 24 June 2013 11:44, Leif Lindholm wrote: > On 24 June 2013 10:08, Ryan Harkin wrote: >> On 24 June 2013 08:44, Dong, Eric wrote: >>> This change is not correct, the "Buffer" variable is not defined. >> >> Hmmm, isn't that curious! I've not

Re: [edk2] [PATCH] MdeModulePkg: fix compilation errors

2013-06-24 Thread Ryan Harkin
I think look into this one properly! Thanks Eric. > > -Original Message- > From: Ryan Harkin [mailto:ryan.har...@linaro.org] > Sent: Friday, June 21, 2013 3:48 PM > To: ryan.har...@linaro.org; edk2-devel@lists.sourceforge.net; > patc...@linaro.org; boot-architec

Re: [edk2] [PATCH 1/9] ArmPlatformPkg/ArmVExpressPkg: Add support for EDK2_OUT_DIR build parameter

2013-06-21 Thread Ryan Harkin
On 21 June 2013 17:12, Andrew Fish wrote: > > On Jun 21, 2013, at 4:23 AM, Ryan Harkin wrote: > >> On 21 June 2013 11:15, Olivier Martin wrote: >>> I am not convinced by this patch. >>> The use case you gave is the automated build system. Nothing prevent you t

Re: [edk2] [PATCH 3/9] ArmPlatformPkg/ArmVExpressPkg: CTA9x4: default to EDK2_ARMVE_STANDALONE=1

2013-06-21 Thread Ryan Harkin
you want to change the OUTPUT_DIRECTORY, you should update the build script to allow you to over-ride the value in the DSC file ;-) (I'll hide now :-)) > > > > >> -----Original Message- >> From: Ryan Harkin [mailto:ryan.har...@linaro.org] >> Sent: 21

Re: [edk2] [PATCH 1/9] ArmPlatformPkg/ArmVExpressPkg: Add support for EDK2_OUT_DIR build parameter

2013-06-21 Thread Ryan Harkin
ll look into the build script if I get some time. Meanwhile, this patch can live in my tree. > >> -Original Message- >> From: Ryan Harkin [mailto:ryan.har...@linaro.org] >> Sent: 21 June 2013 09:05 >> To: ryan.har...@linaro.org; edk2-devel@lists.sourceforge.n

Re: [edk2] [PATCH 5/9] ArmPlatformPkg/Bds: Make ".EFI" files recognizable as EFI applications.

2013-06-21 Thread Ryan Harkin
On 21 June 2013 11:02, Olivier Martin wrote: > I committed this change after fixing the coding style (revision 14437). Yes, your revised version is more appropriate. Thanks. > >> -Original Message----- >> From: Ryan Harkin [mailto:ryan.har...@linaro.org] >> Sent:

Re: [edk2] [PATCH 2/9] ArmPlatformPkg/ArmVExpressPkg: CTA15-A7: Allow EDK2_SKIP_PEICORE to be specified at build time

2013-06-21 Thread Ryan Harkin
aps I should just drop this one? > >> -Original Message- >> From: Ryan Harkin [mailto:ryan.har...@linaro.org] >> Sent: 21 June 2013 09:05 >> To: ryan.har...@linaro.org; edk2-devel@lists.sourceforge.net; >> patc...@linaro.org; boot-architect...@lists.linaro.org

Re: [edk2] [PATCH 1/1] BaseTools: Apply ARM GenFv patch

2013-06-21 Thread Ryan Harkin
t aware of this and I've been carrying this patch, thinking I still needed it. >> -Original Message- >> From: boot-architecture-boun...@lists.linaro.org [mailto:boot- >> architecture-boun...@lists.linaro.org] On Behalf Of Ryan Harkin >> Sent: 21 June 2013 08:

Re: [edk2] ARM specific patches from Linaro

2013-06-21 Thread Ryan Harkin
On 21 June 2013 09:05, Ryan Harkin wrote: > Linaro have been carrying a few small patches in our tree for some months > now; they have proven valuable and stable, so we'd like to push them > upstream. > > The first few patches configure the platforms with some simple defau

[edk2] [PATCH 1/1] BaseTools: Apply ARM GenFv patch

2013-06-21 Thread Ryan Harkin
This patch is needed to fixup some builds, such as Versatile Express A5, otherwise they hang on boot due to the first instruction being zero. Signed-off-by: Ryan Harkin --- BaseTools/Source/C/GenFv/GenFv.c|7 +--- BaseTools/Source/C/GenFv/GenFvInternalLib.c | 48

[edk2] [PATCH] MdeModulePkg: fix compilation errors

2013-06-21 Thread Ryan Harkin
From: Leif Lindholm This patch fixes a minor error that prevents the native ARM compiler from building the code. Signed-off-by: Leif Lindholm --- .../Universal/SetupBrowserDxe/Expression.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/MdeModulePkg/Universal/Set

[edk2] [PATCH 1/9] ArmPlatformPkg/ArmVExpressPkg: Add support for EDK2_OUT_DIR build parameter

2013-06-21 Thread Ryan Harkin
\ -D EDK2_OUT_DIR=Build/vea9 Signed-off-by: Ryan Harkin --- .../ArmVExpressPkg/ArmVExpress-CTA15-A7.dsc|4 .../ArmVExpressPkg/ArmVExpress-CTA9x4.dsc |4 ++-- .../ArmVExpressPkg/ArmVExpress-RTSM-A15.dsc|4 .../ArmVExpressPkg/ArmVExpress-RTSM

[edk2] [PATCH 2/9] ArmPlatformPkg/ArmVExpressPkg: CTA15-A7: Allow EDK2_SKIP_PEICORE to be specified at build time

2013-06-21 Thread Ryan Harkin
The original BSP for TC2 hard codes EDK2_SKIP_PEICORE=1, but this change allows the user to over-ride the value at build time. Signed-off-by: Ryan Harkin --- .../ArmVExpressPkg/ArmVExpress-CTA15-A7.dsc|2 ++ 1 file changed, 2 insertions(+) diff --git a/ArmPlatformPkg/ArmVExpressPkg

[edk2] [PATCH 7/9] ArmPlatformPkg/Bds: Add Reboot and Shutdown options to main menu

2013-06-21 Thread Ryan Harkin
Add Reboot and Shutdown options to the main BDS menu so that users can control the platform remotely without having to physically reboot/shutdown the board. Signed-off-by: Ryan Harkin --- ArmPlatformPkg/Bds/BootMenu.c | 20 1 file changed, 20 insertions(+) diff --git a

[edk2] [PATCH 5/9] ArmPlatformPkg/Bds: Make ".EFI" files recognizable as EFI applications.

2013-06-21 Thread Ryan Harkin
From: Leif Lindholm Currently, only ".efi" files are recognized as valid ARM UEFI applications by BDS. This patch also makes ".EFI" files recognised. Signed-off-by: Leif Lindholm --- ArmPlatformPkg/Bds/BdsHelper.c |3 +++ 1 file changed, 3 insertions(+) diff --git a/ArmPlatformPkg/Bds/Bds

[edk2] [PATCH 3/9] ArmPlatformPkg/ArmVExpressPkg: CTA9x4: default to EDK2_ARMVE_STANDALONE=1

2013-06-21 Thread Ryan Harkin
This patch changes the Versatile Express A9 BSP to build with EDK2_ARMVE_STANDALONE=1 as the default option. The user may then specify zero if they wish to build the non-standalone version. Signed-off-by: Ryan Harkin --- .../ArmVExpressPkg/ArmVExpress-CTA9x4.dsc |3 +++ 1 file

[edk2] [PATCH 8/9] ArmPlatformPkg/Bds: show global FDT config in main menu

2013-06-21 Thread Ryan Harkin
Previously, there was no way to read the global FDT config. This patch displays the global FDT config after the boot devices in the main menu. Signed-off-by: Ryan Harkin --- ArmPlatformPkg/Bds/BootMenu.c | 30 ++ 1 file changed, 30 insertions(+) diff --git a

[edk2] [PATCH 6/9] ArmPlatformPkg/Bds: use letters for hard coded boot menu options

2013-06-21 Thread Ryan Harkin
compatibility, I've left the numbering support in. However, I've re-ordered the Ebl and Boot Manager entries, so the default number for the Boot Manager option is now 2, not 3. Signed-off-by: Ryan Harkin --- ArmPlatformPkg/Bds/BootMenu.c | 17 ++--- 1 file changed, 14 insert

[edk2] ARM specific patches from Linaro

2013-06-21 Thread Ryan Harkin
Linaro have been carrying a few small patches in our tree for some months now; they have proven valuable and stable, so we'd like to push them upstream. The first few patches configure the platforms with some simple default values and some extra build options. [PATCH 1/9] ArmPlatformPkg/ArmVExpre

[edk2] [PATCH 9/9] ArmPlatformPkg/Bds: add local FDT config

2013-06-21 Thread Ryan Harkin
sting configs stored on the board due to the change in structure sizes to store the boot device configurations. Signed-off-by: Ryan Harkin --- ArmPlatformPkg/ArmPlatformPkg.dec |2 + ArmPlatformPkg/Bds/Bds.c | 30 +++- ArmPlatformPkg/Bds/Bds.inf |

[edk2] [PATCH 4/9] ArmPlatformPkg/ArmVExpressPkg: add Shutdown support

2013-06-21 Thread Ryan Harkin
LibResetSystem was performing a cold reboot for both reboot and shutdown. This patch updates LibResetSystem to pass SYS_CFG_SHUTDOWN to the microcontroller when called wit EfiResetShutdown. Signed-off-by: Ryan Harkin --- .../Library/ResetSystemLib/ResetSystemLib.c|9 - 1

Re: [edk2] EDK II git mirror repo updates

2013-06-17 Thread Ryan Harkin
On 18 June 2013 00:21, Jordan Justen wrote: > As part of the recent forced sourceforge project 'upgrades' the url of > the EDK II git mirror has changed: > http://sf.net/p/tianocore/edk2 (web) > http://git.code.sf.net/p/tianocore/edk2 (git) > git://git.code.sf.net/p/tianocore/edk2 > > The EDK II F

Re: [edk2] EDK II svn source control url has changed

2013-06-06 Thread Ryan Harkin
On 7 June 2013 06:33, Keshav Padram wrote: > On 7 June 2013 04:27, Jordan Justen wrote: >> >> On Fri, Apr 19, 2013 at 4:06 PM, Jordan Justen wrote: >> > I will send out notices when the EDK, EDK II and EDK II Buildtools >> > projects are migrated. For other projects, please refer to the wiki >>

Re: [edk2] What do you use for debugging?

2013-05-22 Thread Ryan Harkin
On 22 May 2013 18:28, Richardson, Brian wrote: > I could you give you better guidance if I knew what type of UEFI code you > were trying to debug (UEFI driver? OS loader? UEFI app? Firmware/BIOS?). > And the architecture you're using. I could assume it's x86 and stay out of it ;-) But if you'r

Re: [edk2] EDKII for ARM

2013-03-01 Thread Ryan Harkin
On 28 February 2013 22:36, ap at wrote: > Hi all, > I am a newbie in ARM world and trying to learn the basics. I am very much > familiar with UEFI-EDKII on x86 side and now interested in learning this new > architecture & design. I have downloaded the code from sourceforge.net; but > what’s next;

Re: [edk2] Git as an svn alternative - was [PATCH] Fix broken IA32 ...

2013-02-22 Thread Ryan Harkin
On 22 February 2013 11:14, David Woodhouse wrote: > On Thu, 2013-02-21 at 18:17 -0800, Cameron Esfahani wrote: >> Oh, I'm serious. >> >> Don't take my dislike of git to mean that I don't think we should >> consider the proposal to migrate edk2 away from svn to git. If the >> consensus is to switc

Re: [edk2] [PATCH] MdeModulePkg/Core/Pei: Critical buffer overflow when loading TE Images

2013-02-19 Thread Ryan Harkin
ust expand the memory size to > the PE image size for TE image. Attach my patch; please help to verify this > patch. > Your patch also resolved my issue. Again, I haven't tried to understand the code, but I tested that it works, so you can have my tag for what it's worth: Tested-

Re: [edk2] [PATCH] MdeModulePkg/Core/Pei: Critical buffer overflow when loading TE Images

2013-02-18 Thread Ryan Harkin
> At the least the code in RED must be removed. But some additional code might > be needed to fix the alignment of the code section and the image size > calculation to take in account this alignment. > > But these fixes would go into MdePkg/Library/BasePeCoffLib. > > > >

Re: [edk2] Git as an svn alternative - was [PATCH] Fix broken IA32 ...

2013-02-12 Thread Ryan Harkin
Great discussion :-) On 12 February 2013 11:11, Grant Likely wrote: > On Feb 1, 2013 Jordan Justen wrote: >> On Fri, Feb 1, 2013 at 11:33 AM, Olivier Martin >> wrote: >> > Linaro maintains their own git-svn service: >> > http://git.linaro.org/gitweb?p=mirror/edk2/edk2.git;a=summary >> >> Andrew

Re: [edk2] [PATCH] panda: build.sh: change RELEASE/DEBUG param handling

2013-02-11 Thread Ryan Harkin
On 11 February 2013 16:41, Ryan Harkin wrote: > On 11 February 2013 16:08, Jordan Justen wrote: >> On Mon, Feb 11, 2013 at 12:22 AM, Ryan Harkin wrote: >>> The attached patch changes the PandaBoardPkg/build.sh script RELEASE >>> or DEBUG parameter handling. >>

Re: [edk2] [PATCH] panda: build.sh: change RELEASE/DEBUG param handling

2013-02-11 Thread Ryan Harkin
On 11 February 2013 16:08, Jordan Justen wrote: > On Mon, Feb 11, 2013 at 12:22 AM, Ryan Harkin wrote: >> The attached patch changes the PandaBoardPkg/build.sh script RELEASE >> or DEBUG parameter handling. >> >> Before this patch, build.sh can be called thus: >>

[edk2] [PATCH] panda: build.sh: change RELEASE/DEBUG param handling

2013-02-11 Thread Ryan Harkin
The attached patch changes the PandaBoardPkg/build.sh script RELEASE or DEBUG parameter handling. Before this patch, build.sh can be called thus: ./build.sh ./build.sh -b DEBUG ./build.sh RELEASE This patch simplifies the way you pass the debug/release parameter into build.sh so that build

Re: [edk2] UEFI zImage not loading on Pandaboard

2013-01-18 Thread Ryan Harkin
Hi Keeran, I wish I'd seen this email earlier, I suspect you have either got it working or you've given up by now! On 20 December 2012 09:44, keeran k wrote: > Hello, > > Am trying to get my hands on ARM-UEFI on Pandaboard 4430. I just followed > the instructions given in readme.txt below link;

Re: [edk2] TrustZone support for VE Cortex A15?

2012-10-22 Thread Ryan Harkin
Olivier, Nicola is referring to the A15-TC1 BSP that's in my tree, not to the RTSM version... Nicola, On 22 October 2012 10:58, Nicola Johnsen wrote: > So just that I understand this correctly: if there is no TZ controller on > the board, then secure memory is just emulated? As far as I could