Received: by 2002:a25:ad19:0:0:0:0:0 with SMTP id y25csp11268454ybi; Thu, 25 Jul 2019 13:04:34 -0700 (PDT) X-Google-Smtp-Source: APXvYqwrGd8picYLm2aLGbf01x/QOiiFGj7O4lNytXCydBEcijfSJ0EJ/1zH1wZtD2VTo7P0Kw8I X-Received: by 2002:a65:43c2:: with SMTP id n2mr87807372pgp.110.1564085074462; Thu, 25 Jul 2019 13:04:34 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1564085074; cv=none; d=google.com; s=arc-20160816; b=J1DwQve0duqgOUAF7/UXYIfJoxVxg3hgIL/g2zUmh6rWUdFc95kGyy2+sXEdqs4gcm MTTJvx/4z2kyiMlkkBgv6jjdpIZNMiBZSKU3XQJDXiUk7+OI4KVgUTiH1teEqVl1HFQ+ Kxr9YU8d3tFAbQr4vK92P6UkRUBAHm/80ulPogCI/H4tEBuJKX7ypWIIEhn5VT9ChHI3 ROBp7G8Ay99PPY2RkrdM/GAD8XfsvYUTdhgsjvCWnBCVhy5oXRafX0fuoCOpEGLy5cHM bqXYwCPNtM81i1oEI0H3jgO12L4TQKBhjCjBTWR2iHSRFVa/TprUfzp1rl/Qwmx88uh/ CHOQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date; bh=u5jPjeCGp8Uvi8zyqCGj2KV9DUzDccM5qVHB67dWVVk=; b=NsxGfdnWKbKc3L2xVnhghllyzCAQ+XbMziAwrwlAz+SBANjJir6kjYxW0s4u8wWw2g X+bRer5k40p+Hbe1AwMi3nVZf3ZoLHn9mcW67uPapiQPWEJ2sGg2CRYgD6avlKlOCd/7 Imdhv2wW4QTRRFvo8WzjTuHMwv+gHWLtFfXK+G3N1hDddvk6w5o+l9z+xfQ+uDRCZTxY /BecZINSOgUBEzejKQCHt61WOoibl976ABc6ddlf0hUr/FkBPIKrDTPJcL4FKITl38He 0fasIivbQSiBnteqjUznK7HOP9oVqPmQINhLkLaO4sXXb7wJAzZQHumDAPC57m3bCKda Evfw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=iki.fi Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id e24si17993145pff.125.2019.07.25.13.04.19; Thu, 25 Jul 2019 13:04:34 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=iki.fi Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726545AbfGYUCc (ORCPT + 99 others); Thu, 25 Jul 2019 16:02:32 -0400 Received: from retiisi.org.uk ([95.216.213.190]:37126 "EHLO hillosipuli.retiisi.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726115AbfGYUCb (ORCPT ); Thu, 25 Jul 2019 16:02:31 -0400 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 D7633634C87; Thu, 25 Jul 2019 23:02:11 +0300 (EEST) Received: from sailus by valkosipuli.localdomain with local (Exim 4.92) (envelope-from ) id 1hqjwJ-0001Ev-7L; Thu, 25 Jul 2019 23:02:11 +0300 Date: Thu, 25 Jul 2019 23:02:11 +0300 From: Sakari Ailus To: Luis de Oliveira Cc: "mchehab@kernel.org" , "davem@davemloft.net" , "gregkh@linuxfoundation.org" , "Jonathan.Cameron@huawei.com" , "robh@kernel.org" , "nicolas.ferre@microchip.com" , "paulmck@linux.ibm.com" , "mark.rutland@arm.com" , "kishon@ti.com" , "devicetree@vger.kernel.org" , "linux-media@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "Joao.Pinto@synopsys.com" Subject: Re: [v4 1/6] dt-bindings: media: Document bindings for DW MIPI CSI-2 Host Message-ID: <20190725200211.GK1263@valkosipuli.retiisi.org.uk> References: <1560280855-18085-1-git-send-email-luis.oliveira@synopsys.com> <1560280855-18085-2-git-send-email-luis.oliveira@synopsys.com> <20190628141326.swgl3kg4fj5pmlqx@valkosipuli.retiisi.org.uk> <20190709182500.3x544axnrgy72aje@valkosipuli.retiisi.org.uk> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.10.1 (2018-07-13) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Luis, On Wed, Jul 10, 2019 at 10:20:55AM +0000, Luis de Oliveira wrote: > Hi Sakari, > > From: Sakari Ailus > Date: Tue, Jul 09, 2019 at 19:25:00 > > > Hi Luis, > > > > On Mon, Jul 08, 2019 at 03:21:50PM +0000, Luis de Oliveira wrote: > > > Hi Sakari, > > > > > > Thank you for your feedback. > > > I have my comments inline. > > > > > > From: Sakari Ailus > > > Date: Fri, Jun 28, 2019 at 15:13:26 > > > > > > > Hi Luis, > > > > > > > > Thank you for the patchset. > > > > > > > > On Tue, Jun 11, 2019 at 09:20:50PM +0200, Luis Oliveira wrote: > > > > > From: Luis Oliveira > > > > > > > > > > Add bindings for Synopsys DesignWare MIPI CSI-2 host. > > > > > > > > > > Signed-off-by: Luis Oliveira > > > > > --- > > > > > Changelog > > > > > v3-v4 > > > > > - remove "plat" from the block name @rob @laurent > > > > > - remove "phy-names" when single-entry @rob > > > > > - remove "snps,output-type" -> went to the driver config @laurent > > > > > > > > > > .../devicetree/bindings/media/snps,dw-csi.txt | 41 ++++++++++++++++++++++ > > > > > 1 file changed, 41 insertions(+) > > > > > create mode 100644 Documentation/devicetree/bindings/media/snps,dw-csi.txt > > > > > > > > > > diff --git a/Documentation/devicetree/bindings/media/snps,dw-csi.txt b/Documentation/devicetree/bindings/media/snps,dw-csi.txt > > > > > new file mode 100644 > > > > > index 0000000..613b7f9 > > > > > --- /dev/null > > > > > +++ b/Documentation/devicetree/bindings/media/snps,dw-csi.txt > > > > > @@ -0,0 +1,41 @@ > > > > > +Synopsys DesignWare CSI-2 Host controller > > > > > + > > > > > +Description > > > > > +----------- > > > > > + > > > > > +This HW block is used to receive image coming from an MIPI CSI-2 compatible > > > > > +camera. > > > > > + > > > > > +Required properties: > > > > > +- compatible : shall be "snps,dw-csi" > > > > > +- reg : physical base address and size of the device memory > > > > > + mapped registers; > > > > > +- interrupts : DW CSI-2 Host interrupts > > > > > +- phys : List of one PHY specifier (as defined in > > > > > + Documentation/devicetree/bindings/phy/phy-bindings.txt). > > > > > + This PHY is a MIPI DPHY working in RX mode. > > > > > +- resets : Reference to a reset controller (optional) > > > > > + > > > > > +The per-board settings: > > > > > + - port sub-node describing a single endpoint connected to the camera as > > > > > + described in video-interfaces.txt[1]. > > > > > > > > Which endpoint properties in video-interfaces.txt are relevant for the > > > > hardware? Which values may they have? > > > > > > > > > > Currently I'm using only two properties "data-lanes" and "bus-width", but > > > I have plans to add blanking info also. > > > I will add more info. > > > > Isn't blanking defined by what the transmitter seneds? Or do you have > > hardware limitations on the receiver side? > > > > When we use this IP in prototyping we configure blanking at the receiver > side. > Some cameras don't have blanking configuration capabilities so we > configure it on the RX side. I haven't come across a CSI-2 connected camera without some kind of blanking configuration capabilities. Even if there was one, you couldn't configure blanking from the receiver side. Please document that the data-lanes property is required, and which values are possible. > > > I've only heard of one such case before, and it was a very old parallel > > receiver. > > > > If you have a CSI-2 receiver, bus-width isn't relevant --- it's for paralle > > interfaces only. Please add data-lanes to required endpoint properties. > > > > I used bus-width property in the Synopsys IPI (Image Pixel Interface) > that enables direct video stream access. > This interface is an output that can be 16-bit or 48-bit, that's why I > used bus-width property. Does this device write the image data to system memory, or is it another device? If there's another one, then you should probably have another port to describe that connection. -- Kind regards, Sakari Ailus