* [linux-sunxi] [PATCH v3 2/3] dt-bindings: media: Add Allwinner V3s Camera Sensor Interface (CSI) [not found] <1510558344-45402-1-git-send-email-yong.deng@magewell.com> @ 2017-12-18 8:35 ` Chen-Yu Tsai 2017-12-18 8:49 ` Yong 0 siblings, 1 reply; 5+ messages in thread From: Chen-Yu Tsai @ 2017-12-18 8:35 UTC (permalink / raw) To: linux-arm-kernel On Mon, Nov 13, 2017 at 3:32 PM, Yong Deng <yong.deng@magewell.com> wrote: > Add binding documentation for Allwinner V3s CSI. > > Signed-off-by: Yong Deng <yong.deng@magewell.com> > --- > .../devicetree/bindings/media/sun6i-csi.txt | 51 ++++++++++++++++++++++ > 1 file changed, 51 insertions(+) > create mode 100644 Documentation/devicetree/bindings/media/sun6i-csi.txt > > diff --git a/Documentation/devicetree/bindings/media/sun6i-csi.txt b/Documentation/devicetree/bindings/media/sun6i-csi.txt > new file mode 100644 > index 0000000..f3916a2 > --- /dev/null > +++ b/Documentation/devicetree/bindings/media/sun6i-csi.txt > @@ -0,0 +1,51 @@ > +Allwinner V3s Camera Sensor Interface > +------------------------------ > + > +Required properties: > + - compatible: value must be "allwinner,sun8i-v3s-csi" > + - reg: base address and size of the memory-mapped region. > + - interrupts: interrupt associated to this IP > + - clocks: phandles to the clocks feeding the CSI > + * bus: the CSI interface clock > + * mod: the CSI module clock > + * ram: the CSI DRAM clock > + - clock-names: the clock names mentioned above > + - resets: phandles to the reset line driving the CSI > + > +- ports: A ports node with endpoint definitions as defined in > + Documentation/devicetree/bindings/media/video-interfaces.txt. > + Currently, the driver only support the parallel interface. So, a single port > + node with one endpoint and parallel bus is supported. > + > +Example: > + > + csi1: csi at 01cb4000 { Drop the leading zero in the address part. > + compatible = "allwinner,sun8i-v3s-csi"; > + reg = <0x01cb4000 0x1000>; > + interrupts = <GIC_SPI 84 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&ccu CLK_BUS_CSI>, > + <&ccu CLK_CSI1_SCLK>, CSI also has an MCLK. Do you need that one? ChenYu > + <&ccu CLK_DRAM_CSI>; > + clock-names = "bus", "mod", "ram"; > + resets = <&ccu RST_BUS_CSI>; > + > + port { > + #address-cells = <1>; > + #size-cells = <0>; > + > + /* Parallel bus endpoint */ > + csi1_ep: endpoint { > + remote-endpoint = <&adv7611_ep>; > + bus-width = <16>; > + data-shift = <0>; > + > + /* If hsync-active/vsync-active are missing, > + embedded BT.656 sync is used */ > + hsync-active = <0>; /* Active low */ > + vsync-active = <0>; /* Active low */ > + data-active = <1>; /* Active high */ > + pclk-sample = <1>; /* Rising */ > + }; > + }; > + }; > + > -- > 1.8.3.1 > > -- > You received this message because you are subscribed to the Google Groups "linux-sunxi" group. > To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscribe at googlegroups.com. > For more options, visit https://groups.google.com/d/optout. ^ permalink raw reply [flat|nested] 5+ messages in thread
* [linux-sunxi] [PATCH v3 2/3] dt-bindings: media: Add Allwinner V3s Camera Sensor Interface (CSI) 2017-12-18 8:35 ` [linux-sunxi] [PATCH v3 2/3] dt-bindings: media: Add Allwinner V3s Camera Sensor Interface (CSI) Chen-Yu Tsai @ 2017-12-18 8:49 ` Yong 2017-12-18 9:24 ` Maxime Ripard 0 siblings, 1 reply; 5+ messages in thread From: Yong @ 2017-12-18 8:49 UTC (permalink / raw) To: linux-arm-kernel On Mon, 18 Dec 2017 16:35:51 +0800 Chen-Yu Tsai <wens@csie.org> wrote: > On Mon, Nov 13, 2017 at 3:32 PM, Yong Deng <yong.deng@magewell.com> wrote: > > Add binding documentation for Allwinner V3s CSI. > > > > Signed-off-by: Yong Deng <yong.deng@magewell.com> > > --- > > .../devicetree/bindings/media/sun6i-csi.txt | 51 ++++++++++++++++++++++ > > 1 file changed, 51 insertions(+) > > create mode 100644 Documentation/devicetree/bindings/media/sun6i-csi.txt > > > > diff --git a/Documentation/devicetree/bindings/media/sun6i-csi.txt b/Documentation/devicetree/bindings/media/sun6i-csi.txt > > new file mode 100644 > > index 0000000..f3916a2 > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/media/sun6i-csi.txt > > @@ -0,0 +1,51 @@ > > +Allwinner V3s Camera Sensor Interface > > +------------------------------ > > + > > +Required properties: > > + - compatible: value must be "allwinner,sun8i-v3s-csi" > > + - reg: base address and size of the memory-mapped region. > > + - interrupts: interrupt associated to this IP > > + - clocks: phandles to the clocks feeding the CSI > > + * bus: the CSI interface clock > > + * mod: the CSI module clock > > + * ram: the CSI DRAM clock > > + - clock-names: the clock names mentioned above > > + - resets: phandles to the reset line driving the CSI > > + > > +- ports: A ports node with endpoint definitions as defined in > > + Documentation/devicetree/bindings/media/video-interfaces.txt. > > + Currently, the driver only support the parallel interface. So, a single port > > + node with one endpoint and parallel bus is supported. > > + > > +Example: > > + > > + csi1: csi at 01cb4000 { > > Drop the leading zero in the address part. OK. > > > + compatible = "allwinner,sun8i-v3s-csi"; > > + reg = <0x01cb4000 0x1000>; > > + interrupts = <GIC_SPI 84 IRQ_TYPE_LEVEL_HIGH>; > > + clocks = <&ccu CLK_BUS_CSI>, > > + <&ccu CLK_CSI1_SCLK>, > > CSI also has an MCLK. Do you need that one? MCLK is not needed if the front end is not a sensor (like adv7611). I will add it as an option. > > ChenYu > > > + <&ccu CLK_DRAM_CSI>; > > + clock-names = "bus", "mod", "ram"; > > + resets = <&ccu RST_BUS_CSI>; > > + > > + port { > > + #address-cells = <1>; > > + #size-cells = <0>; > > + > > + /* Parallel bus endpoint */ > > + csi1_ep: endpoint { > > + remote-endpoint = <&adv7611_ep>; > > + bus-width = <16>; > > + data-shift = <0>; > > + > > + /* If hsync-active/vsync-active are missing, > > + embedded BT.656 sync is used */ > > + hsync-active = <0>; /* Active low */ > > + vsync-active = <0>; /* Active low */ > > + data-active = <1>; /* Active high */ > > + pclk-sample = <1>; /* Rising */ > > + }; > > + }; > > + }; > > + > > -- > > 1.8.3.1 > > > > -- > > You received this message because you are subscribed to the Google Groups "linux-sunxi" group. > > To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscribe at googlegroups.com. > > For more options, visit https://groups.google.com/d/optout. > > -- > You received this message because you are subscribed to the Google Groups "linux-sunxi" group. > To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscribe at googlegroups.com. > For more options, visit https://groups.google.com/d/optout. Thanks, Yong ^ permalink raw reply [flat|nested] 5+ messages in thread
* [linux-sunxi] [PATCH v3 2/3] dt-bindings: media: Add Allwinner V3s Camera Sensor Interface (CSI) 2017-12-18 8:49 ` Yong @ 2017-12-18 9:24 ` Maxime Ripard 2017-12-18 9:43 ` Yong 0 siblings, 1 reply; 5+ messages in thread From: Maxime Ripard @ 2017-12-18 9:24 UTC (permalink / raw) To: linux-arm-kernel Hi, On Mon, Dec 18, 2017 at 04:49:21PM +0800, Yong wrote: > > > + compatible = "allwinner,sun8i-v3s-csi"; > > > + reg = <0x01cb4000 0x1000>; > > > + interrupts = <GIC_SPI 84 IRQ_TYPE_LEVEL_HIGH>; > > > + clocks = <&ccu CLK_BUS_CSI>, > > > + <&ccu CLK_CSI1_SCLK>, > > > > CSI also has an MCLK. Do you need that one? > > MCLK is not needed if the front end is not a sensor (like adv7611). > I will add it as an option. I guess this should always be needed then. And the driver will make the decision to enable it or not. Maxime -- Maxime Ripard, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 833 bytes Desc: not available URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20171218/010dbb66/attachment-0001.sig> ^ permalink raw reply [flat|nested] 5+ messages in thread
* [linux-sunxi] [PATCH v3 2/3] dt-bindings: media: Add Allwinner V3s Camera Sensor Interface (CSI) 2017-12-18 9:24 ` Maxime Ripard @ 2017-12-18 9:43 ` Yong 2017-12-18 10:46 ` Chen-Yu Tsai 0 siblings, 1 reply; 5+ messages in thread From: Yong @ 2017-12-18 9:43 UTC (permalink / raw) To: linux-arm-kernel Hi, On Mon, 18 Dec 2017 10:24:37 +0100 Maxime Ripard <maxime.ripard@free-electrons.com> wrote: > Hi, > > On Mon, Dec 18, 2017 at 04:49:21PM +0800, Yong wrote: > > > > + compatible = "allwinner,sun8i-v3s-csi"; > > > > + reg = <0x01cb4000 0x1000>; > > > > + interrupts = <GIC_SPI 84 IRQ_TYPE_LEVEL_HIGH>; > > > > + clocks = <&ccu CLK_BUS_CSI>, > > > > + <&ccu CLK_CSI1_SCLK>, > > > > > > CSI also has an MCLK. Do you need that one? > > > > MCLK is not needed if the front end is not a sensor (like adv7611). > > I will add it as an option. > > I guess this should always be needed then. And the driver will make > the decision to enable it or not. But how the driver to know if it should be enabled? I think MCLK should be added in DT just if the subdev need it. > > Maxime > > -- > Maxime Ripard, Free Electrons > Embedded Linux and Kernel engineering > http://free-electrons.com Thanks, Yong ^ permalink raw reply [flat|nested] 5+ messages in thread
* [linux-sunxi] [PATCH v3 2/3] dt-bindings: media: Add Allwinner V3s Camera Sensor Interface (CSI) 2017-12-18 9:43 ` Yong @ 2017-12-18 10:46 ` Chen-Yu Tsai 0 siblings, 0 replies; 5+ messages in thread From: Chen-Yu Tsai @ 2017-12-18 10:46 UTC (permalink / raw) To: linux-arm-kernel On Mon, Dec 18, 2017 at 5:43 PM, Yong <yong.deng@magewell.com> wrote: > Hi, > > On Mon, 18 Dec 2017 10:24:37 +0100 > Maxime Ripard <maxime.ripard@free-electrons.com> wrote: > >> Hi, >> >> On Mon, Dec 18, 2017 at 04:49:21PM +0800, Yong wrote: >> > > > + compatible = "allwinner,sun8i-v3s-csi"; >> > > > + reg = <0x01cb4000 0x1000>; >> > > > + interrupts = <GIC_SPI 84 IRQ_TYPE_LEVEL_HIGH>; >> > > > + clocks = <&ccu CLK_BUS_CSI>, >> > > > + <&ccu CLK_CSI1_SCLK>, >> > > >> > > CSI also has an MCLK. Do you need that one? >> > >> > MCLK is not needed if the front end is not a sensor (like adv7611). >> > I will add it as an option. >> >> I guess this should always be needed then. And the driver will make >> the decision to enable it or not. > > But how the driver to know if it should be enabled? > I think MCLK should be added in DT just if the subdev need it. Looking around the docs, there doesn't seem to be anything related to MCLK within the CSI section. Furthermore, camera sensor bindings, such as for the OV5640, in fact do have a property for a reference clock, which is called XCLK or XVCLK. Since the clock is already exported from the CCU, I suppose it's just a matter of referencing the clock from the camera node, with the proper pinctrl for that pin. So to summarize, just ignore my comment about the missing MCLK. :) ChenYu ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2017-12-18 10:46 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1510558344-45402-1-git-send-email-yong.deng@magewell.com>
2017-12-18 8:35 ` [linux-sunxi] [PATCH v3 2/3] dt-bindings: media: Add Allwinner V3s Camera Sensor Interface (CSI) Chen-Yu Tsai
2017-12-18 8:49 ` Yong
2017-12-18 9:24 ` Maxime Ripard
2017-12-18 9:43 ` Yong
2017-12-18 10:46 ` Chen-Yu Tsai
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox