Re: [U-Boot] [PATCH v2 02/23] armv8: prevent using THUMB

2016-12-05 Thread Tom Rini 
On Mon, Dec 05, 2016 at 01:52:09AM +, Andre Przywara wrote:

> The predominantely 32-bit ARM targets try to compile the SPL in Thumb
> mode to reduce code size.
> The 64-bit AArch64 instruction set does not know an alternative, concise
> encoding, so the Thumb build option should only be set for 32-bit
> targets.
> Likewise -marm machine options are only valid for ARMv7 targets.
> 
> Signed-off-by: Andre Przywara 
> Reviewed-by: Alexander Graf 

Reviewed-by: Tom Rini 

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v2 02/23] armv8: prevent using THUMB

2016-12-04 Thread Maxime Ripard 
On Mon, Dec 05, 2016 at 01:52:09AM +, Andre Przywara wrote:
> The predominantely 32-bit ARM targets try to compile the SPL in Thumb
> mode to reduce code size.
> The 64-bit AArch64 instruction set does not know an alternative, concise
> encoding, so the Thumb build option should only be set for 32-bit
> targets.
> Likewise -marm machine options are only valid for ARMv7 targets.
> 
> Signed-off-by: Andre Przywara 
> Reviewed-by: Alexander Graf 

Acked-by: Maxime Ripard 

Thanks,
Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v2 02/23] armv8: prevent using THUMB

2016-12-04 Thread Simon Glass 
On 4 December 2016 at 18:52, Andre Przywara  wrote:
> The predominantely 32-bit ARM targets try to compile the SPL in Thumb
> mode to reduce code size.
> The 64-bit AArch64 instruction set does not know an alternative, concise
> encoding, so the Thumb build option should only be set for 32-bit
> targets.
> Likewise -marm machine options are only valid for ARMv7 targets.
>
> Signed-off-by: Andre Przywara 
> Reviewed-by: Alexander Graf 
> ---
>  arch/arm/lib/Makefile  | 2 ++
>  include/configs/sunxi-common.h | 2 +-
>  2 files changed, 3 insertions(+), 1 deletion(-)

Reviewed-by: Simon Glass 
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot