Skip to contentSkip to conversation

mx53loco_defconfig: refresh

PATCH New Vincent Stehlé 4 participants 15 messages Oct 17, 2014 – Nov 26, 2014

lore
Share & tools
lore

Vincent Stehlé

author
#0 lore raw

PATCH 1/1 mx53loco_defconfig: refresh

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? <[email protected]>
Cc Thomas Petazzoni <[email protected]>
Cc Peter Korsgaard <[email protected]>
---
 configs/mx53loco_defconfig | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)
configs/mx53loco_defconfig +5 -1
index 8b12238..f0d13bb 100644
@@ -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"
Signature · 1 line
2.1.1
#1 lore raw

PATCH 1/1 mx53loco_defconfig: refresh

Dear Vincent Stehl?,

Quoted 3 lines · Vincent Stehl?

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

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
#2 lore raw

PATCH 1/1 mx53loco_defconfig: refresh

Hi Thomas, Hi Vincent,

Le Sat, 18 Oct 2014 18:35:34 +0200,
Thomas Petazzoni <[email protected]> a ?crit :

Quoted 12 lines · Vincent Stehl?
Quoted 3 lines

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

#3 lore raw

PATCH 1/1 mx53loco_defconfig: refresh

In reply to Vincent Stehlé

Dear Vincent Stehl?,

Quoted 10 lines · Vincent Stehl?

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? <[email protected]>
Cc Thomas Petazzoni <[email protected]>
Cc Peter Korsgaard <[email protected]>
---
 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

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
author
#4 lore raw

PATCH v2 1/1 mx53loco_defconfig: switch to VFPv3(d32)

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

While at it, switch to EABIHF.

Signed-off-by Vincent Stehl? <[email protected]>
Cc Thomas Petazzoni <[email protected]>
Cc Peter Korsgaard <[email protected]>

---

Hi,

This is v2 of an old patch for mx53loco (patchwork id 400549, "mx53loco_defconfig: refresh").

It boots fine on an i.MX53 QSB on top of latest buildroot, commit ccec35e342a38537fba746a483eec063b5151d3e.

Changes since v1:
~~~~~~~~~~~~~~~~~
- Do not force gcc version anymore.
- Change the title accordingly.
- Switch to EABIHF.

 configs/mx53loco_defconfig | 3 +++
 1 file changed, 3 insertions(+)
configs/mx53loco_defconfig +3 -0
index 942dedf..6af42a7 100644
@@ -1,12 +1,15 @@
 # Architecture
 BR2_arm=y
 BR2_cortex_a8=y
+BR2_ARM_EABIHF=y
+BR2_ARM_FPU_VFPV3=y
 
 # System
 BR2_KERNEL_HEADERS_VERSION=y
 BR2_DEFAULT_KERNEL_VERSION="2.6.35"
 BR2_TARGET_GENERIC_GETTY_PORT="ttymxc0"
 
+
 # Filesystem
 BR2_TARGET_ROOTFS_EXT2=y
Signature · 1 line
2.1.3
#5 lore raw

PATCH v2 1/1 mx53loco_defconfig: switch to VFPv3(d32)

Hi Vincent,

Le Tue, 25 Nov 2014 17:12:23 +0100,
Vincent Stehl? <[email protected]> a ?crit :

Quoted 4 lines

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

While at it, switch to EABIHF.

unless mistaken, switching to hard float prevent the usage of the AMD GPU libraries as these binaries are soft float only.

