Re: [PATCH v2] arm64: dts: renesas: sparrow-hawk: Add overlay for WaveShare 2CH CANFD HAT

Hi Marek,

On Wed, 9 Sept 2026 at 17:14, Marek Vasut
[off-list ref] 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@mailbox.org>

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!

+&{/} {
+       /* 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@glider.be>
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@linux-m68k.org

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