Re: [edk2] [PATCH] ShellBinPkg Arm/AArch64 Shell binary update

2016-08-02 Thread Leif Lindholm 
On Tue, Aug 02, 2016 at 11:21:48AM +0200, Ard Biesheuvel wrote:
> The binaries of ShellBinPkg are generated with ShellPkg from b89919ee8f8c
> ("BaseTools AARCH64: override XIP module linker alignment to 32 bytes")
> 
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Ard Biesheuvel 

For the AArch64 variants:
Tested-by: Leif Lindholm 

And in general:
Reviewed-by: Leif Lindholm 

> ---
> 
> The binaries can be found here:
> https://git.linaro.org/people/ard.biesheuvel/uefi-next.git/shortlog/refs/heads/arm-shell-update
> 
>  ShellBinPkg/MinUefiShell/AArch64/Shell.efi | Bin 392416 -> 341280 bytes
>  ShellBinPkg/MinUefiShell/Arm/Shell.efi | Bin 338592 -> 297888 bytes
>  ShellBinPkg/UefiShell/AArch64/Shell.efi| Bin 893696 -> 827456 bytes
>  ShellBinPkg/UefiShell/Arm/Shell.efi| Bin 786592 -> 731680 bytes
>  4 files changed, 0 insertions(+), 0 deletions(-)
> 
> diff --git a/ShellBinPkg/MinUefiShell/AArch64/Shell.efi 
> b/ShellBinPkg/MinUefiShell/AArch64/Shell.efi
> index 53724a609171..162904634635 100755
> Binary files a/ShellBinPkg/MinUefiShell/AArch64/Shell.efi and 
> b/ShellBinPkg/MinUefiShell/AArch64/Shell.efi differ
> diff --git a/ShellBinPkg/MinUefiShell/Arm/Shell.efi 
> b/ShellBinPkg/MinUefiShell/Arm/Shell.efi
> index 251e0ca38227..c84b17bdba80 100755
> Binary files a/ShellBinPkg/MinUefiShell/Arm/Shell.efi and 
> b/ShellBinPkg/MinUefiShell/Arm/Shell.efi differ
> diff --git a/ShellBinPkg/UefiShell/AArch64/Shell.efi 
> b/ShellBinPkg/UefiShell/AArch64/Shell.efi
> index fdfd51803f93..6f2160bd1f46 100755
> Binary files a/ShellBinPkg/UefiShell/AArch64/Shell.efi and 
> b/ShellBinPkg/UefiShell/AArch64/Shell.efi differ
> diff --git a/ShellBinPkg/UefiShell/Arm/Shell.efi 
> b/ShellBinPkg/UefiShell/Arm/Shell.efi
> index f7a8cca681d7..f8d46d496c05 100755
> Binary files a/ShellBinPkg/UefiShell/Arm/Shell.efi and 
> b/ShellBinPkg/UefiShell/Arm/Shell.efi differ
> -- 
> 2.7.4
> 
___
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel

[edk2] [PATCH] ShellBinPkg Arm/AArch64 Shell binary update

2016-08-02 Thread Ard Biesheuvel 
The binaries of ShellBinPkg are generated with ShellPkg from b89919ee8f8c
("BaseTools AARCH64: override XIP module linker alignment to 32 bytes")

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ard Biesheuvel 
---

The binaries can be found here:
https://git.linaro.org/people/ard.biesheuvel/uefi-next.git/shortlog/refs/heads/arm-shell-update

 ShellBinPkg/MinUefiShell/AArch64/Shell.efi | Bin 392416 -> 341280 bytes
 ShellBinPkg/MinUefiShell/Arm/Shell.efi | Bin 338592 -> 297888 bytes
 ShellBinPkg/UefiShell/AArch64/Shell.efi| Bin 893696 -> 827456 bytes
 ShellBinPkg/UefiShell/Arm/Shell.efi| Bin 786592 -> 731680 bytes
 4 files changed, 0 insertions(+), 0 deletions(-)

diff --git a/ShellBinPkg/MinUefiShell/AArch64/Shell.efi 
b/ShellBinPkg/MinUefiShell/AArch64/Shell.efi
index 53724a609171..162904634635 100755
Binary files a/ShellBinPkg/MinUefiShell/AArch64/Shell.efi and 
b/ShellBinPkg/MinUefiShell/AArch64/Shell.efi differ
diff --git a/ShellBinPkg/MinUefiShell/Arm/Shell.efi 
b/ShellBinPkg/MinUefiShell/Arm/Shell.efi
index 251e0ca38227..c84b17bdba80 100755
Binary files a/ShellBinPkg/MinUefiShell/Arm/Shell.efi and 
b/ShellBinPkg/MinUefiShell/Arm/Shell.efi differ
diff --git a/ShellBinPkg/UefiShell/AArch64/Shell.efi 
b/ShellBinPkg/UefiShell/AArch64/Shell.efi
index fdfd51803f93..6f2160bd1f46 100755
Binary files a/ShellBinPkg/UefiShell/AArch64/Shell.efi and 
b/ShellBinPkg/UefiShell/AArch64/Shell.efi differ
diff --git a/ShellBinPkg/UefiShell/Arm/Shell.efi 
b/ShellBinPkg/UefiShell/Arm/Shell.efi
index f7a8cca681d7..f8d46d496c05 100755
Binary files a/ShellBinPkg/UefiShell/Arm/Shell.efi and 
b/ShellBinPkg/UefiShell/Arm/Shell.efi differ
-- 
2.7.4

