All of lore.kernel.org
 help / color / mirror / Atom feed
From: Martin Jansa <martin.jansa@gmail.com>
To: Ross Burton <ross.burton@intel.com>,
	openembedded-core@lists.openembedded.org
Subject: [PATCH][master][RESEND 12/15] feature-arm-vfp.inc: Further simplify with TUNE_CCARGS_MFLOAT
Date: Thu, 10 Dec 2015 14:48:49 +0100	[thread overview]
Message-ID: <f8842c0a2e8380da5f4be12f0465f1ab92daea9b.1449754344.git.Martin.Jansa@gmail.com> (raw)
In-Reply-To: <cover.1449754344.git.Martin.Jansa@gmail.com>
In-Reply-To: <cover.1449754344.git.Martin.Jansa@gmail.com>

* add TUNE_CCARGS_MFLOAT variable which is used to set -mfloat-abi
  parameter as well as ARMPKGSFX_EABI suffix in TUNE_PKGARCH and
  TARGET_FPU
* TARGET_FPU was using ARMPKGSFX_FPU, but in most cases we use it
  only to distinguish between hard and soft abi, not various -mfpu
  variants which can appear in ARMPKGSFX_FPU

Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
---
 meta/conf/machine/include/arm/arch-arm.inc        | 3 +--
 meta/conf/machine/include/arm/arch-arm64.inc      | 2 +-
 meta/conf/machine/include/arm/feature-arm-vfp.inc | 5 +++--
 3 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/meta/conf/machine/include/arm/arch-arm.inc b/meta/conf/machine/include/arm/arch-arm.inc
index 129fbfb..2e3127c 100644
--- a/meta/conf/machine/include/arm/arch-arm.inc
+++ b/meta/conf/machine/include/arm/arch-arm.inc
@@ -13,5 +13,4 @@ TUNE_PKGARCH = "${ARMPKGARCH}${ARMPKGSFX_THUMB}${ARMPKGSFX_DSP}${ARMPKGSFX_EABI}
 
 ABIEXTENSION = "eabi"
 
-TARGET_FPU = "${@d.getVar('ARMPKGSFX_FPU', True).strip('-') or 'soft'}"
-
+TARGET_FPU = "${@d.getVar('TUNE_CCARGS_MFLOAT', True) or 'soft'}"
diff --git a/meta/conf/machine/include/arm/arch-arm64.inc b/meta/conf/machine/include/arm/arch-arm64.inc
index e098e7a..9440698 100644
--- a/meta/conf/machine/include/arm/arch-arm64.inc
+++ b/meta/conf/machine/include/arm/arch-arm64.inc
@@ -28,7 +28,7 @@ TARGET_FPU_64 = ""
 TUNE_ARCH_32 = "${@bb.utils.contains('TUNE_FEATURES', 'bigendian', 'armeb', 'arm', d)}"
 TUNE_PKGARCH_32 = "${ARMPKGARCH}${ARMPKGSFX_THUMB}${ARMPKGSFX_DSP}${ARMPKGSFX_EABI}${ARMPKGSFX_ENDIAN}${ARMPKGSFX_FPU}"
 ABIEXTENSION_32 = "eabi"
-TARGET_FPU_32 = "${@d.getVar('ARMPKGSFX_FPU', True).strip('-') or 'soft'}"
+TARGET_FPU_32 = "${@d.getVar('TUNE_CCARGS_MFLOAT', True) or 'soft'}"
 
 TUNE_ARCH = "${@bb.utils.contains('TUNE_FEATURES', 'aarch64', '${TUNE_ARCH_64}', '${TUNE_ARCH_32}' ,d)}"
 TUNE_PKGARCH = "${@bb.utils.contains('TUNE_FEATURES', 'aarch64', '${TUNE_PKGARCH_64}', '${TUNE_PKGARCH_32}' ,d)}"
diff --git a/meta/conf/machine/include/arm/feature-arm-vfp.inc b/meta/conf/machine/include/arm/feature-arm-vfp.inc
index 1b39488..4247255 100644
--- a/meta/conf/machine/include/arm/feature-arm-vfp.inc
+++ b/meta/conf/machine/include/arm/feature-arm-vfp.inc
@@ -9,5 +9,6 @@ TUNE_CCARGS  .= "${@ (' -mfpu=%s ' % d.getVar('TUNE_CCARGS_MFPU', True).split()[
 ARMPKGSFX_FPU = "${@ ('-%s'        % d.getVar('TUNE_CCARGS_MFPU', True).split()[-1]) if (d.getVar('TUNE_CCARGS_MFPU', True) != '') else ''}"
 
 TUNEVALID[callconvention-hard] = "Enable EABI hard float call convention, requires VFP."
