Re: [PATCH] arm64: tegra186: Add missing CPU PMUs

From: Marc Zyngier
Date: Tue Nov 10 2020 - 13:08:38 EST


On 2020-11-10 17:36, Thierry Reding wrote:
On Tue, Oct 13, 2020 at 10:58:51AM +0100, Marc Zyngier wrote:
Add the description of CPU PMUs for both the Denver and A57 clusters,
which enables the perf subsystem.

Signed-off-by: Marc Zyngier <maz@xxxxxxxxxx>

[...]


+ pmu_denver {
+ compatible = "nvidia,denver-pmu", "arm,armv8-pmuv3";

checkpatch complains that this isn't documented. Did I miss the DT
bindings patch or do we not have one for this?

We don't. But I don't think adding a compatible string for each
and every micro-architecture makes much sense unless we have something
useful to add to that compatible string. Such as a full description
of the implementation specific events.

Thanks,

M.