From mboxrd@z Thu Jan 1 00:00:00 1970 From: jacopo mondi Subject: Re: [PATCH v4 6/6] dt-bindings: media: rcar-vin: Clarify optional props Date: Wed, 13 Jun 2018 10:54:55 +0200 Message-ID: <20180613085455.GC4952@w540> 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: multipart/mixed; boundary="===============0835768482596142969==" 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, laurent.pinchart@ideasonboard.com, 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 --===============0835768482596142969== Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="BQPnanjtCNWHyqYD" Content-Disposition: inline --BQPnanjtCNWHyqYD Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Hi Sakari, On Tue, Jun 12, 2018 at 06:45:53PM +0300, Sakari Ailus wrote: > Hi Jacopo, > > 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. > There has been quite some debate on this, and please bear with me here for re-proposing it: I started by removing properties in some DT files for older Renesas board which listed endpoint properties not documented in the VIN's bindings and not parsed by the VIN driver [1] Niklas (but Simon and Geert seems to agree here) opposed to that patch, as those properties where described in 'video-interfaces.txt' and even if not parsed by the current driver implementation, they actually describe hardware. I rebated that only properties listed in the device bindings documentation should actually be used, and having properties not parsed by the driver confuses users, which may expect changing them modifies the interface configuration, which does not happens at the moment. This came out as a middle ground from a discussion with Niklas. As stated in the cover letter if this patch makes someone uncomfortable, feel free to drop it not to hold back the rest of the series which has been well received instead. Thanks j [1] https://www.spinics.net/lists/arm-kernel/msg656302.html > > - 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 > > -- > > 2.7.4 > > > > -- > Sakari Ailus > sakari.ailus@linux.intel.com --BQPnanjtCNWHyqYD Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQIcBAEBAgAGBQJbINvfAAoJEHI0Bo8WoVY860oP/3rcThxRrX7Jg1BEvvHG9TO9 Hm4m9hR0+jhSaAiKYOsaGVdTcsvN3Y6PJ4SNfqiR7QXpkNq+QGY/XRkMGPX59vMS z1QaAQM/RgGu7OM0tyJ3nU3tnzmEqb6YIi9FeRPYLmDMRqywUC28qLxKyNLEBUWa AJ3QMFzBzROHVZSPLw4fV90zZOzlZPNcR0vLKS3j93S/nrBnOvwBwuztVHUVMnyE PNhD6GGYVZ9XiWTEyG/6vles1fDFtSKIiN7NrmL/OWCFx1kdoX1JdEdygpyszwjQ 5+RhVfXl/oUy/oPoR9gh3+t0cDdE4O70G+MMUt4IOKjEVr41Ti7/eWGF9hpaGkF0 hYCtKgl1U8eAmwlaybnIWDAVnuNEe2TFyEXmwx8N8PRQ7eSZBhK5leWNrLidsub3 LTNTxBYy5fpnjHNQEivcTmkKiyh2TUHTUEiGbgCdzE/luhkxWXILJnNcr7si8fLy d8dcSjxfUYd/WeeBS5BCSroxnR6Cs9LzYeh4V3y0wUKFfpu//3zdlz0KLqvg/Y3o u4iHBIYBl8PwX3HIo3wR/9zJOwkoo865aucvlm8+oMvhf23cd1l3PTsy+aGOiWR4 ehXr+x1Xd5d3un7lkdT+12o/+eVmcQjvWrnfKrQRNrwITgGjAUAeHR6Bp4WqYZKP 6lk4qIpMerFDGteEOjkH =Xx+b -----END PGP SIGNATURE----- --BQPnanjtCNWHyqYD-- --===============0835768482596142969== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --===============0835768482596142969==--