[OE-core] [PATCH] tune-cortexa*.inc: fix tunings for cortex a5, a7, a8, a9, a15 machines.

2013-06-12 Thread Andy Voltz
Using CORTEX_ID variable reference in the tuning overrides did not work.
This reverts those changes, and adds a tuning file for the cortex-a5.

Revert tune-cortexa5.inc: Add tune file for cortex-a5
Revert tune-cortexa.inc: create a common include for cortex-a armv7a tuning

Signed-off-by: Andy Voltz andy.vo...@timesys.com
---
 meta/conf/machine/include/arm/README |3 ---
 meta/conf/machine/include/tune-cortexa.inc   |   35 --
 meta/conf/machine/include/tune-cortexa15.inc |   35 --
 meta/conf/machine/include/tune-cortexa5.inc  |   35 --
 meta/conf/machine/include/tune-cortexa7.inc  |   35 --
 meta/conf/machine/include/tune-cortexa8.inc  |   35 --
 meta/conf/machine/include/tune-cortexa9.inc  |   35 --
 7 files changed, 165 insertions(+), 48 deletions(-)
 delete mode 100644 meta/conf/machine/include/tune-cortexa.inc

diff --git a/meta/conf/machine/include/arm/README 
b/meta/conf/machine/include/arm/README
index d8d7b64..a1beb75 100644
--- a/meta/conf/machine/include/arm/README
+++ b/meta/conf/machine/include/arm/README
@@ -38,6 +38,3 @@ the core arch-arm.inc file.
 
 ARMPKGSFX_FPU - This is the FPU specific suffix.  The suffix indicates 
 specific FPU optimizations.  'vfp' and 'neon' are both defined.