-TUNE_CCARGS   .= "${@ bb.utils.contains('TUNE_FEATURES', 'callconvention-hard', ' -mfloat-abi=hard', ' -mfloat-abi=softfp', d) if (d.getVar('TUNE_CCARGS_MFPU', True) != '') else '' }"
-ARMPKGSFX_EABI = "${@ bb.utils.contains('TUNE_FEATURES', 'callconvention-hard', 'hf',                '',                    d) if (d.getVar('TUNE_CCARGS_MFPU', True) != '') else '' }"
+TUNE_CCARGS_MFLOAT = "${@ bb.utils.contains('TUNE_FEATURES', 'callconvention-hard', 'hard', 'softfp', d) if (d.getVar('TUNE_CCARGS_MFPU', True) != '') else '' }"
+TUNE_CCARGS   .= "${@ ' -mfloat-abi=${TUNE_CCARGS_MFLOAT}' if (d.getVar('TUNE_CCARGS_MFLOAT', True) != '')     else ''}"
+ARMPKGSFX_EABI = "${@ 'hf'                                 if (d.getVar('TUNE_CCARGS_MFLOAT', True) == 'hard') else ''}"
-- 
2.6.3



  parent reply	other threads:[~2015-12-10 13:47 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-12-10 13:48 [PATCH][master][RESEND 00/15] ARM tune fixes and improvements Martin Jansa
2015-12-10 13:48 ` [PATCH][master][RESEND 01/15] sanity.bbclass: add more information to error message about TUNE_PKGARCH missing in PACKAGE_ARCHS Martin Jansa
2015-12-10 21:11   ` Khem Raj
2015-12-18 13:15     ` [PATCHv2] " Martin Jansa
2015-12-10 13:48 ` [PATCH][master][RESEND 02/15] tune-cortexr4.inc: fix PACKAGE_EXTRA_ARCHS Martin Jansa
2015-12-10 13:48 ` [PATCH][master][RESEND 03/15] arch-armv7a: add missing space before ?= Martin Jansa
2015-12-10 13:48 ` [PATCH][master][RESEND 04/15] arch-armv7a, tune-cortexa*: improve comment VFP -> HF Martin Jansa
2015-12-10 13:48 ` [PATCH][master][RESEND 05/15] arch-armv7a, tune-cortexa*: improve indentation Martin Jansa
2015-12-10 13:48 ` [PATCH][master][RESEND 06/15] arm/arch-arm*, tune-cortexa*, tune-thunderx.inc, powerpac/arch-powerpc64.inc: Use normal assignment Martin Jansa
2015-12-10 13:48 ` [PATCH][master][RESEND 07/15] arch-armv[456]*.inc: improve indentation like armv7a Martin Jansa
2015-12-10 13:48 ` [PATCH][master][RESEND 08/15] arch-armv5.inc: drop duplicate ARMPKGSFX_DSP and PACKAGE_EXTRA_ARCHS_tune-armv5tehf-vfp Martin Jansa
2015-12-10 13:48 ` [PATCH][master][RESEND 09/15] arch-armv7a.inc: Fix PACKAGE_EXTRA_ARCHS for tune-armv7atb-vfpv3, tune-armv7atb-vfpv3d16, cortexa7thf-neon-vfpv4 Martin Jansa
2015-12-10 13:48 ` [PATCH][master][RESEND 10/15] arch-armv7a.inc: add vfpv4 support also to softfp and big endiand tunes Martin Jansa
2015-12-10 13:48 ` [PATCH][master][RESEND 11/15] feature-arm-{neon, vfp}.inc: refactor and fix issues Martin Jansa
2015-12-10 21:14   ` Khem Raj
2015-12-18 13:10     ` Martin Jansa
2015-12-10 13:48 ` Martin Jansa [this message]
2015-12-10 13:48 ` [PATCH][master][RESEND 13/15] cortexa{7, 15, 17}: add VFPv4 tunes Martin Jansa
2015-12-10 13:48 ` [PATCH][master][RESEND 14/15] arch-armv7ve: add tune include for armv7ve and use it from cortexa7 and cortexa15 Martin Jansa
2015-12-18  3:13   ` Andre McCurdy
2015-12-18 13:13     ` Martin Jansa
2015-12-18 20:08       ` Andre McCurdy
2015-12-18 20:13         ` Martin Jansa
2015-12-18 20:29           ` Andre McCurdy
2015-12-18 20:45             ` Martin Jansa
2015-12-18 20:50               ` Andre McCurdy
2015-12-18 20:57                 ` Martin Jansa
2015-12-18 21:18                   ` Andre McCurdy
2015-12-10 13:48 ` [PATCH][master][RESEND 15/15] tune-*: use mcpu instead of mtune for ARM tunes Martin Jansa
2015-12-23 10:10 ` [PATCH][master][RESEND 00/15] ARM tune fixes and improvements Martin Jansa
2016-01-04 16:58   ` Burton, Ross

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=f8842c0a2e8380da5f4be12f0465f1ab92daea9b.1449754344.git.Martin.Jansa@gmail.com \
    --to=martin.jansa@gmail.com \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=ross.burton@intel.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.