___
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel

Re: [edk2] [PATCH] ShellBinPkg: Arm/AArch64 Shell binary update.

2016-02-09 Thread Leif Lindholm 
Thanks!

(Argh, forgot to add your reviewed-by before I pushed!)

So, could I do something like splitting ShellPkg/ShellBinPkg and then:

ShellBinPkg
W: https://github.com/tianocore/tianocore.github.io/wiki/ShellPkg
M: Jaben Carsey <jaben.car...@intel.com>
M: Shumin Qiu <shumin@intel.com>
M: Leif Lindholm <leif.lindh...@linaro.org>(ARM/AArch64)
M: Ard Biesheuvel <ard.biesheu...@linaro.org>  (ARM/AArch64)

(but getting the alignment right when I have a fixed-width font)

/
Leif

On 9 February 2016 at 16:42, Carsey, Jaben <jaben.car...@intel.com> wrote:
> Reviewed-by: Jaben Carsey <jaben.car...@intel.com>
>
> And yes I agree that you both should be added as maintainers of the 
> ShellBinPkg.
>
>> -Original Message-
>> From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Leif
>> Lindholm
>> Sent: Tuesday, February 09, 2016 8:41 AM
>> To: edk2-devel@lists.01.org
>> Cc: Carsey, Jaben <jaben.car...@intel.com>; Qiu, Shumin
>> <shumin@intel.com>; Laszlo Ersek <ler...@redhat.com>; Ard Biesheuvel
>> <ard.biesheu...@linaro.org>
>> Subject: [edk2] [PATCH] ShellBinPkg: Arm/AArch64 Shell binary update.
>> Importance: High
>>
>> The binaries of ShellBinPkg are generated with ShellPkg from
>> 9a8f6fb40712 ("ArmPlatformPkg: remove unused Sec library").
>>
>> This reintroduces the non-standard tftp commands that were
>> unintentionally dropped with svn r18243.
>>
>> Contributed-under: TianoCore Contribution Agreement 1.0
>> Signed-off-by: Leif Lindholm <leif.lindh...@linaro.org>
>> ---
>>
>> Binary diff not included in email, can be found at:
>> https://git.linaro.org/people/leif.lindholm/edk2.git/shortlog/refs/heads/shellb
>> inupdate
>>
>> As a side note - could we add myself and Ard as maintainers on
>> ShellBinPkg, specifically for ARM/AArch64?
>>
>>  ShellBinPkg/MinUefiShell/AArch64/Shell.efi | Bin 382304 -> 392416 bytes
>>  ShellBinPkg/MinUefiShell/Arm/Shell.efi | Bin 330336 -> 338592 bytes
>>  ShellBinPkg/UefiShell/AArch64/Shell.efi| Bin 873664 -> 893696 bytes
>>  ShellBinPkg/UefiShell/Arm/Shell.efi| Bin 769088 -> 786592 bytes
>>  4 files changed, 0 insertions(+), 0 deletions(-)
>>
>> diff --git a/ShellBinPkg/MinUefiShell/AArch64/Shell.efi
>> b/ShellBinPkg/MinUefiShell/AArch64/Shell.efi
>> index 9faeaff..53724a6 100755
>> Binary files a/ShellBinPkg/MinUefiShell/AArch64/Shell.efi and
>> b/ShellBinPkg/MinUefiShell/AArch64/Shell.efi differ
>> diff --git a/ShellBinPkg/MinUefiShell/Arm/Shell.efi
>> b/ShellBinPkg/MinUefiShell/Arm/Shell.efi
>> index 695f72a..251e0ca 100755
>> Binary files a/ShellBinPkg/MinUefiShell/Arm/Shell.efi and
>> b/ShellBinPkg/MinUefiShell/Arm/Shell.efi differ
>> diff --git a/ShellBinPkg/UefiShell/AArch64/Shell.efi
>> b/ShellBinPkg/UefiShell/AArch64/Shell.efi
>> index 76b0a03..fdfd518 100755
>> Binary files a/ShellBinPkg/UefiShell/AArch64/Shell.efi and
>> b/ShellBinPkg/UefiShell/AArch64/Shell.efi differ
>> diff --git a/ShellBinPkg/UefiShell/Arm/Shell.efi
>> b/ShellBinPkg/UefiShell/Arm/Shell.efi
>> index 4d95e08..f7a8cca 100755
>> Binary files a/ShellBinPkg/UefiShell/Arm/Shell.efi and
>> b/ShellBinPkg/UefiShell/Arm/Shell.efi differ
>> --
>> 2.1.4
>>
>> ___
>> edk2-devel mailing list
>> edk2-devel@lists.01.org
>> https://lists.01.org/mailman/listinfo/edk2-devel
___
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel

