Hi Jacopo, Thanks for your work. On 2020-10-14 11:44:43 +0200, Jacopo Mondi wrote: > Add VIN nodes to R8A779A0 R-Car V3U SoC. > > Signed-off-by: Jacopo Mondi <jacopo+renesas@xxxxxxxxxx> > --- > arch/arm64/boot/dts/renesas/r8a779a0.dtsi | 320 ++++++++++++++++++++++ > 1 file changed, 320 insertions(+) > > diff --git a/arch/arm64/boot/dts/renesas/r8a779a0.dtsi b/arch/arm64/boot/dts/renesas/r8a779a0.dtsi > index 83962ad30a1d..bc81e6a761d3 100644 > --- a/arch/arm64/boot/dts/renesas/r8a779a0.dtsi > +++ b/arch/arm64/boot/dts/renesas/r8a779a0.dtsi > @@ -105,6 +105,326 @@ scif0: serial@e6e60000 { > status = "disabled"; > }; > > + vin0: video@e6ef0000 { > + compatible = "renesas,vin-r8a779a0"; How is this tested? There is no driver support and the schema you added the compat string to mandates the 'renesas,id' property for the vin nodes. As far as I can tell the reason for this id is gone with the new pipeline of the V3U. So I guess as long as we don't break dtschema validation all is good, just wanted to make sure ;-) > + reg = <0 0xe6ef0000 0 0x1000>; > + interrupts = <GIC_SPI 160 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cpg CPG_MOD 730>; > + power-domains = <&sysc R8A779A0_PD_ALWAYS_ON>; > + resets = <&cpg 730>; > + status = "disabled"; > + }; > + > + vin1: video@e6ef1000 { > + compatible = "renesas,vin-r8a779a0"; > + reg = <0 0xe6ef1000 0 0x1000>; > + interrupts = <GIC_SPI 161 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cpg CPG_MOD 731>; > + power-domains = <&sysc R8A779A0_PD_ALWAYS_ON>; > + resets = <&cpg 731>; > + status = "disabled"; > + }; > + > + vin2: video@e6ef2000 { > + compatible = "renesas,vin-r8a779a0"; > + reg = <0 0xe6ef2000 0 0x1000>; > + interrupts = <GIC_SPI 162 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cpg CPG_MOD 800>; > + power-domains = <&sysc R8A779A0_PD_ALWAYS_ON>; > + resets = <&cpg 800>; > + status = "disabled"; > + }; > + > + vin3: video@e6ef3000 { > + compatible = "renesas,vin-r8a779a0"; > + reg = <0 0xe6ef3000 0 0x1000>; > + interrupts = <GIC_SPI 163 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cpg CPG_MOD 801>; > + power-domains = <&sysc R8A779A0_PD_ALWAYS_ON>; > + resets = <&cpg 801>; > + status = "disabled"; > + }; > + > + vin4: video@e6ef4000 { > + compatible = "renesas,vin-r8a779a0"; > + reg = <0 0xe6ef4000 0 0x1000>; > + interrupts = <GIC_SPI 164 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cpg CPG_MOD 802>; > + power-domains = <&sysc R8A779A0_PD_ALWAYS_ON>; > + resets = <&cpg 802>; > + status = "disabled"; > + }; > + > + vin5: video@e6ef5000 { > + compatible = "renesas,vin-r8a779a0"; > + reg = <0 0xe6ef5000 0 0x1000>; > + interrupts = <GIC_SPI 165 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cpg CPG_MOD 803>; > + power-domains = <&sysc R8A779A0_PD_ALWAYS_ON>; > + resets = <&cpg 803>; > + status = "disabled"; > + }; > + > + vin6: video@e6ef6000 { > + compatible = "renesas,vin-r8a779a0"; > + reg = <0 0xe6ef6000 0 0x1000>; > + interrupts = <GIC_SPI 166 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cpg CPG_MOD 804>; > + power-domains = <&sysc R8A779A0_PD_ALWAYS_ON>; > + resets = <&cpg 804>; > + status = "disabled"; > + }; > + > + vin7: video@e6ef7000 { > + compatible = "renesas,vin-r8a779a0"; > + reg = <0 0xe6ef7000 0 0x1000>; > + interrupts = <GIC_SPI 167 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cpg CPG_MOD 805>; > + power-domains = <&sysc R8A779A0_PD_ALWAYS_ON>; > + resets = <&cpg 805>; > + status = "disabled"; > + }; > + > + vin8: video@e6ef8000 { > + compatible = "renesas,vin-r8a779a0"; > + reg = <0 0xe6ef8000 0 0x1000>; > + interrupts = <GIC_SPI 168 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cpg CPG_MOD 806>; > + power-domains = <&sysc R8A779A0_PD_ALWAYS_ON>; > + resets = <&cpg 806>; > + status = "disabled"; > + }; > + > + vin9: video@e6ef9000 { > + compatible = "renesas,vin-r8a779a0"; > + reg = <0 0xe6ef9000 0 0x1000>; > + interrupts = <GIC_SPI 169 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cpg CPG_MOD 807>; > + power-domains = <&sysc R8A779A0_PD_ALWAYS_ON>; > + resets = <&cpg 807>; > + status = "disabled"; > + }; > + > + vin10: video@e6efa000 { > + compatible = "renesas,vin-r8a779a0"; > + reg = <0 0xe6efa000 0 0x1000>; > + interrupts = <GIC_SPI 170 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cpg CPG_MOD 808>; > + power-domains = <&sysc R8A779A0_PD_ALWAYS_ON>; > + resets = <&cpg 808>; > + status = "disabled"; > + }; > + > + vin11: video@e6efb000 { > + compatible = "renesas,vin-r8a779a0"; > + reg = <0 0xe6efb000 0 0x1000>; > + interrupts = <GIC_SPI 171 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cpg CPG_MOD 809>; > + power-domains = <&sysc R8A779A0_PD_ALWAYS_ON>; > + resets = <&cpg 809>; > + status = "disabled"; > + }; > + > + vin12: video@e6efc000 { > + compatible = "renesas,vin-r8a779a0"; > + reg = <0 0xe6efc000 0 0x1000>; > + interrupts = <GIC_SPI 172 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cpg CPG_MOD 810>; > + power-domains = <&sysc R8A779A0_PD_ALWAYS_ON>; > + resets = <&cpg 810>; > + status = "disabled"; > + }; > + > + vin13: video@e6efd000 { > + compatible = "renesas,vin-r8a779a0"; > + reg = <0 0xe6efd000 0 0x1000>; > + interrupts = <GIC_SPI 173 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cpg CPG_MOD 811>; > + power-domains = <&sysc R8A779A0_PD_ALWAYS_ON>; > + resets = <&cpg 811>; > + status = "disabled"; > + }; > + > + vin14: video@e6efe000 { > + compatible = "renesas,vin-r8a779a0"; > + reg = <0 0xe6efe000 0 0x1000>; > + interrupts = <GIC_SPI 174 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cpg CPG_MOD 812>; > + power-domains = <&sysc R8A779A0_PD_ALWAYS_ON>; > + resets = <&cpg 812>; > + status = "disabled"; > + }; > + > + vin15: video@e6eff000 { > + compatible = "renesas,vin-r8a779a0"; > + reg = <0 0xe6eff000 0 0x1000>; > + interrupts = <GIC_SPI 175 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cpg CPG_MOD 813>; > + power-domains = <&sysc R8A779A0_PD_ALWAYS_ON>; > + resets = <&cpg 813>; > + status = "disabled"; > + }; > + > + vin16: video@e6ed0000 { > + compatible = "renesas,vin-r8a779a0"; > + reg = <0 0xe6ed0000 0 0x1000>; > + interrupts = <GIC_SPI 176 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cpg CPG_MOD 814>; > + power-domains = <&sysc R8A779A0_PD_ALWAYS_ON>; > + resets = <&cpg 814>; > + status = "disabled"; > + }; > + > + vin17: video@e6ed1000 { > + compatible = "renesas,vin-r8a779a0"; > + reg = <0 0xe6ed1000 0 0x1000>; > + interrupts = <GIC_SPI 177 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cpg CPG_MOD 815>; > + power-domains = <&sysc R8A779A0_PD_ALWAYS_ON>; > + resets = <&cpg 815>; > + status = "disabled"; > + }; > + > + vin18: video@e6ed2000 { > + compatible = "renesas,vin-r8a779a0"; > + reg = <0 0xe6ed2000 0 0x1000>; > + interrupts = <GIC_SPI 178 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cpg CPG_MOD 816>; > + power-domains = <&sysc R8A779A0_PD_ALWAYS_ON>; > + resets = <&cpg 816>; > + status = "disabled"; > + }; > + > + vin19: video@e6ed3000 { > + compatible = "renesas,vin-r8a779a0"; > + reg = <0 0xe6ed3000 0 0x1000>; > + interrupts = <GIC_SPI 179 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cpg CPG_MOD 817>; > + power-domains = <&sysc R8A779A0_PD_ALWAYS_ON>; > + resets = <&cpg 817>; > + status = "disabled"; > + }; > + > + vin20: video@e6ed4000 { > + compatible = "renesas,vin-r8a779a0"; > + reg = <0 0xe6ed4000 0 0x1000>; > + interrupts = <GIC_SPI 180 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cpg CPG_MOD 818>; > + power-domains = <&sysc R8A779A0_PD_ALWAYS_ON>; > + resets = <&cpg 818>; > + status = "disabled"; > + }; > + > + vin21: video@e6ed5000 { > + compatible = "renesas,vin-r8a779a0"; > + reg = <0 0xe6ed5000 0 0x1000>; > + interrupts = <GIC_SPI 181 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cpg CPG_MOD 819>; > + power-domains = <&sysc R8A779A0_PD_ALWAYS_ON>; > + resets = <&cpg 819>; > + status = "disabled"; > + }; > + > + vin22: video@e6ed6000 { > + compatible = "renesas,vin-r8a779a0"; > + reg = <0 0xe6ed6000 0 0x1000>; > + interrupts = <GIC_SPI 182 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cpg CPG_MOD 820>; > + power-domains = <&sysc R8A779A0_PD_ALWAYS_ON>; > + resets = <&cpg 820>; > + status = "disabled"; > + }; > + > + vin23: video@e6ed7000 { > + compatible = "renesas,vin-r8a779a0"; > + reg = <0 0xe6ed7000 0 0x1000>; > + interrupts = <GIC_SPI 183 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cpg CPG_MOD 821>; > + power-domains = <&sysc R8A779A0_PD_ALWAYS_ON>; > + resets = <&cpg 821>; > + status = "disabled"; > + }; > + > + vin24: video@e6ed8000 { > + compatible = "renesas,vin-r8a779a0"; > + reg = <0 0xe6ed8000 0 0x1000>; > + interrupts = <GIC_SPI 184 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cpg CPG_MOD 822>; > + power-domains = <&sysc R8A779A0_PD_ALWAYS_ON>; > + resets = <&cpg 822>; > + status = "disabled"; > + }; > + > + vin25: video@e6ed9000 { > + compatible = "renesas,vin-r8a779a0"; > + reg = <0 0xe6ed9000 0 0x1000>; > + interrupts = <GIC_SPI 185 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cpg CPG_MOD 823>; > + power-domains = <&sysc R8A779A0_PD_ALWAYS_ON>; > + resets = <&cpg 823>; > + status = "disabled"; > + }; > + > + vin26: video@e6eda000 { > + compatible = "renesas,vin-r8a779a0"; > + reg = <0 0xe6eda000 0 0x1000>; > + interrupts = <GIC_SPI 186 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cpg CPG_MOD 824>; > + power-domains = <&sysc R8A779A0_PD_ALWAYS_ON>; > + resets = <&cpg 824>; > + status = "disabled"; > + }; > + > + vin27: video@e6edb000 { > + compatible = "renesas,vin-r8a779a0"; > + reg = <0 0xe6edb000 0 0x1000>; > + interrupts = <GIC_SPI 187 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cpg CPG_MOD 825>; > + power-domains = <&sysc R8A779A0_PD_ALWAYS_ON>; > + resets = <&cpg 825>; > + status = "disabled"; > + }; > + > + vin28: video@e6edc000 { > + compatible = "renesas,vin-r8a779a0"; > + reg = <0 0xe6edc000 0 0x1000>; > + interrupts = <GIC_SPI 188 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cpg CPG_MOD 826>; > + power-domains = <&sysc R8A779A0_PD_ALWAYS_ON>; > + resets = <&cpg 826>; > + status = "disabled"; > + }; > + > + vin29: video@e6edd000 { > + compatible = "renesas,vin-r8a779a0"; > + reg = <0 0xe6edd000 0 0x1000>; > + interrupts = <GIC_SPI 189 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cpg CPG_MOD 827>; > + power-domains = <&sysc R8A779A0_PD_ALWAYS_ON>; > + resets = <&cpg 827>; > + status = "disabled"; > + }; > + > + vin30: video@e6ede000 { > + compatible = "renesas,vin-r8a779a0"; > + reg = <0 0xe6ede000 0 0x1000>; > + interrupts = <GIC_SPI 190 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cpg CPG_MOD 828>; > + power-domains = <&sysc R8A779A0_PD_ALWAYS_ON>; > + resets = <&cpg 828>; > + status = "disabled"; > + }; > + > + vin31: video@e6edf000 { > + compatible = "renesas,vin-r8a779a0"; > + reg = <0 0xe6edf000 0 0x1000>; > + interrupts = <GIC_SPI 191 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cpg CPG_MOD 829>; > + power-domains = <&sysc R8A779A0_PD_ALWAYS_ON>; > + resets = <&cpg 829>; > + status = "disabled"; > + }; > + > csi40: csi2@feaa0000 { > compatible = "renesas,r8a779a0-csi2"; > reg = <0 0xfeaa0000 0 0x10000>; > -- > 2.28.0 > -- Regards, Niklas Söderlund