-
-CORTEX_ID - This is the cortex processor type, and is used to setup the
-proper tuning (-mtune=cortex-${CORTEX_ID}) in tune-cortexa.inc.
diff --git a/meta/conf/machine/include/tune-cortexa.inc 
b/meta/conf/machine/include/tune-cortexa.inc
deleted file mode 100644
index 987e4d9..000
--- a/meta/conf/machine/include/tune-cortexa.inc
+++ /dev/null
@@ -1,35 +0,0 @@
-DEFAULTTUNE ?= armv7a-neon
-
-require conf/machine/include/arm/arch-armv7a.inc
-
-TUNE_CCARGS .= ${@bb.utils.contains(TUNE_FEATURES, cortex${CORTEX_ID},  
-mcpu=cortex-${CORTEX_ID}, , d)}
-
-# Little Endian base configs
-AVAILTUNES += cortex${CORTEX_ID} cortex${CORTEX_ID}t cortex${CORTEX_ID}-neon 
cortex${CORTEX_ID}t-neon
-ARMPKGARCH_tune-cortex${CORTEX_ID} = cortex${CORTEX_ID}
-ARMPKGARCH_tune-cortex${CORTEX_ID}t = cortex${CORTEX_ID}
-ARMPKGARCH_tune-cortex${CORTEX_ID}-neon = cortex${CORTEX_ID}
-ARMPKGARCH_tune-cortex${CORTEX_ID}t-neon = cortex${CORTEX_ID}
-TUNE_FEATURES_tune-cortex${CORTEX_ID} = ${TUNE_FEATURES_tune-armv7a} 
cortex${CORTEX_ID}
-TUNE_FEATURES_tune-cortex${CORTEX_ID}t = ${TUNE_FEATURES_tune-armv7at} 
cortex${CORTEX_ID}
-TUNE_FEATURES_tune-cortex${CORTEX_ID}-neon = 
${TUNE_FEATURES_tune-armv7a-neon} cortex${CORTEX_ID}
-TUNE_FEATURES_tune-cortex${CORTEX_ID}t-neon = 
${TUNE_FEATURES_tune-armv7at-neon} cortex${CORTEX_ID}
-PACKAGE_EXTRA_ARCHS_tune-cortex${CORTEX_ID} = 
${PACKAGE_EXTRA_ARCHS_tune-armv7a} cortex${CORTEX_ID}-vfp
-PACKAGE_EXTRA_ARCHS_tune-cortex${CORTEX_ID}t = 
${PACKAGE_EXTRA_ARCHS_tune-armv7at} cortex${CORTEX_ID}-vfp 
cortex${CORTEX_ID}t2-vfp
-PACKAGE_EXTRA_ARCHS_tune-cortex${CORTEX_ID}-neon = 
${PACKAGE_EXTRA_ARCHS_tune-armv7a-neon} cortex${CORTEX_ID}-vfp 
cortex${CORTEX_ID}-vfp-neon
-PACKAGE_EXTRA_ARCHS_tune-cortex${CORTEX_ID}t-neon = 
${PACKAGE_EXTRA_ARCHS_tune-armv7at-neon} cortex${CORTEX_ID}-vfp 
cortex${CORTEX_ID}-vfp-neon cortex${CORTEX_ID}t2-vfp 
cortex${CORTEX_ID}t2-vfp-neon
-
-# VFP Tunes
-AVAILTUNES += cortex${CORTEX_ID}hf cortex${CORTEX_ID}thf 
cortex${CORTEX_ID}hf-neon cortex${CORTEX_ID}thf-neon
-ARMPKGARCH_tune-cortex${CORTEX_ID}hf = cortex${CORTEX_ID}
-ARMPKGARCH_tune-cortex${CORTEX_ID}thf = cortex${CORTEX_ID}
-ARMPKGARCH_tune-cortex${CORTEX_ID}hf-neon = cortex${CORTEX_ID}
-ARMPKGARCH_tune-cortex${CORTEX_ID}thf-neon = cortex${CORTEX_ID}
-TUNE_FEATURES_tune-cortex${CORTEX_ID}hf ?= ${TUNE_FEATURES_tune-armv7ahf} 
cortex${CORTEX_ID}
-TUNE_FEATURES_tune-cortex${CORTEX_ID}thf ?= ${TUNE_FEATURES_tune-armv7athf} 
cortex${CORTEX_ID}
-TUNE_FEATURES_tune-cortex${CORTEX_ID}hf-neon ?= 
${TUNE_FEATURES_tune-armv7ahf-neon} cortex${CORTEX_ID}
-TUNE_FEATURES_tune-cortex${CORTEX_ID}thf-neon ?= 
${TUNE_FEATURES_tune-armv7athf-neon} cortex${CORTEX_ID}
-PACKAGE_EXTRA_ARCHS_tune-cortex${CORTEX_ID}hf = 
${PACKAGE_EXTRA_ARCHS_tune-armv7ahf} cortex${CORTEX_ID}hf-vfp
-PACKAGE_EXTRA_ARCHS_tune-cortex${CORTEX_ID}thf = 
${PACKAGE_EXTRA_ARCHS_tune-armv7athf} cortex${CORTEX_ID}hf-vfp 
cortex${CORTEX_ID}t2hf-vfp
-PACKAGE_EXTRA_ARCHS_tune-cortex${CORTEX_ID}hf-neon = 
${PACKAGE_EXTRA_ARCHS_tune-armv7ahf-neon} cortex${CORTEX_ID}hf-vfp 
cortex${CORTEX_ID}hf-vfp-neon
-PACKAGE_EXTRA_ARCHS_tune-cortex${CORTEX_ID}thf-neon = 
${PACKAGE_EXTRA_ARCHS_tune-armv7athf-neon} cortex${CORTEX_ID}hf-vfp 
cortex${CORTEX_ID}hf-vfp-neon cortex${CORTEX_ID}t2hf-vfp 
cortex${CORTEX_ID}t2hf-vfp-neon
diff --git a/meta/conf/machine/include/tune-cortexa15.inc 
b/meta/conf/machine/include/tune-cortexa15.inc
index c89bacc..5529777 100644
--- a/meta/conf/machine/include/tune-cortexa15.inc
+++ b/meta/conf/machine/include/tune-cortexa15.inc
@@ -1,5 +1,36 @@
-CORTEX_ID = a15
+DEFAULTTUNE ?= armv7a-neon
+
+require 

Re: [OE-core] [PATCH] tune-cortexa*.inc: fix tunings for cortex a5, a7, a8, a9, a15 machines.

2013-06-12 Thread Martin Jansa
On Wed, Jun 12, 2013 at 12:38:01PM -0400, Andy Voltz wrote:
 Using CORTEX_ID variable reference in the tuning overrides did not work.
 This reverts those changes, and adds a tuning file for the cortex-a5.
 
 Revert tune-cortexa5.inc: Add tune file for cortex-a5
 Revert tune-cortexa.inc: create a common include for cortex-a armv7a tuning
 
 Signed-off-by: Andy Voltz andy.vo...@timesys.com

Acked-by: Martin Jansa martin.ja...@gmail.com

Thanks

I have it reverted since it was merged..

 ---
  meta/conf/machine/include/arm/README |3 ---
  meta/conf/machine/include/tune-cortexa.inc   |   35 
 --
  meta/conf/machine/include/tune-cortexa15.inc |   35 
 --
  meta/conf/machine/include/tune-cortexa5.inc  |   35 
 --
  meta/conf/machine/include/tune-cortexa7.inc  |   35 
 --
  meta/conf/machine/include/tune-cortexa8.inc  |   35 
 --
  meta/conf/machine/include/tune-cortexa9.inc  |   35 
 --
  7 files changed, 165 insertions(+), 48 deletions(-)
  delete mode 100644 meta/conf/machine/include/tune-cortexa.inc
 
 diff --git a/meta/conf/machine/include/arm/README 
 b/meta/conf/machine/include/arm/README
 index d8d7b64..a1beb75 100644
 --- a/meta/conf/machine/include/arm/README
 +++ b/meta/conf/machine/include/arm/README
 @@ -38,6 +38,3 @@ the core arch-arm.inc file.
  
  ARMPKGSFX_FPU - This is the FPU specific suffix.  The suffix indicates 
  specific FPU optimizations.  'vfp' and 'neon' are both defined.
 -
 -CORTEX_ID - This is the cortex processor type, and is used to setup the
 -proper tuning (-mtune=cortex-${CORTEX_ID}) in tune-cortexa.inc.
 diff --git a/meta/conf/machine/include/tune-cortexa.inc 
 b/meta/conf/machine/include/tune-cortexa.inc
 deleted file mode 100644
 index 987e4d9..000
 --- a/meta/conf/machine/include/tune-cortexa.inc
 +++ /dev/null
 @@ -1,35 +0,0 @@
 -DEFAULTTUNE ?= armv7a-neon
 -
 -require conf/machine/include/arm/arch-armv7a.inc
 -
 -TUNE_CCARGS .= ${@bb.utils.contains(TUNE_FEATURES, cortex${CORTEX_ID}, 
  -mcpu=cortex-${CORTEX_ID}, , d)}
 -
 -# Little Endian base configs
 -AVAILTUNES += cortex${CORTEX_ID} cortex${CORTEX_ID}t 
 cortex${CORTEX_ID}-neon cortex${CORTEX_ID}t-neon
 -ARMPKGARCH_tune-cortex${CORTEX_ID} = cortex${CORTEX_ID}
 -ARMPKGARCH_tune-cortex${CORTEX_ID}t = cortex${CORTEX_ID}
 -ARMPKGARCH_tune-cortex${CORTEX_ID}-neon = cortex${CORTEX_ID}
 -ARMPKGARCH_tune-cortex${CORTEX_ID}t-neon = cortex${CORTEX_ID}
 -TUNE_FEATURES_tune-cortex${CORTEX_ID} = ${TUNE_FEATURES_tune-armv7a} 
 cortex${CORTEX_ID}
 -TUNE_FEATURES_tune-cortex${CORTEX_ID}t = ${TUNE_FEATURES_tune-armv7at} 
 cortex${CORTEX_ID}
 -TUNE_FEATURES_tune-cortex${CORTEX_ID}-neon = 
 ${TUNE_FEATURES_tune-armv7a-neon} cortex${CORTEX_ID}
 -TUNE_FEATURES_tune-cortex${CORTEX_ID}t-neon = 
 ${TUNE_FEATURES_tune-armv7at-neon} cortex${CORTEX_ID}
 -PACKAGE_EXTRA_ARCHS_tune-cortex${CORTEX_ID} = 
 ${PACKAGE_EXTRA_ARCHS_tune-armv7a} cortex${CORTEX_ID}-vfp
 -PACKAGE_EXTRA_ARCHS_tune-cortex${CORTEX_ID}t = 
 ${PACKAGE_EXTRA_ARCHS_tune-armv7at} cortex${CORTEX_ID}-vfp 
 cortex${CORTEX_ID}t2-vfp
 -PACKAGE_EXTRA_ARCHS_tune-cortex${CORTEX_ID}-neon = 
 ${PACKAGE_EXTRA_ARCHS_tune-armv7a-neon} cortex${CORTEX_ID}-vfp 
 cortex${CORTEX_ID}-vfp-neon
 -PACKAGE_EXTRA_ARCHS_tune-cortex${CORTEX_ID}t-neon = 
 ${PACKAGE_EXTRA_ARCHS_tune-armv7at-neon} cortex${CORTEX_ID}-vfp 
 cortex${CORTEX_ID}-vfp-neon cortex${CORTEX_ID}t2-vfp 
 cortex${CORTEX_ID}t2-vfp-neon
 -
 -# VFP Tunes
 -AVAILTUNES += cortex${CORTEX_ID}hf cortex${CORTEX_ID}thf 
 cortex${CORTEX_ID}hf-neon cortex${CORTEX_ID}thf-neon
 -ARMPKGARCH_tune-cortex${CORTEX_ID}hf = cortex${CORTEX_ID}
 -ARMPKGARCH_tune-cortex${CORTEX_ID}thf = cortex${CORTEX_ID}
 -ARMPKGARCH_tune-cortex${CORTEX_ID}hf-neon = cortex${CORTEX_ID}
 -ARMPKGARCH_tune-cortex${CORTEX_ID}thf-neon = cortex${CORTEX_ID}
 -TUNE_FEATURES_tune-cortex${CORTEX_ID}hf ?= ${TUNE_FEATURES_tune-armv7ahf} 
 cortex${CORTEX_ID}
 -TUNE_FEATURES_tune-cortex${CORTEX_ID}thf ?= ${TUNE_FEATURES_tune-armv7athf} 
 cortex${CORTEX_ID}
 -TUNE_FEATURES_tune-cortex${CORTEX_ID}hf-neon ?= 
 ${TUNE_FEATURES_tune-armv7ahf-neon} cortex${CORTEX_ID}
 -TUNE_FEATURES_tune-cortex${CORTEX_ID}thf-neon ?= 
 ${TUNE_FEATURES_tune-armv7athf-neon} cortex${CORTEX_ID}
 -PACKAGE_EXTRA_ARCHS_tune-cortex${CORTEX_ID}hf = 
 ${PACKAGE_EXTRA_ARCHS_tune-armv7ahf} cortex${CORTEX_ID}hf-vfp
 -PACKAGE_EXTRA_ARCHS_tune-cortex${CORTEX_ID}thf = 
 ${PACKAGE_EXTRA_ARCHS_tune-armv7athf} cortex${CORTEX_ID}hf-vfp 
 cortex${CORTEX_ID}t2hf-vfp
 -PACKAGE_EXTRA_ARCHS_tune-cortex${CORTEX_ID}hf-neon = 
 ${PACKAGE_EXTRA_ARCHS_tune-armv7ahf-neon} cortex${CORTEX_ID}hf-vfp 
 cortex${CORTEX_ID}hf-vfp-neon
 -PACKAGE_EXTRA_ARCHS_tune-cortex${CORTEX_ID}thf-neon = 
 ${PACKAGE_EXTRA_ARCHS_tune-armv7athf-neon} cortex${CORTEX_ID}hf-vfp 
 cortex${CORTEX_ID}hf-vfp-neon cortex${CORTEX_ID}t2hf-vfp 
 cortex${CORTEX_ID}t2hf-vfp-neon
 diff --git 

