Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1678208

Re: [PATCH RFC 2/2] dt-bindings: add binding documentation for Allwinner CSI

Path csiph.com!news.redatomik.org!aioe.org!bofh.it!news.nic.it!robomod
From Rob Herring <robh@kernel.org>
Newsgroups linux.kernel
Subject Re: [PATCH RFC 2/2] dt-bindings: add binding documentation for Allwinner CSI
Date Thu, 29 Jun 2017 23:30:02 +0200
Message-ID <tXPaG-4Ds-7@gated-at.bofh.it> (permalink)
References <tWWxA-84J-11@gated-at.bofh.it> <tWWxA-84J-25@gated-at.bofh.it>
X-Original-To Yong Deng <yong.deng@magewell.com>
X-Google-Dkim-Signature v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=w0OTrWKunlQcLCvFUeAYgczA19JaTyJ91pIcOXBJA84=; b=ZL1vF5Sx0F6ILmtMarR3JV/5YFs+fvUWbsIg+j5p6RkzRJsurpo/T4gdPmIuQMpy+Y JIjI9LCjnY0vYuj3+bXBdl/0rzKuONLVAySWW8DS4q6Kq4HUPkuNHs3n+nUA0bK0OpV3 HzrySHd9bump/UwAbP5bMKzwRo47UXYQKoabd0J9a4FQVi2zL8uUCWcRpSPJRbZgMx9R +RGZfgPbS/Lx89Dw0kgxUdY2io2kIRkTrp26+DVrKh/lQQwh3j313HBNr4E1IdzVy7Dd BE9uDs1KCxToKrz4Vr1dinPzMFRp8RZRxijrb1IJ2HXivI8iVh5I43r8Hi6UNjSxqh0Z V/Tw==
X-Gm-Message-State AKS2vOwEyC51tE+GymiQypeYE19su7GBc8FZlg1KeR9GhYevjhRa6/NM tc8AINxc01HSTA==
X-Received by 10.107.3.234 with SMTP id e103mr4946124ioi.136.1498771199166; Thu, 29 Jun 2017 14:19:59 -0700 (PDT)
MIME-Version 1.0
Content-Type text/plain; charset=us-ascii
Content-Disposition inline
User-Agent NeoMutt/20170113 (1.7.2)
Sender robomod@news.nic.it
List-ID <linux-kernel.vger.kernel.org>
X-Mailing-List linux-kernel@vger.kernel.org
Approved robomod@news.nic.it
Lines 87
Organization linux.* mail to news gateway
X-Original-Cc mchehab@kernel.org, mark.rutland@arm.com, maxime.ripard@free-electrons.com, wens@csie.org, hans.verkuil@cisco.com, peter.griffin@linaro.org, hugues.fruchet@st.com, krzk@kernel.org, bparrot@ti.com, arnd@arndb.de, jean-christophe.trotin@st.com, benjamin.gaignard@linaro.org, tiffany.lin@mediatek.com, kamil@wypas.org, kieran+renesas@ksquared.org.uk, andrew-ct.chen@mediatek.com, linux-media@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-sunxi@googlegroups.com
X-Original-Date Thu, 29 Jun 2017 16:19:57 -0500
X-Original-Message-ID <20170629211957.uz7jijkuoxr2vohc@rob-hp-laptop>
X-Original-References <1498561654-14658-1-git-send-email-yong.deng@magewell.com> <1498561654-14658-3-git-send-email-yong.deng@magewell.com>
X-Original-Sender linux-kernel-owner@vger.kernel.org
Xref csiph.com linux.kernel:1678208

Show key headers only | View raw


On Tue, Jun 27, 2017 at 07:07:34PM +0800, Yong Deng wrote:
> Add binding documentation for Allwinner CSI.

For the subject:

dt-bindings: media: Add Allwinner Camera Sensor Interface (CSI)

"binding documentation" is redundant.

> 
> Signed-off-by: Yong Deng <yong.deng@magewell.com>
> ---
>  .../devicetree/bindings/media/sunxi-csi.txt        | 51 ++++++++++++++++++++++
>  1 file changed, 51 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/media/sunxi-csi.txt
> 
> diff --git a/Documentation/devicetree/bindings/media/sunxi-csi.txt b/Documentation/devicetree/bindings/media/sunxi-csi.txt
> new file mode 100644
> index 0000000..770be0e
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/media/sunxi-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
> +    * ahb: 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. The
> +  first port should be the input endpoints, the second one the outputs

Is there more than one endpoint for each port? If so, need to define 
that numbering too.

> +
> +Example:
> +
> +	csi1: csi@01cb4000 {
> +		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>,
> +			 <&ccu CLK_DRAM_CSI>;
> +		clock-names = "ahb", "mod", "ram";
> +		resets = <&ccu RST_BUS_CSI>;
> +
> +		port {
> +			#address-cells = <1>;
> +			#size-cells = <0>;
> +
> +			/* Parallel bus endpoint */
> +			csi1_0: endpoint@0 {
> +				reg = <0>;

Don't need this and everything associated with it for a single endpoint. 

> +				remote = <&adv7611_1>;
> +				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
> 
> --
> To unsubscribe from this list: send the line "unsubscribe devicetree" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH RFC 0/2] Initial Allwinner V3s CSI Support Yong Deng <yong.deng@magewell.com> - 2017-06-27 13:10 +0200
  [PATCH RFC 2/2] dt-bindings: add binding documentation for Allwinner CSI Yong Deng <yong.deng@magewell.com> - 2017-06-27 13:10 +0200
    Re: [PATCH RFC 2/2] dt-bindings: add binding documentation for  Allwinner CSI Rob Herring <robh@kernel.org> - 2017-06-29 23:30 +0200
      Re: [PATCH RFC 2/2] dt-bindings: add binding documentation for  Allwinner CSI Chen-Yu Tsai <wens@csie.org> - 2017-06-30 05:50 +0200
        Re: [PATCH RFC 2/2] dt-bindings: add binding documentation for  Allwinner CSI Yong <yong.deng@magewell.com> - 2017-06-30 10:00 +0200
        Re: [PATCH RFC 2/2] dt-bindings: add binding documentation for  Allwinner CSI Rob Herring <robh@kernel.org> - 2017-06-30 18:10 +0200
  Re: [PATCH RFC 1/2] media: V3s: Add support for Allwinner CSI. Hans Verkuil <hverkuil@xs4all.nl> - 2017-07-03 12:30 +0200
    Re: [PATCH RFC 1/2] media: V3s: Add support for Allwinner CSI. Yong <yong.deng@magewell.com> - 2017-07-03 13:10 +0200
      Re: [PATCH RFC 1/2] media: V3s: Add support for Allwinner CSI. Maxime Ripard <maxime.ripard@free-electrons.com> - 2017-07-03 13:30 +0200

csiph-web