From mboxrd@z Thu Jan 1 00:00:00 1970 From: Laurent Pinchart Subject: Re: [PATCH v4 6/6] dt-bindings: media: rcar-vin: Clarify optional props Date: Tue, 12 Jun 2018 19:15:06 +0300 Message-ID: <2802458.0ZPzlQSr2q@avalon> References: <1528813566-17927-1-git-send-email-jacopo+renesas@jmondi.org> <1528813566-17927-7-git-send-email-jacopo+renesas@jmondi.org> <20180612154553.kgqnqkwv3y6srivg@kekkonen.localdomain> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20180612154553.kgqnqkwv3y6srivg@kekkonen.localdomain> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Sakari Ailus Cc: devicetree@vger.kernel.org, robh+dt@kernel.org, linux-renesas-soc@vger.kernel.org, horms@verge.net.au, geert@glider.be, niklas.soderlund@ragnatech.se, Jacopo Mondi , mchehab@kernel.org, hans.verkuil@cisco.com, linux-arm-kernel@lists.infradead.org, linux-media@vger.kernel.org List-Id: devicetree@vger.kernel.org Hello, On Tuesday, 12 June 2018 18:45:53 EEST Sakari Ailus wrote: > On Tue, Jun 12, 2018 at 04:26:06PM +0200, Jacopo Mondi wrote: > > Add a note to the R-Car VIN interface bindings to clarify that all > > properties listed as generic properties in video-interfaces.txt can > > be included in port@0 endpoint, but if not explicitly listed in the > > interface bindings documentation, they do not modify it behaviour. > > > > Signed-off-by: Jacopo Mondi > > --- > > > > Documentation/devicetree/bindings/media/rcar_vin.txt | 6 ++++++ > > 1 file changed, 6 insertions(+) > > > > diff --git a/Documentation/devicetree/bindings/media/rcar_vin.txt > > b/Documentation/devicetree/bindings/media/rcar_vin.txt index > > 8130849..03544c7 100644 > > --- a/Documentation/devicetree/bindings/media/rcar_vin.txt > > +++ b/Documentation/devicetree/bindings/media/rcar_vin.txt > > @@ -55,6 +55,12 @@ from local SoC CSI-2 receivers (port1) depending on > > SoC. > > > > instances that are connected to external pins should have port 0. > > > > - Optional properties for endpoint nodes of port@0: > > + > > + All properties described in [1] and which apply to the selected > > + media bus type could be optionally listed here to better describe > > + the current hardware configuration, but only the following ones > > do > > + actually modify the VIN interface behaviour: > > + > > I don't think this should be needed. You should only have properties that > describe the hardware configuration in a given system. I agree. Please list explicitly all properties that are applicable for this device, and don't tell which one(s) are used by the driver. > > - hsync-active: see [1] for description. Default is active high. > > - vsync-active: see [1] for description. Default is active high. > > - data-enable-active: polarity of CLKENB signal, see [1] for -- Regards, Laurent Pinchart