Re: [PATCH] IA64, PPC, SPARC: minor irq handler cleanups

[Jeff Garzik <jeff@garzik.org>]

Kumar Gala wrote:
> On Oct 26, 2007, at 4:40 AM, Jeff Garzik wrote:
>
> > ia64/sn/kernel/huberror.c:
> >     - remove pointless void* cast
> >     - add KERN_xxx prefix
> >
> > ia64/sn/pci/tioce_provider.c: start functions at column zero
> >
> > ppc/8xx_io/fec.c: kill prototype, remove extra whitespace
> >
> > ppc/platforms/85xx/*
> >     'irq' argument in irq handler is used purely as a temporary
> >     variable.  Update the function to reflect this usage, changing
> >     the first arg's name from 'irq' to 'dummy'
> >
> > ppc/platforms/sbc82xx.c: ditto
> >
> > sparc/kernel/time.c: mark timer_interrupt() static (from DaveM)
> >
> > Signed-off-by: Jeff Garzik <jgarzik@redhat.com>
> > ---
> >  arch/ia64/sn/kernel/huberror.c               |    4 ++--
> >  arch/ia64/sn/pci/tioce_provider.c            |    6 ++++--
> >  arch/ppc/8xx_io/fec.c                        |    3 +--
> >  arch/ppc/platforms/85xx/mpc8560_ads.c        |    4 +++-
> >  arch/ppc/platforms/85xx/mpc85xx_cds_common.c |    4 +++-
> >  arch/ppc/platforms/85xx/stx_gp3.c            |    4 +++-
> >  arch/ppc/platforms/85xx/tqm85xx.c            |    4 +++-
> >  arch/ppc/platforms/sbc82xx.c                 |    4 +++-
> >  arch/sparc/kernel/time.c                     |    2 +-
> >  9 files changed, 23 insertions(+), 12 deletions(-)
>
> I haven't been following this with detail.  Is the intent that this 
> would be for 2.6.24 or .25?

These patches are all cleanups and minor fixes I found while iterating 
through each interrupt handler in the kernel, in pursuit of a related 
project.

So they can stand by themselves, and can integrate into whatever flow 
maintainers find most comfortable.

I'm not in any rush, but neither would I like them to sit forever...

	Jeff


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/