All of lore.kernel.org
 help / color / mirror / Atom feed
From: Arnout Vandecappelle <arnout@mind.be>
To: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
Cc: Kory Maincent <kory.maincent@bootlin.com>, buildroot@buildroot.org
Subject: Re: [Buildroot] [PATCH] package/arm-gnu-a-toolchain: bump to version 10.3-2021.07
Date: Mon, 20 Sep 2021 11:18:34 +0200	[thread overview]
Message-ID: <f8756bf0-c955-f301-299e-f5d8c0aabe04@mind.be> (raw)
In-Reply-To: <20210919150904.04051568@windsurf>



On 19/09/2021 15:09, Thomas Petazzoni wrote:
> Hello,
> 
> On Sun, 19 Sep 2021 14:35:56 +0200
> Arnout Vandecappelle <arnout@mind.be> wrote:
> 
>>> -ARM_GNU_A_TOOLCHAIN_VERSION = 9.2-2019.12
>>> -ARM_GNU_A_TOOLCHAIN_SITE = https://developer.arm.com/-/media/Files/downloads/gnu-a/$(ARM_GNU_A_TOOLCHAIN_VERSION)/binrel
>>> -ARM_GNU_A_TOOLCHAIN_SOURCE = gcc-arm-$(ARM_GNU_A_TOOLCHAIN_VERSION)-x86_64-arm-none-eabi.tar.xz
>>> +ARM_GNU_A_TOOLCHAIN_VERSION = 10.3-2021.07
>>> +ARM_GNU_A_TOOLCHAIN_SITE = https://developer.arm.com/-/media/Files/downloads/gnu-rm/$(ARM_GNU_A_TOOLCHAIN_VERSION)
>>> +ARM_GNU_A_TOOLCHAIN_SOURCE = gcc-arm-none-eabi-$(ARM_GNU_A_TOOLCHAIN_VERSION)-x86_64-linux.tar.bz2
>>
>>    Err, this can't be right... 'gnu-rm' stands for "GNU-based toolchain for
>> Cortex-R and Cortex-M", and "arm-none-eabi" means it's a toolchain without libc...
>>
>>    So either someone made a huge mistake at ARM, or you haven't actually tested
>> this patch :-)

  Or, of course, I can't read and I see 
toolchain/toolchain-external/toolchain-external-arm-arm instead of 
package/arm-gnu-a-toolchain...

  Worse still: I didn't see that the old one already *was* arm-none-eabi, it was 
just specified at the end of the tarball name instead of at the beginning.


> It was tested to build code for a Cortex-M4.
> 
> This package was originally introduced to build ARM 32-bit code in the
> context of TF-A (boot/arm-trusted-firmware/). At the very beginning,
> was submitted by a Marvell person, who also had to build code for a
> Cortex-M3 co-processor IIRC, but then was taken over by a person from
> Amarula, who needed it to build ARM 32-bit code running on a Cortex-A.
> 
> If it's truly a bare-metal toolchain with no libraries, it should not
> matter, right? You just pass to gcc -march=XYZ, and it will produce the
> right code. However, my understanding is that those bare-metal
> toolchains do have some libraries built for the target, which can
> potentially get linked with the code produced.

  Yeah, you get libgcc and libstdc++ from the gcc source. libstdc++ is probably 
not used . libgcc may not be linked in either if -nostdlib is used (which it 
probably is).


> So, long story short: do we need arm-gnu-a-toolchaind and
> arm-gnu-rm-toolchain packages ?

  Easy to check: build a bunch of the defconfigs which set 
BR2_TARGET_ARM_TRUSTED_FIRMWARE_NEEDS_ARM32_TOOLCHAIN=y. And maybe double-check 
if indeed ATF doesn't link with libgcc.

  Regards,
  Arnout
_______________________________________________
buildroot mailing list
buildroot@lists.buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

  reply	other threads:[~2021-09-20  9:18 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-09-08  8:16 [Buildroot] [PATCH] package/arm-gnu-a-toolchain: bump to version 10.3-2021.07 Kory Maincent
2021-09-19 12:35 ` Arnout Vandecappelle
2021-09-19 13:09   ` Thomas Petazzoni
2021-09-20  9:18     ` Arnout Vandecappelle [this message]
2021-09-20 12:01       ` Thomas Petazzoni
2021-09-20 18:52         ` Arnout Vandecappelle
2021-09-23 19:55           ` Thomas Petazzoni
2021-09-23 21:20             ` Arnout Vandecappelle
2021-11-30 10:37               ` Köry Maincent
2021-12-30 15:59 ` Thomas Petazzoni

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=f8756bf0-c955-f301-299e-f5d8c0aabe04@mind.be \
    --to=arnout@mind.be \
    --cc=buildroot@buildroot.org \
    --cc=kory.maincent@bootlin.com \
    --cc=thomas.petazzoni@bootlin.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.