From mboxrd@z Thu Jan 1 00:00:00 1970 From: Laurent Pinchart Date: Fri, 20 Nov 2015 02:46:50 +0000 Subject: Re: [PATCH 00/04] pinctrl: sh-pfc: r8a7794: DU support Message-Id: <2997631.z1xUhmAZzM@avalon> List-Id: References: <20151117031832.24924.60685.sendpatchset@little-apple> In-Reply-To: <20151117031832.24924.60685.sendpatchset@little-apple> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Magnus Damm Cc: linux-sh@vger.kernel.org, linus.walleij@linaro.org, linux-kernel@vger.kernel.org, koji.matsuoka.xm@renesas.com, linux-gpio@vger.kernel.org, horms@verge.net.au, geert@glider.be Hi Magnus, Thank you for the patch. On Tuesday 17 November 2015 12:18:32 Magnus Damm wrote: > pinctrl: sh-pfc: r8a7794: DU support > > [PATCH 01/04] pinctrl: sh-pfc: r8a7794: Add DU pin groups > [PATCH 02/04] pinctrl: sh-pfc: r8a7794: Separate DU CDE and DISP > [PATCH 03/04] pinctrl: sh-pfc: r8a7794: Add missing dot clock signals > [PATCH 04/04] pinctrl: sh-pfc: r8a7794: Break out ODDF from sync > > These patches take the r8a7794 PFC DU support code from the BSP > and reworks it to fit the r8a7794 ALT board. Tested with the ALT > VGA port - by default PFC is not used however enabling PFC using > an incremental (yet to be posted) patch works well. > > It is worth noting that patch 2-4 modifies the pin groups. This > means that the upstream DT ABI for PFC DU will differ compared > to the unreviewed BSP code. > > In general it is not considered good practice to change the pin > groups and break compatibility since they are part of the DT ABI. > > For this particular case upstream never have had PFC DU support > for r8a7794, so treating the BSP bindings as experimental and > migrate away seems reasonable. If we start considering DT bindings that never went upstream as stable we'll have a big problem. I mean even bigger than the upstream DT bindings stability problem :-) > Signed-off-by: Magnus Damm Wouldn't it make sense to merge the 4 patches together ? > --- > > Written against v4.4-rc1 > > drivers/pinctrl/sh-pfc/pfc-r8a7794.c | 282 +++++++++++++++++++++++++++---- > 1 file changed, 252 insertions(+), 30 deletions(-) -- Regards, Laurent Pinchart