Patchwork tune-ppce500: correct the -mfloat-gprs compiler option

login
register
mail settings
Submitter Christopher Larson
Date March 30, 2012, 10:50 p.m.
Message ID <1333147814-4990-1-git-send-email-kergoth@gmail.com>
Download mbox | patch
Permalink /patch/24959/
State New
Headers show

Comments

Christopher Larson - March 30, 2012, 10:50 p.m.
From: Christopher Larson <chris_larson@mentor.com>

Per Edmar at Freescale, e500v2 needs -mfloat-gprs=double, but e500v1 needs
-mfloat-gprs=single.

Signed-off-by: Christopher Larson <chris_larson@mentor.com>
---
 meta/conf/machine/include/tune-ppce500.inc |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
Mark Hatle - March 30, 2012, 10:57 p.m.
I noticed the same problem in the tuning rework that I've gone pending.  I'm 
currently working on validating it and hope to have something for merging once I 
get done validating all 120 tunes.

--Mark

On 3/30/12 5:50 PM, Christopher Larson wrote:
> From: Christopher Larson<chris_larson@mentor.com>
>
> Per Edmar at Freescale, e500v2 needs -mfloat-gprs=double, but e500v1 needs
> -mfloat-gprs=single.
>
> Signed-off-by: Christopher Larson<chris_larson@mentor.com>
> ---
>   meta/conf/machine/include/tune-ppce500.inc |    2 +-
>   1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/meta/conf/machine/include/tune-ppce500.inc b/meta/conf/machine/include/tune-ppce500.inc
> index feb1967..c966560 100644
> --- a/meta/conf/machine/include/tune-ppce500.inc
> +++ b/meta/conf/machine/include/tune-ppce500.inc
> @@ -7,7 +7,7 @@ TUNE_CCARGS += "${@bb.utils.contains("TUNE_FEATURES", "ppce500", "-mcpu=8540", "
>   TUNE_PKGARCH_tune-ppce500 = "ppce500"
>
>   TUNEVALID[spe] = "Enable SPE ABI extensions"
> -TUNE_CCARGS += "${@bb.utils.contains("TUNE_FEATURES", "spe", "-mabi=spe -mspe -mfloat-gprs=double", "", d)}"
> +TUNE_CCARGS += "${@bb.utils.contains("TUNE_FEATURES", "spe", "-mabi=spe -mspe -mfloat-gprs=single", "", d)}"
>
>   TARGET_FPU = "ppc-efs"
>

Patch

diff --git a/meta/conf/machine/include/tune-ppce500.inc b/meta/conf/machine/include/tune-ppce500.inc
index feb1967..c966560 100644
--- a/meta/conf/machine/include/tune-ppce500.inc
+++ b/meta/conf/machine/include/tune-ppce500.inc
@@ -7,7 +7,7 @@  TUNE_CCARGS += "${@bb.utils.contains("TUNE_FEATURES", "ppce500", "-mcpu=8540", "
 TUNE_PKGARCH_tune-ppce500 = "ppce500"
 
 TUNEVALID[spe] = "Enable SPE ABI extensions"
-TUNE_CCARGS += "${@bb.utils.contains("TUNE_FEATURES", "spe", "-mabi=spe -mspe -mfloat-gprs=double", "", d)}"
+TUNE_CCARGS += "${@bb.utils.contains("TUNE_FEATURES", "spe", "-mabi=spe -mspe -mfloat-gprs=single", "", d)}"
 
 TARGET_FPU = "ppc-efs"