Re: [edk2] [PATCH] ShellBinPkg: Arm/AArch64 Shell binary update.

2016-02-09 Thread Laszlo Ersek 
On 02/09/16 17:53, Leif Lindholm wrote:
> Thanks!
> 
> (Argh, forgot to add your reviewed-by before I pushed!)
> 
> So, could I do something like splitting ShellPkg/ShellBinPkg and then:
> 
> ShellBinPkg
> W: https://github.com/tianocore/tianocore.github.io/wiki/ShellPkg
> M: Jaben Carsey <jaben.car...@intel.com>
> M: Shumin Qiu <shumin@intel.com>
> M: Leif Lindholm <leif.lindh...@linaro.org>(ARM/AArch64)
> M: Ard Biesheuvel <ard.biesheu...@linaro.org>  (ARM/AArch64)

Something like this would be especially important for MdeModulePkg
maintenance, which is absolutely huge.

Based on QEMU customs though, I think pathnames (directories, and
filenames with wildcards) would be more appropriate:

F: ShellBinPkg/UefiShell/AArch64
F: ShellBinPkg/UefiShell/Arm

Thanks
Laszlo

> 
> (but getting the alignment right when I have a fixed-width font)
> 
> /
> Leif
> 
> On 9 February 2016 at 16:42, Carsey, Jaben <jaben.car...@intel.com> wrote:
>> Reviewed-by: Jaben Carsey <jaben.car...@intel.com>
>>
>> And yes I agree that you both should be added as maintainers of the 
>> ShellBinPkg.
>>
>>> -Original Message-
>>> From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Leif
>>> Lindholm
>>> Sent: Tuesday, February 09, 2016 8:41 AM
>>> To: edk2-devel@lists.01.org
>>> Cc: Carsey, Jaben <jaben.car...@intel.com>; Qiu, Shumin
>>> <shumin@intel.com>; Laszlo Ersek <ler...@redhat.com>; Ard Biesheuvel
>>> <ard.biesheu...@linaro.org>
>>> Subject: [edk2] [PATCH] ShellBinPkg: Arm/AArch64 Shell binary update.
>>> Importance: High
>>>
>>> The binaries of ShellBinPkg are generated with ShellPkg from
>>> 9a8f6fb40712 ("ArmPlatformPkg: remove unused Sec library").
>>>
>>> This reintroduces the non-standard tftp commands that were
>>> unintentionally dropped with svn r18243.
>>>
>>> Contributed-under: TianoCore Contribution Agreement 1.0
>>> Signed-off-by: Leif Lindholm <leif.lindh...@linaro.org>
>>> ---
>>>
>>> Binary diff not included in email, can be found at:
>>> https://git.linaro.org/people/leif.lindholm/edk2.git/shortlog/refs/heads/shellb
>>> inupdate
>>>
>>> As a side note - could we add myself and Ard as maintainers on
>>> ShellBinPkg, specifically for ARM/AArch64?
>>>
>>>  ShellBinPkg/MinUefiShell/AArch64/Shell.efi | Bin 382304 -> 392416 bytes
>>>  ShellBinPkg/MinUefiShell/Arm/Shell.efi | Bin 330336 -> 338592 bytes
>>>  ShellBinPkg/UefiShell/AArch64/Shell.efi| Bin 873664 -> 893696 bytes
>>>  ShellBinPkg/UefiShell/Arm/Shell.efi| Bin 769088 -> 786592 bytes
>>>  4 files changed, 0 insertions(+), 0 deletions(-)
>>>
>>> diff --git a/ShellBinPkg/MinUefiShell/AArch64/Shell.efi
>>> b/ShellBinPkg/MinUefiShell/AArch64/Shell.efi
>>> index 9faeaff..53724a6 100755
>>> Binary files a/ShellBinPkg/MinUefiShell/AArch64/Shell.efi and
>>> b/ShellBinPkg/MinUefiShell/AArch64/Shell.efi differ
>>> diff --git a/ShellBinPkg/MinUefiShell/Arm/Shell.efi
>>> b/ShellBinPkg/MinUefiShell/Arm/Shell.efi
>>> index 695f72a..251e0ca 100755
>>> Binary files a/ShellBinPkg/MinUefiShell/Arm/Shell.efi and
>>> b/ShellBinPkg/MinUefiShell/Arm/Shell.efi differ
>>> diff --git a/ShellBinPkg/UefiShell/AArch64/Shell.efi
>>> b/ShellBinPkg/UefiShell/AArch64/Shell.efi
>>> index 76b0a03..fdfd518 100755
>>> Binary files a/ShellBinPkg/UefiShell/AArch64/Shell.efi and
>>> b/ShellBinPkg/UefiShell/AArch64/Shell.efi differ
>>> diff --git a/ShellBinPkg/UefiShell/Arm/Shell.efi
>>> b/ShellBinPkg/UefiShell/Arm/Shell.efi
>>> index 4d95e08..f7a8cca 100755
>>> Binary files a/ShellBinPkg/UefiShell/Arm/Shell.efi and
>>> b/ShellBinPkg/UefiShell/Arm/Shell.efi differ
>>> --
>>> 2.1.4
>>>
>>> ___
>>> edk2-devel mailing list
>>> edk2-devel@lists.01.org
>>> https://lists.01.org/mailman/listinfo/edk2-devel

