PM / devfreq: Show the all available frequencies (416b46a2) · Commits · 戴 / test

The commit ("thermal: Add devfreq cooling") allows the devfreq device to use the cooling device. When the cooling down are required, the devfreq_cooling.c disables the OPP entry with the dev_pm_opp_disable(). In result, 'available_frequencies'[1] sysfs node never came to show the all available frequencies. [1] /sys/class/devfreq/.../available_frequencies So, this patch uses the 'freq_table' in the 'struct devfreq_dev_profile' in order to show the all available frequencies. - If 'freq_table' is NULL, devfreq core initializes them by using OPP values. - If 'freq_table' is initialized, devfreq core just uses the 'freq_table'. And this patch adds some comment about the sort way of 'freq_table'. Signed-off-by:

Chanwoo Choi <cw00.choi@samsung.com> Signed-off-by:

MyungJoo Ham <myungjoo.ham@samsung.com>

drivers/devfreq/devfreq.c

+5 −11

Original line number	Diff line number	Diff line
		@@ -1194,22 +1194,16 @@ static ssize_t available_frequencies_show(struct device *d,
		char *buf)
		{
		struct devfreq *df = to_devfreq(d);
		struct device *dev = df->dev.parent;
		struct dev_pm_opp *opp;
		ssize_t count = 0;
		unsigned long freq = 0;
		int i;

		do {
		opp = dev_pm_opp_find_freq_ceil(dev, &freq);
		if (IS_ERR(opp))
		break;
		mutex_lock(&df->lock);

		dev_pm_opp_put(opp);
		for (i = 0; i < df->profile->max_state; i++)
		count += scnprintf(&buf[count], (PAGE_SIZE - count - 2),
		"%lu ", freq);
		freq++;
		} while (1);
		"%lu ", df->profile->freq_table[i]);

		mutex_unlock(&df->lock);
		/* Truncate the trailing space */
		if (count)
		count--;

include/linux/devfreq.h

+3 −2

Original line number	Diff line number	Diff line
		@@ -84,7 +84,8 @@ struct devfreq_dev_status {
		* from devfreq_remove_device() call. If the user
		* has registered devfreq->nb at a notifier-head,
		* this is the time to unregister it.
		* @freq_table: Optional list of frequencies to support statistics.
		* @freq_table: Optional list of frequencies to support statistics
		* and freq_table must be generated in ascending order.
		* @max_state: The size of freq_table.
		*/
		struct devfreq_dev_profile {

Admin message