Hi Jacopo, Thanks for your patch. On 2018-05-16 15:42:08 +0200, Jacopo Mondi wrote: > Describe CVBS video input through analog video decoder ADV7180 > connected to video input interface VIN4. > > The video input signal path is shared with HDMI video input, and > selected by on-board switches SW-53 and SW-54 with CVBS input selected > by the default switches configuration. You are missing your SoB line :-) Reviewed-by: Niklas Söderlund <niklas.soderlund+renesas@xxxxxxxxxxxx> > --- > arch/arm64/boot/dts/renesas/r8a77995-draak.dts | 36 ++++++++++++++++++++++++++ > 1 file changed, 36 insertions(+) > > diff --git a/arch/arm64/boot/dts/renesas/r8a77995-draak.dts b/arch/arm64/boot/dts/renesas/r8a77995-draak.dts > index d03f194..9aba28f 100644 > --- a/arch/arm64/boot/dts/renesas/r8a77995-draak.dts > +++ b/arch/arm64/boot/dts/renesas/r8a77995-draak.dts > @@ -142,6 +142,11 @@ > groups = "usb0"; > function = "usb0"; > }; > + > + vin4_pins_cvbs: vin4 { > + groups = "vin4_data8", "vin4_sync", "vin4_clk"; > + function = "vin4"; > + }; > }; > > &i2c0 { > @@ -154,6 +159,17 @@ > reg = <0x50>; > pagesize = <8>; > }; > + > + analog-video@20 { > + compatible = "adi,adv7180"; > + reg = <0x20>; > + > + port { > + adv7180_out: endpoint { > + remote-endpoint = <&vin4_in>; > + }; > + }; > + }; > }; > > &i2c1 { > @@ -246,3 +262,23 @@ > timeout-sec = <60>; > status = "okay"; > }; > + > +&vin4 { > + pinctrl-0 = <&vin4_pins_cvbs>; > + pinctrl-names = "default"; > + > + status = "okay"; > + > + ports { > + #address-cells = <1>; > + #size-cells = <0>; > + > + port@0 { > + reg = <0>; > + > + vin4_in: endpoint { > + remote-endpoint = <&adv7180_out>; > + }; > + }; > + }; > +}; > -- > 2.7.4 > -- Regards, Niklas Söderlund -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html