lkml.org 
[lkml]   [2008]   [Jan]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
FromAndi Kleen <>
SubjectRe: [PATCH x86] [4/16] Add /proc/irq/*/spurious to dump the spurious irq debugging state
DateFri, 4 Jan 2008 13:22:01 +0100
ix it:

You mean the new line after local variables?

I never liked this rule. Newlines may make sense on very large functions, but
on short functions they just obscure more. And I actually checked CodingStyle now
and it says nothing about that. So please apply the patch anyways.

-Andi

> > +static int irq_spurious_read(char *page, char **start, off_t off,
> > + int count, int *eof, void *data)
> > +{
> > + struct irq_desc *d = &irq_desc[(long) data];
> > + return sprintf(page, "count %u\n"
> > + "unhandled %u\n"
>
> Ingo
>




\
 
 \ /
  Last update: 2008-01-04 13:29    [from the cache]
©2003-2010