From mboxrd@z Thu Jan 1 00:00:00 1970 From: Laurent Pinchart Date: Wed, 27 Nov 2013 11:29:19 +0000 Subject: Re: [PATCH RFC v2 15/17] ARM: shmobile: Remove non-multiplatform Marzen reference support Message-Id: <5414839.e4PRTtIDTx@avalon> List-Id: References: <1385451139-29686-1-git-send-email-horms+renesas@verge.net.au> <1385451139-29686-16-git-send-email-horms+renesas@verge.net.au> In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: linux-arm-kernel@lists.infradead.org Hi Magnus, On Wednesday 27 November 2013 17:51:16 Magnus Damm wrote: > On Tue, Nov 26, 2013 at 4:32 PM, Simon Horman wrote: > > Now that r8a7779 has CCF support remove the legacy Marzen reference > > Kconfig bits CONFIG_MACH_MARZEN_REFERENCE for the non-multiplatform > > case. > > > > Starting from this commit Marzen board support is always enabled via > > CONFIG_MACH_MARZEN, and CONFIG_ARCH_MULTIPLATFORM is used to select > > between board-marzen.c and board-marzen-reference.c > > > > The file board-marzen-reference.c can no longer be used together with > > the legacy sh-clk clock framework, instead CCF is used. > > > > Based on work for the Koelsch board by Laurent Pinchart. > > > > Cc: Laurent Pinchart > > Signed-off-by: Simon Horman > > > > --- > > v2 > > * Correct changelog to refer to marzen instead of koelsch > > * Do not add typo to Makefile > > I think we need to discuss _when_ to do this in more detail. I know > this follows the same style as Laurent's patches, but I don't think > we're ready to flip the switch yet. I propose that we let DT reference > coexist with multiplatform and non-multiplatform cases for a while to > make testing easier and give time to migrate. I understand your point, but as I've stated in a private e-mail, I believe that having three platforms (DT reference with and without multiplatform, and legacy) for each board is a step backward. It will be error-prone and time consuming. -- Regards, Laurent Pinchart