Re: [OE-core] [PATCH] tune-cortexa*.inc: fix tunings for cortex a5, a7, a8, a9, a15 machines.

2013-06-12 Thread Otavio Salvador
On Wed, Jun 12, 2013 at 1:38 PM, Andy Voltz andy.vo...@timesys.com wrote:

 Using CORTEX_ID variable reference in the tuning overrides did not work.
 This reverts those changes, and adds a tuning file for the cortex-a5.

 Revert tune-cortexa5.inc: Add tune file for cortex-a5
 Revert tune-cortexa.inc: create a common include for cortex-a armv7a
 tuning

 Signed-off-by: Andy Voltz andy.vo...@timesys.com


Thanks for fixing this.

Acked-by: Otavio Salvador ota...@ossystems.com.br

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH] tune-cortexa*.inc: fix tunings for cortex a5, a7, a8, a9, a15 machines.

2013-06-12 Thread Saul Wold

On 06/12/2013 09:38 AM, Andy Voltz wrote:

Using CORTEX_ID variable reference in the tuning overrides did not work.
This reverts those changes, and adds a tuning file for the cortex-a5.

Revert tune-cortexa5.inc: Add tune file for cortex-a5
Revert tune-cortexa.inc: create a common include for cortex-a armv7a tuning

Andy, does this patch supersede your -mtune not -mcpu patch?  I seem to 
have some trouble applying this patch.


