Re: [PATCH] ARM: dts: renesas: koelsch: Update ADV7180 binding

[Geert Uytterhoeven <geert@linux-m68k.org>]

Hi Niklas,

On Fri, 3 Oct 2025 at 23:53, Niklas Söderlund
<niklas.soderlund+renesas@ragnatech.se> wrote:
> Use the more modern adi,adv7180cp compatible for the CVBS input found on
> R-Car Gen2 Koelsch boards. This aligns the bindings with the other Gen2
> board with the same setup Gose.
>
> Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>

Thanks for your patch!

> --- a/arch/arm/boot/dts/renesas/r8a7791-koelsch.dts
> +++ b/arch/arm/boot/dts/renesas/r8a7791-koelsch.dts
> @@ -295,6 +295,16 @@ hdmi_con_in: endpoint {
>                 };
>         };
>
> +       composite-in {
> +               compatible = "composite-video-connector";
> +
> +               port {
> +                       composite_con_in: endpoint {
> +                               remote-endpoint = <&adv7180_in>;
> +                       };
> +               };
> +       };
> +

Moving this below cec-clock while applying, to "preserve" (sic) sort order...

>         cec_clock: cec-clock {
>                 compatible = "fixed-clock";
>                 #clock-cells = <0>;

Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
i.e. will queue in renesas-devel for v6.19.

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