arch/mips: inverse the mfpxx logic
authorYann E. MORIN <yann.morin.1998@free.fr>
Sun, 3 Sep 2017 09:53:51 +0000 (11:53 +0200)
committerThomas Petazzoni <thomas.petazzoni@free-electrons.com>
Fri, 24 Nov 2017 21:45:48 +0000 (22:45 +0100)
Currently, the possibility to choose the floating point mode (32, xx or
64) is conditional on having a sufficiently recent gcc version.

Which means that the architecture selection depends on the gcc version.

But that's opposite to what we've always done in Buildroot: the software
versions are conditional to the architecture options. There is nothing
we can do about the hardware: it is there, we can't change it, while we
can restrict ourselves to using software that is working on said
hardware.

Thus, we inverse the logic, to move the condition onto the software
side: whenever mfpxx is selected, we restrict the toolchain selection to
at least a gcc-5.

And now, the blind BR2_TOOLCHAIN_HAS_MFPXX_OPTION symbol is no longer
needed, so we get rid of it.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Cc: Vicente Olivert Riera <Vincent.Riera@imgtec.com>
Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
arch/Config.in.mips
toolchain/toolchain-common.in

index 13d1a477e64c8c17746ba78144c9905391a05861..f8e57bab3396253477df5f7eb28f78fb2ee2f253 100644 (file)
@@ -146,7 +146,7 @@ config BR2_MIPS_SOFT_FLOAT
 choice
        prompt "FP mode"
        depends on !BR2_ARCH_IS_64 && !BR2_MIPS_SOFT_FLOAT
-       default BR2_MIPS_FP32_MODE_XX if BR2_TOOLCHAIN_HAS_MFPXX_OPTION
+       default BR2_MIPS_FP32_MODE_XX
        help
          MIPS32 supports different FP modes (32,xx,64). Information about FP
          modes can be found here:
@@ -159,7 +159,7 @@ config BR2_MIPS_FP32_MODE_32
 
 config BR2_MIPS_FP32_MODE_XX
        bool "xx"
-       depends on BR2_TOOLCHAIN_HAS_MFPXX_OPTION
+       select BR2_ARCH_NEEDS_GCC_AT_LEAST_5
 
 config BR2_MIPS_FP32_MODE_64
        bool "64"
index 15553772fd2a6871c835ad1b57206333d1010cc1..d743c637ceee05b510b5118cedf21f882099d54b 100644 (file)
@@ -356,10 +356,6 @@ config BR2_TOOLCHAIN_HAS_MNAN_OPTION
        bool
        default y if BR2_TOOLCHAIN_GCC_AT_LEAST_4_9
 
-config BR2_TOOLCHAIN_HAS_MFPXX_OPTION
-       bool
-       default y if BR2_TOOLCHAIN_GCC_AT_LEAST_5
-
 config BR2_TOOLCHAIN_HAS_SYNC_1
        bool
        default y