May you please ask inside Freescale to provide AMD GPU libraries compiled for hard float ? (we don't need support nor warranty, we just need the binaries compiled for hard float to try if that can work).

Many thanks
Eric

author
#6 lore raw

PATCH v2 1/1 mx53loco_defconfig: switch to VFPv3(d32)

On 11/25/2014 05:54 PM, Eric B?nard wrote:
(switching i.mx53 to hard floats)

Quoted 2 lines

unless mistaken, switching to hard float prevent the usage of the AMD GPU libraries as these binaries are soft float only.

You are right of course; silly me :-S

Quoted 3 lines

May you please ask inside Freescale to provide AMD GPU libraries compiled for hard float ? (we don't need support nor warranty, we just need the binaries compiled for hard float to try if that can work).

Ok, I'll ask. No promise that you will get an answer, though :)

...also I wonder, shall I maybe submit a "v3" of the patch, with no EABIHF?

Best regards,

V.

#7 lore raw

PATCH v2 1/1 mx53loco_defconfig: switch to VFPv3(d32)

Dear Vincent Stehl?,

Quoted 7 lines · Vincent Stehl?
Quoted 3 lines

May you please ask inside Freescale to provide AMD GPU libraries compiled for hard float ? (we don't need support nor warranty, we just need the binaries compiled for hard float to try if that can work).

Ok, I'll ask. No promise that you will get an answer, though :)

...also I wonder, shall I maybe submit a "v3" of the patch, with no EABIHF?

I'd say yes. Is 2.6.35 the latest kernel available for this platform, or could we switch to a more recent version.

Also, Eric, you mention the AMD GPU libraries, but do we have packages for those libraries in Buildroot? I know we have the ones for i.MX6, but what about i.MX53 ?

Thanks,

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
#8 lore raw

PATCH v2 1/1 mx53loco_defconfig: switch to VFPv3(d32)

Hi Thomas,

Le Tue, 25 Nov 2014 18:30:17 +0100,
Thomas Petazzoni <[email protected]> a ?crit :

Quoted 8 lines · Vincent Stehl?
Quoted 7 lines
Quoted 4 lines

May you please ask inside Freescale to provide AMD GPU libraries compiled for hard float ? (we don't need support nor warranty, we just need the binaries compiled for hard float to try if that can work).

Ok, I'll ask. No promise that you will get an answer, though :)

Quoted 8 lines
Quoted 1 line

...also I wonder, shall I maybe submit a "v3" of the patch, with no EABIHF?

I'd say yes. Is 2.6.35 the latest kernel available for this platform, or could we switch to a more recent version.

Also, Eric, you mention the AMD GPU libraries, but do we have packages for those libraries in Buildroot? I know we have the ones for i.MX6, but what about i.MX53 ?

a patch adding the AMD GPU libraries was sent earlier this month by J?r?me Pouiller : http://patchwork.ozlabs.org/patch/408174/
I can't find the new version but a patch depending on this one is available so I assume a v2 will follow :
http://patchwork.ozlabs.org/patch/412521/

Eric

#9 lore raw

PATCH v2 1/1 mx53loco_defconfig: switch to VFPv3(d32)

Hello Eric,

Quoted 1 line · On Tuesday 25 November 2014 18:44:29 Eric B?nard wrote:

Le Tue, 25 Nov 2014 18:30:17 +0100,

[...]

Quoted 9 lines
Quoted 3 lines

Also, Eric, you mention the AMD GPU libraries, but do we have packages for those libraries in Buildroot? I know we have the ones for i.MX6, but what about i.MX53 ?

a patch adding the AMD GPU libraries was sent earlier this month by J?r?me Pouiller : http://patchwork.ozlabs.org/patch/408174/
I can't find the new version but a patch depending on this one is available so I assume a v2 will follow :
http://patchwork.ozlabs.org/patch/412521/

They are two separate series. First one introduce iMX5 GPU driver while second one introduce a new helper function to extract FSL archives.

I currently do not plan to make a v2 of iMX 5 GPU series.

BR,

Signature · 3 lines
J?r?me Pouiller, Sysmic
Embedded Linux specialist
http://www.sysmic.fr
#10 lore raw

PATCH v2 1/1 mx53loco_defconfig: switch to VFPv3(d32)

Hi J?r?me,

Le Tue, 25 Nov 2014 12:57:00 -0800 (PST),
J?r?me Pouiller <[email protected]> a ?crit :

Quoted 7 lines · On Tuesday 25 November 2014 18:44:29 Eric B?nard wrote:
Quoted 1 line

Le Tue, 25 Nov 2014 18:30:17 +0100,

[...]

Quoted 4 lines
Quoted 3 lines

Also, Eric, you mention the AMD GPU libraries, but do we have packages for those libraries in Buildroot? I know we have the ones for i.MX6, but what about i.MX53 ?

Quoted 8 lines
Quoted 5 lines

a patch adding the AMD GPU libraries was sent earlier this month by J?r?me Pouiller : http://patchwork.ozlabs.org/patch/408174/
I can't find the new version but a patch depending on this one is available so I assume a v2 will follow :
http://patchwork.ozlabs.org/patch/412521/

They are two separate series. First one introduce iMX5 GPU driver while second one introduce a new helper function to extract FSL archives.

Yes that's why I wrote the second depends on the first one ;-)

Quoted 2 lines

I currently do not plan to make a v2 of iMX 5 GPU series.

v1 is marked as superseded in patchwork and doesn't seems to be in next : is that intended ?

Best regards,
Eric

#11 lore raw

PATCH v2 1/1 mx53loco_defconfig: switch to VFPv3(d32)

Quoted 9 lines · On Tuesday 25 November 2014 22:10:51 Eric B?nard wrote:

Hi J?r?me,

Le Tue, 25 Nov 2014 12:57:00 -0800 (PST),

J?r?me Pouiller <[email protected]> a ?crit :

Quoted 4 lines · On Tuesday 25 November 2014 18:44:29 Eric B?nard wrote:
Quoted 1 line

Le Tue, 25 Nov 2014 18:30:17 +0100,

[...]

[...]

Quoted 4 lines
Quoted 1 line

I currently do not plan to make a v2 of iMX 5 GPU series.

v1 is marked as superseded in patchwork and doesn't seems to be in next : is that intended ?

I understand better why you expected a v2 :-) .

It seems to be a mistake. I revert statuses of iMX 5 GPU series.

BR,

Signature · 3 lines
J?r?me Pouiller, Sysmic
Embedded Linux specialist
http://www.sysmic.fr
author
#12 lore raw

PATCH v2 1/1 mx53loco_defconfig: switch to VFPv3(d32)

In reply to Thomas Petazzoni

On 11/25/2014 06:30 PM, Thomas Petazzoni wrote:
(re-submit a v3)

Quoted 1 line

I'd say yes.

Ok, will do.

Quoted 2 lines

Is 2.6.35 the latest kernel available for this platform, or could we switch to a more recent version.

The kernel b3912bb used in buildroot for mx53loco is more advanced than the latest official release 2.6.35 11.09 from Freescale on the Quick Start Board [1]. Nothing to update there I would say.

Best regards,

V.

[1]
http://www.freescale.com/webapp/sps/site/prod_summary.jsp?code=IMX53QSB&fpsp=1&tab=Design_Tools_Tab

author
#13 lore raw

PATCH v3 1/1 mx53loco_defconfig: switch to VFPv3(d32)

In reply to Thomas Petazzoni

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

Signed-off-by Vincent Stehl? <[email protected]>
Cc Thomas Petazzoni <[email protected]>
Cc Peter Korsgaard <[email protected]>
Cc Eric B?nard <[email protected]>

---

Here is v3; sorry for the noise.

Changes since v2:
~~~~~~~~~~~~~~~~~
- Switch back to EABI.

Changes since v1:
~~~~~~~~~~~~~~~~~
- Do not force gcc version anymore.
- Change the title accordingly.
- Switch to EABIHF.

 configs/mx53loco_defconfig | 1 +
 1 file changed, 1 insertion(+)
configs/mx53loco_defconfig +1 -0
index 942dedf..d2a2b66 100644
@@ -1,6 +1,7 @@
 # Architecture
 BR2_arm=y
 BR2_cortex_a8=y
+BR2_ARM_FPU_VFPV3=y
 
 # System
 BR2_KERNEL_HEADERS_VERSION=y
Signature · 1 line
2.1.3
Applied
#14 lore raw

PATCH v3 1/1 mx53loco_defconfig: switch to VFPv3(d32)

Dear Vincent Stehl?,

Quoted 6 lines · Vincent Stehl?

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

Signed-off-by Vincent Stehl? <[email protected]>
Cc Thomas Petazzoni <[email protected]>
Cc Peter Korsgaard <[email protected]>
Cc Eric B?nard <[email protected]>

Applied to next, thanks.

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com