linux-pm.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Zhang Rui <rui.zhang@intel.com>
To: linux-pm@vger.kernel.org
Cc: amit.daniel@samsung.com, gu1@aeroxteam.fr,
	hongbo.zhang@linaro.org, Zhang Rui <rui.zhang@intel.com>
Subject: [PATCH 0/5] Thermal: cpu cooling fix
Date: Wed,  6 Feb 2013 12:30:01 +0800	[thread overview]
Message-ID: <1360125006-25018-1-git-send-email-rui.zhang@intel.com> (raw)

the per_cpu cpufreq_frequency_table contains all the frequency entries
which are reported by platform code.
But this table is not clean enough because it may contain invalid,
duplicate entries, and it may be in either descending or asceding order.

converting between cpu frequency and cooling state using this table
in cpufreq cooling code is painful as each cooling state of
cpu cooling device must have a valid and unique frequency.

thus cpu cooling table is introduced in this patch set.
CPU cooling table is a table that
1. each entry represents a cpu cooling state,
   aka, a different cpu frequency.
2. no invalid entry in this table
3. no duplicate entry in this table
4. the frequency of all the entries are in descending order, aka,
   table[0] equals maximum frequency and cooling state 0.

And this is the table we are using inside cpufreq cooling generic code.

Besides, all the cpufreq cooling users, e.g. exynos, should touch
neither per_cpu cpufreq_frequency_table nor the cpu cooling table.
it should invoke cpufreq cooling API to do cooling_state/frequency
transition.

I just run build test here. It would be great that some of you
can test it on exynos.
any comments?

thanks,
rui

             reply	other threads:[~2013-02-06  4:30 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-02-06  4:30 Zhang Rui [this message]
2013-02-06  4:30 ` [PATCH 1/5] Thermal: fix a wrong comment Zhang Rui
2013-02-06  4:30 ` [PATCH 2/5] Thermal: Introduce cpu cooling table Zhang Rui
2013-02-07  9:36   ` Hongbo Zhang
2013-02-08  3:30     ` Zhang Rui
2013-02-08 13:09       ` Hongbo Zhang
2013-02-06  4:30 ` [PATCH 3/5] Thermal: Get cpu frequency via " Zhang Rui
2013-02-06  4:30 ` [PATCH 4/5] Thermal: Get max cpu cooling states " Zhang Rui
2013-02-06  4:30 ` [PATCH 5/5] Thermal: Get cooling state " Zhang Rui

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=1360125006-25018-1-git-send-email-rui.zhang@intel.com \
    --to=rui.zhang@intel.com \
    --cc=amit.daniel@samsung.com \
    --cc=gu1@aeroxteam.fr \
    --cc=hongbo.zhang@linaro.org \
    --cc=linux-pm@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).