arm64: topology: Avoid the have_policy check
authorViresh Kumar <viresh.kumar@linaro.org>
Fri, 8 Jan 2021 11:16:51 +0000 (16:46 +0530)
committerWill Deacon <will@kernel.org>
Wed, 20 Jan 2021 12:49:40 +0000 (12:49 +0000)
commit384e5699e101b1ee184590a39ee60f10ec0d36b6
tree3661320231187739860e0e459cd6b50e7f27e50b
parent19c329f6808995b142b3966301f217c831e7cf31
arm64: topology: Avoid the have_policy check

Every time I have stumbled upon this routine, I get confused with the
way 'have_policy' is used and I have to dig in to understand why is it
so. Here is an attempt to make it easier to understand, and hopefully it
is an improvement.

The 'have_policy' check was just an optimization to avoid writing
to amu_fie_cpus in case we don't have to, but that optimization itself
is creating more confusion than the real work. Lets just do that if all
the CPUs support AMUs. It is much cleaner that way.

Reviewed-by: Ionela Voinescu <ionela.voinescu@arm.com>
Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
Tested-by: Ionela Voinescu <ionela.voinescu@arm.com>
Link: https://lore.kernel.org/r/c125766c4be93461772015ac7c9a6ae45d5756f6.1610104461.git.viresh.kumar@linaro.org
Signed-off-by: Will Deacon <will@kernel.org>
arch/arm64/kernel/topology.c