All of lore.kernel.org
 help / color / mirror / Atom feed
From: Viresh Kumar <viresh.kumar@linaro.org>
To: Lee Jones <lee.jones@linaro.org>, rob.herring@linaro.org
Cc: linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org, kernel@stlinux.com,
	rjw@rjwysocki.net, linux-pm@vger.kernel.org,
	devicetree@vger.kernel.org, ajitpal.singh@st.com, sre@kernel.org,
	dbaryshkov@gmail.com
Subject: Re: [PATCH v4 1/2] dt: cpufreq: st: Provide bindings for ST's CPUFreq implementation
Date: Tue, 28 Jul 2015 13:20:27 +0530	[thread overview]
Message-ID: <20150728075027.GB13710@linux> (raw)
In-Reply-To: <20150728074104.GL21114@x1>

Cc'ing Rob as well..

On 28-07-15, 08:41, Lee Jones wrote:
> I have two issues with that.  Firstly, although the driver uses the
> OPP API (it also uses the Regulator and Clock API too), it is
> fundamentally a CPUFreq driver, so I think it should have a CPUFreq
> DT entry.  Secondly, if someone doesn't know the history of the
> ST CPUFreq set, they will look here for an accompanying document.  I
> personally wouldn't think to look in power/*opp* for a CPUFreq
> binding.
> 
> Perhaps, as all of the CPUFreq drivers use the OPP API, everything
> should be moved to drivers/base/power or drivers/power?

Okay, looks fine :)

-- 
viresh

WARNING: multiple messages have this Message-ID (diff)
From: viresh.kumar@linaro.org (Viresh Kumar)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v4 1/2] dt: cpufreq: st: Provide bindings for ST's CPUFreq implementation
Date: Tue, 28 Jul 2015 13:20:27 +0530	[thread overview]
Message-ID: <20150728075027.GB13710@linux> (raw)
In-Reply-To: <20150728074104.GL21114@x1>

Cc'ing Rob as well..

On 28-07-15, 08:41, Lee Jones wrote:
> I have two issues with that.  Firstly, although the driver uses the
> OPP API (it also uses the Regulator and Clock API too), it is
> fundamentally a CPUFreq driver, so I think it should have a CPUFreq
> DT entry.  Secondly, if someone doesn't know the history of the
> ST CPUFreq set, they will look here for an accompanying document.  I
> personally wouldn't think to look in power/*opp* for a CPUFreq
> binding.
> 
> Perhaps, as all of the CPUFreq drivers use the OPP API, everything
> should be moved to drivers/base/power or drivers/power?

Okay, looks fine :)

-- 
viresh

  reply	other threads:[~2015-07-28  7:50 UTC|newest]