It might be best to provide a series that includes the reverts with 
appropriate commit messages.


Thanks
Sau!



Signed-off-by: Andy Voltz andy.vo...@timesys.com
---
  meta/conf/machine/include/arm/README |3 ---
  meta/conf/machine/include/tune-cortexa.inc   |   35 --
  meta/conf/machine/include/tune-cortexa15.inc |   35 --
  meta/conf/machine/include/tune-cortexa5.inc  |   35 --
  meta/conf/machine/include/tune-cortexa7.inc  |   35 --
  meta/conf/machine/include/tune-cortexa8.inc  |   35 --
  meta/conf/machine/include/tune-cortexa9.inc  |   35 --
  7 files changed, 165 insertions(+), 48 deletions(-)
  delete mode 100644 meta/conf/machine/include/tune-cortexa.inc

diff --git a/meta/conf/machine/include/arm/README 
b/meta/conf/machine/include/arm/README
index d8d7b64..a1beb75 100644
--- a/meta/conf/machine/include/arm/README
+++ b/meta/conf/machine/include/arm/README
@@ -38,6 +38,3 @@ the core arch-arm.inc file.

  ARMPKGSFX_FPU - This is the FPU specific suffix.  The suffix indicates
  specific FPU optimizations.  'vfp' and 'neon' are both defined.
-
-CORTEX_ID - This is the cortex processor type, and is used to setup the
-proper tuning (-mtune=cortex-${CORTEX_ID}) in tune-cortexa.inc.
diff --git a/meta/conf/machine/include/tune-cortexa.inc 
b/meta/conf/machine/include/tune-cortexa.inc
deleted file mode 100644
index 987e4d9..000
--- a/meta/conf/machine/include/tune-cortexa.inc
+++ /dev/null
@@ -1,35 +0,0 @@
-DEFAULTTUNE ?= armv7a-neon
-
-require conf/machine/include/arm/arch-armv7a.inc
-
-TUNE_CCARGS .= ${@bb.utils.contains(TUNE_FEATURES, cortex${CORTEX_ID},  
-mcpu=cortex-${CORTEX_ID}, , d)}
-
-# Little Endian base configs
-AVAILTUNES += cortex${CORTEX_ID} cortex${CORTEX_ID}t cortex${CORTEX_ID}-neon 
cortex${CORTEX_ID}t-neon
-ARMPKGARCH_tune-cortex${CORTEX_ID} = cortex${CORTEX_ID}
-ARMPKGARCH_tune-cortex${CORTEX_ID}t = cortex${CORTEX_ID}
-ARMPKGARCH_tune-cortex${CORTEX_ID}-neon = cortex${CORTEX_ID}
-ARMPKGARCH_tune-cortex${CORTEX_ID}t-neon = cortex${CORTEX_ID}
-TUNE_FEATURES_tune-cortex${CORTEX_ID} = ${TUNE_FEATURES_tune-armv7a} 
cortex${CORTEX_ID}
-TUNE_FEATURES_tune-cortex${CORTEX_ID}t = ${TUNE_FEATURES_tune-armv7at} 
cortex${CORTEX_ID}
-TUNE_FEATURES_tune-cortex${CORTEX_ID}-neon = ${TUNE_FEATURES_tune-armv7a-neon} 
cortex${CORTEX_ID}
-TUNE_FEATURES_tune-cortex${CORTEX_ID}t-neon = ${TUNE_FEATURES_tune-armv7at-neon} 
cortex${CORTEX_ID}
-PACKAGE_EXTRA_ARCHS_tune-cortex${CORTEX_ID} = ${PACKAGE_EXTRA_ARCHS_tune-armv7a} 
cortex${CORTEX_ID}-vfp
-PACKAGE_EXTRA_ARCHS_tune-cortex${CORTEX_ID}t = ${PACKAGE_EXTRA_ARCHS_tune-armv7at} 
cortex${CORTEX_ID}-vfp cortex${CORTEX_ID}t2-vfp
-PACKAGE_EXTRA_ARCHS_tune-cortex${CORTEX_ID}-neon = 
${PACKAGE_EXTRA_ARCHS_tune-armv7a-neon} cortex${CORTEX_ID}-vfp 
cortex${CORTEX_ID}-vfp-neon
-PACKAGE_EXTRA_ARCHS_tune-cortex${CORTEX_ID}t-neon = 
${PACKAGE_EXTRA_ARCHS_tune-armv7at-neon} cortex${CORTEX_ID}-vfp 
cortex${CORTEX_ID}-vfp-neon cortex${CORTEX_ID}t2-vfp cortex${CORTEX_ID}t2-vfp-neon
-
-# VFP Tunes
-AVAILTUNES += cortex${CORTEX_ID}hf cortex${CORTEX_ID}thf cortex${CORTEX_ID}hf-neon 
cortex${CORTEX_ID}thf-neon
-ARMPKGARCH_tune-cortex${CORTEX_ID}hf = cortex${CORTEX_ID}
-ARMPKGARCH_tune-cortex${CORTEX_ID}thf = cortex${CORTEX_ID}
-ARMPKGARCH_tune-cortex${CORTEX_ID}hf-neon = cortex${CORTEX_ID}
-ARMPKGARCH_tune-cortex${CORTEX_ID}thf-neon = cortex${CORTEX_ID}
-TUNE_FEATURES_tune-cortex${CORTEX_ID}hf ?= ${TUNE_FEATURES_tune-armv7ahf} 
cortex${CORTEX_ID}
-TUNE_FEATURES_tune-cortex${CORTEX_ID}thf ?= ${TUNE_FEATURES_tune-armv7athf} 
cortex${CORTEX_ID}
-TUNE_FEATURES_tune-cortex${CORTEX_ID}hf-neon ?= 
${TUNE_FEATURES_tune-armv7ahf-neon} cortex${CORTEX_ID}
-TUNE_FEATURES_tune-cortex${CORTEX_ID}thf-neon ?= 
${TUNE_FEATURES_tune-armv7athf-neon} cortex${CORTEX_ID}
-PACKAGE_EXTRA_ARCHS_tune-cortex${CORTEX_ID}hf = 
${PACKAGE_EXTRA_ARCHS_tune-armv7ahf} cortex${CORTEX_ID}hf-vfp
-PACKAGE_EXTRA_ARCHS_tune-cortex${CORTEX_ID}thf = 
${PACKAGE_EXTRA_ARCHS_tune-armv7athf} cortex${CORTEX_ID}hf-vfp 
cortex${CORTEX_ID}t2hf-vfp
-PACKAGE_EXTRA_ARCHS_tune-cortex${CORTEX_ID}hf-neon = 
${PACKAGE_EXTRA_ARCHS_tune-armv7ahf-neon} cortex${CORTEX_ID}hf-vfp 
cortex${CORTEX_ID}hf-vfp-neon
-PACKAGE_EXTRA_ARCHS_tune-cortex${CORTEX_ID}thf-neon = 
${PACKAGE_EXTRA_ARCHS_tune-armv7athf-neon} cortex${CORTEX_ID}hf-vfp 
cortex${CORTEX_ID}hf-vfp-neon cortex${CORTEX_ID}t2hf-vfp 
cortex${CORTEX_ID}t2hf-vfp-neon
diff --git 

