* [Buildroot] [PATCH] package/binutils: extend the 2.28 default to thumb mode
@ 2018-06-01 17:26 Yann E. MORIN
2018-06-01 17:32 ` Peter Korsgaard
2018-06-17 15:03 ` Peter Korsgaard
0 siblings, 2 replies; 3+ messages in thread
From: Yann E. MORIN @ 2018-06-01 17:26 UTC (permalink / raw)
To: buildroot
Commit 17f352ac (package/binutils: default to 2.29 for Cortex-M targets)
made the default version 2.28 (and not 2.29!) when the target is an
arm-v7m CPU.
However, the real trigger is compilation in Thumb mode, not the fact
that the target is v7m.
The fact that it was noticed on a v7m target is because Thumb is the
only mode valid on those CPUs.
Tighten the defaults to 2.28 for Thumb and Thumb2 modes.
Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Cc: Laurent GONZALEZ <br22@gezedo.com>
Cc: Christophe Priouzeau <christophe.priouzeau@st.com>
Cc: Peter Korsgaard <peter@korsgaard.com>
Cc: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
Cc: Arnout Vandecappelle <arnout@mind.be>
---
package/binutils/Config.in.host | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/package/binutils/Config.in.host b/package/binutils/Config.in.host
index 4e578ef2ef..de229895c1 100644
--- a/package/binutils/Config.in.host
+++ b/package/binutils/Config.in.host
@@ -2,7 +2,8 @@ comment "Binutils Options"
choice
prompt "Binutils Version"
- default BR2_BINUTILS_VERSION_2_28_X if BR2_ARM_CPU_ARMV7M
+ default BR2_BINUTILS_VERSION_2_28_X if BR2_ARM_INSTRUCTIONS_THUMB
+ default BR2_BINUTILS_VERSION_2_28_X if BR2_ARM_INSTRUCTIONS_THUMB2
default BR2_BINUTILS_VERSION_2_29_X if !BR2_arc
default BR2_BINUTILS_VERSION_ARC if BR2_arc
help
--
2.14.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [Buildroot] [PATCH] package/binutils: extend the 2.28 default to thumb mode
2018-06-01 17:26 [Buildroot] [PATCH] package/binutils: extend the 2.28 default to thumb mode Yann E. MORIN
@ 2018-06-01 17:32 ` Peter Korsgaard
2018-06-17 15:03 ` Peter Korsgaard
1 sibling, 0 replies; 3+ messages in thread
From: Peter Korsgaard @ 2018-06-01 17:32 UTC (permalink / raw)
To: buildroot
>>>>> "Yann" == Yann E MORIN <yann.morin.1998@free.fr> writes:
> Commit 17f352ac (package/binutils: default to 2.29 for Cortex-M targets)
> made the default version 2.28 (and not 2.29!) when the target is an
> arm-v7m CPU.
> However, the real trigger is compilation in Thumb mode, not the fact
> that the target is v7m.
> The fact that it was noticed on a v7m target is because Thumb is the
> only mode valid on those CPUs.
> Tighten the defaults to 2.28 for Thumb and Thumb2 modes.
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> Cc: Laurent GONZALEZ <br22@gezedo.com>
> Cc: Christophe Priouzeau <christophe.priouzeau@st.com>
> Cc: Peter Korsgaard <peter@korsgaard.com>
> Cc: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
> Cc: Arnout Vandecappelle <arnout@mind.be>
> ---
> package/binutils/Config.in.host | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
> diff --git a/package/binutils/Config.in.host b/package/binutils/Config.in.host
> index 4e578ef2ef..de229895c1 100644
> --- a/package/binutils/Config.in.host
> +++ b/package/binutils/Config.in.host
> @@ -2,7 +2,8 @@ comment "Binutils Options"
> choice
> prompt "Binutils Version"
> - default BR2_BINUTILS_VERSION_2_28_X if BR2_ARM_CPU_ARMV7M
> + default BR2_BINUTILS_VERSION_2_28_X if BR2_ARM_INSTRUCTIONS_THUMB
> + default BR2_BINUTILS_VERSION_2_28_X if BR2_ARM_INSTRUCTIONS_THUMB2
> default BR2_BINUTILS_VERSION_2_29_X if !BR2_arc
> default BR2_BINUTILS_VERSION_ARC if BR2_arc
> help
Committed, thanks. Will you also send a patch to update the comment in
linux/Config.in?
--
Bye, Peter Korsgaard
^ permalink raw reply [flat|nested] 3+ messages in thread
* [Buildroot] [PATCH] package/binutils: extend the 2.28 default to thumb mode
2018-06-01 17:26 [Buildroot] [PATCH] package/binutils: extend the 2.28 default to thumb mode Yann E. MORIN
2018-06-01 17:32 ` Peter Korsgaard
@ 2018-06-17 15:03 ` Peter Korsgaard
1 sibling, 0 replies; 3+ messages in thread
From: Peter Korsgaard @ 2018-06-17 15:03 UTC (permalink / raw)
To: buildroot
>>>>> "Yann" == Yann E MORIN <yann.morin.1998@free.fr> writes:
> Commit 17f352ac (package/binutils: default to 2.29 for Cortex-M targets)
> made the default version 2.28 (and not 2.29!) when the target is an
> arm-v7m CPU.
> However, the real trigger is compilation in Thumb mode, not the fact
> that the target is v7m.
> The fact that it was noticed on a v7m target is because Thumb is the
> only mode valid on those CPUs.
> Tighten the defaults to 2.28 for Thumb and Thumb2 modes.
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> Cc: Laurent GONZALEZ <br22@gezedo.com>
> Cc: Christophe Priouzeau <christophe.priouzeau@st.com>
> Cc: Peter Korsgaard <peter@korsgaard.com>
> Cc: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
> Cc: Arnout Vandecappelle <arnout@mind.be>
Committed to 2018.02.x, thanks.
--
Bye, Peter Korsgaard
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-06-17 15:03 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-06-01 17:26 [Buildroot] [PATCH] package/binutils: extend the 2.28 default to thumb mode Yann E. MORIN
2018-06-01 17:32 ` Peter Korsgaard
2018-06-17 15:03 ` Peter Korsgaard
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.