Thread overview: 103+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-07-27 15:20 [PATCH v4 1/2] dt: cpufreq: st: Provide bindings for ST's CPUFreq implementation Lee Jones
2015-07-27 15:20 ` Lee Jones
2015-07-27 15:20 ` [PATCH v4 2/2] dt: power: st: Provide bindings for ST's OPPs Lee Jones
2015-07-27 15:20   ` Lee Jones
2015-07-28  2:29   ` Viresh Kumar
2015-07-28  2:29     ` Viresh Kumar
2015-07-28  7:34     ` Lee Jones
2015-07-28  7:34       ` Lee Jones
2015-07-28  7:47       ` Viresh Kumar
2015-07-28  7:47         ` Viresh Kumar
2015-07-28  8:30         ` Lee Jones
2015-07-28  8:30           ` Lee Jones
2015-07-28 22:55     ` Stephen Boyd
2015-07-28 22:55       ` Stephen Boyd
2015-07-29  8:14       ` Lee Jones
2015-07-29  8:14         ` Lee Jones
2015-07-29 22:15         ` Stephen Boyd
2015-07-29 22:15           ` Stephen Boyd
2015-07-30  8:46           ` Lee Jones
2015-07-30  8:46             ` Lee Jones
2015-07-30 16:16             ` Rob Herring
2015-07-30 16:16               ` Rob Herring
2015-07-30 16:16               ` Rob Herring
2015-07-31 16:37               ` Stephen Boyd
2015-07-31 16:37                 ` Stephen Boyd
2015-08-01 11:36                 ` Viresh Kumar
2015-08-01 11:36                   ` Viresh Kumar
2015-08-03  3:46                 ` Viresh Kumar
2015-08-03  3:46                   ` Viresh Kumar
2015-08-10 13:22                   ` Lee Jones
2015-08-10 13:22                     ` Lee Jones
2015-08-11  8:00                     ` Viresh Kumar
2015-08-11  8:00                       ` Viresh Kumar
2015-08-11  8:00                       ` Viresh Kumar
2015-08-11  9:30                       ` Lee Jones
2015-08-11  9:30                         ` Lee Jones
2015-08-11 10:09                         ` Viresh Kumar
2015-08-11 10:09                           ` Viresh Kumar
2015-08-11 10:09                           ` Viresh Kumar
2015-08-11 11:54                           ` Lee Jones
2015-08-11 11:54                             ` Lee Jones
2015-08-11 12:01                             ` Viresh Kumar
2015-08-11 12:01                               ` Viresh Kumar
2015-08-11 13:27                               ` Lee Jones
2015-08-11 13:27                                 ` Lee Jones
2015-08-11 14:28                                 ` Viresh Kumar
2015-08-11 14:28                                   ` Viresh Kumar
2015-08-11 15:17                                   ` Lee Jones
2015-08-11 15:17                                     ` Lee Jones
2015-08-12 11:08                                     ` Viresh Kumar
2015-08-12 11:08                                       ` Viresh Kumar
2015-08-26 12:06                                       ` Lee Jones
2015-08-26 12:06                                         ` Lee Jones
2015-09-02  8:06                                         ` Viresh Kumar
2015-09-02  8:06                                           ` Viresh Kumar
2015-09-02  8:06                                           ` Viresh Kumar
2015-09-02 18:58                                           ` Rob Herring
2015-09-02 18:58                                             ` Rob Herring
2015-09-09  6:27                                             ` Viresh Kumar
2015-09-09  6:27                                               ` Viresh Kumar
2015-09-09  7:59                                               ` Lee Jones
2015-09-09  7:59                                                 ` Lee Jones
2015-09-09  8:30                                                 ` Viresh Kumar
2015-09-09  8:30                                                   ` Viresh Kumar
2015-09-09  8:30                                                   ` Viresh Kumar
2015-09-09 13:39                                                   ` Lee Jones
2015-09-09 13:39                                                     ` Lee Jones
2015-09-09 16:02                                                     ` Viresh Kumar
2015-09-09 16:02                                                       ` Viresh Kumar
2015-09-09 16:36                                                       ` Lee Jones
2015-09-09 16:36                                                         ` Lee Jones
2015-09-09 23:50                                                         ` Rob Herring
2015-09-09 23:50                                                           ` Rob Herring
2015-09-10  0:57                                                           ` Stephen Boyd
2015-09-10  0:57                                                             ` Stephen Boyd
2015-09-10  1:04                                                             ` Viresh Kumar
2015-09-10  1:04                                                               ` Viresh Kumar
2015-09-10  8:31                                                               ` Lee Jones
2015-09-10  8:31                                                                 ` Lee Jones
2015-09-16  4:33                                                                 ` Viresh Kumar
2015-09-16  4:33                                                                   ` Viresh Kumar
2015-09-16  6:52                                                                   ` Lee Jones
2015-09-16  6:52                                                                     ` Lee Jones
     [not found]   ` <1438010430-5802-2-git-send-email-lee.jones-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2015-07-28 13:55     ` Rob Herring
2015-07-28 13:55       ` Rob Herring
2015-07-28 13:55       ` Rob Herring
     [not found]       ` <CAL_JsqL=e+fL_67_GPKjt_7wJ81GfFx7m9gjxmBDvW_JBXWpfQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-07-28 14:39         ` Lee Jones
2015-07-28 14:39           ` Lee Jones
2015-07-28 14:39           ` Lee Jones
2015-07-28 15:35           ` Rob Herring
2015-07-28 15:35             ` Rob Herring
2015-07-28 15:43             ` Lee Jones
2015-07-28 15:43               ` Lee Jones
2015-07-28  2:23 ` [PATCH v4 1/2] dt: cpufreq: st: Provide bindings for ST's CPUFreq implementation Viresh Kumar
2015-07-28  2:23   ` Viresh Kumar
2015-07-28  7:41   ` Lee Jones
2015-07-28  7:41     ` Lee Jones
2015-07-28  7:50     ` Viresh Kumar [this message]
2015-07-28  7:50       ` Viresh Kumar
2015-07-28  8:35 ` Viresh Kumar
2015-07-28  8:35   ` Viresh Kumar
2015-07-28  8:55   ` Lee Jones
2015-07-28  8:55     ` Lee Jones

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=20150728075027.GB13710@linux \
    --to=viresh.kumar@linaro.org \
    --cc=ajitpal.singh@st.com \
    --cc=dbaryshkov@gmail.com \
    --cc=devicetree@vger.kernel.org \
    --cc=kernel@stlinux.com \
    --cc=lee.jones@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pm@vger.kernel.org \
    --cc=rjw@rjwysocki.net \
    --cc=rob.herring@linaro.org \
    --cc=sre@kernel.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.