From mboxrd@z Thu Jan 1 00:00:00 1970 From: Laurent Pinchart Subject: Re: [PATCH 01/10] dt-bindings: display: renesas: Add R-Car LVDS encoder DT bindings Date: Mon, 15 Jan 2018 10:26:56 +0200 Message-ID: <2172881.iFUds2YuFP@avalon> References: <20180112005858.26472-1-laurent.pinchart+renesas@ideasonboard.com> <2213619.LpHAiGQTjv@avalon> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Return-path: In-Reply-To: Sender: linux-renesas-soc-owner@vger.kernel.org To: Geert Uytterhoeven Cc: Simon Horman , Laurent Pinchart , DRI Development , Linux-Renesas , Sergei Shtylyov , devicetree@vger.kernel.org List-Id: devicetree@vger.kernel.org Hi Geert, On Monday, 15 January 2018 10:05:59 EET Geert Uytterhoeven wrote: > On Mon, Jan 15, 2018 at 7:59 AM, Laurent Pinchart wrote: > > On Monday, 15 January 2018 08:55:29 EET Simon Horman wrote: > >> On Fri, Jan 12, 2018 at 03:29:48PM +0200, Laurent Pinchart wrote: > >>> On Friday, 12 January 2018 12:13:18 EET Geert Uytterhoeven wrote: > >>>> As this is a new binding, please use "renesas,-lvds". > >>> > >>> I've recently been thinking that we made the wrong choice, - > >>> would be better in my opinion as it aligns with -, but > >>> it's too late to change that, so I'll change the order here. > >> > >> My recollection is that in the beginning we had a bit of a mixture but > >> leaned towards -, which made sense in my opinion. However, after > >> some discussion it was agreed that the best-practice for upstream was to > >> use -. Unless that situation has changed lets stock with using > >> - for new bindings. > > > > Sure, that was my plan, and it seems I failed to explain it clearly. I too > > believe that - would be better, but as we have standardized on > > - and as there's no strong reason to reconsider that decision > > at the moment, the next version of this patch will use -. It was > > a mistake in v1, not an attempt to change what we had agreed on. > > Note that I believe you have to consider the full tuple > ",-" to see the light: is more closely tied to > , than is. I suppose there are pros and cons for both options :-) I see more as a version qualifier. -- Regards, Laurent Pinchart