___
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel

[edk2] [PATCH] ShellBinPkg: Arm/AArch64 Shell binary update.

2016-02-09 Thread Leif Lindholm 
The binaries of ShellBinPkg are generated with ShellPkg from
9a8f6fb40712 ("ArmPlatformPkg: remove unused Sec library").

This reintroduces the non-standard tftp commands that were
unintentionally dropped with svn r18243.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Leif Lindholm 
---

Binary diff not included in email, can be found at:
https://git.linaro.org/people/leif.lindholm/edk2.git/shortlog/refs/heads/shellbinupdate

As a side note - could we add myself and Ard as maintainers on
ShellBinPkg, specifically for ARM/AArch64?

 ShellBinPkg/MinUefiShell/AArch64/Shell.efi | Bin 382304 -> 392416 bytes
 ShellBinPkg/MinUefiShell/Arm/Shell.efi | Bin 330336 -> 338592 bytes
 ShellBinPkg/UefiShell/AArch64/Shell.efi| Bin 873664 -> 893696 bytes
 ShellBinPkg/UefiShell/Arm/Shell.efi| Bin 769088 -> 786592 bytes
 4 files changed, 0 insertions(+), 0 deletions(-)

diff --git a/ShellBinPkg/MinUefiShell/AArch64/Shell.efi 
b/ShellBinPkg/MinUefiShell/AArch64/Shell.efi
index 9faeaff..53724a6 100755
Binary files a/ShellBinPkg/MinUefiShell/AArch64/Shell.efi and 
b/ShellBinPkg/MinUefiShell/AArch64/Shell.efi differ
diff --git a/ShellBinPkg/MinUefiShell/Arm/Shell.efi 
b/ShellBinPkg/MinUefiShell/Arm/Shell.efi
index 695f72a..251e0ca 100755
Binary files a/ShellBinPkg/MinUefiShell/Arm/Shell.efi and 
b/ShellBinPkg/MinUefiShell/Arm/Shell.efi differ
diff --git a/ShellBinPkg/UefiShell/AArch64/Shell.efi 
b/ShellBinPkg/UefiShell/AArch64/Shell.efi
index 76b0a03..fdfd518 100755
Binary files a/ShellBinPkg/UefiShell/AArch64/Shell.efi and 
b/ShellBinPkg/UefiShell/AArch64/Shell.efi differ
diff --git a/ShellBinPkg/UefiShell/Arm/Shell.efi 
b/ShellBinPkg/UefiShell/Arm/Shell.efi
index 4d95e08..f7a8cca 100755
Binary files a/ShellBinPkg/UefiShell/Arm/Shell.efi and 
b/ShellBinPkg/UefiShell/Arm/Shell.efi differ
-- 
2.1.4

___
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel

Re: [edk2] [PATCH] ShellBinPkg: Arm/AArch64 Shell binary update.

2016-02-09 Thread Carsey, Jaben 
I would actually say mark Shumin and myself as "(Ia32/X64)" similar to marking 
you and Ard as ARM/AArch64.  I am not really involved in ARM binaries at all.