Re: [OE-core] [PATCH] tune-cortexa*.inc: fix tunings for cortex a5, a7, a8, a9, a15 machines.

2013-06-12 Thread Andy Voltz
Hi Saul,

On Wed, Jun 12, 2013 at 12:57:20PM -0400, Saul Wold wrote:
 On 06/12/2013 09:38 AM, Andy Voltz wrote:
  Using CORTEX_ID variable reference in the tuning overrides did not work.
  This reverts those changes, and adds a tuning file for the cortex-a5.
 
  Revert tune-cortexa5.inc: Add tune file for cortex-a5
  Revert tune-cortexa.inc: create a common include for cortex-a armv7a 
  tuning
 
 Andy, does this patch supersede your -mtune not -mcpu patch?  I seem to
 have some trouble applying this patch.
 

Yes. Sorry I thought that patch was NACK/dead.

 It might be best to provide a series that includes the reverts with
 appropriate commit messages.
 

Sure, I can send as a series.

Regards
-- 
Andy Voltz
Timesys Corporation
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH] tune-cortexa*.inc: fix tunings for cortex a5, a7, a8, a9, a15 machines.

2013-06-12 Thread Saul Wold

On 06/12/2013 10:13 AM, Andy Voltz wrote:

Hi Saul,

On Wed, Jun 12, 2013 at 12:57:20PM -0400, Saul Wold wrote:

On 06/12/2013 09:38 AM, Andy Voltz wrote:

Using CORTEX_ID variable reference in the tuning overrides did not work.
This reverts those changes, and adds a tuning file for the cortex-a5.

Revert tune-cortexa5.inc: Add tune file for cortex-a5
Revert tune-cortexa.inc: create a common include for cortex-a armv7a tuning


Andy, does this patch supersede your -mtune not -mcpu patch?  I seem to
have some trouble applying this patch.



Yes. Sorry I thought that patch was NACK/dead.


It might be best to provide a series that includes the reverts with
appropriate commit messages.



Sure, I can send as a series.



Don't worry about it, seems RP already merged it into master, I must 
have had too much junk in my Master_under_test branch and had not 
reverted the right stuff.


Sau!


Regards


___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core