Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1736294

[PATCH v2 3/8] PM / devfreq: Show the available min/max frequency through sysfs node

Show all headers | View raw

The existing {min|max}_freq sysfs nodes don't consider whether min/max_freq
are available or not. Those sysfs nodes show just the stored value
in the struct devfreq.

The devfreq uses the OPP interface and then dev_pm_opp_{disable|add}()
might change the state of the device's supported frequency. This patch
shows the available minimum and maximum frequency through sysfs node.

Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com>
---
 drivers/devfreq/devfreq.c | 18 ++++++++++++++++--
 1 file changed, 16 insertions(+), 2 deletions(-)

diff --git a/drivers/devfreq/devfreq.c b/drivers/devfreq/devfreq.c
index 2ce1fd0a1324..799a0cf75d39 100644
--- a/drivers/devfreq/devfreq.c
+++ b/drivers/devfreq/devfreq.c
@@ -1128,7 +1128,14 @@ static ssize_t min_freq_store(struct device *dev, struct device_attribute *attr,
 static ssize_t min_freq_show(struct device *dev, struct device_attribute *attr,
 			     char *buf)
 {
-	return sprintf(buf, "%lu\n", to_devfreq(dev)->min_freq);
+	struct devfreq *df = to_devfreq(dev);
+	unsigned long min_freq = to_devfreq(dev)->min_freq;
+	unsigned long available_min_freq = find_available_min_freq(df);
+
+	if (available_min_freq != 0 && min_freq < available_min_freq)
+		min_freq = available_min_freq;
+
+	return sprintf(buf, "%lu\n", min_freq);
 }
 
 static ssize_t max_freq_store(struct device *dev, struct device_attribute *attr,
@@ -1162,7 +1169,14 @@ static ssize_t max_freq_store(struct device *dev, struct device_attribute *attr,
 static ssize_t max_freq_show(struct device *dev, struct device_attribute *attr,
 			     char *buf)
 {
-	return sprintf(buf, "%lu\n", to_devfreq(dev)->max_freq);
+	struct devfreq *df = to_devfreq(dev);
+	unsigned long max_freq = to_devfreq(dev)->max_freq;
+	unsigned long available_max_freq = find_available_max_freq(df);
+
+	if (available_max_freq != 0 && max_freq > available_max_freq)
+		max_freq = available_max_freq;
+
+	return sprintf(buf, "%lu\n", max_freq);
 }
 static DEVICE_ATTR_RW(max_freq);
 
-- 
1.9.1

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH v2 0/8] PM / devfreq: Use OPP interface to handle the  frequency Chanwoo Choi <cw00.choi@samsung.com> - 2017-09-21 02:40 +0200
  [PATCH v2 1/8] PM / devfreq: Set min/max_freq when adding the  devfreq device Chanwoo Choi <cw00.choi@samsung.com> - 2017-09-21 02:40 +0200
  [PATCH v2 3/8] PM / devfreq: Show the available min/max frequency  through sysfs node Chanwoo Choi <cw00.choi@samsung.com> - 2017-09-21 02:40 +0200
  [PATCH v2 8/8] PM / devfreq: exynos-bus: Register cooling device Chanwoo Choi <cw00.choi@samsung.com> - 2017-09-21 02:40 +0200
    Re: [PATCH v2 8/8] PM / devfreq: exynos-bus: Register cooling  device Chanwoo Choi <cw00.choi@samsung.com> - 2017-09-21 07:00 +0200
  [PATCH v2 6/8] PM / devfreq: Remove unneeded conditional statement Chanwoo Choi <cw00.choi@samsung.com> - 2017-09-21 02:40 +0200
  [PATCH v2.1] PM / devfreq: exynos-bus: Register cooling device Chanwoo Choi <cw00.choi@samsung.com> - 2017-09-26 04:40 +0200

csiph-web