Re: [PATCH v3 2/2] arm64: dts: renesas: Add LCDC overlays for RZ/T2H and RZ/N2H EVKs

On Wed, 19 Aug 2026 at 20:39, Prabhakar <[email protected]> wrote:
> From: Lad Prabhakar <[email protected]>
>
> Add DT overlay support for enabling the DU/LCDC pipeline on the RZ/T2H
> and RZ/N2H evaluation kits when fitted with an ADV7513 HDMI transmitter
> on CN15/CN20.
>
> Share the common ADV7513 and LCDC pin configuration between the two
> overlays. Disable board functions whose pins are reassigned to LCDC,
> including the affected LEDs, SDHI0 on the RZ/T2H EVK, and I2C0 on the
> RZ/N2H EVK.
>
> Move the LED8 and LED9 preprocessor conditionals inside the node
> definitions so that the nodes remain present in the base DTS. This
> allows the LCDC overlays to reference and disable the LEDs when their
> pins are reassigned to display functions.
>
> Configure the LCDC data and clock pins with slew-rate setting 1 as
> recommended by the hardware manual. Use slew-rate setting 0 for the
> HSYNC, VSYNC and DE pins, as testing with the ADV7513 showed unstable
> display output and visible flicker when these synchronization signals
> were configured for fast slew rate.
>
> Signed-off-by: Lad Prabhakar <[email protected]>
> ---
> v2->v3:
> - Renamed HEADER_IRQ8 to IRQ8
> - Added irq support for the ADV7513 in the overlays
> - Corrected the DIP switch settings for I2C_SDA1/I2C_SCL1
> - Added missing DIP switch settings for LCDC_DATG3
> - MOved the header include at the end
> - Added a comment for i2c0 as to why it is disabled in the overlay
> - Split up the du_pinctrl into a group of pins as only sync/de pins
>   needed to set slew-rate=0, the rest of the pins can use slew-rate=1
> - Updated commit message

Reviewed-by: Geert Uytterhoeven <[email protected]>
i.e. will queue in renesas-devel for v7.4.

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]

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