All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] checkpatch.pl: fix 2 of 2 errors in ./kernel/user.c
@ 2010-12-14 12:55 ` Toralf Förster
  0 siblings, 0 replies; 6+ messages in thread
From: Toralf Förster @ 2010-12-14 12:55 UTC (permalink / raw)
  To: kernel-janitors; +Cc: linux-kernel, Toralf Förster


Signed-off-by: Toralf Förster <toralf.foerster@gmx.de>
---
 kernel/user.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/kernel/user.c b/kernel/user.c
index 2c7d8d5..6db8ed9 100644
--- a/kernel/user.c
+++ b/kernel/user.c
@@ -5,7 +5,7 @@
  *
  * We have a per-user structure to keep track of how many
  * processes, files etc the user has claimed, in order to be
- * able to have per-user limits for system resources. 
+ * able to have per-user limits for system resources.
  */
 
 #include <linux/init.h>
@@ -181,7 +181,7 @@ static int __init uid_cache_init(void)
 	uid_cachep = kmem_cache_create("uid_cache", sizeof(struct user_struct),
 			0, SLAB_HWCACHE_ALIGN|SLAB_PANIC, NULL);
 
-	for(n = 0; n < UIDHASH_SZ; ++n)
+	for (n = 0; n < UIDHASH_SZ; ++n)
 		INIT_HLIST_HEAD(init_user_ns.uidhash_table + n);
 
 	/* Insert the root user immediately (init already runs as root) */
-- 
1.7.3.3


^ permalink raw reply related	[flat|nested] 6+ messages in thread
* [PATCH] checkpatch.pl: fix 2 of 2 errors in ./kernel/user.c
@ 2010-12-14 12:55 ` Toralf Förster
  0 siblings, 0 replies; 6+ messages in thread
From: Toralf Förster @ 2010-12-14 12:55 UTC (permalink / raw)
  To: kernel-janitors; +Cc: linux-kernel, Toralf Förster


Signed-off-by: Toralf Förster <toralf.foerster@gmx.de>
---
 kernel/user.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/kernel/user.c b/kernel/user.c
index 2c7d8d5..6db8ed9 100644
--- a/kernel/user.c
+++ b/kernel/user.c
@@ -5,7 +5,7 @@
  *
  * We have a per-user structure to keep track of how many
  * processes, files etc the user has claimed, in order to be
- * able to have per-user limits for system resources. 
+ * able to have per-user limits for system resources.
  */
 
 #include <linux/init.h>
@@ -181,7 +181,7 @@ static int __init uid_cache_init(void)
 	uid_cachep = kmem_cache_create("uid_cache", sizeof(struct user_struct),
 			0, SLAB_HWCACHE_ALIGN|SLAB_PANIC, NULL);
 
-	for(n = 0; n < UIDHASH_SZ; ++n)
+	for (n = 0; n < UIDHASH_SZ; ++n)
 		INIT_HLIST_HEAD(init_user_ns.uidhash_table + n);
 
 	/* Insert the root user immediately (init already runs as root) */
-- 
1.7.3.3

--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply related	[flat|nested] 6+ messages in thread
* [PATCH] checkpatch.pl: fix 2 of 2 errors in ./kernek/panic.c
  2010-12-14 12:55 ` Toralf Förster
@ 2010-12-14 13:33 ` Toralf Förster
  -1 siblings, 0 replies; 6+ messages in thread
From: Toralf Förster @ 2010-12-14 13:33 UTC (permalink / raw)
  To: kernel-janitors; +Cc: akpm, linux-kernel, Toralf Förster


Signed-off-by: Toralf Förster <toralf.foerster@gmx.de>
---
 kernel/panic.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/kernel/panic.c b/kernel/panic.c
index 4c13b1a..13623e4 100644
--- a/kernel/panic.c
+++ b/kernel/panic.c
@@ -56,7 +56,7 @@ EXPORT_SYMBOL(panic_blink);
  *
  *	This function never returns.
  */
-NORET_TYPE void panic(const char * fmt, ...)
+NORET_TYPE void panic(const char *fmt, ...)
 {
 	static char buf[1024];
 	va_list args;
@@ -75,7 +75,7 @@ NORET_TYPE void panic(const char * fmt, ...)
 	va_start(args, fmt);
 	vsnprintf(buf, sizeof(buf), fmt, args);
 	va_end(args);
-	printk(KERN_EMERG "Kernel panic - not syncing: %s\n",buf);
+	printk(KERN_EMERG "Kernel panic - not syncing: %s\n", buf);
 #ifdef CONFIG_DEBUG_BUGVERBOSE
 	dump_stack();
 #endif
-- 
1.7.3.3


^ permalink raw reply related	[flat|nested] 6+ messages in thread
* [PATCH] checkpatch.pl: fix 2 of 2 errors in ./kernek/panic.c
@ 2010-12-14 13:33 ` Toralf Förster
  0 siblings, 0 replies; 6+ messages in thread
From: Toralf Förster @ 2010-12-14 13:33 UTC (permalink / raw)
  To: kernel-janitors; +Cc: akpm, linux-kernel, Toralf Förster


Signed-off-by: Toralf Förster <toralf.foerster@gmx.de>
---
 kernel/panic.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/kernel/panic.c b/kernel/panic.c
index 4c13b1a..13623e4 100644
--- a/kernel/panic.c
+++ b/kernel/panic.c
@@ -56,7 +56,7 @@ EXPORT_SYMBOL(panic_blink);
  *
  *	This function never returns.
  */
-NORET_TYPE void panic(const char * fmt, ...)
+NORET_TYPE void panic(const char *fmt, ...)
 {
 	static char buf[1024];
 	va_list args;
@@ -75,7 +75,7 @@ NORET_TYPE void panic(const char * fmt, ...)
 	va_start(args, fmt);
 	vsnprintf(buf, sizeof(buf), fmt, args);
 	va_end(args);
-	printk(KERN_EMERG "Kernel panic - not syncing: %s\n",buf);
+	printk(KERN_EMERG "Kernel panic - not syncing: %s\n", buf);
 #ifdef CONFIG_DEBUG_BUGVERBOSE
 	dump_stack();
 #endif
-- 
1.7.3.3

--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply related	[flat|nested] 6+ messages in thread
* Re: [PATCH] checkpatch.pl: fix 2 of 2 errors in ./kernek/panic.c
  2010-12-14 13:33 ` Toralf Förster
@ 2010-12-14 16:17   ` Thiago Farina
  -1 siblings, 0 replies; 6+ messages in thread
From: Thiago Farina @ 2010-12-14 16:17 UTC (permalink / raw)
  To: Toralf Förster; +Cc: kernel-janitors, akpm, linux-kernel

2010/12/14 Toralf Förster <toralf.foerster@gmx.de>:
>
> Signed-off-by: Toralf Förster <toralf.foerster@gmx.de>
> ---
>  kernel/panic.c |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/kernel/panic.c b/kernel/panic.c
> index 4c13b1a..13623e4 100644
> --- a/kernel/panic.c
> +++ b/kernel/panic.c
> @@ -56,7 +56,7 @@ EXPORT_SYMBOL(panic_blink);
>  *
>  *     This function never returns.
>  */
> -NORET_TYPE void panic(const char * fmt, ...)
> +NORET_TYPE void panic(const char *fmt, ...)
>  {
>        static char buf[1024];
>        va_list args;
> @@ -75,7 +75,7 @@ NORET_TYPE void panic(const char * fmt, ...)
>        va_start(args, fmt);
>        vsnprintf(buf, sizeof(buf), fmt, args);
>        va_end(args);
> -       printk(KERN_EMERG "Kernel panic - not syncing: %s\n",buf);
> +       printk(KERN_EMERG "Kernel panic - not syncing: %s\n", buf);
>  #ifdef CONFIG_DEBUG_BUGVERBOSE
>        dump_stack();
>  #endif

Can you put in the change log message the description of those errors
and resend?  Looking at this patch it seems they are just whitespace
changes or something very trivial like this.

The subject line could be something more like:
kernek/panic.c: Fix two checkpatch.pl errors:

^ permalink raw reply	[flat|nested] 6+ messages in thread
* Re: [PATCH] checkpatch.pl: fix 2 of 2 errors in ./kernek/panic.c
@ 2010-12-14 16:17   ` Thiago Farina
  0 siblings, 0 replies; 6+ messages in thread
From: Thiago Farina @ 2010-12-14 16:17 UTC (permalink / raw)
  To: Toralf Förster; +Cc: kernel-janitors, akpm, linux-kernel

2010/12/14 Toralf Förster <toralf.foerster@gmx.de>:
>
> Signed-off-by: Toralf Förster <toralf.foerster@gmx.de>
> ---
>  kernel/panic.c |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/kernel/panic.c b/kernel/panic.c
> index 4c13b1a..13623e4 100644
> --- a/kernel/panic.c
> +++ b/kernel/panic.c
> @@ -56,7 +56,7 @@ EXPORT_SYMBOL(panic_blink);
>  *
>  *     This function never returns.
>  */
> -NORET_TYPE void panic(const char * fmt, ...)
> +NORET_TYPE void panic(const char *fmt, ...)
>  {
>        static char buf[1024];
>        va_list args;
> @@ -75,7 +75,7 @@ NORET_TYPE void panic(const char * fmt, ...)
>        va_start(args, fmt);
>        vsnprintf(buf, sizeof(buf), fmt, args);
>        va_end(args);
> -       printk(KERN_EMERG "Kernel panic - not syncing: %s\n",buf);
> +       printk(KERN_EMERG "Kernel panic - not syncing: %s\n", buf);
>  #ifdef CONFIG_DEBUG_BUGVERBOSE
>        dump_stack();
>  #endif

Can you put in the change log message the description of those errors
and resend?  Looking at this patch it seems they are just whitespace
changes or something very trivial like this.

The subject line could be something more like:
kernek/panic.c: Fix two checkpatch.pl errors:

^ permalink raw reply	[flat|nested] 6+ messages in thread
end of thread, other threads:[~2010-12-14 16:17 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-12-14 12:55 [PATCH] checkpatch.pl: fix 2 of 2 errors in ./kernel/user.c Toralf Förster
2010-12-14 12:55 ` Toralf Förster
2010-12-14 13:33 [PATCH] checkpatch.pl: fix 2 of 2 errors in ./kernek/panic.c Toralf Förster
2010-12-14 13:33 ` Toralf Förster
2010-12-14 16:17 ` Thiago Farina
2010-12-14 16:17   ` Thiago Farina

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.