All of lore.kernel.org
 help / color / mirror / Atom feed
From: akuster808 <akuster808@gmail.com>
To: Mark Hatle <mark.hatle@windriver.com>,
	 openembedded-core@lists.openembedded.org
Subject: Re: [PATCH 0/3] Add new tune files for IBM POWER5,6,7
Date: Mon, 25 Aug 2014 12:53:46 -0700	[thread overview]
Message-ID: <53FB944A.5080000@gmail.com> (raw)
In-Reply-To: <53FB8AFB.7090406@windriver.com>



On 08/25/2014 12:14 PM, Mark Hatle wrote:
> On 8/24/14, 4:41 PM, Armin Kuster wrote:
>> I believe these are new tune files. They add support for
>> IBM's POWER[5,6,7] cpus. Minimally tested on qemuppc64.
>> This is the first set of patches to add qemuppc64 support.
>>
>> Let me know if there are any issues with the format of these tune files.
>
> Only a minor comment.. I'd prefer if the tunes were named
> 'tune-power5.inc', etc.. instead of 'ppcpower...'

easy enough for me to change. 'ppcpower*' seemed odd to me too but I was 
following the convention of the other ppc tune files.

>
> But I can't say I care enough to say reject this.  :)  So if the
> existing names are already in use, I think thats enough of reason to let
> it in..

thanks for taking the time to respond.

- Armin

>
> --Mark
>
>>
>>
>> Armin Kuster (3):
>>    IBM Power5: Add new tune file for PPC power5 cpu
>>    IBM power6: Add new tune file for PPC power6
>>    IBM power7: Add new tune file for PPC power7
>>
>>   meta/conf/machine/include/tune-ppcpower5.inc | 21 +++++++++++++++++++++
>>   meta/conf/machine/include/tune-ppcpower6.inc | 21 +++++++++++++++++++++
>>   meta/conf/machine/include/tune-ppcpower7.inc | 21 +++++++++++++++++++++
>>   3 files changed, 63 insertions(+)
>>   create mode 100644 meta/conf/machine/include/tune-ppcpower5.inc
>>   create mode 100644 meta/conf/machine/include/tune-ppcpower6.inc
>>   create mode 100644 meta/conf/machine/include/tune-ppcpower7.inc
>>
>


      reply	other threads:[~2014-08-25 19:53 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-08-24 21:41 [PATCH 0/3] Add new tune files for IBM POWER5,6,7 Armin Kuster
2014-08-24 21:41 ` [PATCH 1/3] IBM Power5: Add new tune file for PPC power5 cpu Armin Kuster
2014-08-24 21:41 ` [PATCH 2/3] IBM power6: Add new tune file for PPC power6 Armin Kuster
2014-08-24 21:41 ` [PATCH 3/3] IBM power7: Add new tune file for PPC power7 Armin Kuster
2014-08-25 19:14 ` [PATCH 0/3] Add new tune files for IBM POWER5,6,7 Mark Hatle
2014-08-25 19:53   ` akuster808 [this message]

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=53FB944A.5080000@gmail.com \
    --to=akuster808@gmail.com \
    --cc=mark.hatle@windriver.com \
    --cc=openembedded-core@lists.openembedded.org \
    /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.