From patchwork Wed Aug 28 15:55:55 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Sylwester Nawrocki/Kernel \\(PLT\\) /SRPOL/Staff Engineer/Samsung Electronics" X-Patchwork-Id: 2850865 Return-Path: X-Original-To: patchwork-linux-media@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork2.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.19.201]) by patchwork2.web.kernel.org (Postfix) with ESMTP id 656F9BF546 for ; Wed, 28 Aug 2013 15:57:01 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 29D6020412 for ; Wed, 28 Aug 2013 15:57:00 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id B58E9203E6 for ; Wed, 28 Aug 2013 15:56:58 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754291Ab3H1P44 (ORCPT ); Wed, 28 Aug 2013 11:56:56 -0400 Received: from mailout3.samsung.com ([203.254.224.33]:16202 "EHLO mailout3.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753815Ab3H1P44 (ORCPT ); Wed, 28 Aug 2013 11:56:56 -0400 Received: from epcpsbgm2.samsung.com (epcpsbgm2 [203.254.230.27]) by mailout3.samsung.com (Oracle Communications Messaging Server 7u4-24.01(7.0.4.24.0) 64bit (built Nov 17 2011)) with ESMTP id <0MS900BND0AUK420@mailout3.samsung.com> for linux-media@vger.kernel.org; Thu, 29 Aug 2013 00:56:55 +0900 (KST) X-AuditID: cbfee61b-b7f776d0000016c8-c7-521e1dc626d3 Received: from epmmp1.local.host ( [203.254.227.16]) by epcpsbgm2.samsung.com (EPCPMTA) with SMTP id 86.11.05832.6CD1E125; Thu, 29 Aug 2013 00:56:54 +0900 (KST) Received: from amdc1344.digital.local ([106.116.147.32]) by mmp1.samsung.com (Oracle Communications Messaging Server 7u4-24.01 (7.0.4.24.0) 64bit (built Nov 17 2011)) with ESMTPA id <0MS900LE309EAJB0@mmp1.samsung.com>; Thu, 29 Aug 2013 00:56:54 +0900 (KST) From: Sylwester Nawrocki To: linux-media@vger.kernel.org Cc: mturquette@linaro.org, g.liakhovetski@gmx.de, laurent.pinchart@ideasonboard.com, arun.kk@samsung.com, hverkuil@xs4all.nl, sakari.ailus@iki.fi, a.hajda@samsung.com, kyungmin.park@samsung.com, t.figa@samsung.com, linux-arm-kernel@lists.infradead.org, Sylwester Nawrocki Subject: [PATCH v2 2/7] V4L: s5k6a3: Add DT binding documentation Date: Wed, 28 Aug 2013 17:55:55 +0200 Message-id: <1377705360-12197-3-git-send-email-s.nawrocki@samsung.com> X-Mailer: git-send-email 1.7.9.5 In-reply-to: <1377705360-12197-1-git-send-email-s.nawrocki@samsung.com> References: <1377705360-12197-1-git-send-email-s.nawrocki@samsung.com> X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFtrJLMWRmVeSWpSXmKPExsVy+t9jAd1jsnJBBhu3W1rcWneO1eLjqdus Fu83zmOyODX5GZPF2aY37BadE5ewW2x6fI3VomfDVlaLpxMuslkcftPOanFm/0o2i/UzXrM4 8Hh8+BjnMbtjJqvH4a8LWTzuXNvD5rF5Sb1H35ZVjB6fN8l5nPr6mT2AI4rLJiU1J7MstUjf LoEr4+f52+wFW6Ur+ifNY2xgvCDWxcjJISFgIrFq/UdGCFtM4sK99WxdjFwcQgKLGCXWfbzI AuF0MEnc39bIDlLFJmAo0Xu0D6xDREBe4knvDbAOZoENTBJ3V9wDKxIWcJR4vOMaC4jNIqAq caj1MFgDr4CbRNuU00xdjBxA6xQk5kyyATE5Bdwl7lyQAKkQAqp4dLuZbQIj7wJGhlWMoqkF yQXFSem5RnrFibnFpXnpesn5uZsYwQH6THoH46oGi0OMAhyMSjy8Eb9lg4RYE8uKK3MPMUpw MCuJ8P4WkQsS4k1JrKxKLcqPLyrNSS0+xCjNwaIkznuw1TpQSCA9sSQ1OzW1ILUIJsvEwSnV wOj40kDEZM6O1f6i0fUlJzpkkqdPEBLqPxypn8kh/cT4gViEBZPsp7InzcWTf+55oJS2635v 7tPZ989vCP56J6n2a7PnizRf1sxneu4XVPvZLhiezVwryh25yTiHX0B7mV/PetveiTGPd8k6 TrPWFdZ9lLySaWrI9zU32+4/3pktaRcZw3UvU4mlOCPRUIu5qDgRAAzzYQlMAgAA Sender: linux-media-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-media@vger.kernel.org X-Spam-Status: No, score=-9.4 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_HI, RP_MATCHES_RCVD, UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP This patch adds binding documentation for the Samsung S5K6A3(YX) raw image sensor. Signed-off-by: Sylwester Nawrocki Signed-off-by: Kyungmin Park --- The binding of this sensors shows some issue in the generic video-interfaces binding. Namely The video bus type (serial MIPI CSI-2, parallel ITU-R BT.656, etc.) is being determined by the binding parser (v4l2-of.c) depending on what properties are found in an enddpoint node. Please have a look at the data-lanes property description. The sensor supports MIPI CSI-2 and SMIA CCP2 interfaces which both use one data lane. One data lane is everything this sensors supports. During our discussions on the generic bidings in the past I proposed to introduce a property in the endpoint node that would indicate what bus type (standard/protocol) is used, e.g. MIPI CSI-2, ITU-R BT.656, SMIA CCP2, etc. It was argued though that we can well determine bus type based on properties found in the endpoint node. So now in case of this sensor I'm not sure how it can be differentiated whether MIPI CSI-2 or CCP2 bus is used. There is no CCP2 specific generic properties yet. Anyway I'm not really happy there is no property like bus_type that would clearly indicate what data bus type is used. Then would would for instance not specify "data-lanes" in endpoint node just to differentiate between MIPI CSI-2 and the parallel busses. The main issue for this particular binding is that even with data-lanes = <1>; it is still impossible to figure out whether MIPI CSI-2 or SMIA CCP2 data bus is used. So how about introducing, e.g. a string type "bus_type" common property ? I'm considering starting a separate thread for discussing this. --- .../devicetree/bindings/media/samsung-s5k6a3.txt | 31 ++++++++++++++++++++ 1 file changed, 31 insertions(+) create mode 100644 Documentation/devicetree/bindings/media/samsung-s5k6a3.txt diff --git a/Documentation/devicetree/bindings/media/samsung-s5k6a3.txt b/Documentation/devicetree/bindings/media/samsung-s5k6a3.txt new file mode 100644 index 0000000..a51fbe8 --- /dev/null +++ b/Documentation/devicetree/bindings/media/samsung-s5k6a3.txt @@ -0,0 +1,31 @@ +Samsung S5K6A3(YX) raw image sensor +--------------------------------- + +S5K6A3YX is a raw image sensor with MIPI CSI-2 and CCP2 image data interfaces +and CCI (I2C compatible) control bus. + +Required properties: + +- compatible : "samsung,s5k6a3yx"; +- reg : I2C slave address of the sensor; +- svdda-supply : core voltage supply; +- svddio-supply : I/O voltage supply; +- gpios : specifier of a GPIO connected to the RESET pin; +- clocks : should contain the sensor's EXTCLK clock specifier, from + the common clock bindings. +- clock-names : should contain "extclk" entry; + +Optional properties: + +- clock-frequency : the frequency at which the "extclk" clock should be + configured to operate, in Hz; if this property is not + specified default 24 MHz value will be used. + +The common video interfaces bindings (see video-interfaces.txt) should be +used to specify link to the image data receiver. The S5K6A3(YX) device +node should contain one 'port' child node with an 'endpoint' subnode. + +Following properties are valid for the endpoint node: + +- data-lanes : (optional) specifies MIPI CSI-2 data lanes as covered in + video-interfaces.txt. The sensor supports only one data lane.