diff mbox

mx53loco_defconfig: refresh

Message ID 1413558308-30081-1-git-send-email-vincent.stehle@freescale.com
State Changes Requested
Headers show

Commit Message

Vincent Stehlé Oct. 17, 2014, 3:05 p.m. UTC
Force gcc to version 4.5, as versions 4.7 and 4.8 break the boot.

Switch to VFPv3(d32), as i.MX53 Cortex-A8 supports it (MVFR0 = 0x11110222).

Signed-off-by: Vincent Stehlé <vincent.stehle@freescale.com>
Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Cc: Peter Korsgaard <jacmet@sunsite.dk>
---
 configs/mx53loco_defconfig | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

Comments

Thomas Petazzoni Oct. 18, 2014, 4:35 p.m. UTC | #1
Dear Vincent Stehlé,

On Fri, 17 Oct 2014 17:05:08 +0200, Vincent Stehlé wrote:
> Force gcc to version 4.5, as versions 4.7 and 4.8 break the boot.
> 
> Switch to VFPv3(d32), as i.MX53 Cortex-A8 supports it (MVFR0 = 0x11110222).

This is quite worrying. Gcc 4.5 is very old now, and we would like to
get rid of it in Buildroot rather than adding more defconfig that use
it.

Is there some hope to get a fix in a more recent gcc such as 4.9.x ?
Has the specific gcc bug been identified and reported ?

Thanks,

Thomas
Eric Benard Oct. 18, 2014, 8:05 p.m. UTC | #2
Hi Thomas, Hi Vincent,

Le Sat, 18 Oct 2014 18:35:34 +0200,
Thomas Petazzoni <thomas.petazzoni@free-electrons.com> a écrit :

> On Fri, 17 Oct 2014 17:05:08 +0200, Vincent Stehlé wrote:
> > Force gcc to version 4.5, as versions 4.7 and 4.8 break the boot.
> > 
> > Switch to VFPv3(d32), as i.MX53 Cortex-A8 supports it (MVFR0 = 0x11110222).
> 
> This is quite worrying. Gcc 4.5 is very old now, and we would like to
> get rid of it in Buildroot rather than adding more defconfig that use
> it.
> 
> Is there some hope to get a fix in a more recent gcc such as 4.9.x ?
> Has the specific gcc bug been identified and reported ?
> 
if you switch to imx_2.6.35_maintain branch instead of older
rel_imx_2.6.35_11.09.01 actually used in the defconfig, you will get
(among other fixes) these patches :
http://git.freescale.com/git/cgit.cgi/imx/linux-2.6-imx.git/commit/?h=imx_2.6.35_maintain&id=96a7591a6f72db6d7bb337e78d61c53f670d9793
http://git.freescale.com/git/cgit.cgi/imx/linux-2.6-imx.git/commit/?h=imx_2.6.35_maintain&id=a0de0f94dd9868d2280aa8cdb8295231e00d5ef6
and I believe the kernel built with recent gcc will run as it runs fine
on i.MX53 when built with Yocto.

Best regards,
Eric
Thomas Petazzoni Oct. 19, 2014, 2:39 p.m. UTC | #3
Dear Vincent Stehlé,

On Fri, 17 Oct 2014 17:05:08 +0200, Vincent Stehlé wrote:
> Force gcc to version 4.5, as versions 4.7 and 4.8 break the boot.
> 
> Switch to VFPv3(d32), as i.MX53 Cortex-A8 supports it (MVFR0 = 0x11110222).
> 
> Signed-off-by: Vincent Stehlé <vincent.stehle@freescale.com>
> Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> Cc: Peter Korsgaard <jacmet@sunsite.dk>
> ---
>  configs/mx53loco_defconfig | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)

Can you update the patch according to Eric Benard's comment?

In the mean time, I'll mark your patch as "Changes Requested" in our
patch tracking system.

Thanks!

Thomas
diff mbox

Patch

diff --git a/configs/mx53loco_defconfig b/configs/mx53loco_defconfig
index 8b12238..f0d13bb 100644
--- a/configs/mx53loco_defconfig
+++ b/configs/mx53loco_defconfig
@@ -1,7 +1,11 @@ 
 # Architecture
 BR2_arm=y
 BR2_cortex_a8=y
-BR2_ARM_FPU_VFPV3D16=y
+BR2_ARM_FPU_VFPV3=y
+
+# toolchain
+# Use gcc 4.5, as gcc 4.7 & 4.8 break the boot.
+BR2_GCC_VERSION_4_5_X=y
 
 # System
 BR2_TARGET_GENERIC_GETTY_PORT="ttymxc0"