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