On Wed, Jun 7, 2017 at 11:33 AM, Steve Longerbeam <slongerbeam@xxxxxxxxx> wrote: > Enables the ADV7180 decoder sensor. The ADV7180 connects to the > parallel-bus mux input on ipu1_csi0_mux. > > The ADV7180 power pin is via max7310_b port expander. > > Signed-off-by: Steve Longerbeam <steve_longerbeam@xxxxxxxxxx> > > - Use IRQ_TYPE_LEVEL_LOW instead of 0x8 for interrupt type for clarity. > - For 8-bit parallel IPU1-CSI0 bus connection only data[12-19] are used. > > Signed-off-by: Tim Harvey <tharvey@xxxxxxxxxxxxx> > --- > arch/arm/boot/dts/imx6qdl-sabreauto.dtsi | 50 ++++++++++++++++++++++++++++++++ > 1 file changed, 50 insertions(+) > > diff --git a/arch/arm/boot/dts/imx6qdl-sabreauto.dtsi b/arch/arm/boot/dts/imx6qdl-sabreauto.dtsi > index 1212f82..c24af28 100644 > --- a/arch/arm/boot/dts/imx6qdl-sabreauto.dtsi > +++ b/arch/arm/boot/dts/imx6qdl-sabreauto.dtsi > @@ -124,6 +124,21 @@ > #size-cells = <0>; > reg = <1>; > > + adv7180: camera@21 { > + compatible = "adi,adv7180"; > + reg = <0x21>; > + powerdown-gpios = <&max7310_b 2 GPIO_ACTIVE_LOW>; > + interrupt-parent = <&gpio1>; > + interrupts = <27 IRQ_TYPE_LEVEL_LOW>; > + > + port { > + adv7180_to_ipu1_csi0_mux: endpoint { > + remote-endpoint = <&ipu1_csi0_mux_from_parallel_sensor>; > + bus-width = <8>; > + }; > + }; > + }; > + > max7310_a: gpio@30 { > compatible = "maxim,max7310"; > reg = <0x30>; > @@ -151,6 +166,25 @@ > }; > }; > > +&ipu1_csi0_from_ipu1_csi0_mux { > + bus-width = <8>; > +}; > + > +&ipu1_csi0_mux_from_parallel_sensor { > + remote-endpoint = <&adv7180_to_ipu1_csi0_mux>; > + bus-width = <8>; > +}; > + > +&ipu1_csi0 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_ipu1_csi0>; > + > + /* enable frame interval monitor on this port */ > + fim { > + status = "okay"; > + }; Steve, You need to remove the fim node now that you've moved this to V4L2 controls. Regards, Tim _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel