perf: starfive: fix 64-bit only COMPILE_TEST condition
authorConor Dooley <conor.dooley@microchip.com>
Mon, 18 Mar 2024 15:35:04 +0000 (15:35 +0000)
committerCatalin Marinas <catalin.marinas@arm.com>
Tue, 19 Mar 2024 19:01:39 +0000 (19:01 +0000)
commit1d63d1d9e5c5cb2e7c7ca75751a5eaf67c5623a7
tree02a0833815cbfd24dee5991fc14cfae38cac5b77
parent3fbd56f0e7c14e7c7a7597fd4a368753fe70d76f
perf: starfive: fix 64-bit only COMPILE_TEST condition

ARCH_STARFIVE is not restricted to 64-bit platforms, so while Will's
addition of a 64-bit only condition satisfied the build robots doing
COMPILE_TEST builds, Palmer ran into the same problems with writeq()
being undefined during regular rv32 builds.

Promote the dependency on 64-bit to its own `depends on` so that the
driver can never be included in 32-bit builds.

Reported-by: Palmer Dabbelt <palmer@rivosinc.com>
Fixes: c2b24812f7bc ("perf: starfive: Add StarLink PMU support")
Fixes: f0dbc6d0de38 ("perf: starfive: Only allow COMPILE_TEST for 64-bit architectures")
Signed-off-by: Conor Dooley <conor.dooley@microchip.com>
Acked-by: Will Deacon <will@kernel.org>
Reviewed-by: Palmer Dabbelt <palmer@rivosinc.com>
Acked-by: Palmer Dabbelt <palmer@rivosinc.com>
Acked-by: Ji Sheng Teoh <jisheng.teoh@starfivetech.com>
Acked-by: Emil Renner Berthing <emil.renner.berthing@canonical.com>
Link: https://lore.kernel.org/r/20240318-emphatic-rally-f177a4fe1bdc@spud
Signed-off-by: Catalin Marinas <catalin.marinas@arm.com>
drivers/perf/Kconfig