Skip to content

Commit 1271d6d

Browse files
rafaeljwrafaeljw
committed
Merge branch 'pm-opp'
* pm-opp: cpufreq: OMAP: Register an Energy Model cpufreq: imx6q: Register an Energy Model opp: no need to check return value of debugfs_create functions cpufreq: mediatek: Register an Energy Model cpufreq: scmi: Register an Energy Model cpufreq: arm_big_little: Register an Energy Model cpufreq: scpi: Register an Energy Model cpufreq: dt: Register an Energy Model
2 parents 6a558c7 + 78317ed commit 1271d6d

File tree

10 files changed

+87
-102
lines changed

10 files changed

+87
-102
lines changed

‎drivers/cpufreq/arm_big_little.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -487,6 +487,8 @@ static int bL_cpufreq_init(struct cpufreq_policy *policy)
487487
policy->cpuinfo.transition_latency =
488488
arm_bL_ops->get_transition_latency(cpu_dev);
489489

490+
dev_pm_opp_of_register_em(policy->cpus);
491+
490492
if (is_bL_switching_enabled())
491493
per_cpu(cpu_last_req_freq, policy->cpu) = clk_get_cpu_rate(policy->cpu);
492494

‎drivers/cpufreq/cpufreq-dt.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -278,6 +278,8 @@ static int cpufreq_init(struct cpufreq_policy *policy)
278278
policy->cpuinfo.transition_latency = transition_latency;
279279
policy->dvfs_possible_from_any_cpu = true;
280280

281+
dev_pm_opp_of_register_em(policy->cpus);
282+
281283
return 0;
282284

283285
out_free_cpufreq_table:

‎drivers/cpufreq/imx6q-cpufreq.c

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -198,6 +198,7 @@ static int imx6q_cpufreq_init(struct cpufreq_policy *policy)
198198
policy->clk = clks[ARM].clk;
199199
ret = cpufreq_generic_init(policy, freq_table, transition_latency);
200200
policy->suspend_freq = max_freq;
201+
dev_pm_opp_of_register_em(policy->cpus);
201202

202203
return ret;
203204
}

‎drivers/cpufreq/mediatek-cpufreq.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -456,6 +456,8 @@ static int mtk_cpufreq_init(struct cpufreq_policy *policy)
456456
policy->driver_data = info;
457457
policy->clk = info->cpu_clk;
458458

459+
dev_pm_opp_of_register_em(policy->cpus);
460+
459461
return 0;
460462
}
461463

‎drivers/cpufreq/omap-cpufreq.c

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -133,8 +133,10 @@ static int omap_cpu_init(struct cpufreq_policy *policy)
133133

134134
/* FIXME: what's the actual transition time? */
135135
result = cpufreq_generic_init(policy, freq_table, 300 * 1000);
136-
if (!result)
136+
if (!result) {
137+
dev_pm_opp_of_register_em(policy->cpus);
137138
return 0;
139+
}
138140

139141
freq_table_free();
140142
fail:

‎drivers/cpufreq/scmi-cpufreq.c

Lines changed: 36 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@
1111
#include <linux/cpu.h>
1212
#include <linux/cpufreq.h>
1313
#include <linux/cpumask.h>
14+
#include <linux/energy_model.h>
1415
#include <linux/export.h>
1516
#include <linux/module.h>
1617
#include <linux/pm_opp.h>
@@ -101,13 +102,42 @@ scmi_get_sharing_cpus(struct device *cpu_dev, struct cpumask *cpumask)
101102
return 0;
102103
}
103104

105+
static int __maybe_unused
106+
scmi_get_cpu_power(unsigned long *power, unsigned long *KHz, int cpu)
107+
{
108+
struct device *cpu_dev = get_cpu_device(cpu);
109+
unsigned long Hz;
110+
int ret, domain;
111+
112+
if (!cpu_dev) {
113+
pr_err("failed to get cpu%d device\n", cpu);
114+
return -ENODEV;
115+
}
116+
117+
domain = handle->perf_ops->device_domain_id(cpu_dev);
118+
if (domain < 0)
119+
return domain;
120+
121+
/* Get the power cost of the performance domain. */
122+
Hz = *KHz * 1000;
123+
ret = handle->perf_ops->est_power_get(handle, domain, &Hz, power);
124+
if (ret)
125+
return ret;
126+
127+
/* The EM framework specifies the frequency in KHz. */
128+
*KHz = Hz / 1000;
129+
130+
return 0;
131+
}
132+
104133
static int scmi_cpufreq_init(struct cpufreq_policy *policy)
105134
{
106-
int ret;
135+
int ret, nr_opp;
107136
unsigned int latency;
108137
struct device *cpu_dev;
109138
struct scmi_data *priv;
110139
struct cpufreq_frequency_table *freq_table;
140+
struct em_data_callback em_cb = EM_DATA_CB(scmi_get_cpu_power);
111141

112142
cpu_dev = get_cpu_device(policy->cpu);
113143
if (!cpu_dev) {
@@ -134,8 +164,8 @@ static int scmi_cpufreq_init(struct cpufreq_policy *policy)
134164
return ret;
135165
}
136166

137-
ret = dev_pm_opp_get_opp_count(cpu_dev);
138-
if (ret <= 0) {
167+
nr_opp = dev_pm_opp_get_opp_count(cpu_dev);
168+
if (nr_opp <= 0) {
139169
dev_dbg(cpu_dev, "OPP table is not ready, deferring probe\n");
140170
ret = -EPROBE_DEFER;
141171
goto out_free_opp;
@@ -169,6 +199,9 @@ static int scmi_cpufreq_init(struct cpufreq_policy *policy)
169199
policy->cpuinfo.transition_latency = latency;
170200

171201
policy->fast_switch_possible = true;
202+
203+
em_register_perf_domain(policy->cpus, nr_opp, &em_cb);
204+
172205
return 0;
173206

174207
out_free_priv:

‎drivers/cpufreq/scpi-cpufreq.c

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -168,6 +168,9 @@ static int scpi_cpufreq_init(struct cpufreq_policy *policy)
168168
policy->cpuinfo.transition_latency = latency;
169169

170170
policy->fast_switch_possible = false;
171+
172+
dev_pm_opp_of_register_em(policy->cpus);
173+
171174
return 0;
172175

173176
out_free_cpufreq_table:

‎drivers/opp/core.c

Lines changed: 2 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -792,7 +792,6 @@ static struct opp_device *_add_opp_dev_unlocked(const struct device *dev,
792792
struct opp_table *opp_table)
793793
{
794794
struct opp_device *opp_dev;
795-
int ret;
796795

797796
opp_dev = kzalloc(sizeof(*opp_dev), GFP_KERNEL);
798797
if (!opp_dev)
@@ -804,10 +803,7 @@ static struct opp_device *_add_opp_dev_unlocked(const struct device *dev,
804803
list_add(&opp_dev->node, &opp_table->dev_list);
805804

806805
/* Create debugfs entries for the opp_table */
807-
ret = opp_debug_register(opp_dev, opp_table);
808-
if (ret)
809-
dev_err(dev, "%s: Failed to register opp debugfs (%d)\n",
810-
__func__, ret);
806+
opp_debug_register(opp_dev, opp_table);
811807

812808
return opp_dev;
813809
}
@@ -1228,10 +1224,7 @@ int _opp_add(struct device *dev, struct dev_pm_opp *new_opp,
12281224
new_opp->opp_table = opp_table;
12291225
kref_init(&new_opp->kref);
12301226

1231-
ret = opp_debug_create_one(new_opp, opp_table);
1232-
if (ret)
1233-
dev_err(dev, "%s: Failed to register opp to debugfs (%d)\n",
1234-
__func__, ret);
1227+
opp_debug_create_one(new_opp, opp_table);
12351228

12361229
if (!_opp_supported_by_regulators(new_opp, opp_table)) {
12371230
new_opp->available = false;

‎drivers/opp/debugfs.c

Lines changed: 29 additions & 81 deletions
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ void opp_debug_remove_one(struct dev_pm_opp *opp)
3535
debugfs_remove_recursive(opp->dentry);
3636
}
3737

38-
static bool opp_debug_create_supplies(struct dev_pm_opp *opp,
38+
static void opp_debug_create_supplies(struct dev_pm_opp *opp,
3939
struct opp_table *opp_table,
4040
struct dentry *pdentry)
4141
{
@@ -50,30 +50,21 @@ static bool opp_debug_create_supplies(struct dev_pm_opp *opp,
5050
/* Create per-opp directory */
5151
d = debugfs_create_dir(name, pdentry);
5252

53-
if (!d)
54-
return false;
53+
debugfs_create_ulong("u_volt_target", S_IRUGO, d,
54+
&opp->supplies[i].u_volt);
5555

56-
if (!debugfs_create_ulong("u_volt_target", S_IRUGO, d,
57-
&opp->supplies[i].u_volt))
58-
return false;
56+
debugfs_create_ulong("u_volt_min", S_IRUGO, d,
57+
&opp->supplies[i].u_volt_min);
5958

60-
if (!debugfs_create_ulong("u_volt_min", S_IRUGO, d,
61-
&opp->supplies[i].u_volt_min))
62-
return false;
59+
debugfs_create_ulong("u_volt_max", S_IRUGO, d,
60+
&opp->supplies[i].u_volt_max);
6361

64-
if (!debugfs_create_ulong("u_volt_max", S_IRUGO, d,
65-
&opp->supplies[i].u_volt_max))
66-
return false;
67-
68-
if (!debugfs_create_ulong("u_amp", S_IRUGO, d,
69-
&opp->supplies[i].u_amp))
70-
return false;
62+
debugfs_create_ulong("u_amp", S_IRUGO, d,
63+
&opp->supplies[i].u_amp);
7164
}
72-
73-
return true;
7465
}
7566

76-
int opp_debug_create_one(struct dev_pm_opp *opp, struct opp_table *opp_table)
67+
void opp_debug_create_one(struct dev_pm_opp *opp, struct opp_table *opp_table)
7768
{
7869
struct dentry *pdentry = opp_table->dentry;
7970
struct dentry *d;
@@ -95,40 +86,23 @@ int opp_debug_create_one(struct dev_pm_opp *opp, struct opp_table *opp_table)
9586

9687
/* Create per-opp directory */
9788
d = debugfs_create_dir(name, pdentry);
98-
if (!d)
99-
return -ENOMEM;
100-
101-
if (!debugfs_create_bool("available", S_IRUGO, d, &opp->available))
102-
return -ENOMEM;
103-
104-
if (!debugfs_create_bool("dynamic", S_IRUGO, d, &opp->dynamic))
105-
return -ENOMEM;
106-
107-
if (!debugfs_create_bool("turbo", S_IRUGO, d, &opp->turbo))
108-
return -ENOMEM;
109-
110-
if (!debugfs_create_bool("suspend", S_IRUGO, d, &opp->suspend))
111-
return -ENOMEM;
112-
113-
if (!debugfs_create_u32("performance_state", S_IRUGO, d, &opp->pstate))
114-
return -ENOMEM;
11589

116-
if (!debugfs_create_ulong("rate_hz", S_IRUGO, d, &opp->rate))
117-
return -ENOMEM;
90+
debugfs_create_bool("available", S_IRUGO, d, &opp->available);
91+
debugfs_create_bool("dynamic", S_IRUGO, d, &opp->dynamic);
92+
debugfs_create_bool("turbo", S_IRUGO, d, &opp->turbo);
93+
debugfs_create_bool("suspend", S_IRUGO, d, &opp->suspend);
94+
debugfs_create_u32("performance_state", S_IRUGO, d, &opp->pstate);
95+
debugfs_create_ulong("rate_hz", S_IRUGO, d, &opp->rate);
96+
debugfs_create_ulong("clock_latency_ns", S_IRUGO, d,
97+
&opp->clock_latency_ns);
11898

119-
if (!opp_debug_create_supplies(opp, opp_table, d))
120-
return -ENOMEM;
121-
122-
if (!debugfs_create_ulong("clock_latency_ns", S_IRUGO, d,
123-
&opp->clock_latency_ns))
124-
return -ENOMEM;
99+
opp_debug_create_supplies(opp, opp_table, d);
125100

126101
opp->dentry = d;
127-
return 0;
128102
}
129103

130-
static int opp_list_debug_create_dir(struct opp_device *opp_dev,
131-
struct opp_table *opp_table)
104+
static void opp_list_debug_create_dir(struct opp_device *opp_dev,
105+
struct opp_table *opp_table)
132106
{
133107
const struct device *dev = opp_dev->dev;
134108
struct dentry *d;
@@ -137,36 +111,21 @@ static int opp_list_debug_create_dir(struct opp_device *opp_dev,
137111

138112
/* Create device specific directory */
139113
d = debugfs_create_dir(opp_table->dentry_name, rootdir);
140-
if (!d) {
141-
dev_err(dev, "%s: Failed to create debugfs dir\n", __func__);
142-
return -ENOMEM;
143-
}
144114

145115
opp_dev->dentry = d;
146116
opp_table->dentry = d;
147-
148-
return 0;
149117
}
150118

151-
static int opp_list_debug_create_link(struct opp_device *opp_dev,
152-
struct opp_table *opp_table)
119+
static void opp_list_debug_create_link(struct opp_device *opp_dev,
120+
struct opp_table *opp_table)
153121
{
154-
const struct device *dev = opp_dev->dev;
155122
char name[NAME_MAX];
156-
struct dentry *d;
157123

158124
opp_set_dev_name(opp_dev->dev, name);
159125

160126
/* Create device specific directory link */
161-
d = debugfs_create_symlink(name, rootdir, opp_table->dentry_name);
162-
if (!d) {
163-
dev_err(dev, "%s: Failed to create link\n", __func__);
164-
return -ENOMEM;
165-
}
166-
167-
opp_dev->dentry = d;
168-
169-
return 0;
127+
opp_dev->dentry = debugfs_create_symlink(name, rootdir,
128+
opp_table->dentry_name);
170129
}
171130

172131
/**
@@ -177,20 +136,13 @@ static int opp_list_debug_create_link(struct opp_device *opp_dev,
177136
* Dynamically adds device specific directory in debugfs 'opp' directory. If the
178137
* device-opp is shared with other devices, then links will be created for all
179138
* devices except the first.
180-
*
181-
* Return: 0 on success, otherwise negative error.
182139
*/
183-
int opp_debug_register(struct opp_device *opp_dev, struct opp_table *opp_table)
140+
void opp_debug_register(struct opp_device *opp_dev, struct opp_table *opp_table)
184141
{
185-
if (!rootdir) {
186-
pr_debug("%s: Uninitialized rootdir\n", __func__);
187-
return -EINVAL;
188-
}
189-
190142
if (opp_table->dentry)
191-
return opp_list_debug_create_link(opp_dev, opp_table);
192-
193-
return opp_list_debug_create_dir(opp_dev, opp_table);
143+
opp_list_debug_create_link(opp_dev, opp_table);
144+
else
145+
opp_list_debug_create_dir(opp_dev, opp_table);
194146
}
195147

196148
static void opp_migrate_dentry(struct opp_device *opp_dev,
@@ -252,10 +204,6 @@ static int __init opp_debug_init(void)
252204
{
253205
/* Create /sys/kernel/debug/opp directory */
254206
rootdir = debugfs_create_dir("opp", NULL);
255-
if (!rootdir) {
256-
pr_err("%s: Failed to create root directory\n", __func__);
257-
return -ENOMEM;
258-
}
259207

260208
return 0;
261209
}

‎drivers/opp/opp.h

Lines changed: 7 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -236,18 +236,17 @@ static inline void _of_opp_free_required_opps(struct opp_table *opp_table,
236236

237237
#ifdef CONFIG_DEBUG_FS
238238
void opp_debug_remove_one(struct dev_pm_opp *opp);
239-
int opp_debug_create_one(struct dev_pm_opp *opp, struct opp_table *opp_table);
240-
int opp_debug_register(struct opp_device *opp_dev, struct opp_table *opp_table);
239+
void opp_debug_create_one(struct dev_pm_opp *opp, struct opp_table *opp_table);
240+
void opp_debug_register(struct opp_device *opp_dev, struct opp_table *opp_table);
241241
void opp_debug_unregister(struct opp_device *opp_dev, struct opp_table *opp_table);
242242
#else
243243
static inline void opp_debug_remove_one(struct dev_pm_opp *opp) {}
244244

245-
static inline int opp_debug_create_one(struct dev_pm_opp *opp,
246-
struct opp_table *opp_table)
247-
{ return 0; }
248-
static inline int opp_debug_register(struct opp_device *opp_dev,
249-
struct opp_table *opp_table)
250-
{ return 0; }
245+
static inline void opp_debug_create_one(struct dev_pm_opp *opp,
246+
struct opp_table *opp_table) { }
247+
248+
static inline void opp_debug_register(struct opp_device *opp_dev,
249+
struct opp_table *opp_table) { }
251250

252251
static inline void opp_debug_unregister(struct opp_device *opp_dev,
253252
struct opp_table *opp_table)

0 commit comments

Comments
 (0)