> -Original Message-
> From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Leif
> Lindholm
> Sent: Tuesday, February 09, 2016 8:54 AM
> To: Carsey, Jaben <jaben.car...@intel.com>
> Cc: edk2-devel@lists.01.org; Laszlo Ersek <ler...@redhat.com>; Qiu, Shumin
> <shumin@intel.com>; Ard Biesheuvel <ard.biesheu...@linaro.org>
> Subject: Re: [edk2] [PATCH] ShellBinPkg: Arm/AArch64 Shell binary update.
> Importance: High
> 
> Thanks!
> 
> (Argh, forgot to add your reviewed-by before I pushed!)
> 
> So, could I do something like splitting ShellPkg/ShellBinPkg and then:
> 
> ShellBinPkg
> W: https://github.com/tianocore/tianocore.github.io/wiki/ShellPkg
> M: Jaben Carsey <jaben.car...@intel.com>
> M: Shumin Qiu <shumin@intel.com>
> M: Leif Lindholm <leif.lindh...@linaro.org>(ARM/AArch64)
> M: Ard Biesheuvel <ard.biesheu...@linaro.org>  (ARM/AArch64)
> 
> (but getting the alignment right when I have a fixed-width font)
> 
> /
> Leif
> 
> On 9 February 2016 at 16:42, Carsey, Jaben <jaben.car...@intel.com>
> wrote:
> > Reviewed-by: Jaben Carsey <jaben.car...@intel.com>
> >
> > And yes I agree that you both should be added as maintainers of the
> ShellBinPkg.
> >
> >> -Original Message-
> >> From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of
> Leif
> >> Lindholm
> >> Sent: Tuesday, February 09, 2016 8:41 AM
> >> To: edk2-devel@lists.01.org
> >> Cc: Carsey, Jaben <jaben.car...@intel.com>; Qiu, Shumin
> >> <shumin@intel.com>; Laszlo Ersek <ler...@redhat.com>; Ard
> Biesheuvel
> >> <ard.biesheu...@linaro.org>
> >> Subject: [edk2] [PATCH] ShellBinPkg: Arm/AArch64 Shell binary update.
> >> Importance: High
> >>
> >> The binaries of ShellBinPkg are generated with ShellPkg from
> >> 9a8f6fb40712 ("ArmPlatformPkg: remove unused Sec library").
> >>
> >> This reintroduces the non-standard tftp commands that were
> >> unintentionally dropped with svn r18243.
> >>
> >> Contributed-under: TianoCore Contribution Agreement 1.0
> >> Signed-off-by: Leif Lindholm <leif.lindh...@linaro.org>
> >> ---
> >>
> >> Binary diff not included in email, can be found at:
> >>
> https://git.linaro.org/people/leif.lindholm/edk2.git/shortlog/refs/heads/shellb
> >> inupdate
> >>
> >> As a side note - could we add myself and Ard as maintainers on
> >> ShellBinPkg, specifically for ARM/AArch64?
> >>
> >>  ShellBinPkg/MinUefiShell/AArch64/Shell.efi | Bin 382304 -> 392416
> bytes
> >>  ShellBinPkg/MinUefiShell/Arm/Shell.efi | Bin 330336 -> 338592 bytes
> >>  ShellBinPkg/UefiShell/AArch64/Shell.efi| Bin 873664 -> 893696 bytes
> >>  ShellBinPkg/UefiShell/Arm/Shell.efi| Bin 769088 -> 786592 bytes
> >>  4 files changed, 0 insertions(+), 0 deletions(-)
> >>
> >> diff --git a/ShellBinPkg/MinUefiShell/AArch64/Shell.efi
> >> b/ShellBinPkg/MinUefiShell/AArch64/Shell.efi
> >> index 9faeaff..53724a6 100755
> >> Binary files a/ShellBinPkg/MinUefiShell/AArch64/Shell.efi and
> >> b/ShellBinPkg/MinUefiShell/AArch64/Shell.efi differ
> >> diff --git a/ShellBinPkg/MinUefiShell/Arm/Shell.efi
> >> b/ShellBinPkg/MinUefiShell/Arm/Shell.efi
> >> index 695f72a..251e0ca 100755
> >> Binary files a/ShellBinPkg/MinUefiShell/Arm/Shell.efi and
> >> b/ShellBinPkg/MinUefiShell/Arm/Shell.efi differ
> >> diff --git a/ShellBinPkg/UefiShell/AArch64/Shell.efi
> >> b/ShellBinPkg/UefiShell/AArch64/Shell.efi
> >> index 76b0a03..fdfd518 100755
> >> Binary files a/ShellBinPkg/UefiShell/AArch64/Shell.efi and
> >> b/ShellBinPkg/UefiShell/AArch64/Shell.efi differ
> >> diff --git a/ShellBinPkg/UefiShell/Arm/Shell.efi
> >> b/ShellBinPkg/UefiShell/Arm/Shell.efi
> >> index 4d95e08..f7a8cca 100755
> >> Binary files a/ShellBinPkg/UefiShell/Arm/Shell.efi and
> >> b/ShellBinPkg/UefiShell/Arm/Shell.efi differ
> >> --
> >> 2.1.4
> >>
> >> ___
> >> edk2-devel mailing list
> >> edk2-devel@lists.01.org
> >> https://lists.01.org/mailman/listinfo/edk2-devel
> ___
> edk2-devel mailing list
> edk2-devel@lists.01.org
> https://lists.01.org/mailman/listinfo/edk2-devel
___
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel

Re: [edk2] [PATCH] ShellBinPkg: Arm/AArch64 Shell binary update.

2015-11-23 Thread Leif Lindholm 
I'll take that as a reviewed-by, thanks :)

Pushed as r18925.

On Mon, Nov 23, 2015 at 05:21:16PM +, Carsey, Jaben wrote:
> Not sure how to RB this ... it's fine with me to commit it.
> 
> > -Original Message-
> > From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org]
> > Sent: Monday, November 23, 2015 5:22 AM
> > To: Leif Lindholm 
> > Cc: edk2-devel@lists.01.org; Carsey, Jaben ; Qiu,
> > Shumin 
> > Subject: Re: [PATCH] ShellBinPkg: Arm/AArch64 Shell binary update.
> > Importance: High
> > 
> > On 23 November 2015 at 14:20, Leif Lindholm 
> > wrote:
> > > The binaries of ShellBinPkg are generated with ShellPkg from r18915.
> > >
> > > Contributed-under: TianoCore Contribution Agreement 1.0
> > > Signed-off-by: Leif Lindholm 
> > > ---
> > >
> > > Patch generated with --no-binary, actual binaries available from
> > > https://git.linaro.org/people/leif.lindholm/edk2.git/shortlog/refs/hea
> > > ds/shellbinupdate
> > >
> > 
> > Acked-by: Ard Biesheuvel 
> > 
> > >  ShellBinPkg/MinUefiShell/AArch64/Shell.efi | Bin 387808 -> 382304 bytes
> > >  ShellBinPkg/MinUefiShell/Arm/Shell.efi | Bin 330208 -> 330336 bytes
> > >  ShellBinPkg/UefiShell/AArch64/Shell.efi| Bin 887136 -> 873664 bytes
> > >  ShellBinPkg/UefiShell/Arm/Shell.efi| Bin 769536 -> 769088 bytes
> > >  4 files changed, 0 insertions(+), 0 deletions(-)
> > >
> > > diff --git a/ShellBinPkg/MinUefiShell/AArch64/Shell.efi
> > > b/ShellBinPkg/MinUefiShell/AArch64/Shell.efi
> > > index d5756e1..9faeaff 100755
> > > Binary files a/ShellBinPkg/MinUefiShell/AArch64/Shell.efi and
> > > b/ShellBinPkg/MinUefiShell/AArch64/Shell.efi differ diff --git
> > > a/ShellBinPkg/MinUefiShell/Arm/Shell.efi
> > > b/ShellBinPkg/MinUefiShell/Arm/Shell.efi
> > > index 8a7d7e5..695f72a 100755
> > > Binary files a/ShellBinPkg/MinUefiShell/Arm/Shell.efi and
> > > b/ShellBinPkg/MinUefiShell/Arm/Shell.efi differ diff --git
> > > a/ShellBinPkg/UefiShell/AArch64/Shell.efi
> > > b/ShellBinPkg/UefiShell/AArch64/Shell.efi
> > > index 056da8d..76b0a03 100755
> > > Binary files a/ShellBinPkg/UefiShell/AArch64/Shell.efi and
> > > b/ShellBinPkg/UefiShell/AArch64/Shell.efi differ diff --git
> > > a/ShellBinPkg/UefiShell/Arm/Shell.efi
> > > b/ShellBinPkg/UefiShell/Arm/Shell.efi
> > > index afa91c4..4d95e08 100755
> > > Binary files a/ShellBinPkg/UefiShell/Arm/Shell.efi and
> > > b/ShellBinPkg/UefiShell/Arm/Shell.efi differ
> > > --
> > > 2.1.4
> > >
___
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel

Re: [edk2] [PATCH] ShellBinPkg: Arm/AArch64 Shell binary update.

2015-11-23 Thread Carsey, Jaben 
Not sure how to RB this ... it's fine with me to commit it.

> -Original Message-
> From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org]
> Sent: Monday, November 23, 2015 5:22 AM
> To: Leif Lindholm 
> Cc: edk2-devel@lists.01.org; Carsey, Jaben ; Qiu,
> Shumin 
> Subject: Re: [PATCH] ShellBinPkg: Arm/AArch64 Shell binary update.
> Importance: High
> 
> On 23 November 2015 at 14:20, Leif Lindholm 
> wrote:
> > The binaries of ShellBinPkg are generated with ShellPkg from r18915.
> >
> > Contributed-under: TianoCore Contribution Agreement 1.0
> > Signed-off-by: Leif Lindholm 
> > ---
> >
> > Patch generated with --no-binary, actual binaries available from
> > https://git.linaro.org/people/leif.lindholm/edk2.git/shortlog/refs/hea
> > ds/shellbinupdate
> >
> 
> Acked-by: Ard Biesheuvel 
> 
> >  ShellBinPkg/MinUefiShell/AArch64/Shell.efi | Bin 387808 -> 382304 bytes
> >  ShellBinPkg/MinUefiShell/Arm/Shell.efi | Bin 330208 -> 330336 bytes
> >  ShellBinPkg/UefiShell/AArch64/Shell.efi| Bin 887136 -> 873664 bytes
> >  ShellBinPkg/UefiShell/Arm/Shell.efi| Bin 769536 -> 769088 bytes
> >  4 files changed, 0 insertions(+), 0 deletions(-)
> >
> > diff --git a/ShellBinPkg/MinUefiShell/AArch64/Shell.efi
> > b/ShellBinPkg/MinUefiShell/AArch64/Shell.efi
> > index d5756e1..9faeaff 100755
> > Binary files a/ShellBinPkg/MinUefiShell/AArch64/Shell.efi and
> > b/ShellBinPkg/MinUefiShell/AArch64/Shell.efi differ diff --git
> > a/ShellBinPkg/MinUefiShell/Arm/Shell.efi
> > b/ShellBinPkg/MinUefiShell/Arm/Shell.efi
> > index 8a7d7e5..695f72a 100755
> > Binary files a/ShellBinPkg/MinUefiShell/Arm/Shell.efi and
> > b/ShellBinPkg/MinUefiShell/Arm/Shell.efi differ diff --git
> > a/ShellBinPkg/UefiShell/AArch64/Shell.efi
> > b/ShellBinPkg/UefiShell/AArch64/Shell.efi
> > index 056da8d..76b0a03 100755
> > Binary files a/ShellBinPkg/UefiShell/AArch64/Shell.efi and
> > b/ShellBinPkg/UefiShell/AArch64/Shell.efi differ diff --git
> > a/ShellBinPkg/UefiShell/Arm/Shell.efi
> > b/ShellBinPkg/UefiShell/Arm/Shell.efi
> > index afa91c4..4d95e08 100755
> > Binary files a/ShellBinPkg/UefiShell/Arm/Shell.efi and
> > b/ShellBinPkg/UefiShell/Arm/Shell.efi differ
> > --
> > 2.1.4
> >
___
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel

[edk2] [PATCH] ShellBinPkg: Arm/AArch64 Shell binary update.

2015-08-19 Thread Ard Biesheuvel 
The binaries of ShellBinPkg are generated with ShellPkg project 18222.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org
---

These are built with the tiny code model, note the code size reduction
for AARCH64.

Branch:
https://git.linaro.org/people/ard.biesheuvel/uefi-next.git/shortlog/refs/heads/binshell

 ShellBinPkg/MinUefiShell/AArch64/Shell.efi | Bin 408416 - 387808 bytes
 ShellBinPkg/MinUefiShell/Arm/Shell.efi | Bin 324480 - 330208 bytes
 ShellBinPkg/UefiShell/AArch64/Shell.efi| Bin 945152 - 887136 bytes
 ShellBinPkg/UefiShell/Arm/Shell.efi| Bin 757728 - 769536 bytes
 4 files changed, 0 insertions(+), 0 deletions(-)

diff --git a/ShellBinPkg/MinUefiShell/AArch64/Shell.efi 
b/ShellBinPkg/MinUefiShell/AArch64/Shell.efi
index 7f63d86230e8..d5756e101342 100755
Binary files a/ShellBinPkg/MinUefiShell/AArch64/Shell.efi and 
b/ShellBinPkg/MinUefiShell/AArch64/Shell.efi differ
diff --git a/ShellBinPkg/MinUefiShell/Arm/Shell.efi 
b/ShellBinPkg/MinUefiShell/Arm/Shell.efi
index 782489af6960..8a7d7e5b7946 100755
Binary files a/ShellBinPkg/MinUefiShell/Arm/Shell.efi and 
b/ShellBinPkg/MinUefiShell/Arm/Shell.efi differ
diff --git a/ShellBinPkg/UefiShell/AArch64/Shell.efi 
b/ShellBinPkg/UefiShell/AArch64/Shell.efi
index 5a44a8ca6189..056da8d2932a 100755
Binary files a/ShellBinPkg/UefiShell/AArch64/Shell.efi and 
b/ShellBinPkg/UefiShell/AArch64/Shell.efi differ
diff --git a/ShellBinPkg/UefiShell/Arm/Shell.efi 
b/ShellBinPkg/UefiShell/Arm/Shell.efi
index 5a61df621047..afa91c4d6caf 100755
Binary files a/ShellBinPkg/UefiShell/Arm/Shell.efi and 
b/ShellBinPkg/UefiShell/Arm/Shell.efi differ
-- 
1.9.1

___
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel

Re: [edk2] [PATCH] ShellBinPkg: Arm/AArch64 Shell binary update.

2015-08-19 Thread Leif Lindholm 
On Wed, Aug 19, 2015 at 01:46:42PM +0200, Ard Biesheuvel wrote:
 The binaries of ShellBinPkg are generated with ShellPkg project 18222.
 
 Contributed-under: TianoCore Contribution Agreement 1.0
 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org
 ---
 
 These are built with the tiny code model, note the code size reduction
 for AARCH64.

Hmm...

I would prefer to not update the binaries with this until the
-mcmode=tiny patch for BaseTools goes in, though.

 Branch:
 https://git.linaro.org/people/ard.biesheuvel/uefi-next.git/shortlog/refs/heads/binshell
 
  ShellBinPkg/MinUefiShell/AArch64/Shell.efi | Bin 408416 - 387808 bytes
  ShellBinPkg/MinUefiShell/Arm/Shell.efi | Bin 324480 - 330208 bytes
  ShellBinPkg/UefiShell/AArch64/Shell.efi| Bin 945152 - 887136 bytes
  ShellBinPkg/UefiShell/Arm/Shell.efi| Bin 757728 - 769536 bytes
  4 files changed, 0 insertions(+), 0 deletions(-)

Very nice!

 diff --git a/ShellBinPkg/MinUefiShell/AArch64/Shell.efi 
 b/ShellBinPkg/MinUefiShell/AArch64/Shell.efi
 index 7f63d86230e8..d5756e101342 100755
 Binary files a/ShellBinPkg/MinUefiShell/AArch64/Shell.efi and 
 b/ShellBinPkg/MinUefiShell/AArch64/Shell.efi differ
 diff --git a/ShellBinPkg/MinUefiShell/Arm/Shell.efi 
 b/ShellBinPkg/MinUefiShell/Arm/Shell.efi
 index 782489af6960..8a7d7e5b7946 100755
 Binary files a/ShellBinPkg/MinUefiShell/Arm/Shell.efi and 
 b/ShellBinPkg/MinUefiShell/Arm/Shell.efi differ
 diff --git a/ShellBinPkg/UefiShell/AArch64/Shell.efi 
 b/ShellBinPkg/UefiShell/AArch64/Shell.efi
 index 5a44a8ca6189..056da8d2932a 100755
 Binary files a/ShellBinPkg/UefiShell/AArch64/Shell.efi and 
 b/ShellBinPkg/UefiShell/AArch64/Shell.efi differ
 diff --git a/ShellBinPkg/UefiShell/Arm/Shell.efi 
 b/ShellBinPkg/UefiShell/Arm/Shell.efi
 index 5a61df621047..afa91c4d6caf 100755
 Binary files a/ShellBinPkg/UefiShell/Arm/Shell.efi and 
 b/ShellBinPkg/UefiShell/Arm/Shell.efi differ
 -- 
 1.9.1
 
___
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel

Re: [edk2] [PATCH] ShellBinPkg: Arm/AArch64 Shell binary update.

2015-08-19 Thread Carsey, Jaben 
Reviewed-by: Jaben Carsey jaben.car...@intel.com

 -Original Message-
 From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of
 Leif Lindholm
 Sent: Wednesday, August 19, 2015 8:53 AM
 To: Ard Biesheuvel ard.biesheu...@linaro.org
 Cc: edk2-devel@lists.01.org; ryan.har...@linaro.org
 Subject: Re: [edk2] [PATCH] ShellBinPkg: Arm/AArch64 Shell binary update.
 Importance: High
 
 On Wed, Aug 19, 2015 at 01:46:42PM +0200, Ard Biesheuvel wrote:
  The binaries of ShellBinPkg are generated with ShellPkg project 18222.
 
  Contributed-under: TianoCore Contribution Agreement 1.0
  Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org
  ---
 
  These are built with the tiny code model, note the code size reduction
  for AARCH64.
 
 Hmm...
 
 I would prefer to not update the binaries with this until the
 -mcmode=tiny patch for BaseTools goes in, though.
 
  Branch:
  https://git.linaro.org/people/ard.biesheuvel/uefi-
 next.git/shortlog/refs/heads/binshell
 
   ShellBinPkg/MinUefiShell/AArch64/Shell.efi | Bin 408416 - 387808 bytes
   ShellBinPkg/MinUefiShell/Arm/Shell.efi | Bin 324480 - 330208 bytes
   ShellBinPkg/UefiShell/AArch64/Shell.efi| Bin 945152 - 887136 bytes
   ShellBinPkg/UefiShell/Arm/Shell.efi| Bin 757728 - 769536 bytes
   4 files changed, 0 insertions(+), 0 deletions(-)
 
 Very nice!
 
  diff --git a/ShellBinPkg/MinUefiShell/AArch64/Shell.efi
 b/ShellBinPkg/MinUefiShell/AArch64/Shell.efi
  index 7f63d86230e8..d5756e101342 100755
  Binary files a/ShellBinPkg/MinUefiShell/AArch64/Shell.efi and
 b/ShellBinPkg/MinUefiShell/AArch64/Shell.efi differ
  diff --git a/ShellBinPkg/MinUefiShell/Arm/Shell.efi
 b/ShellBinPkg/MinUefiShell/Arm/Shell.efi
  index 782489af6960..8a7d7e5b7946 100755
  Binary files a/ShellBinPkg/MinUefiShell/Arm/Shell.efi and
 b/ShellBinPkg/MinUefiShell/Arm/Shell.efi differ
  diff --git a/ShellBinPkg/UefiShell/AArch64/Shell.efi
 b/ShellBinPkg/UefiShell/AArch64/Shell.efi
  index 5a44a8ca6189..056da8d2932a 100755
  Binary files a/ShellBinPkg/UefiShell/AArch64/Shell.efi and
 b/ShellBinPkg/UefiShell/AArch64/Shell.efi differ
  diff --git a/ShellBinPkg/UefiShell/Arm/Shell.efi
 b/ShellBinPkg/UefiShell/Arm/Shell.efi
  index 5a61df621047..afa91c4d6caf 100755
  Binary files a/ShellBinPkg/UefiShell/Arm/Shell.efi and
 b/ShellBinPkg/UefiShell/Arm/Shell.efi differ
  --
  1.9.1
 
 ___
 edk2-devel mailing list
 edk2-devel@lists.01.org
 https://lists.01.org/mailman/listinfo/edk2-devel
___
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel