On 14/10/2021 16.03, Hector Martin wrote:
On 14/10/2021 15.56, Viresh Kumar wrote:
+ /*
+ * Attach the CPU device to its genpd domain (if any), to allow OPP
+ * dependencies to be satisfied.
+ */
+ ret = genpd_dev_pm_attach(cpu_dev);
+ if (ret <= 0) {
+ dev_err(cpu_dev, "Failed to attach CPU device to genpd\n");
+ goto out;
+ }
+
Other platform do this from some other place I think.
Ulf, where should this code be moved ? cpu-clk driver ?
I see one driver that does this is drivers/clk/qcom/apcs-sdx55.c (via
dev_pm_domain_attach). Though it only does it for CPU#0; we need to do
it for all CPUs.
Looking into this further, I'm not sure I like the idea of doing this in
the clocks driver. There might be locking issues since it gets
instantiated twice and yet doesn't really itself know what subset of
CPUs it applies to.
There's another driver that does this:
drivers/cpuidle/cpuidle-psci-domain.c. That one specifically looks for a
power domain called "psci". Perhaps it would make sense to make this
generic in cpufreq-dt as per my prior patch, but explicitly request a
"cpufreq" domain? That way only devicetrees that opt in to having this
handled by cpufreq by naming it that way would get this behavior.
--
Hector Martin (marcan@xxxxxxxxx)
Public Key: https://mrcn.st/pub
