On Oct 26, 2007, at 4:49 AM, Jeff Garzik wrote:
quoted text > 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...
Ok. Not to be a pain, but it would be easier if this one patch might
be easier if it was split down the arch lines so each maintainer can
deal with it.
I say this especially for the ppc code because the intent in 2.6.25
is to actually try and remove some of the files you are fixing up for
their arch/powerpc variants.
- k
-
unsubscribe notice 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/
Messages in current thread:
Re: [PATCH] IA64, PPC, SPARC: minor irq handler cleanups , Kumar Gala , (Fri Oct 26, 10:55 am)