Message ID | 20240826144352.3026980-3-niklas.soderlund+renesas@ragnatech.se (mailing list archive) |
---|---|
State | New |
Headers | show |
Series | rcar-isp: Add support for R-Car V4M | expand |
Hi Niklas, Thank you for the patch. On Mon, Aug 26, 2024 at 04:43:48PM +0200, Niklas Söderlund wrote: > To make it easier to support new Gen4 SoCs add a family fallback > compatible similar to what was done for VIN on Gen4. > > There is no functional change, but the addition of the family fallback > in the bindings produces warnings for V4H for DTS checks if they are not > added. > > Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se> Assuming 1/6 gets accepted, Reviewed-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> > --- > * Changes since v1 > - New in v2. > --- > arch/arm64/boot/dts/renesas/r8a779g0.dtsi | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/arch/arm64/boot/dts/renesas/r8a779g0.dtsi b/arch/arm64/boot/dts/renesas/r8a779g0.dtsi > index ac3d4062c2fa..ded805f0e633 100644 > --- a/arch/arm64/boot/dts/renesas/r8a779g0.dtsi > +++ b/arch/arm64/boot/dts/renesas/r8a779g0.dtsi > @@ -2075,7 +2075,8 @@ du_out_dsi1: endpoint { > }; > > isp0: isp@fed00000 { > - compatible = "renesas,r8a779g0-isp"; > + compatible = "renesas,r8a779g0-isp", > + "renesas,rcar-gen4-isp"; > reg = <0 0xfed00000 0 0x10000>; > interrupts = <GIC_SPI 473 IRQ_TYPE_LEVEL_LOW>; > clocks = <&cpg CPG_MOD 612>; > @@ -2158,7 +2159,8 @@ isp0vin07: endpoint { > }; > > isp1: isp@fed20000 { > - compatible = "renesas,r8a779g0-isp"; > + compatible = "renesas,r8a779g0-isp", > + "renesas,rcar-gen4-isp"; > reg = <0 0xfed20000 0 0x10000>; > interrupts = <GIC_SPI 474 IRQ_TYPE_LEVEL_LOW>; > clocks = <&cpg CPG_MOD 613>;
On Mon, Aug 26, 2024 at 4:44 PM Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se> wrote: > To make it easier to support new Gen4 SoCs add a family fallback > compatible similar to what was done for VIN on Gen4. > > There is no functional change, but the addition of the family fallback > in the bindings produces warnings for V4H for DTS checks if they are not > added. > > Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se> > --- > * Changes since v1 > - New in v2. Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> Gr{oetje,eeting}s, Geert
diff --git a/arch/arm64/boot/dts/renesas/r8a779g0.dtsi b/arch/arm64/boot/dts/renesas/r8a779g0.dtsi index ac3d4062c2fa..ded805f0e633 100644 --- a/arch/arm64/boot/dts/renesas/r8a779g0.dtsi +++ b/arch/arm64/boot/dts/renesas/r8a779g0.dtsi @@ -2075,7 +2075,8 @@ du_out_dsi1: endpoint { }; isp0: isp@fed00000 { - compatible = "renesas,r8a779g0-isp"; + compatible = "renesas,r8a779g0-isp", + "renesas,rcar-gen4-isp"; reg = <0 0xfed00000 0 0x10000>; interrupts = <GIC_SPI 473 IRQ_TYPE_LEVEL_LOW>; clocks = <&cpg CPG_MOD 612>; @@ -2158,7 +2159,8 @@ isp0vin07: endpoint { }; isp1: isp@fed20000 { - compatible = "renesas,r8a779g0-isp"; + compatible = "renesas,r8a779g0-isp", + "renesas,rcar-gen4-isp"; reg = <0 0xfed20000 0 0x10000>; interrupts = <GIC_SPI 474 IRQ_TYPE_LEVEL_LOW>; clocks = <&cpg CPG_MOD 613>;
To make it easier to support new Gen4 SoCs add a family fallback compatible similar to what was done for VIN on Gen4. There is no functional change, but the addition of the family fallback in the bindings produces warnings for V4H for DTS checks if they are not added. Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se> --- * Changes since v1 - New in v2. --- arch/arm64/boot/dts/renesas/r8a779g0.dtsi | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-)