Hi Marek, On Mon, May 20, 2019 at 10:18:11PM +0200, Marek Vasut wrote: > Add bindings for the Intersil ISL7998x BT656-to-MIPI-CSI2 decoder. > > Signed-off-by: Marek Vasut <marex@xxxxxxx> > Cc: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> > Cc: Mauro Carvalho Chehab <mchehab+samsung@xxxxxxxxxx> > Cc: Rob Herring <robh+dt@xxxxxxxxxx> > Cc: devicetree@xxxxxxxxxxxxxxx > To: linux-media@xxxxxxxxxxxxxxx > --- > .../bindings/media/i2c/isl7998x.txt | 37 +++++++++++++++++++ > 1 file changed, 37 insertions(+) > create mode 100644 Documentation/devicetree/bindings/media/i2c/isl7998x.txt > > diff --git a/Documentation/devicetree/bindings/media/i2c/isl7998x.txt b/Documentation/devicetree/bindings/media/i2c/isl7998x.txt > new file mode 100644 > index 000000000000..c21703983360 > --- /dev/null > +++ b/Documentation/devicetree/bindings/media/i2c/isl7998x.txt > @@ -0,0 +1,37 @@ > +Intersil ISL7998x BT656-to-MIPI-CSI2 decoder > + > +The Intersil ISL7998x is a BT656-to-MIPI-CSI decoder which, capable of > +receiving up to four analog stream and multiplexing them into up to four > +MIPI CSI2 virtual channels, using one MIPI clock lane and 1/2 data lanes. > + > +Required Properties: > +- compatible: value should be "isil,isl79987" > +- pd-gpios: a GPIO spec for the Power Down pin (active high) > + > +Option Properties: > +- isil,num-inputs: Number of connected inputs (1, 2 or 4) The presence of ports describing connected Bt.656 inputs tells this. > + > +For further reading on port node refer to > +Documentation/devicetree/bindings/media/video-interfaces.txt. Which endpoint properties are relevant for the endpoint(s) in the CSI-2 port? How about the ports describing the Bt.656 interfaces? You should have those, too... > + > +Example: > + > + i2c_master { > + isl7998x_mipi@44 { > + compatible = "isil,isl79987"; > + reg = <0x44>; > + isil,num-inputs = <4>; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_videoadc>; > + pd-gpios = <&gpio3 27 GPIO_ACTIVE_HIGH>; > + status = "okay"; > + > + port { > + isl79987_to_mipi_csi2: endpoint { > + remote-endpoint = <&mipi_csi2_in>; > + clock-lanes = <0>; > + data-lanes = <1 2>; > + }; > + }; > + }; > + }; > -- Regards, Sakari Ailus sakari.ailus@xxxxxxxxxxxxxxx