2020-02-11 21:32:46

by Dietmar Eggemann

[permalink] [raw]
Subject: [PATCH 0/2] drivers base/arch_topology: Remove 'struct sched_domain' forward declaration & reformat function names

This patch-set contains the requested split of
https://lore.kernel.org/r/[email protected]

Dietmar Eggemann (2):
drivers base/arch_topology: Remove 'struct sched_domain' forward
declaration
drivers base/arch_topology: Reformat topology_get_[cpu/freq]_scale()
function name

include/linux/arch_topology.h | 7 ++-----
1 file changed, 2 insertions(+), 5 deletions(-)

--
2.17.1


2020-02-11 21:32:46

by Dietmar Eggemann

[permalink] [raw]
Subject: [PATCH 1/2] drivers base/arch_topology: Remove 'struct sched_domain' forward declaration

The sched domain pointer argument from topology_get_freq_scale() and
topology_get_cpu_scale() got removed by commit 7673c8a4c75d
("sched/cpufreq: Remove arch_scale_freq_capacity()'s 'sd' parameter")
and commit 8ec59c0f5f49 ("sched/topology: Remove unused 'sd' parameter
from arch_scale_cpu_capacity()").

So the 'struct sched_domain' forward declaration is no longer needed.
Remove it.

Signed-off-by: Dietmar Eggemann <[email protected]>
Reviewed-by: Sudeep Holla <[email protected]>
---
include/linux/arch_topology.h | 1 -
1 file changed, 1 deletion(-)

diff --git a/include/linux/arch_topology.h b/include/linux/arch_topology.h
index 3015ecbb90b1..f4b1d4fd7efb 100644
--- a/include/linux/arch_topology.h
+++ b/include/linux/arch_topology.h
@@ -16,7 +16,6 @@ bool topology_parse_cpu_capacity(struct device_node *cpu_node, int cpu);

DECLARE_PER_CPU(unsigned long, cpu_scale);

-struct sched_domain;
static inline
unsigned long topology_get_cpu_scale(int cpu)
{
--
2.17.1

2020-02-11 21:32:55

by Dietmar Eggemann

[permalink] [raw]
Subject: [PATCH 2/2] drivers base/arch_topology: Reformat topology_get_[cpu/freq]_scale() function name

The storage class and inline definition as well as the return type,
function name and parameter list fit all into one line.

Signed-off-by: Dietmar Eggemann <[email protected]>
Reviewed-by: Sudeep Holla <[email protected]>
---
include/linux/arch_topology.h | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/include/linux/arch_topology.h b/include/linux/arch_topology.h
index f4b1d4fd7efb..c507e9ddd909 100644
--- a/include/linux/arch_topology.h
+++ b/include/linux/arch_topology.h
@@ -16,8 +16,7 @@ bool topology_parse_cpu_capacity(struct device_node *cpu_node, int cpu);

DECLARE_PER_CPU(unsigned long, cpu_scale);

-static inline
-unsigned long topology_get_cpu_scale(int cpu)
+static inline unsigned long topology_get_cpu_scale(int cpu)
{
return per_cpu(cpu_scale, cpu);
}
@@ -26,8 +25,7 @@ void topology_set_cpu_scale(unsigned int cpu, unsigned long capacity);

DECLARE_PER_CPU(unsigned long, freq_scale);

-static inline
-unsigned long topology_get_freq_scale(int cpu)
+static inline unsigned long topology_get_freq_scale(int cpu)
{
return per_cpu(freq_scale, cpu);
}
--
2.17.1