Re: [PATCH] 2.4.30 PicoPower IRQ router

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



A few more style comments below. Nice cleanup compared to the first 
version though. Now let's get it perfect :-)

On Wed, 27 Apr 2005, Sune Mølgaard wrote:

> Domen Puncer wrote:
> > On 26/04/05 21:00 +0200, Sune Mølgaard wrote:
> > 
> > Signed-off-by? And weird indentification, try to use tabs.
> > 
> 
> Signed-off-by Sune Molgaard [email protected]

Proper form would be:  Signed-off-by: Sune Molgaard <[email protected]>


> +static int pirq_pico_set(struct pci_dev *router, struct pci_dev *dev, int
> pirq, int irq)
> +{
> +        outb(0x10+((pirq-1)>>1), 0x24);
   ^^^^^^^^
   Proper indentation depth (8 chars)

> +       unsigned int x;
   ^^^^^^^
   Only 7 char indentation.

This mismatch is found several places in the patch. Besides, the patch 
seems whitespace damaged - the lines seem to be indented with spaces, not 
tabs. Perhaps your email client mangled it?

(note: Chapter 1 of Documentation/CodingStyle describes proper 
 indentation)


> +        switch(device)
> +       {

The opening brace goes on the same line as the switch statement.

See Documentation/CodingStyle Chapter 2


-- 
Jesper Juhl


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

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux