On Sat, Feb 15, 2025 at 03:48:43PM +0100, Uwe Kleine-König wrote: > Hello, > > On Mon, Jan 27, 2025 at 11:50:01AM +0100, Uwe Kleine-König wrote: > > 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 > > --- > > 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. > > Applied now without further feedback to > https://git.kernel.org/pub/scm/linux/kernel/git/ukleinek/linux.git pwm/for-next Bit late, but ye I think that's fair.