All of lore.kernel.org
 help / color / mirror / Atom feed
From: Masahiro Yamada <yamada.masahiro@socionext.com>
To: Simon Horman <horms@verge.net.au>
Cc: Viresh Kumar <viresh.kumar@linaro.org>,
	Rafael Wysocki <rjw@rjwysocki.net>,
	linux-pm@vger.kernel.org,
	Vincent Guittot <vincent.guittot@linaro.org>,
	Geert Uytterhoeven <geert@linux-m68k.org>,
	Maxime Ripard <maxime.ripard@free-electrons.com>,
	Chen-Yu Tsai <wens@csie.org>, Wei Xu <xuwei5@hisilicon.com>,
	Kukjin Kim <kgene@kernel.org>,
	Krzysztof Kozlowski <krzk@kernel.org>,
	Jun Nie <jun.nie@linaro.org>,
	baoyou.xie@linaro.org,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: [1/2] cpufreq: dt-platdev: Automatically create cpufreq device with OPP v2
Date: Wed, 23 Aug 2017 12:48:43 +0900	[thread overview]
Message-ID: <CAK7LNAQSrHA2_5Az6D1xxFUgm-FHaeQqVnn5O+7GOb6kgKL4vw@mail.gmail.com> (raw)
In-Reply-To: <20170821131759.GA15330@verge.net.au>

2017-08-21 22:17 GMT+09:00 Simon Horman <horms@verge.net.au>:
> On Wed, Aug 16, 2017 at 11:07:27AM +0530, Viresh Kumar wrote:
>> The initial idea of creating the cpufreq-dt-platdev.c file was to keep a
>> list of platforms that use the "operating-points" (V1) bindings and
>> create cpufreq device for them only, as we weren't sure which platforms
>> would want the device to get created automatically as some had their own
>> cpufreq drivers as well, or wanted to initialize cpufreq after doing
>> some stuff from platform code.
>>
>> But that wasn't the case with platforms using "operating-points-v2"
>> property. We wanted the device to get created automatically without the
>> need of adding them to the whitelist. Though, we will still have some
>> exceptions where we don't want to create the device automatically.
>>
>> Rename the earlier platform list as *whitelist* and create a new
>> *blacklist* as well.
>>
>> The cpufreq-dt device will get created if:
>> - The platform is there in the whitelist OR
>> - The platform has "operating-points-v2" property in CPU0's DT node and
>>   isn't part of the blacklist .
>>
>> Reported-by: Geert Uytterhoeven <geert@linux-m68k.org>
>> Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
>
> I have exercised this on the r8a7795 and r8a7795 with the following
> reverted:
>
> * 034def597bb7 ("cpufreq: rcar: Add support for R8A7795 SoC")
> * bea2ebca6b91 ("cpufreq: dt: Add r8a7796 support to to use generic
> cpufreq driver")
>
> Tested-by: Simon Horman <horms+renesas@verge.net.au>



Reviewed-by: Masahiro Yamada <yamada.masahiro@socionext.com>




-- 
Best Regards
Masahiro Yamada

  reply	other threads:[~2017-08-23  3:49 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-08-16  5:37 [PATCH 1/2] cpufreq: dt-platdev: Automatically create cpufreq device with OPP v2 Viresh Kumar
2017-08-16  5:37 ` [PATCH 2/2] cpufreq: dt-platdev: Drop few entries from whitelist Viresh Kumar
2017-08-16  6:51   ` Geert Uytterhoeven
2017-08-17  7:53     ` Simon Horman
2017-08-17  8:40       ` Viresh Kumar
2017-08-21 13:19       ` Simon Horman
2017-08-16  8:53   ` Chen-Yu Tsai
2017-08-16 11:24     ` Viresh Kumar
2017-08-17 17:00       ` Chen-Yu Tsai
2017-08-23  3:33   ` Masahiro Yamada
2017-08-16  8:49 ` [PATCH 1/2] cpufreq: dt-platdev: Automatically create cpufreq device with OPP v2 Chen-Yu Tsai
2017-08-16 11:23   ` Viresh Kumar
2017-08-21 13:17 ` [1/2] " Simon Horman
2017-08-23  3:48   ` Masahiro Yamada [this message]
2017-08-21 13:13 Viresh Kumar
2017-08-21 13:17 ` Simon Horman
2017-08-29 10:11   ` Viresh Kumar
2017-09-11 12:13     ` Keerthy
2017-09-11 12:13       ` Keerthy
2017-09-11 14:18       ` Dave Gerlach
2017-09-11 14:18         ` Dave Gerlach
2017-09-19 18:46         ` Viresh Kumar

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=CAK7LNAQSrHA2_5Az6D1xxFUgm-FHaeQqVnn5O+7GOb6kgKL4vw@mail.gmail.com \
    --to=yamada.masahiro@socionext.com \
    --cc=baoyou.xie@linaro.org \
    --cc=geert@linux-m68k.org \
    --cc=horms@verge.net.au \
    --cc=jun.nie@linaro.org \
    --cc=kgene@kernel.org \
    --cc=krzk@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pm@vger.kernel.org \
    --cc=maxime.ripard@free-electrons.com \
    --cc=rjw@rjwysocki.net \
    --cc=vincent.guittot@linaro.org \
    --cc=viresh.kumar@linaro.org \
    --cc=wens@csie.org \
    --cc=xuwei5@hisilicon.com \
    /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.