20 Jan
2020
20 Jan
'20
3:11 p.m.
On 20/01/2020 15:53, Dietmar Eggemann wrote:
On 16/01/2020 16:20, lukasz.luba@arm.com wrote:
From: Lukasz Luba lukasz.luba@arm.com
[...]
+enum em_type {
- EM_SIMPLE,
- EM_CPU,
+};
s/EM_SIMPLE/EM_DEV ?
Right now I only see energy models and _one_ specific type (the CPU EM). So a tag 'is a CPU EM' would suffice. No need for EM_SIMPE ...
Wait, you even have
struct em_device { struct em_perf_domain *em_pd; struct device *dev; ... }
static bool _is_cpu_device(struct device *dev)
Shouldn't this be enough to distinguish between EM and special CPU EM under the API? Even when required to use container_of() to get from em_perf_domain to device.