Re: [PATCH v2] ARM: shmobile: r8a7779 Marzen legacy fix V2

[Simon Horman <horms@verge.net.au>]

On Wed, Jul 22, 2015 at 09:34:38AM +0900, Simon Horman wrote:
> On Tue, Jul 14, 2015 at 04:25:22PM +0900, Magnus Damm wrote:
> > From: Magnus Damm <damm+renesas@opensource.se>
> > 
> > Unbreak the r8a7779 Marzen legacy code provided by marzen_defconfig.
> > 
> > As it is today Marzen multiplatform is working, but the legacy code
> > is broken. This patch intends to leave multiplatform as-is but do
> > a simple one-shot fix to unbreak the legacy code base.
> > 
> > Without this patch there is no preset delay and the TWD is defined
> > both in C code and in DT. Solve the delay by invoking shmobile_init_delay()
> > and simply disable TWD in case of building for legacy.
> > 
> > Marzen legacy code will be removed in the near future, but until then
> > we may as well avoid breaking it.
> > 
> > Signed-off-by: Magnus Damm <damm+renesas@opensource.se>
> 
> I currently have this patch queued up.
> However, as "[PATCH 00/03] ARM: shmobile: r8a7779 Marzen legacy removal" is
> also queued up I have it in mind to drop this patch. The reasoning is
> twofold:
> 
> * There seems little benefit to fix legacy code that is being removed
> * There is a tedious merge conflict between fixing and removing the code

I have gone ahead and dropped this patch.