From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-8.3 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 0B4C8C32750 for ; Tue, 13 Aug 2019 11:47:03 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id D4C4520844 for ; Tue, 13 Aug 2019 11:47:02 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="iUFRB628" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org D4C4520844 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=iki.fi Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=Jl4pvu33DZ49qQtURufZXSLs01bEsEUwu5HGW59hERM=; b=iUFRB6285L12wF t+LSI/8MbObPuvkMrCRDo28BexqybMsNWIEnnUDLA/0juoXM2MXavT3Sz2SyDG9u3+aCI0vZp2ET8 BhrAFlXFOMHZvNx4sBbyGJZ4U20/9m/7j5Gh84T44A2agw1CL/Z0aB0zbaAIcx3QZJ/iMztZIfdnR +jP5NyXhumQxfYIvP/myGLZ3dXPAEr9vMOqYBItDz2cQgR33qiwWEMpJ+VlZGpof2vOGH9wdtyapY UVnXYL+BZ/x8rrgmlgffMPAXITt19dF13JV0Zk24dX4hF65A4Q7p2B+UJKU74v/DocVxzX4SczGB7 /0a3arOBvq8LtiYhcrRg==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.92 #3 (Red Hat Linux)) id 1hxVGS-0007aO-8M; Tue, 13 Aug 2019 11:46:56 +0000 Received: from hillosipuli.retiisi.org.uk ([2a01:4f9:c010:4572::81:2]) by bombadil.infradead.org with esmtps (Exim 4.92 #3 (Red Hat Linux)) id 1hxVGO-0007ZT-HX for linux-arm-kernel@lists.infradead.org; Tue, 13 Aug 2019 11:46:54 +0000 Received: from valkosipuli.localdomain (valkosipuli.retiisi.org.uk [IPv6:2a01:4f9:c010:4572::80:2]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (P-256) server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by hillosipuli.retiisi.org.uk (Postfix) with ESMTPS id D8AB3634C88; Tue, 13 Aug 2019 14:46:43 +0300 (EEST) Received: from sailus by valkosipuli.localdomain with local (Exim 4.92) (envelope-from ) id 1hxVGF-0000fC-7e; Tue, 13 Aug 2019 14:46:43 +0300 Date: Tue, 13 Aug 2019 14:46:43 +0300 From: Sakari Ailus To: Manivannan Sadhasivam Subject: Re: [PATCH v2 1/3] dt-bindings: media: i2c: Add IMX290 CMOS sensor binding Message-ID: <20190813114643.GA2527@valkosipuli.retiisi.org.uk> References: <20190806130938.19916-1-manivannan.sadhasivam@linaro.org> <20190806130938.19916-2-manivannan.sadhasivam@linaro.org> <20190813094526.GG835@valkosipuli.retiisi.org.uk> <20190813113358.GA28877@Mani-XPS-13-9360> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20190813113358.GA28877@Mani-XPS-13-9360> User-Agent: Mutt/1.10.1 (2018-07-13) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20190813_044652_945808_CA5FFEDE X-CRM114-Status: GOOD ( 28.08 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: devicetree@vger.kernel.org, c.barrett@framos.com, linux-kernel@vger.kernel.org, a.brela@framos.com, robh+dt@kernel.org, mchehab@kernel.org, linux-arm-kernel@lists.infradead.org, linux-media@vger.kernel.org Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi Manivannan, On Tue, Aug 13, 2019 at 05:03:58PM +0530, Manivannan Sadhasivam wrote: > Hi Sakari, > > Thanks for the review! > > On Tue, Aug 13, 2019 at 12:45:26PM +0300, Sakari Ailus wrote: > > Hi Manivannan, > > > > On Tue, Aug 06, 2019 at 06:39:36PM +0530, Manivannan Sadhasivam wrote: > > > Add devicetree binding for IMX290 CMOS image sensor. > > > > > > Signed-off-by: Manivannan Sadhasivam > > > Reviewed-by: Rob Herring > > > --- > > > .../devicetree/bindings/media/i2c/imx290.txt | 51 +++++++++++++++++++ > > > 1 file changed, 51 insertions(+) > > > create mode 100644 Documentation/devicetree/bindings/media/i2c/imx290.txt > > > > > > diff --git a/Documentation/devicetree/bindings/media/i2c/imx290.txt b/Documentation/devicetree/bindings/media/i2c/imx290.txt > > > new file mode 100644 > > > index 000000000000..7535b5b5b24b > > > --- /dev/null > > > +++ b/Documentation/devicetree/bindings/media/i2c/imx290.txt > > > @@ -0,0 +1,51 @@ > > > +* Sony IMX290 1/2.8-Inch CMOS Image Sensor > > > + > > > +The Sony IMX290 is a 1/2.8-Inch CMOS Solid-state image sensor with > > > +Square Pixel for Color Cameras. It is programmable through I2C and 4-wire > > > +interfaces. The sensor output is available via CMOS logic parallel SDR output, > > > +Low voltage LVDS DDR output and CSI-2 serial data output. > > > > If there are three to choose from, then you should specify which one is in > > use. Given that I think chances remain slim we'd add support for the other > > two (it's certainly not ruled out though), CSI-2 could be the default. But > > this needs to be documented. > > > > Hmm... I'm not sure here. Bindings should describe the hardware and not the > limitations of the driver. Here as you said, the sensor can output frames > in 3 different modes/formats but the driver only supports CSI2. I can add a > note in the driver but not sure whether dt-binding is the right place or not! I guess alternatively you could document the necessary bindings for the other two busses. But what I'm saying here is that it's highly unlikely they'll be ever needed, and it'd be mostly a waste of time to implement that. (That said, I have nothing against the use of these busses, but I've never seen anyone using them.) Many other devices use defaults for more contentious settings. > > > > + > > > +Required Properties: > > > +- compatible: Should be "sony,imx290" > > > +- reg: I2C bus address of the device > > > +- clocks: Reference to the xclk clock. > > > +- clock-names: Should be "xclk". > > > +- clock-frequency: Frequency of the xclk clock. > > > > ...in Hz. > > > > Ack. > > > > +- vdddo-supply: Sensor digital IO regulator. > > > +- vdda-supply: Sensor analog regulator. > > > +- vddd-supply: Sensor digital core regulator. > > > + > > > +Optional Properties: > > > +- reset-gpios: Sensor reset GPIO > > > + > > > +The imx290 device node should contain one 'port' child node with > > > +an 'endpoint' subnode. For further reading on port node refer to > > > +Documentation/devicetree/bindings/media/video-interfaces.txt. > > > > Which other properties are relevant for the device? > > Not much other than, clock/data lanes. Please document data-lanes, and which values it may have. > > > I suppose you can't change the lane order, so clock-lanes is redundant > > (don't use it in the example) and data-lanes should be monotonically > > incrementing series from 1 to 4. > > > > We can change the order and the example here illustrates how it has been > wired in FRAMOS module. If I change the lane order like you said, it won't > work. I highly doubt that. Neither the driver nor the sensor uses the lane ordering information. And even if the driver only supported four lanes, then it should check the number of lanes is actually four. -- Regards, Sakari Ailus _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel