All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Srivatsa S. Bhat" <srivatsa.bhat@linux.vnet.ibm.com>
To: Viresh Kumar <viresh.kumar@linaro.org>
Cc: "Gautham R. Shenoy" <ego@linux.vnet.ibm.com>,
	linuxppc-dev@ozlabs.org, Linux PM list <linux-pm@vger.kernel.org>,
	benh@kernel.crashing.org, svaidy@linux.vnet.ibm.com,
	Anton Blanchard <anton@samba.org>
Subject: Re: [PATCH v3 1/5] powernv: cpufreq driver for powernv platform
Date: Fri, 21 Mar 2014 17:15:03 +0530	[thread overview]
Message-ID: <532C263F.4090504@linux.vnet.ibm.com> (raw)
In-Reply-To: <CAOh2x=kp65vrKamtxxR9iCX+t-S3nDJP05+1mfpOiWzbWpfGYA@mail.gmail.com>

On 03/21/2014 02:11 PM, Viresh Kumar wrote:
> On Thu, Mar 20, 2014 at 5:40 PM, Gautham R. Shenoy
> <ego@linux.vnet.ibm.com> wrote:
>> From: Vaidyanathan Srinivasan <svaidy@linux.vnet.ibm.com>
> 
> Hi Vaidy,
> 
>> diff --git a/drivers/cpufreq/Kconfig b/drivers/cpufreq/Kconfig
>> index 4b029c0..4ba1632 100644
>> --- a/drivers/cpufreq/Kconfig
>> +++ b/drivers/cpufreq/Kconfig
>> @@ -48,6 +48,7 @@ config CPU_FREQ_STAT_DETAILS
>>  choice
[...]
>> +static struct cpufreq_driver powernv_cpufreq_driver = {
>> +       .verify         = powernv_cpufreq_verify,
>> +       .target         = powernv_cpufreq_target,
> 
> I think you have Srivatsa there who has seen lots of cpufreq code and
> could have helped you a lot :)
>

:-)

I have followed the locking and synchronization aspects of cpufreq
closely, but unfortunately, I haven't been able to keep up with some of
the other details that well :-( Besides, its hard to keep up with the
rate of your changes, you are way too fast! ;-)

Regards,
Srivatsa S. Bhat


WARNING: multiple messages have this Message-ID (diff)
From: "Srivatsa S. Bhat" <srivatsa.bhat@linux.vnet.ibm.com>
To: Viresh Kumar <viresh.kumar@linaro.org>
Cc: "Gautham R. Shenoy" <ego@linux.vnet.ibm.com>,
	Linux PM list <linux-pm@vger.kernel.org>,
	linuxppc-dev@ozlabs.org, Anton Blanchard <anton@samba.org>
Subject: Re: [PATCH v3 1/5] powernv: cpufreq driver for powernv platform
Date: Fri, 21 Mar 2014 17:15:03 +0530	[thread overview]
Message-ID: <532C263F.4090504@linux.vnet.ibm.com> (raw)
In-Reply-To: <CAOh2x=kp65vrKamtxxR9iCX+t-S3nDJP05+1mfpOiWzbWpfGYA@mail.gmail.com>

On 03/21/2014 02:11 PM, Viresh Kumar wrote:
> On Thu, Mar 20, 2014 at 5:40 PM, Gautham R. Shenoy
> <ego@linux.vnet.ibm.com> wrote:
>> From: Vaidyanathan Srinivasan <svaidy@linux.vnet.ibm.com>
> 
> Hi Vaidy,
> 
>> diff --git a/drivers/cpufreq/Kconfig b/drivers/cpufreq/Kconfig
>> index 4b029c0..4ba1632 100644
>> --- a/drivers/cpufreq/Kconfig
>> +++ b/drivers/cpufreq/Kconfig
>> @@ -48,6 +48,7 @@ config CPU_FREQ_STAT_DETAILS
>>  choice
[...]
>> +static struct cpufreq_driver powernv_cpufreq_driver = {
>> +       .verify         = powernv_cpufreq_verify,
>> +       .target         = powernv_cpufreq_target,
> 
> I think you have Srivatsa there who has seen lots of cpufreq code and
> could have helped you a lot :)
>

:-)

I have followed the locking and synchronization aspects of cpufreq
closely, but unfortunately, I haven't been able to keep up with some of
the other details that well :-( Besides, its hard to keep up with the
rate of your changes, you are way too fast! ;-)

Regards,
Srivatsa S. Bhat

  parent reply	other threads:[~2014-03-21 11:45 UTC|newest]

Thread overview: 80+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-03-20 12:10 [PATCH v3 0/5] powernv: Enable Dynamic Frequency Gautham R. Shenoy
2014-03-20 12:10 ` Gautham R. Shenoy
2014-03-20 12:10 ` [PATCH v3 1/5] powernv: cpufreq driver for powernv platform Gautham R. Shenoy
2014-03-20 12:10   ` Gautham R. Shenoy
2014-03-21  8:41   ` Viresh Kumar
2014-03-21  8:41     ` Viresh Kumar
2014-03-21 10:43     ` Gautham R Shenoy
2014-03-21 10:43       ` Gautham R Shenoy
2014-03-21 10:54       ` Viresh Kumar
2014-03-21 10:54         ` Viresh Kumar
2014-03-21 11:40         ` Srivatsa S. Bhat
2014-03-21 11:40           ` Srivatsa S. Bhat
2014-03-21 13:23         ` Gautham R Shenoy
2014-03-21 13:23           ` Gautham R Shenoy
2014-03-21 13:34           ` Viresh Kumar
2014-03-21 13:34             ` Viresh Kumar
2014-03-21 14:54         ` Gautham R Shenoy
2014-03-21 14:54           ` Gautham R Shenoy
2014-03-22  3:43           ` Viresh Kumar
2014-03-22  3:43             ` Viresh Kumar
2014-03-21 14:48       ` Vaidyanathan Srinivasan
2014-03-21 14:48         ` Vaidyanathan Srinivasan
2014-03-22  3:43         ` Viresh Kumar
2014-03-22  3:43           ` Viresh Kumar
2014-03-21 11:45     ` Srivatsa S. Bhat [this message]
2014-03-21 11:45       ` Srivatsa S. Bhat
2014-03-21 11:47       ` Viresh Kumar
2014-03-21 11:47         ` Viresh Kumar
2014-03-20 12:10 ` [PATCH v3 2/5] powernv,cpufreq:Add per-core locking to serialize frequency transitions Gautham R. Shenoy
2014-03-20 12:10   ` [PATCH v3 2/5] powernv, cpufreq:Add " Gautham R. Shenoy
2014-03-21  6:24   ` [PATCH v3 2/5] powernv,cpufreq:Add " Gautham R Shenoy
2014-03-21  6:24     ` [PATCH v3 2/5] powernv, cpufreq:Add " Gautham R Shenoy
2014-03-21  8:42   ` Viresh Kumar
2014-03-21  8:42     ` Viresh Kumar
2014-03-21  9:56     ` [PATCH v3 2/5] powernv,cpufreq:Add " Srivatsa S. Bhat
2014-03-21  9:56       ` [PATCH v3 2/5] powernv, cpufreq:Add " Srivatsa S. Bhat
2014-03-20 12:10 ` [PATCH v3 3/5] powernv:cpufreq: Create pstate_id_to_freq() helper Gautham R. Shenoy
2014-03-20 12:10   ` Gautham R. Shenoy
2014-03-21  6:24   ` Gautham R Shenoy
2014-03-21  6:24     ` Gautham R Shenoy
2014-03-20 12:10 ` [PATCH v3 4/5] powernv:cpufreq: Export nominal frequency via sysfs Gautham R. Shenoy
2014-03-20 12:10   ` Gautham R. Shenoy
2014-03-21  8:47   ` Viresh Kumar
2014-03-21  8:47     ` Viresh Kumar
2014-03-21  9:55     ` Gautham R Shenoy
2014-03-21  9:55       ` Gautham R Shenoy
2014-03-21  9:57       ` Viresh Kumar
2014-03-21  9:57         ` Viresh Kumar
2014-03-20 12:11 ` [PATCH v3 5/5] powernv:cpufreq: Implement the driver->get() method Gautham R. Shenoy
2014-03-20 12:11   ` Gautham R. Shenoy
2014-03-21  6:25   ` Gautham R Shenoy
2014-03-21  6:25     ` Gautham R Shenoy
2014-03-21  9:31   ` Viresh Kumar
2014-03-21  9:31     ` Viresh Kumar
2014-03-21 11:04     ` Gautham R Shenoy
2014-03-21 11:04       ` Gautham R Shenoy
2014-03-21 11:45       ` Viresh Kumar
2014-03-21 11:45         ` Viresh Kumar
2014-03-21 12:01         ` David Laight
2014-03-21 12:01           ` David Laight
2014-03-21 12:31           ` Gautham R Shenoy
2014-03-21 12:31             ` Gautham R Shenoy
2014-03-21 12:34           ` Viresh Kumar
2014-03-21 12:34             ` Viresh Kumar
2014-03-21 15:01             ` David Laight
2014-03-21 15:01               ` David Laight
2014-03-21 13:04         ` Gautham R Shenoy
2014-03-21 13:04           ` Gautham R Shenoy
2014-03-21 13:12           ` Viresh Kumar
2014-03-21 13:12             ` Viresh Kumar
2014-03-21 14:25             ` Gautham R Shenoy
2014-03-21 14:25               ` Gautham R Shenoy
2014-03-21 22:56       ` Benjamin Herrenschmidt
2014-03-21 22:56         ` Benjamin Herrenschmidt
2014-03-22  7:53         ` Gautham R Shenoy
2014-03-22  7:53           ` Gautham R Shenoy
2014-03-21  7:46 ` [PATCH v3 0/5] powernv: Enable Dynamic Frequency Viresh Kumar
2014-03-21  7:46   ` Viresh Kumar
2014-03-21  8:19   ` Gautham R Shenoy
2014-03-21  8:19     ` Gautham R Shenoy

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=532C263F.4090504@linux.vnet.ibm.com \
    --to=srivatsa.bhat@linux.vnet.ibm.com \
    --cc=anton@samba.org \
    --cc=benh@kernel.crashing.org \
    --cc=ego@linux.vnet.ibm.com \
    --cc=linux-pm@vger.kernel.org \
    --cc=linuxppc-dev@ozlabs.org \
    --cc=svaidy@linux.vnet.ibm.com \
    --cc=viresh.kumar@linaro.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.