[OE-core] [PATCH 1/1] tune-cortexa9.inc: add vfpv3 tunes

[Ioan-Adrian Ratiu]

Hello

On Thu, 11 Feb 2016 17:41:54 +0100
Martin Jansa <martin.jansa at gmail.com> wrote:

> On Thu, Feb 11, 2016 at 05:18:56PM +0200, Ioan-Adrian Ratiu wrote:
> > From: Richard Tollerton <rich.tollerton at ni.com>
> > 
> > Define tunnings to enable 32 register VFPv3 for cortexa9 processor cores  
> 
> Can you execute the test from:
> http://git.openembedded.org/openembedded-core-contrib/log/?h=jansa/tune2-test
> just to verify that all these new combinations work ok and to update the
> test results for next person modifying the tune files?

 I've tested with that script and everything looks ok (it took a while for all those openssl builds to run the first time).

I also had to modify that hardcoded path to the openembedded-core location in the script. I asume this test script was created for the poky layout?
(also I've done a  grep "\(-vfpv3\)" on the AVAILTUNES var, but I don't think it matters that much though, run the test without it also)

There is also a discrepancy between the value of TUNE_PKGARCH == armv7athf and the file name from the env.fake-cortexa9.armv7athf-vfpv3. I believe there may be a typo in one of the tunes, ommited the "t" for thumb.

> 
> Thanks
> 
> > More details: http://www.arm.com/products/processors/technologies/vector-floating-point.php
> > 
> > Signed-off-by: Richard Tollerton <rich.tollerton at ni.com>
> > Signed-off-by: Ioan-Adrian Ratiu <adrian.ratiu at ni.com>
> > ---
> >  meta/conf/machine/include/tune-cortexa9.inc | 15 +++++++++++++++
> >  1 file changed, 15 insertions(+)
> > 
> > diff --git a/meta/conf/machine/include/tune-cortexa9.inc b/meta/conf/machine/include/tune-cortexa9.inc
> > index 965f2d9..3d8dc1d 100644
> > --- a/meta/conf/machine/include/tune-cortexa9.inc
> > +++ b/meta/conf/machine/include/tune-cortexa9.inc
> > @@ -34,3 +34,18 @@ PACKAGE_EXTRA_ARCHS_tune-cortexa9hf             = "${PACKAGE_EXTRA_ARCHS_tune-ar
> >  PACKAGE_EXTRA_ARCHS_tune-cortexa9thf            = "${PACKAGE_EXTRA_ARCHS_tune-armv7athf} cortexa9hf-vfp cortexa9t2hf-vfp"
> >  PACKAGE_EXTRA_ARCHS_tune-cortexa9hf-neon        = "${PACKAGE_EXTRA_ARCHS_tune-armv7ahf-neon}  cortexa9hf-vfp cortexa9hf-neon"
> >  PACKAGE_EXTRA_ARCHS_tune-cortexa9thf-neon       = "${PACKAGE_EXTRA_ARCHS_tune-armv7athf-neon} cortexa9hf-vfp cortexa9hf-neon cortexa9t2hf-vfp cortexa9t2hf-neon"
> > +
> > +# VFPv3 Tunes
> > +AVAILTUNES += "cortexa9-vfpv3 cortexa9t-vfpv3 cortexa9hf-vfpv3 cortexa9thf-vfpv3"
> > +ARMPKGARCH_tune-cortexa9-vfpv3          = "cortexa9"
> > +ARMPKGARCH_tune-cortexa9t-vfpv3         = "cortexa9"
> > +ARMPKGARCH_tune-cortexa9hf-vfpv3        = "cortexa9"
> > +ARMPKGARCH_tune-cortexa9thf-vfpv3       = "cortexa9"
> > +TUNE_FEATURES_tune-cortexa9-vfpv3           = "${TUNE_FEATURES_tune-armv7a-vfpv3}          cortexa9"
> > +TUNE_FEATURES_tune-cortexa9t-vfpv3          = "${TUNE_FEATURES_tune-armv7at-vfpv3}         cortexa9"
> > +TUNE_FEATURES_tune-cortexa9hf-vfpv3         = "${TUNE_FEATURES_tune-armv7ahf-vfpv3}        cortexa9"
> > +TUNE_FEATURES_tune-cortexa9thf-vfpv3        = "${TUNE_FEATURES_tune-armv7athf-vfpv3}       cortexa9"
> > +PACKAGE_EXTRA_ARCHS_tune-cortexa9-vfpv3           = "${PACKAGE_EXTRA_ARCHS_tune-armv7a-vfpv3}  cortexa9-vfp cortexa9-vfpv3"
> > +PACKAGE_EXTRA_ARCHS_tune-cortexa9t-vfpv3          = "${PACKAGE_EXTRA_ARCHS_tune-armv7at-vfpv3} cortexa9-vfp cortexa9-vfpv3 cortexa9t2-vfp cortexa9t2-vfpv3"
> > +PACKAGE_EXTRA_ARCHS_tune-cortexa9hf-vfpv3         = "${PACKAGE_EXTRA_ARCHS_tune-armv7ahf-vfpv3}  cortexa9hf-vfp cortexa9hf-vfpv3"
> > +PACKAGE_EXTRA_ARCHS_tune-cortexa9thf-vfpv3        = "${PACKAGE_EXTRA_ARCHS_tune-armv7athf-vfpv3} cortexa9hf-vfp cortexa9hf-vfpv3 cortexa9t2hf-vfp cortexa9t2hf-vfpv3"
> > -- 
> > 2.7.0
> > 
> > -- 
> > _______________________________________________
> > Openembedded-core mailing list
> > Openembedded-core at lists.openembedded.org
> > http://lists.openembedded.org/mailman/listinfo/openembedded-core  
>