From: Sudeep Holla <sudeep.holla@arm.com>
To: "Javi Merino" <javi.merino@arm.com>,
"\"Chung-Yih Wang (王崇懿)\"" <cywang@google.com>,
"Zhang Rui" <rui.zhang@intel.com>,
"Eduardo Valentin" <edubezval@gmail.com>
Cc: Sudeep Holla <sudeep.holla@arm.com>,
Mark Rutland <Mark.Rutland@arm.com>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
Rob Herring <robh+dt@kernel.org>, Pawel Moll <Pawel.Moll@arm.com>,
Ian Campbell <ijc+devicetree@hellion.org.uk>,
Kumar Gala <galak@codeaurora.org>,
"linux-pm@vger.kernel.org" <linux-pm@vger.kernel.org>,
"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>
Subject: Re: [PATCH] thermal: of: Introduce governor selection in dts
Date: Tue, 11 Aug 2015 10:07:31 +0100 [thread overview]
Message-ID: <55C9BB53.1090001@arm.com> (raw)
In-Reply-To: <20150810174723.GC2747@e104805>
On 10/08/15 18:47, Javi Merino wrote:
> On Mon, Aug 10, 2015 at 09:00:49AM +0100, Chung-Yih Wang (王崇懿) wrote:
>> This patch was originally introduced when we made power_allocator the
>> default governor where we had issues in binding a thermal zone w/o
>> parameters to. Then we came out this facility for binding a specific
>> governor to a thermal zone in dts instead of the default governor.
>> Javi seems like this idea much.
>
> While I can understand why this is not suitable for devicetree, we
> should have a way in the kernel to configure different governors for
> different thermal zones defined in device tree. Thermal zones defined
> from platform code can choose the thermal governor when they are
> registered.
>
> If this information can't go in device tree, where can we put it? As
> an additional parameter to thermal_zone_of_sensor_register()?
>
Why can't it be via sysfs allowing users to select their choice of
governor ? (like cpuidle/freq or even devfreq I assume)
Regards,
Sudeep
next prev parent reply other threads:[~2015-08-11 9:07 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-08-07 7:09 [PATCH] thermal: of: Introduce governor selection in dts Chung-Yih Wang (王崇懿)
2015-08-07 7:18 ` Krzysztof Kozlowski
[not found] ` <CAM2ehZYXsqCoS9ygXNfPHxt9-b5cFxH_ovrgjvM-BfDO6REcUg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-08-07 10:31 ` Mark Rutland
2015-08-07 10:31 ` Mark Rutland
2015-08-10 8:00 ` Chung-Yih Wang (王崇懿)
2015-08-10 17:47 ` Javi Merino
2015-08-11 9:07 ` Sudeep Holla [this message]
[not found] ` <55C9BB53.1090001-5wv7dgnIgG8@public.gmane.org>
2015-11-02 21:24 ` Eduardo Valentin
2015-11-02 21:24 ` Eduardo Valentin
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=55C9BB53.1090001@arm.com \
--to=sudeep.holla@arm.com \
--cc=Mark.Rutland@arm.com \
--cc=Pawel.Moll@arm.com \
--cc=cywang@google.com \
--cc=devicetree@vger.kernel.org \
--cc=edubezval@gmail.com \
--cc=galak@codeaurora.org \
--cc=ijc+devicetree@hellion.org.uk \
--cc=javi.merino@arm.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=robh+dt@kernel.org \
--cc=rui.zhang@intel.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.