[PATCH] pwm: Strengthen dependency for PWM_SIFIVE
Uwe Kleine-König
u.kleine-koenig at baylibre.com
Mon Jan 27 02:50:01 PST 2025
Back when the sifive pwm driver was added there was no symbol for sifive
SoCs yet. Today there is ARCH_SIFIVE however. Let PWM_SIFIVE depend on
that to ensure the driver is only build for platforms where there is a
chance that the hardware is available.
Signed-off-by: Uwe Kleine-König <u.kleine-koenig at baylibre.com>
---
Hello,
a quick grep suggests that the driver is only used on machines that include
arch/riscv/boot/dts/sifive/fu540-c000.dtsi or
arch/riscv/boot/dts/sifive/fu740-c000.dtsi and for these ARCH_SIFIVE is
enabled. So I'd guess this patch is fine. Still an ack from someone who
knows this arch better would be very welcome.
Best regards
Uwe
drivers/pwm/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/pwm/Kconfig b/drivers/pwm/Kconfig
index 0915c1e7df16..be0db73003e4 100644
--- a/drivers/pwm/Kconfig
+++ b/drivers/pwm/Kconfig
@@ -567,7 +567,7 @@ config PWM_SIFIVE
tristate "SiFive PWM support"
depends on OF
depends on COMMON_CLK && HAS_IOMEM
- depends on RISCV || COMPILE_TEST
+ depends on ARCH_SIFIVE || COMPILE_TEST
help
Generic PWM framework driver for SiFive SoCs.
--
2.47.1
More information about the linux-riscv
mailing list