* [PATCH] Add tune for Cortex-A5 chips
@ 2013-10-21 8:59 Ulf Samuelsson
2013-10-21 11:17 ` Martin Jansa
0 siblings, 1 reply; 4+ messages in thread
From: Ulf Samuelsson @ 2013-10-21 8:59 UTC (permalink / raw)
To: openembedded-core; +Cc: Ulf Samuelsson
Signed-off-by: Ulf Samuelsson <ulf@emagii.com>
---
meta/conf/machine/include/tune-cortexa5.inc | 24 ++++++++++++++++++++++++
1 file changed, 24 insertions(+)
create mode 100644 meta/conf/machine/include/tune-cortexa5.inc
diff --git a/meta/conf/machine/include/tune-cortexa5.inc b/meta/conf/machine/include/tune-cortexa5.inc
new file mode 100644
index 0000000..69c945e
--- /dev/null
+++ b/meta/conf/machine/include/tune-cortexa5.inc
@@ -0,0 +1,24 @@
+DEFAULTTUNE ?= "cortexa5thf"
+# TUNE_FEATURES = "armv7a vfp callconvention-hard"
+# TARGET_FPU = "vfp"
+
+require conf/machine/include/arm/arch-armv7a.inc
+
+TUNEVALID[cortexa5] = "Enable Cortex-A5 specific processor optimizations"
+TUNE_CCARGS += "${@bb.utils.contains("TUNE_FEATURES", "cortexa5", "-mtune=cortex-a5", "", d)}"
+
+# Little Endian base configs
+AVAILTUNES += "cortexa5 cortexa5t"
+TUNE_FEATURES_tune-cortexa5 = "${TUNE_FEATURES_tune-armv7a} cortexa5"
+TUNE_FEATURES_tune-cortexa5t = "${TUNE_FEATURES_tune-armv7at} cortexa5"
+
+PACKAGE_EXTRA_ARCHS_tune-cortexa5 = "${PACKAGE_EXTRA_ARCHS_tune-armv7a}"
+PACKAGE_EXTRA_ARCHS_tune-cortexa5t = "${PACKAGE_EXTRA_ARCHS_tune-armv7at}"
+
+# VFP Tunes
+AVAILTUNES += "cortexa5hf cortexa5thf"
+TUNE_FEATURES_tune-cortexa5hf ?= "${TUNE_FEATURES_tune-armv7ahf} cortexa5"
+TUNE_FEATURES_tune-cortexa5thf ?= "${TUNE_FEATURES_tune-armv7athf} cortexa5"
+PACKAGE_EXTRA_ARCHS_tune-cortexa5hf = "${PACKAGE_EXTRA_ARCHS_tune-armv7ahf}"
+PACKAGE_EXTRA_ARCHS_tune-cortexa5thf = "${PACKAGE_EXTRA_ARCHS_tune-armv7athf}"
+
--
1.7.9.5
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [PATCH] Add tune for Cortex-A5 chips
2013-10-21 8:59 [PATCH] Add tune for Cortex-A5 chips Ulf Samuelsson
@ 2013-10-21 11:17 ` Martin Jansa
2013-10-21 12:55 ` Ulf Samuelsson
0 siblings, 1 reply; 4+ messages in thread
From: Martin Jansa @ 2013-10-21 11:17 UTC (permalink / raw)
To: Ulf Samuelsson; +Cc: openembedded-core
[-- Attachment #1: Type: text/plain, Size: 2740 bytes --]
On Mon, Oct 21, 2013 at 10:59:57AM +0200, Ulf Samuelsson wrote:
> Signed-off-by: Ulf Samuelsson <ulf@emagii.com>
> ---
> meta/conf/machine/include/tune-cortexa5.inc | 24 ++++++++++++++++++++++++
> 1 file changed, 24 insertions(+)
> create mode 100644 meta/conf/machine/include/tune-cortexa5.inc
>
> diff --git a/meta/conf/machine/include/tune-cortexa5.inc b/meta/conf/machine/include/tune-cortexa5.inc
> new file mode 100644
> index 0000000..69c945e
> --- /dev/null
> +++ b/meta/conf/machine/include/tune-cortexa5.inc
> @@ -0,0 +1,24 @@
> +DEFAULTTUNE ?= "cortexa5thf"
wasn't cortexa5 added already in:
commit 50bc63c5c377d9fbb87b3efefc8c6f5473ba642a
Author: Andy Voltz <andy.voltz@timesys.com>
Date: Wed Jun 5 11:52:30 2013 -0400
tune-cortexa5.inc: Add tune file for cortex-a5
I think that one was better, because it had optional neon support and
default DEFAULTTUNE should be something more generic (at least to stay
consistent with other tunes) it's DISTRO decision to select common
denominator (or more specific tunes for each MACHINE) for binary feeds
they want to manage.
see
commit a11bdc36a1be18cc5aa14682b2a2c9ee83141f51
Author: Martin Jansa <Martin.Jansa@gmail.com>
Date: Tue Oct 2 13:21:08 2012 +0000
tune-*: define more generic DEFAULTTUNE to share feed between
machines
> +# TUNE_FEATU4RES = "armv7a vfp callconvention-hard"
> +# TARGET_FPU = "vfp"
> +
> +require conf/machine/include/arm/arch-armv7a.inc
> +
> +TUNEVALID[cortexa5] = "Enable Cortex-A5 specific processor optimizations"
> +TUNE_CCARGS += "${@bb.utils.contains("TUNE_FEATURES", "cortexa5", "-mtune=cortex-a5", "", d)}"
> +
> +# Little Endian base configs
> +AVAILTUNES += "cortexa5 cortexa5t"
> +TUNE_FEATURES_tune-cortexa5 = "${TUNE_FEATURES_tune-armv7a} cortexa5"
> +TUNE_FEATURES_tune-cortexa5t = "${TUNE_FEATURES_tune-armv7at} cortexa5"
> +
> +PACKAGE_EXTRA_ARCHS_tune-cortexa5 = "${PACKAGE_EXTRA_ARCHS_tune-armv7a}"
> +PACKAGE_EXTRA_ARCHS_tune-cortexa5t = "${PACKAGE_EXTRA_ARCHS_tune-armv7at}"
> +
> +# VFP Tunes
> +AVAILTUNES += "cortexa5hf cortexa5thf"
> +TUNE_FEATURES_tune-cortexa5hf ?= "${TUNE_FEATURES_tune-armv7ahf} cortexa5"
> +TUNE_FEATURES_tune-cortexa5thf ?= "${TUNE_FEATURES_tune-armv7athf} cortexa5"
> +PACKAGE_EXTRA_ARCHS_tune-cortexa5hf = "${PACKAGE_EXTRA_ARCHS_tune-armv7ahf}"
> +PACKAGE_EXTRA_ARCHS_tune-cortexa5thf = "${PACKAGE_EXTRA_ARCHS_tune-armv7athf}"
> +
> --
> 1.7.9.5
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
--
Martin 'JaMa' Jansa jabber: Martin.Jansa@gmail.com
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH] Add tune for Cortex-A5 chips
2013-10-21 11:17 ` Martin Jansa
@ 2013-10-21 12:55 ` Ulf Samuelsson
2013-10-21 21:46 ` Khem Raj
0 siblings, 1 reply; 4+ messages in thread
From: Ulf Samuelsson @ 2013-10-21 12:55 UTC (permalink / raw)
To: Martin Jansa; +Cc: openembedded-core
On 2013-10-21 13:17, Martin Jansa wrote:
> On Mon, Oct 21, 2013 at 10:59:57AM +0200, Ulf Samuelsson wrote:
>> Signed-off-by: Ulf Samuelsson <ulf@emagii.com>
>> ---
>> meta/conf/machine/include/tune-cortexa5.inc | 24 ++++++++++++++++++++++++
>> 1 file changed, 24 insertions(+)
>> create mode 100644 meta/conf/machine/include/tune-cortexa5.inc
>>
>> diff --git a/meta/conf/machine/include/tune-cortexa5.inc b/meta/conf/machine/include/tune-cortexa5.inc
>> new file mode 100644
>> index 0000000..69c945e
>> --- /dev/null
>> +++ b/meta/conf/machine/include/tune-cortexa5.inc
>> @@ -0,0 +1,24 @@
>> +DEFAULTTUNE ?= "cortexa5thf"
> wasn't cortexa5 added already in:
>
> commit 50bc63c5c377d9fbb87b3efefc8c6f5473ba642a
> Author: Andy Voltz <andy.voltz@timesys.com>
> Date: Wed Jun 5 11:52:30 2013 -0400
>
> tune-cortexa5.inc: Add tune file for cortex-a5
>
> I think that one was better, because it had optional neon support and
> default DEFAULTTUNE should be something more generic (at least to stay
> consistent with other tunes) it's DISTRO decision to select common
> denominator (or more specific tunes for each MACHINE) for binary feeds
> they want to manage.
Looks like you are right.
I looked at the openembedded-core in Angstrom master, and it wasn't there.
(
because Angstrom master checks out
"openembedded-core/remotes/origin/angstrom-staging-yocto1.3"
)
Sorry.
/Ulf
>
> see
> commit a11bdc36a1be18cc5aa14682b2a2c9ee83141f51
> Author: Martin Jansa <Martin.Jansa@gmail.com>
> Date: Tue Oct 2 13:21:08 2012 +0000
>
> tune-*: define more generic DEFAULTTUNE to share feed between
> machines
>
>> +# TUNE_FEATU4RES = "armv7a vfp callconvention-hard"
>> +# TARGET_FPU = "vfp"
>> +
>> +require conf/machine/include/arm/arch-armv7a.inc
>> +
>> +TUNEVALID[cortexa5] = "Enable Cortex-A5 specific processor optimizations"
>> +TUNE_CCARGS += "${@bb.utils.contains("TUNE_FEATURES", "cortexa5", "-mtune=cortex-a5", "", d)}"
>> +
>> +# Little Endian base configs
>> +AVAILTUNES += "cortexa5 cortexa5t"
>> +TUNE_FEATURES_tune-cortexa5 = "${TUNE_FEATURES_tune-armv7a} cortexa5"
>> +TUNE_FEATURES_tune-cortexa5t = "${TUNE_FEATURES_tune-armv7at} cortexa5"
>> +
>> +PACKAGE_EXTRA_ARCHS_tune-cortexa5 = "${PACKAGE_EXTRA_ARCHS_tune-armv7a}"
>> +PACKAGE_EXTRA_ARCHS_tune-cortexa5t = "${PACKAGE_EXTRA_ARCHS_tune-armv7at}"
>> +
>> +# VFP Tunes
>> +AVAILTUNES += "cortexa5hf cortexa5thf"
>> +TUNE_FEATURES_tune-cortexa5hf ?= "${TUNE_FEATURES_tune-armv7ahf} cortexa5"
>> +TUNE_FEATURES_tune-cortexa5thf ?= "${TUNE_FEATURES_tune-armv7athf} cortexa5"
>> +PACKAGE_EXTRA_ARCHS_tune-cortexa5hf = "${PACKAGE_EXTRA_ARCHS_tune-armv7ahf}"
>> +PACKAGE_EXTRA_ARCHS_tune-cortexa5thf = "${PACKAGE_EXTRA_ARCHS_tune-armv7athf}"
>> +
>> --
>> 1.7.9.5
>>
>> _______________________________________________
>> Openembedded-core mailing list
>> Openembedded-core@lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-core
--
Best Regards
Ulf Samuelsson
ulf@emagii.com
+46 722 427437
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH] Add tune for Cortex-A5 chips
2013-10-21 12:55 ` Ulf Samuelsson
@ 2013-10-21 21:46 ` Khem Raj
0 siblings, 0 replies; 4+ messages in thread
From: Khem Raj @ 2013-10-21 21:46 UTC (permalink / raw)
To: ulf@emagii.com; +Cc: openembedded-core@lists.openembedded.org
[-- Attachment #1: Type: text/plain, Size: 3622 bytes --]
On Monday, October 21, 2013, Ulf Samuelsson <ulf@emagii.com> wrote:
> On 2013-10-21 13:17, Martin Jansa wrote:
>>
>> On Mon, Oct 21, 2013 at 10:59:57AM +0200, Ulf Samuelsson wrote:
>>>
>>> Signed-off-by: Ulf Samuelsson <ulf@emagii.com>
>>> ---
>>> meta/conf/machine/include/tune-cortexa5.inc | 24
++++++++++++++++++++++++
>>> 1 file changed, 24 insertions(+)
>>> create mode 100644 meta/conf/machine/include/tune-cortexa5.inc
>>>
>>> diff --git a/meta/conf/machine/include/tune-cortexa5.inc
b/meta/conf/machine/include/tune-cortexa5.inc
>>> new file mode 100644
>>> index 0000000..69c945e
>>> --- /dev/null
>>> +++ b/meta/conf/machine/include/tune-cortexa5.inc
>>> @@ -0,0 +1,24 @@
>>> +DEFAULTTUNE ?= "cortexa5thf"
>>
>> wasn't cortexa5 added already in:
>>
>> commit 50bc63c5c377d9fbb87b3efefc8c6f5473ba642a
>> Author: Andy Voltz <andy.voltz@timesys.com>
>> Date: Wed Jun 5 11:52:30 2013 -0400
>>
>> tune-cortexa5.inc: Add tune file for cortex-a5
>>
>> I think that one was better, because it had optional neon support and
>> default DEFAULTTUNE should be something more generic (at least to stay
>> consistent with other tunes) it's DISTRO decision to select common
>> denominator (or more specific tunes for each MACHINE) for binary feeds
>> they want to manage.
>
> Looks like you are right.
> I looked at the openembedded-core in Angstrom master, and it wasn't there.
> (
> because Angstrom master checks out
> "openembedded-core/remotes/origin/angstrom-staging-yocto1.3"
> )
>
thats default checkout of setup hopefully will change to 2013.12 release
soon which will be based on dora release
> Sorry.
>
> /Ulf
>
>>
>> see
>> commit a11bdc36a1be18cc5aa14682b2a2c9ee83141f51
>> Author: Martin Jansa <Martin.Jansa@gmail.com>
>> Date: Tue Oct 2 13:21:08 2012 +0000
>>
>> tune-*: define more generic DEFAULTTUNE to share feed between
>> machines
>>
>>> +# TUNE_FEATU4RES = "armv7a vfp callconvention-hard"
>>> +# TARGET_FPU = "vfp"
>>> +
>>> +require conf/machine/include/arm/arch-armv7a.inc
>>> +
>>> +TUNEVALID[cortexa5] = "Enable Cortex-A5 specific processor
optimizations"
>>> +TUNE_CCARGS += "${@bb.utils.contains("TUNE_FEATURES", "cortexa5",
"-mtune=cortex-a5", "", d)}"
>>> +
>>> +# Little Endian base configs
>>> +AVAILTUNES += "cortexa5 cortexa5t"
>>> +TUNE_FEATURES_tune-cortexa5 = "${TUNE_FEATURES_tune-armv7a} cortexa5"
>>> +TUNE_FEATURES_tune-cortexa5t = "${TUNE_FEATURES_tune-armv7at} cortexa5"
>>> +
>>> +PACKAGE_EXTRA_ARCHS_tune-cortexa5 =
"${PACKAGE_EXTRA_ARCHS_tune-armv7a}"
>>> +PACKAGE_EXTRA_ARCHS_tune-cortexa5t =
"${PACKAGE_EXTRA_ARCHS_tune-armv7at}"
>>> +
>>> +# VFP Tunes
>>> +AVAILTUNES += "cortexa5hf cortexa5thf"
>>> +TUNE_FEATURES_tune-cortexa5hf ?= "${TUNE_FEATURES_tune-armv7ahf}
cortexa5"
>>> +TUNE_FEATURES_tune-cortexa5thf ?= "${TUNE_FEATURES_tune-armv7athf}
cortexa5"
>>> +PACKAGE_EXTRA_ARCHS_tune-cortexa5hf =
"${PACKAGE_EXTRA_ARCHS_tune-armv7ahf}"
>>> +PACKAGE_EXTRA_ARCHS_tune-cortexa5thf =
"${PACKAGE_EXTRA_ARCHS_tune-armv7athf}"
>>> +
>>> --
>>> 1.7.9.5
>>>
>>> _______________________________________________
>>> Openembedded-core mailing list
>>> Openembedded-core@lists.openembedded.org
>>> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>
>
> --
> Best Regards
> Ulf Samuelsson
> ulf@emagii.com
> +46 722 427437
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>
[-- Attachment #2: Type: text/html, Size: 5211 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2013-10-21 21:46 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-10-21 8:59 [PATCH] Add tune for Cortex-A5 chips Ulf Samuelsson
2013-10-21 11:17 ` Martin Jansa
2013-10-21 12:55 ` Ulf Samuelsson
2013-10-21 21:46 ` Khem Raj
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox