Re: [PATCH] This patch to printk.c fixes a few errors reported bycheckpatch.pl

From: Randy Dunlap
Date: Thu Jan 03 2008 - 18:49:49 EST


On Fri, 4 Jan 2008 00:14:46 +0100 Paolo Ciarrocchi wrote:

> This patch to printk.c fixes a few errors reported by checkpatch.pl
>
> Before:
> total: 18 errors, 17 warnings, 1306 lines checked
>
> After:
> total: 1 errors, 17 warnings, 1305 lines checked
>
>
> Signed-off-by: Paolo Ciarrocchi <paolo.ciarrocchi@xxxxxxxxx>
> ---
>
> Adrian, I'm sending this patch to you because according to git shortlog -e
> you are the most active modifier of this file.
>
>
> kernel/printk.c | 25 ++++++++++++-------------
> 1 files changed, 12 insertions(+), 13 deletions(-)
>
> diff --git a/kernel/printk.c b/kernel/printk.c
> index 89011bf..00f784f 100644
> --- a/kernel/printk.c
> +++ b/kernel/printk.c

Hi,

Please read Andrew's "The Perfect Patch," especially the section
about Subject: lines.

http://www.zip.com.au/~akpm/linux/patches/stuff/tpp.txt

---
~Randy
desserts: http://www.xenotime.net/linux/recipes/
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/