From: Matthew Longnecker <mlongnecker@nvidia.com>
To: linux-pm@vger.kernel.org
Cc: linux-tegra@vger.kernel.org
Subject: Re: [PATCH v2 0/2] Support to tune governor in run time
Date: Mon, 13 Jan 2014 11:01:36 -0800 [thread overview]
Message-ID: <52D43810.6030801@nvidia.com> (raw)
In-Reply-To: <52D4095C.205@ti.com>
On 1/13/2014 7:42 AM, Eduardo Valentin wrote:
>> This serie can support to turn governor for thermal zone in
>> run time.
>
> Can you please explain why this is needed? Are you facing troubles with
> current way to switch governors? If yes, can you please report them?
>
....
>> Adds thermal_update_governor() function, so the thermal platform
>> driver can use it to update governor.
>
> Here I cannot see why the platform driver would need to update a
> governor, instead of a zone. Platform drivers are not supposed to be
> aware of governors. For updating a zone we already have an API for that,
> please check documentation.
>
I think we have a miscommunication. The purpose of
thermal_update_governor is to *switch* governors at runtime (from within
the kernel).
Wei has used the term "update" in the sense of switch rather than
"update" in the sense used by thermal_zone_device_update.
Eduardo, what is your recommended technique for setting the governor of
a thermal zone device created via device tree?
thanks,
Matt
next prev parent reply other threads:[~2014-01-13 19:01 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-01-13 11:17 [PATCH v2 0/2] Support to tune governor in run time Wei Ni
2014-01-13 11:17 ` [PATCH v2 1/2] thermal: add available policies attribut Wei Ni
[not found] ` <1389611863-7812-2-git-send-email-wni-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2014-01-13 15:58 ` Eduardo Valentin
2014-01-15 11:26 ` Wei Ni
[not found] ` <52D67066.8010403-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2014-01-15 14:00 ` Eduardo Valentin
[not found] ` <1389611863-7812-1-git-send-email-wni-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2014-01-13 11:17 ` [PATCH v2 2/2] thermal: add interface to support tune governor in run-time Wei Ni
[not found] ` <1389611863-7812-3-git-send-email-wni-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2014-01-17 8:22 ` Zhang Rui
2014-01-17 9:32 ` Wei Ni
[not found] ` <52D8F8BD.1080700-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2014-01-20 1:41 ` Zhang Rui
2014-01-20 8:47 ` Wei Ni
2014-01-17 16:01 ` Eduardo Valentin
2014-01-13 15:42 ` [PATCH v2 0/2] Support to tune governor in run time Eduardo Valentin
2014-01-13 19:01 ` Matthew Longnecker [this message]
2014-01-13 21:33 ` Eduardo Valentin
2014-01-14 4:17 ` Wei Ni
2014-01-14 17:44 ` Eduardo Valentin
[not found] ` <52D57762.8070609-l0cyMroinI0@public.gmane.org>
2014-01-15 11:35 ` Wei Ni
[not found] ` <52D67296.7050107-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2014-01-15 14:06 ` Eduardo Valentin
[not found] ` <52D4BA76.4040003-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2014-01-14 17:50 ` Eduardo Valentin
[not found] ` <52D57901.5050300-l0cyMroinI0@public.gmane.org>
2014-01-15 11:43 ` Wei Ni
2014-01-15 14:04 ` 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=52D43810.6030801@nvidia.com \
--to=mlongnecker@nvidia.com \
--cc=linux-pm@vger.kernel.org \
--cc=linux-tegra@vger.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.