Re: [PATCH] arm64: dts: renesas: Add pixpaper display overlay for RZ/V2H Kakip board

[Geert Uytterhoeven]

Hi Wig,

On Sat, 24 Jan 2026 at 17:36, Wig Cheng <onlywig@xxxxxxxxx> wrote:
> Add device tree overlay to support the MayQueen PixPaper e-paper display
> on the Renesas RZ/V2H EVK (KAKIP board). The display is connected via
> SPI0 interface and uses GPIO pins for reset, busy, and DC control.
>
> The overlay configures:
>     - RSPI0 pinmux for SPI communication (MOSI, MISO, CLK, CE0)
>     - PixPaper display device with proper GPIO assignments
>     - SPI frequency set to 1MHz for stable operation
>
> This enables support for the Open-EP Community pixpaper-213-c module on
> the RZ/V2H platform.
>
> Signed-off-by: Wig Cheng <onlywig@xxxxxxxxx>

Thanks for your patch!

> --- /dev/null
> +++ b/arch/arm64/boot/dts/renesas/r9a09g057h48-kakip-pixpaper.dtso
> @@ -0,0 +1,40 @@
> +// SPDX-License-Identifier: GPL-2.0
> +/*
> + * Device Tree Overlay for Mayqueen (Open-EP Community) pixpaper display
> + * support on Renesas RZ/V2H platform (KAKIP board).
> + *
> + * Copyright (C) 2026 Wig Cheng <onlywig@xxxxxxxxx>
> + */
> +
> +/dts-v1/;
> +/plugin/;
> +
> +#include <dt-bindings/pinctrl/renesas,r9a09g057-pinctrl.h>
> +#include <dt-bindings/gpio/gpio.h>
> +#include <dt-bindings/interrupt-controller/irq.h>

This include is not needed, so I will drop it while applying.

> +
> +&pinctrl {
> +       rspi0_pins: rspi0 {
> +               pinmux = <RZV2H_PORT_PINMUX(9, 0, 1)>, /* SPI0 MOSI */
> +                        <RZV2H_PORT_PINMUX(9, 1, 1)>, /* SPI0 MISO */
> +                        <RZV2H_PORT_PINMUX(9, 2, 1)>, /* SPI0 CLK */
> +                        <RZV2H_PORT_PINMUX(9, 3, 1)>; /* SPI0 CE0 */
> +       };
> +};
> +
> +&rspi0 {
> +       pinctrl-0 = <&rspi0_pins>;
> +       pinctrl-names = "default";
> +       #address-cells = <1>;
> +       #size-cells = <0>;
> +       status = "okay";
> +
> +       display0: display@0 {

The label is not needed, so I will drop it while applying.

> +               compatible = "mayqueen,pixpaper";
> +               reg = <0>;
> +               spi-max-frequency = <1000000>;
> +               reset-gpios = <&pinctrl RZV2H_GPIO(A, 7) GPIO_ACTIVE_HIGH>;
> +               busy-gpios = <&pinctrl RZV2H_GPIO(B, 3) GPIO_ACTIVE_HIGH>;
> +               dc-gpios = <&pinctrl RZV2H_GPIO(7, 4) GPIO_ACTIVE_HIGH>;
> +       };
> +};

The rest LGTM, so
Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>
i.e. will queue in renesas-devel for v7.1 with the above fixed.
No need to resend.

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