Re: [PATCH v2] arm64: dts: renesas: sparrow-hawk: Add overlay for WaveShare 2CH CANFD HAT
From: Geert Uytterhoeven
Date: Wed Sep 09 2026 - 12:49:31 EST
Hi Marek,
On Wed, 9 Sept 2026 at 17:14, Marek Vasut
<marek.vasut+renesas@xxxxxxxxxxx> wrote:
> Add a DT overlay to bind the WaveShare 2-Channel Isolated CAN FD
> Expansion HAT on the Retronix R-Car V4H Sparrow Hawk board.
>
> Current device link is at https://www.waveshare.com/2-ch-can-fd-hat.htm
>
> Signed-off-by: Marek Vasut <marek.vasut+renesas@xxxxxxxxxxx>
> V2: - Drop the unique spi controller node identifier
> - Switch IRQ to active low
> - Sort compatible and reg properties
> - Add "CN7 pin 22" comment
> - Move pwm1 node to the top and add comment
> - Rename MSIOF0 PFC node
Thanks for the update!
> --- /dev/null
> +++ b/arch/arm64/boot/dts/renesas/r8a779g3-sparrow-hawk-ws-2ch-canfd.dtso
> +&{/} {
> + /* Fixed clock dedicated to SPI CAN on 2-CH CAN FD HAT */
> + clk_can0: clk-ws-2ch-canfd0 {
> + compatible = "fixed-clock";
> + #clock-cells = <0>;
> + clock-frequency = <40000000>;
> + };
> +
> + clk_can1: clk-ws-2ch-canfd1 {
> + compatible = "fixed-clock";
> + #clock-cells = <0>;
> + clock-frequency = <40000000>;
> + };
> +
> + /*
> + * SPI_1 is operated as GPIO SPI, because MSIOF5 can not be
> + * muxed on the pins allocated for SPI_1 on this CANFD HAT.
> + */
> + spi { /* SPI_1 */
As this lives at the root node, I'd rather change it back to
spi-ws-2ch-canfd while applying, if you don't mind?
Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>
i.e. will queue in renesas-devel for v7.4, with the above fixed.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds