cpufreq: arm_scmi: Constify scmi_perf_ops pointers
authorRikard Falkeborn <rikard.falkeborn@gmail.com>
Sun, 6 Sep 2020 23:04:50 +0000 (01:04 +0200)
committerSudeep Holla <sudeep.holla@arm.com>
Mon, 7 Sep 2020 11:43:27 +0000 (12:43 +0100)
The perf_ops are not modified through this pointer. Make them const to
indicate that. This is in preparation to make the scmi-ops pointers in
scmi_handle const.

Link: https://lore.kernel.org/r/20200906230452.33410-2-rikard.falkeborn@gmail.com
Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Rikard Falkeborn <rikard.falkeborn@gmail.com>
Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>
drivers/cpufreq/scmi-cpufreq.c

index fb42e33..46b2680 100644 (file)
@@ -29,7 +29,7 @@ static const struct scmi_handle *handle;
 static unsigned int scmi_cpufreq_get_rate(unsigned int cpu)
 {
        struct cpufreq_policy *policy = cpufreq_cpu_get_raw(cpu);
-       struct scmi_perf_ops *perf_ops = handle->perf_ops;
+       const struct scmi_perf_ops *perf_ops = handle->perf_ops;
        struct scmi_data *priv = policy->driver_data;
        unsigned long rate;
        int ret;
@@ -50,7 +50,7 @@ scmi_cpufreq_set_target(struct cpufreq_policy *policy, unsigned int index)
 {
        int ret;
        struct scmi_data *priv = policy->driver_data;
-       struct scmi_perf_ops *perf_ops = handle->perf_ops;
+       const struct scmi_perf_ops *perf_ops = handle->perf_ops;
        u64 freq = policy->freq_table[index].frequency;
 
        ret = perf_ops->freq_set(handle, priv->domain_id, freq * 1000, false);
@@ -64,7 +64,7 @@ static unsigned int scmi_cpufreq_fast_switch(struct cpufreq_policy *policy,
                                             unsigned int target_freq)
 {
        struct scmi_data *priv = policy->driver_data;
-       struct scmi_perf_ops *perf_ops = handle->perf_ops;
+       const struct scmi_perf_ops *perf_ops = handle->perf_ops;
 
        if (!perf_ops->freq_set(handle, priv->domain_id,
                                target_freq * 1000, true)) {