Re: [PATCH] sh: remove voyagergx

[Paul Mundt <lethal@linux-sh.org>]

On Mon, Jan 14, 2008 at 01:51:08AM -0800, Kristoffer Ericson wrote:
> On Mon, 14 Jan 2008 01:35:16 +0900
> Paul Mundt <lethal@linux-sh.org> wrote:
> 
> > On Sun, Jan 13, 2008 at 10:38:14PM +0900, Magnus Damm wrote:
> > > sh: remove voyagergx
> > > 
> > > This patch removes redundant irq handling code together with unused
> > > consistent alloc code. R2D uart setup code is changed to use sm501-regs.h
> > > and unused header files are removed.
> > > 
> > > Signed-off-by: Magnus Damm <damm@igel.co.jp>
> > > ---
> > > 
> > >  arch/sh/Makefile                          |    1 
> > >  arch/sh/boards/renesas/rts7751r2d/irq.c   |    4 
> > >  arch/sh/boards/renesas/rts7751r2d/setup.c |   73 +++---
> > >  arch/sh/cchips/voyagergx/Makefile         |    9 
> > >  arch/sh/cchips/voyagergx/consistent.c     |  121 ----------
> > >  arch/sh/cchips/voyagergx/irq.c            |  101 --------
> > >  include/asm-sh/voyagergx.h                |  341 -----------------------------
> > >  7 files changed, 39 insertions(+), 611 deletions(-)
> > > 
> > Nice, this code has been crap for longer than I can remember (either a
> > testament to how long this code has sucked or that my memory isn't very
> > good, either way), it's good to finally be rid of it. Now if only we
> > could move the hd64461x atrocity in to drivers/mfd, we can be rid of
> > arch/sh/cchips/ entirely.
> 
> Hey! I've just spent a couple of hours merging hd64461 & hd64465 into hd6446x, and now you
> tell me you want to rip it out :D
> 
No, I want to move it somewhere sane and make it behave like the other
MFD drivers. That's been the plan all along, it just hasn't happened yet.