All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2] module: remove trailing whitespace
@ 2016-10-19 23:12 Anson Jacob
  2016-10-19 23:57 ` Jessica Yu
  0 siblings, 1 reply; 4+ messages in thread
From: Anson Jacob @ 2016-10-19 23:12 UTC (permalink / raw)
  To: rusty; +Cc: Kernel List, Jessica Yu

Fix checkpatch.pl warning:
ERROR: trailing whitespace

Signed-off-by: Anson Jacob <ansonjacob.aj@gmail.com>
---
Changes in v2:
  - Add Checkpatch warning info to the commit message.
 
 include/linux/module.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/linux/module.h b/include/linux/module.h
index 0c3207d..91df94f 100644
--- a/include/linux/module.h
+++ b/include/linux/module.h
@@ -412,7 +412,7 @@ struct module {
 	/* Protected by RCU and/or module_mutex: use rcu_dereference() */
 	struct mod_kallsyms *kallsyms;
 	struct mod_kallsyms core_kallsyms;
-	
+
 	/* Section attributes */
 	struct module_sect_attrs *sect_attrs;
 
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: module: remove trailing whitespace
  2016-10-19 23:12 [PATCH v2] module: remove trailing whitespace Anson Jacob
@ 2016-10-19 23:57 ` Jessica Yu
  0 siblings, 0 replies; 4+ messages in thread
From: Jessica Yu @ 2016-10-19 23:57 UTC (permalink / raw)
  To: Anson Jacob; +Cc: rusty, Kernel List

+++ Anson Jacob [19/10/16 19:12 -0400]:
>Fix checkpatch.pl warning:
>ERROR: trailing whitespace
>
>Signed-off-by: Anson Jacob <ansonjacob.aj@gmail.com>

Thanks for the quick turnaround,

Acked-by: Jessica Yu <jeyu@redhat.com>

>---
>Changes in v2:
>  - Add Checkpatch warning info to the commit message.
>
> include/linux/module.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/include/linux/module.h b/include/linux/module.h
>index 0c3207d..91df94f 100644
>--- a/include/linux/module.h
>+++ b/include/linux/module.h
>@@ -412,7 +412,7 @@ struct module {
> 	/* Protected by RCU and/or module_mutex: use rcu_dereference() */
> 	struct mod_kallsyms *kallsyms;
> 	struct mod_kallsyms core_kallsyms;
>-	
>+
> 	/* Section attributes */
> 	struct module_sect_attrs *sect_attrs;
>
>-- 
>2.7.4
>

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: module: remove trailing whitespace
  2016-10-19 22:11 ` Jessica Yu
@ 2016-10-19 23:05   ` Anson Jacob
  0 siblings, 0 replies; 4+ messages in thread
From: Anson Jacob @ 2016-10-19 23:05 UTC (permalink / raw)
  To: Jessica Yu; +Cc: rusty, linux-kernel

On Wed, Oct 19, 2016 at 06:11:59PM -0400, Jessica Yu wrote:
> It would be nice to mention the exact checkpatch.pl error this fixes,
> and include that in the changelog. Other than that, looks good to me.

Thank you for your comments, will send a revised patch.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: module: remove trailing whitespace
  2016-10-07  5:48 [PATCH] " Anson Jacob
@ 2016-10-19 22:11 ` Jessica Yu
  2016-10-19 23:05   ` Anson Jacob
  0 siblings, 1 reply; 4+ messages in thread
From: Jessica Yu @ 2016-10-19 22:11 UTC (permalink / raw)
  To: Anson Jacob; +Cc: rusty, linux-kernel

+++ Anson Jacob [07/10/16 01:48 -0400]:
>remove whitespace on blank line
>
>Signed-off-by: Anson Jacob <ansonjacob.aj@gmail.com>

It would be nice to mention the exact checkpatch.pl error this fixes,
and include that in the changelog. Other than that, looks good to me.

>---
> include/linux/module.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/include/linux/module.h b/include/linux/module.h
>index 0c3207d..91df94f 100644
>--- a/include/linux/module.h
>+++ b/include/linux/module.h
>@@ -412,7 +412,7 @@ struct module {
> 	/* Protected by RCU and/or module_mutex: use rcu_dereference() */
> 	struct mod_kallsyms *kallsyms;
> 	struct mod_kallsyms core_kallsyms;
>-	
>+
> 	/* Section attributes */
> 	struct module_sect_attrs *sect_attrs;
>
>-- 
>2.7.4
>

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2016-10-19 23:57 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-10-19 23:12 [PATCH v2] module: remove trailing whitespace Anson Jacob
2016-10-19 23:57 ` Jessica Yu
  -- strict thread matches above, loose matches on Subject: below --
2016-10-07  5:48 [PATCH] " Anson Jacob
2016-10-19 22:11 ` Jessica Yu
2016-10-19 23:05   ` Anson Jacob

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.