* [Buildroot] [PATCH] configs/kontron_pitx_imx8m_defconfig: remove BR2_ARM_FPU_VFPV3
@ 2022-01-20 13:52 Heiko Thiery
2022-01-22 12:15 ` Thomas Petazzoni
0 siblings, 1 reply; 3+ messages in thread
From: Heiko Thiery @ 2022-01-20 13:52 UTC (permalink / raw)
To: buildroot
Cc: Heiko Thiery, Giulio Benetti, Michael Walle, Thomas Petazzoni,
Yann E . MORIN
The FP setting is not used for aarch64 in buildroot. The FPU options are
set implicit.
Signed-off-by: Heiko Thiery <heiko.thiery@gmail.com>
---
configs/kontron_pitx_imx8m_defconfig | 1 -
1 file changed, 1 deletion(-)
diff --git a/configs/kontron_pitx_imx8m_defconfig b/configs/kontron_pitx_imx8m_defconfig
index 7f9c0eeae8..8f756f5246 100644
--- a/configs/kontron_pitx_imx8m_defconfig
+++ b/configs/kontron_pitx_imx8m_defconfig
@@ -1,6 +1,5 @@
# Architecture
BR2_aarch64=y
-BR2_ARM_FPU_VFPV3=y
# System
BR2_TARGET_GENERIC_GETTY_PORT="ttymxc2"
--
2.30.2
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [Buildroot] [PATCH] configs/kontron_pitx_imx8m_defconfig: remove BR2_ARM_FPU_VFPV3
2022-01-20 13:52 [Buildroot] [PATCH] configs/kontron_pitx_imx8m_defconfig: remove BR2_ARM_FPU_VFPV3 Heiko Thiery
@ 2022-01-22 12:15 ` Thomas Petazzoni
2022-01-22 15:31 ` Heiko Thiery
0 siblings, 1 reply; 3+ messages in thread
From: Thomas Petazzoni @ 2022-01-22 12:15 UTC (permalink / raw)
To: Heiko Thiery; +Cc: Giulio Benetti, Michael Walle, Yann E . MORIN, buildroot
On Thu, 20 Jan 2022 14:52:52 +0100
Heiko Thiery <heiko.thiery@gmail.com> wrote:
> The FP setting is not used for aarch64 in buildroot. The FPU options are
> set implicit.
>
> Signed-off-by: Heiko Thiery <heiko.thiery@gmail.com>
> ---
> configs/kontron_pitx_imx8m_defconfig | 1 -
> 1 file changed, 1 deletion(-)
Applied to master, thanks. That being said the FPU handling on AArch64
is not very clear to me. I'm not sure we're handling it correctly in
Buildroot today.
Thomas
--
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [Buildroot] [PATCH] configs/kontron_pitx_imx8m_defconfig: remove BR2_ARM_FPU_VFPV3
2022-01-22 12:15 ` Thomas Petazzoni
@ 2022-01-22 15:31 ` Heiko Thiery
0 siblings, 0 replies; 3+ messages in thread
From: Heiko Thiery @ 2022-01-22 15:31 UTC (permalink / raw)
To: Thomas Petazzoni; +Cc: Giulio Benetti, Michael Walle, Yann E . MORIN, buildroot
Hi Thomas,
Am Sa., 22. Jan. 2022 um 13:15 Uhr schrieb Thomas Petazzoni
<thomas.petazzoni@bootlin.com>:
>
> On Thu, 20 Jan 2022 14:52:52 +0100
> Heiko Thiery <heiko.thiery@gmail.com> wrote:
>
> > The FP setting is not used for aarch64 in buildroot. The FPU options are
> > set implicit.
> >
> > Signed-off-by: Heiko Thiery <heiko.thiery@gmail.com>
> > ---
> > configs/kontron_pitx_imx8m_defconfig | 1 -
> > 1 file changed, 1 deletion(-)
>
> Applied to master, thanks. That being said the FPU handling on AArch64
> is not very clear to me. I'm not sure we're handling it correctly in
> Buildroot today.
I must admit that I have not fully understood the topic FPU either.
What I am sure of is that the kconfig selection of the "Floating point
stategy" option for aarch64 has no effect, since this setting is made
implicitly depending on the target architecture used.
Therefore, a patch could look something like this to fix this:
diff --git a/arch/Config.in.arm b/arch/Config.in.arm
index 4c0910e4f8..05acaa9157 100644
--- a/arch/Config.in.arm
+++ b/arch/Config.in.arm
@@ -618,6 +618,7 @@ endchoice
choice
prompt "Floating point strategy"
+ depends on BR2_arm || BR2_armeb
default BR2_ARM_FPU_FP_ARMV8 if BR2_ARM_CPU_HAS_FP_ARMV8
default BR2_ARM_FPU_FPV5D16 if BR2_ARM_CPU_HAS_FPV5
default BR2_ARM_FPU_FPV4D16 if BR2_ARM_CPU_HAS_FPV4
--
Heiko
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply related [flat|nested] 3+ messages in thread
end of thread, other threads:[~2022-01-22 15:31 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-01-20 13:52 [Buildroot] [PATCH] configs/kontron_pitx_imx8m_defconfig: remove BR2_ARM_FPU_VFPV3 Heiko Thiery
2022-01-22 12:15 ` Thomas Petazzoni
2022-01-22 15:31 ` Heiko Thiery
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.