Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] Binutils version used on MIPS
@ 2014-07-01 18:14 Thomas Petazzoni
  2014-07-02  7:51 ` Markos Chandras
  0 siblings, 1 reply; 5+ messages in thread
From: Thomas Petazzoni @ 2014-07-01 18:14 UTC (permalink / raw)
  To: buildroot

Vicente, Markos,

For the MIPS architecture, the current default binutils version in the
Buildroot internal toolchain backend is 2.21:

        prompt "Binutils Version"
        depends on !BR2_arc
        default BR2_BINUTILS_VERSION_2_21 if (BR2_mips || BR2_mipsel || BR2_sh)
        default BR2_BINUTILS_VERSION_2_24 if BR2_microblaze || BR2_powerpc64le
        default BR2_BINUTILS_VERSION_2_22

Do you know if there's a good reason to keep using 2.21, as opposed to
a more recent version such as 2.23 or 2.24 ? It would be good to have a
little bit of testing done with binutils 2.23 or 2.24 and if it's
successful, bump the default version used on MIPS.

Thanks!

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

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [Buildroot] Binutils version used on MIPS
  2014-07-01 18:14 [Buildroot] Binutils version used on MIPS Thomas Petazzoni
@ 2014-07-02  7:51 ` Markos Chandras
  2014-07-02 15:23   ` Peter Korsgaard
  0 siblings, 1 reply; 5+ messages in thread
From: Markos Chandras @ 2014-07-02  7:51 UTC (permalink / raw)
  To: buildroot

On 07/01/2014 07:14 PM, Thomas Petazzoni wrote:
> Vicente, Markos,
> 
> For the MIPS architecture, the current default binutils version in the
> Buildroot internal toolchain backend is 2.21:
> 
>         prompt "Binutils Version"
>         depends on !BR2_arc
>         default BR2_BINUTILS_VERSION_2_21 if (BR2_mips || BR2_mipsel || BR2_sh)
>         default BR2_BINUTILS_VERSION_2_24 if BR2_microblaze || BR2_powerpc64le
>         default BR2_BINUTILS_VERSION_2_22
> 
> Do you know if there's a good reason to keep using 2.21, as opposed to
> a more recent version such as 2.23 or 2.24 ? It would be good to have a
> little bit of testing done with binutils 2.23 or 2.24 and if it's
> successful, bump the default version used on MIPS.
> 
> Thanks!
> 
> Thomas
> 
Hi Thomas,

No good reason and thanks for noticing. We will send a patch to use the
latest one by default.

-- 
markos

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [Buildroot] Binutils version used on MIPS
  2014-07-02  7:51 ` Markos Chandras
@ 2014-07-02 15:23   ` Peter Korsgaard
  2014-07-02 15:46     ` Thomas Petazzoni
  0 siblings, 1 reply; 5+ messages in thread
From: Peter Korsgaard @ 2014-07-02 15:23 UTC (permalink / raw)
  To: buildroot

>>>>> "Markos" == Markos Chandras <Markos.Chandras@imgtec.com> writes:

Hi,

 > No good reason and thanks for noticing. We will send a patch to use the
 > latest one by default.

FYI, the current (arch independent) default binutils version is 2.22,
not 2.24.

We moved from 2.21 to 2.22 back in october:

commit 9777dbedb012f68e54e14f9567d9dc0187d53b17
Author: Peter Korsgaard <peter@korsgaard.com>
Date:   Sun Oct 6 23:45:10 2013 +0200

    binutils: default to 2.22
    
    2.21.1 is getting quite old, the linker issues with 2.22 seems to be
    (mostly?) sorted out by now and armhf needs 2.22+, so let's bump the
    default version.
    
    Signed-off-by: Peter Korsgaard <peter@korsgaard.com>


Time has perhaps come to move to 2.23 or 2.24, I don't really know.

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [Buildroot] Binutils version used on MIPS
  2014-07-02 15:23   ` Peter Korsgaard
@ 2014-07-02 15:46     ` Thomas Petazzoni
  2014-07-02 19:51       ` Peter Korsgaard
  0 siblings, 1 reply; 5+ messages in thread
From: Thomas Petazzoni @ 2014-07-02 15:46 UTC (permalink / raw)
  To: buildroot

Dear Peter Korsgaard,

On Wed, 02 Jul 2014 17:23:26 +0200, Peter Korsgaard wrote:

> FYI, the current (arch independent) default binutils version is 2.22,
> not 2.24.
> 
> We moved from 2.21 to 2.22 back in october:
> 
> commit 9777dbedb012f68e54e14f9567d9dc0187d53b17
> Author: Peter Korsgaard <peter@korsgaard.com>
> Date:   Sun Oct 6 23:45:10 2013 +0200
> 
>     binutils: default to 2.22
>     
>     2.21.1 is getting quite old, the linker issues with 2.22 seems to be
>     (mostly?) sorted out by now and armhf needs 2.22+, so let's bump the
>     default version.
>     
>     Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
> 
> 
> Time has perhaps come to move to 2.23 or 2.24, I don't really know.

Right, but Vicente patch simply removes the usage of a special version
for MIPS, so MIPS now just uses whatever is the default version used
for all architectures in Buildroot. Which I think is the right thing to
do, unless there are special reasons to use a different version than
the default one (such as AVR32 need of an old version, or
Microblaze/PowerPC64LE need of a new version).

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

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [Buildroot] Binutils version used on MIPS
  2014-07-02 15:46     ` Thomas Petazzoni
@ 2014-07-02 19:51       ` Peter Korsgaard
  0 siblings, 0 replies; 5+ messages in thread
From: Peter Korsgaard @ 2014-07-02 19:51 UTC (permalink / raw)
  To: buildroot

>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni@free-electrons.com> writes:

Hi,

 >> Time has perhaps come to move to 2.23 or 2.24, I don't really know.

 > Right, but Vicente patch simply removes the usage of a special version
 > for MIPS, so MIPS now just uses whatever is the default version used
 > for all architectures in Buildroot. Which I think is the right thing to
 > do, unless there are special reasons to use a different version than
 > the default one (such as AVR32 need of an old version, or
 > Microblaze/PowerPC64LE need of a new version).

Yes, I completely agree, my comment was in regard to the references to
2.23 / 2.24.

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2014-07-02 19:51 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-07-01 18:14 [Buildroot] Binutils version used on MIPS Thomas Petazzoni
2014-07-02  7:51 ` Markos Chandras
2014-07-02 15:23   ` Peter Korsgaard
2014-07-02 15:46     ` Thomas Petazzoni
2014-07-02 19:51       ` Peter Korsgaard

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox