Hi Tommaso, On Wed, 26 Feb 2025 at 16:25, Tommaso Merciai <tommaso.merciai.xr@xxxxxxxxxxxxxx> wrote: > From: Lad Prabhakar <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx> > > The MIPI CSI-2 block on the Renesas RZ/V2H(P) SoC is similar to the one > found on the Renesas RZ/G2L SoC, with the following differences: > - A different D-PHY > - Additional registers for the MIPI CSI-2 link > - Only two clocks > > Add a new compatible string, `renesas,r9a09g057-csi2`, for the RZ/V2H(P) > SoC. > > Reviewed-by: Rob Herring (Arm) <robh@xxxxxxxxxx> > Reviewed-by: Laurent Pinchart <laurent.pinchart+renesas@xxxxxxxxxxxxxxxx> > Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx> > Signed-off-by: Tommaso Merciai <tommaso.merciai.xr@xxxxxxxxxxxxxx> Thanks for your patch! > --- a/Documentation/devicetree/bindings/media/renesas,rzg2l-csi2.yaml > +++ b/Documentation/devicetree/bindings/media/renesas,rzg2l-csi2.yaml > @@ -48,7 +58,8 @@ properties: > resets: > items: > - description: CRU_PRESETN reset terminal > - - description: CRU_CMN_RSTB reset terminal > + - description: > + CRU_CMN_RSTB reset terminal (all but RZ/V2H(P)) or D-PHY reset (RZ/V2H(P)) According to the flowcharts[1] in the RZ/G2L, RZ/V2L, and RZ/G2UL documentation, CPG_RST_CRU.CRU_CMN_RSTB is the D-PHY reset on all of them, so I think you can just write "D-PHY reset". Or "D-PHY reset (CRU_CMN_RSTB or CRU_n_S_RESETN)"? [1] Section 35. Camera Data Receiving Unit (CRU) 35.3.1 Starting Reception for the MIPI CSI-2 Input 35.3.2.2Stopping the D-PHY (MIPI CSI-2 Input) 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