linux-kernel - Re: [PATCH] 8250

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20071119105848.51a20901@weaponx>
Date:	Mon, 19 Nov 2007 10:58:48 -0600
From:	Josh Boyer <jwboyer@...il.com>
To:	Alan Cox <alan@...rguk.ukuu.org.uk>
Cc:	akpm@...l.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] 8250_gsc: Coding style

On Mon, 19 Nov 2007 15:07:51 +0000
Alan Cox <alan@...rguk.ukuu.org.uk> wrote:

You forgot the Signed-off-by line.

josh

> diff -u --new-file --recursive --exclude-from /usr/src/exclude linux.vanilla-2.6.24-rc2-mm1/drivers/serial/8250_gsc.c linux-2.6.24-rc2-mm1/drivers/serial/8250_gsc.c
> --- linux.vanilla-2.6.24-rc2-mm1/drivers/serial/8250_gsc.c	2007-11-16 17:20:48.000000000 +0000
> +++ linux-2.6.24-rc2-mm1/drivers/serial/8250_gsc.c	2007-11-16 18:35:39.000000000 +0000
> @@ -25,8 +25,7 @@
>  
>  #include "8250.h"
>  
> -static int __init 
> -serial_init_chip(struct parisc_device *dev)
> +static int __init serial_init_chip(struct parisc_device *dev)
>  {
>  	struct uart_port port;
>  	unsigned long address;
> @@ -38,18 +37,17 @@
>  		 * what we have here is a missing parent device, so tell
>  		 * the user what they're missing.
>  		 */
> -		if (parisc_parent(dev)->id.hw_type != HPHW_IOA) {
> -			printk(KERN_INFO "Serial: device 0x%lx not configured.\n"
> +		if (parisc_parent(dev)->id.hw_type != HPHW_IOA)
> +			printk(KERN_INFO
> +				"Serial: device 0x%lx not configured.\n"
>  				"Enable support for Wax, Lasi, Asp or Dino.\n",
>  				dev->hpa.start);
> -		}
>  		return -ENODEV;
>  	}
>  
>  	address = dev->hpa.start;
> -	if (dev->id.sversion != 0x8d) {
> +	if (dev->id.sversion != 0x8d)
>  		address += 0x800;
> -	}
>  
>  	memset(&port, 0, sizeof(port));
>  	port.iotype	= UPIO_MEM;
> @@ -63,11 +61,12 @@
>  
>  	err = serial8250_register_port(&port);
>  	if (err < 0) {
> -		printk(KERN_WARNING "serial8250_register_port returned error %d\n", err);
> +		printk(KERN_WARNING
> +			"serial8250_register_port returned error %d\n", err);
>  		iounmap(port.membase);
>  		return err;
>  	}
> -        
> +
>  	return 0;
>  }
>  
> -
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@...r.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/