All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 149/199] arch/ia64/hp/sim/boot/ssc.h: Checkpatch cleanup
@ 2010-05-23 20:01 Andrea Gelmini
  2010-05-23 20:01 ` [PATCH 159/199] arch/m68k/hp300/hp300map.map: " Andrea Gelmini
                   ` (9 more replies)
  0 siblings, 10 replies; 15+ messages in thread
From: Andrea Gelmini @ 2010-05-23 20:01 UTC (permalink / raw)
  To: linux-ia64

arch/ia64/hp/sim/boot/ssc.h:33: WARNING: space prohibited between function name and open parenthesis '('

Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
---
 arch/ia64/hp/sim/boot/ssc.h |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/ia64/hp/sim/boot/ssc.h b/arch/ia64/hp/sim/boot/ssc.h
index 3b94c03..87400ed 100644
--- a/arch/ia64/hp/sim/boot/ssc.h
+++ b/arch/ia64/hp/sim/boot/ssc.h
@@ -30,6 +30,6 @@
 /*
  * Simulator system call.
  */
-extern long ssc (long arg0, long arg1, long arg2, long arg3, int nr);
+extern long ssc(long arg0, long arg1, long arg2, long arg3, int nr);
 
 #endif /* ssc_h */
-- 
1.7.1.251.gf80a2


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

* [PATCH 159/199] arch/m68k/hp300/hp300map.map: Checkpatch cleanup
  2010-05-23 20:01 [PATCH 149/199] arch/ia64/hp/sim/boot/ssc.h: Checkpatch cleanup Andrea Gelmini
@ 2010-05-23 20:01 ` Andrea Gelmini
  2010-06-06 11:08   ` Geert Uytterhoeven
  2010-06-06 11:30   ` Andreas Schwab
  2010-05-23 20:01 ` [PATCH 160/199] arch/m68k/sun3/leds.c: " Andrea Gelmini
                   ` (8 subsequent siblings)
  9 siblings, 2 replies; 15+ messages in thread
From: Andrea Gelmini @ 2010-05-23 20:01 UTC (permalink / raw)
  To: andrea.gelmini
  Cc: Philip Blundell, Geert Uytterhoeven, Roman Zippel, linux-m68k

arch/m68k/hp300/hp300map.map:185: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:186: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:187: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:188: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:189: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:190: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:191: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:192: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:193: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:194: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:195: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:196: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:197: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:198: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:199: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:200: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:201: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:202: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:203: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:204: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:205: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:206: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:207: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:208: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:209: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:210: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:211: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:212: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:213: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:214: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:215: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:216: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:217: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:218: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:219: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:220: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:221: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:222: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:223: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:224: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:225: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:226: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:227: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:228: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:229: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:230: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:231: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:232: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:233: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:234: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:235: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:236: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:237: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:238: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:239: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:240: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:241: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:242: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:243: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:244: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:245: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:246: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:247: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:248: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:249: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:250: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:251: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
arch/m68k/hp300/hp300map.map:252: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8

Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
---
 arch/m68k/hp300/hp300map.map |  136 +++++++++++++++++++++---------------------
 1 files changed, 68 insertions(+), 68 deletions(-)

diff --git a/arch/m68k/hp300/hp300map.map b/arch/m68k/hp300/hp300map.map
index 6b45f0a..621c076 100644
--- a/arch/m68k/hp300/hp300map.map
+++ b/arch/m68k/hp300/hp300map.map
@@ -182,71 +182,71 @@ string Prior = "\033[5~"
 string Next = "\033[6~"
 string Macro = "\033[M"
 string Pause = "\033[P"
-compose '`' 'A' to 'À'
-compose '`' 'a' to 'à'
-compose '\'' 'A' to 'Á'
-compose '\'' 'a' to 'á'
-compose '^' 'A' to 'Â'
-compose '^' 'a' to 'â'
-compose '~' 'A' to 'Ã'
-compose '~' 'a' to 'ã'
-compose '"' 'A' to 'Ä'
-compose '"' 'a' to 'ä'
-compose 'O' 'A' to 'Å'
-compose 'o' 'a' to 'å'
-compose '0' 'A' to 'Å'
-compose '0' 'a' to 'å'
-compose 'A' 'A' to 'Å'
-compose 'a' 'a' to 'å'
-compose 'A' 'E' to 'Æ'
-compose 'a' 'e' to 'æ'
-compose ',' 'C' to 'Ç'
-compose ',' 'c' to 'ç'
-compose '`' 'E' to 'È'
-compose '`' 'e' to 'è'
-compose '\'' 'E' to 'É'
-compose '\'' 'e' to 'é'
-compose '^' 'E' to 'Ê'
-compose '^' 'e' to 'ê'
-compose '"' 'E' to 'Ë'
-compose '"' 'e' to 'ë'
-compose '`' 'I' to 'Ì'
-compose '`' 'i' to 'ì'
-compose '\'' 'I' to 'Í'
-compose '\'' 'i' to 'í'
-compose '^' 'I' to 'Î'
-compose '^' 'i' to 'î'
-compose '"' 'I' to 'Ï'
-compose '"' 'i' to 'ï'
-compose '-' 'D' to 'Ð'
-compose '-' 'd' to 'ð'
-compose '~' 'N' to 'Ñ'
-compose '~' 'n' to 'ñ'
-compose '`' 'O' to 'Ò'
-compose '`' 'o' to 'ò'
-compose '\'' 'O' to 'Ó'
-compose '\'' 'o' to 'ó'
-compose '^' 'O' to 'Ô'
-compose '^' 'o' to 'ô'
-compose '~' 'O' to 'Õ'
-compose '~' 'o' to 'õ'
-compose '"' 'O' to 'Ö'
-compose '"' 'o' to 'ö'
-compose '/' 'O' to 'Ø'
-compose '/' 'o' to 'ø'
-compose '`' 'U' to 'Ù'
-compose '`' 'u' to 'ù'
-compose '\'' 'U' to 'Ú'
-compose '\'' 'u' to 'ú'
-compose '^' 'U' to 'Û'
-compose '^' 'u' to 'û'
-compose '"' 'U' to 'Ü'
-compose '"' 'u' to 'ü'
-compose '\'' 'Y' to 'Ý'
-compose '\'' 'y' to 'ý'
-compose 'T' 'H' to 'Þ'
-compose 't' 'h' to 'þ'
-compose 's' 's' to 'ß'
-compose '"' 'y' to 'ÿ'
-compose 's' 'z' to 'ß'
-compose 'i' 'j' to 'ÿ'
+compose '`' 'A' to 'À'
+compose '`' 'a' to 'à'
+compose '\'' 'A' to 'Á'
+compose '\'' 'a' to 'á'
+compose '^' 'A' to 'Â'
+compose '^' 'a' to 'â'
+compose '~' 'A' to 'Ã'
+compose '~' 'a' to 'ã'
+compose '"' 'A' to 'Ä'
+compose '"' 'a' to 'ä'
+compose 'O' 'A' to 'Ã…'
+compose 'o' 'a' to 'Ã¥'
+compose '0' 'A' to 'Ã…'
+compose '0' 'a' to 'Ã¥'
+compose 'A' 'A' to 'Ã…'
+compose 'a' 'a' to 'Ã¥'
+compose 'A' 'E' to 'Æ'
+compose 'a' 'e' to 'æ'
+compose ',' 'C' to 'Ç'
+compose ',' 'c' to 'ç'
+compose '`' 'E' to 'È'
+compose '`' 'e' to 'è'
+compose '\'' 'E' to 'É'
+compose '\'' 'e' to 'é'
+compose '^' 'E' to 'Ê'
+compose '^' 'e' to 'ê'
+compose '"' 'E' to 'Ë'
+compose '"' 'e' to 'ë'
+compose '`' 'I' to 'Ì'
+compose '`' 'i' to 'ì'
+compose '\'' 'I' to 'Í'
+compose '\'' 'i' to 'í'
+compose '^' 'I' to 'ÃŽ'
+compose '^' 'i' to 'î'
+compose '"' 'I' to 'Ï'
+compose '"' 'i' to 'ï'
+compose '-' 'D' to 'Äž'
+compose '-' 'd' to 'ÄŸ'
+compose '~' 'N' to 'Ñ'
+compose '~' 'n' to 'ñ'
+compose '`' 'O' to 'Ã’'
+compose '`' 'o' to 'ò'
+compose '\'' 'O' to 'Ó'
+compose '\'' 'o' to 'ó'
+compose '^' 'O' to 'Ô'
+compose '^' 'o' to 'ô'
+compose '~' 'O' to 'Õ'
+compose '~' 'o' to 'õ'
+compose '"' 'O' to 'Ö'
+compose '"' 'o' to 'ö'
+compose '/' 'O' to 'Ø'
+compose '/' 'o' to 'ø'
+compose '`' 'U' to 'Ù'
+compose '`' 'u' to 'ù'
+compose '\'' 'U' to 'Ú'
+compose '\'' 'u' to 'ú'
+compose '^' 'U' to 'Û'
+compose '^' 'u' to 'û'
+compose '"' 'U' to 'Ü'
+compose '"' 'u' to 'ü'
+compose '\'' 'Y' to 'İ'
+compose '\'' 'y' to 'ı'
+compose 'T' 'H' to 'Åž'
+compose 't' 'h' to 'ÅŸ'
+compose 's' 's' to 'ß'
+compose '"' 'y' to 'ÿ'
+compose 's' 'z' to 'ß'
+compose 'i' 'j' to 'ÿ'
-- 
1.7.1.251.gf80a2

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

* [PATCH 160/199] arch/m68k/sun3/leds.c: Checkpatch cleanup
  2010-05-23 20:01 [PATCH 149/199] arch/ia64/hp/sim/boot/ssc.h: Checkpatch cleanup Andrea Gelmini
  2010-05-23 20:01 ` [PATCH 159/199] arch/m68k/hp300/hp300map.map: " Andrea Gelmini
@ 2010-05-23 20:01 ` Andrea Gelmini
  2010-05-23 20:01 ` [PATCH 172/199] arch/parisc/kernel/pa7300lc.c: " Andrea Gelmini
                   ` (7 subsequent siblings)
  9 siblings, 0 replies; 15+ messages in thread
From: Andrea Gelmini @ 2010-05-23 20:01 UTC (permalink / raw)
  To: andrea.gelmini; +Cc: Sam Creasey, Geert Uytterhoeven, Roman Zippel, linux-m68k

arch/m68k/sun3/leds.c:10: ERROR: code indent should use tabs where possible
arch/m68k/sun3/leds.c:11: ERROR: space required after that ',' (ctx:VxV)

Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
---
 arch/m68k/sun3/leds.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/m68k/sun3/leds.c b/arch/m68k/sun3/leds.c
index a3e9484..aad2e0a 100644
--- a/arch/m68k/sun3/leds.c
+++ b/arch/m68k/sun3/leds.c
@@ -7,7 +7,7 @@ void sun3_leds(unsigned char byte)
 	unsigned char dfc;
 
 	GET_DFC(dfc);
-        SET_DFC(FC_CONTROL);
-	SET_CONTROL_BYTE(AC_LEDS,byte);
+	SET_DFC(FC_CONTROL);
+	SET_CONTROL_BYTE(AC_LEDS, byte);
 	SET_DFC(dfc);
 }
-- 
1.7.1.251.gf80a2

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

* [PATCH 172/199] arch/parisc/kernel/pa7300lc.c: Checkpatch cleanup
  2010-05-23 20:01 [PATCH 149/199] arch/ia64/hp/sim/boot/ssc.h: Checkpatch cleanup Andrea Gelmini
  2010-05-23 20:01 ` [PATCH 159/199] arch/m68k/hp300/hp300map.map: " Andrea Gelmini
  2010-05-23 20:01 ` [PATCH 160/199] arch/m68k/sun3/leds.c: " Andrea Gelmini
@ 2010-05-23 20:01 ` Andrea Gelmini
  2010-05-23 20:02 ` [PATCH 173/199] arch/parisc/kernel/sys32.h: " Andrea Gelmini
                   ` (6 subsequent siblings)
  9 siblings, 0 replies; 15+ messages in thread
From: Andrea Gelmini @ 2010-05-23 20:01 UTC (permalink / raw)
  To: andrea.gelmini
  Cc: Kyle McMartin, Helge Deller, James E.J. Bottomley, linux-parisc

arch/parisc/kernel/pa7300lc.c:3: ERROR: trailing whitespace

Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
---
 arch/parisc/kernel/pa7300lc.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/parisc/kernel/pa7300lc.c b/arch/parisc/kernel/pa7300lc.c
index 8a89780..fa6ca3a 100644
--- a/arch/parisc/kernel/pa7300lc.c
+++ b/arch/parisc/kernel/pa7300lc.c
@@ -1,6 +1,6 @@
 /*
  *   linux/arch/parisc/kernel/pa7300lc.c
- *	- PA7300LC-specific functions	
+ *	- PA7300LC-specific functions
  *
  *   Copyright (C) 2000 Philipp Rumpf */
 
-- 
1.7.1.251.gf80a2


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

* [PATCH 173/199] arch/parisc/kernel/sys32.h: Checkpatch cleanup
  2010-05-23 20:01 [PATCH 149/199] arch/ia64/hp/sim/boot/ssc.h: Checkpatch cleanup Andrea Gelmini
                   ` (2 preceding siblings ...)
  2010-05-23 20:01 ` [PATCH 172/199] arch/parisc/kernel/pa7300lc.c: " Andrea Gelmini
@ 2010-05-23 20:02 ` Andrea Gelmini
  2010-05-23 20:02 ` [PATCH 174/199] arch/parisc/lib/io.c: " Andrea Gelmini
                   ` (5 subsequent siblings)
  9 siblings, 0 replies; 15+ messages in thread
From: Andrea Gelmini @ 2010-05-23 20:02 UTC (permalink / raw)
  To: andrea.gelmini
  Cc: Kyle McMartin, Helge Deller, James E.J. Bottomley, linux-parisc

arch/parisc/kernel/sys32.h:1: ERROR: trailing whitespace
arch/parisc/kernel/sys32.h:33: WARNING: space prohibited between function name and open parenthesis '('

Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
---
 arch/parisc/kernel/sys32.h |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/parisc/kernel/sys32.h b/arch/parisc/kernel/sys32.h
index 06c2090..4bcda8d 100644
--- a/arch/parisc/kernel/sys32.h
+++ b/arch/parisc/kernel/sys32.h
@@ -1,4 +1,4 @@
-/* 
+/*
  *    Copyright (C) 2002 Richard Hirst <rhirst at parisc-linux.org>
  *    Copyright (C) 2003 James Bottomley <jejb at parisc-linux.org>
  *    Copyright (C) 2003 Randolph Chung <tausq with parisc-linux.org>
@@ -30,7 +30,7 @@
     mm_segment_t old_fs = get_fs(); \
     set_fs(KERNEL_DS); \
     ret = syscall(args); \
-    set_fs (old_fs); \
+    set_fs(old_fs); \
 }
 
 #ifdef CONFIG_COMPAT
-- 
1.7.1.251.gf80a2


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

* [PATCH 174/199] arch/parisc/lib/io.c: Checkpatch cleanup
  2010-05-23 20:01 [PATCH 149/199] arch/ia64/hp/sim/boot/ssc.h: Checkpatch cleanup Andrea Gelmini
                   ` (3 preceding siblings ...)
  2010-05-23 20:02 ` [PATCH 173/199] arch/parisc/kernel/sys32.h: " Andrea Gelmini
@ 2010-05-23 20:02 ` Andrea Gelmini
  2010-05-23 20:02 ` [PATCH 175/199] arch/parisc/lib/memset.c: " Andrea Gelmini
                   ` (4 subsequent siblings)
  9 siblings, 0 replies; 15+ messages in thread
From: Andrea Gelmini @ 2010-05-23 20:02 UTC (permalink / raw)
  To: andrea.gelmini
  Cc: Kyle McMartin, Helge Deller, James E.J. Bottomley, linux-parisc

arch/parisc/lib/io.c:55: ERROR: trailing whitespace
arch/parisc/lib/io.c:68: ERROR: trailing statements should be on next line
arch/parisc/lib/io.c:117: WARNING: braces {} are not necessary for single statement blocks
arch/parisc/lib/io.c:126: WARNING: space prohibited between function name and open parenthesis '('
arch/parisc/lib/io.c:166: WARNING: space prohibited between function name and open parenthesis '('
arch/parisc/lib/io.c:172: ERROR: trailing whitespace
arch/parisc/lib/io.c:175: ERROR: trailing whitespace
arch/parisc/lib/io.c:176: ERROR: that open brace { should be on the previous line
arch/parisc/lib/io.c:179: ERROR: spaces required around that '>=' (ctx:VxV)
arch/parisc/lib/io.c:180: ERROR: trailing whitespace
arch/parisc/lib/io.c:187: WARNING: braces {} are not necessary for single statement blocks
arch/parisc/lib/io.c:191: ERROR: trailing whitespace
arch/parisc/lib/io.c:196: ERROR: spaces required around that '>=' (ctx:VxV)
arch/parisc/lib/io.c:197: ERROR: trailing whitespace
arch/parisc/lib/io.c:204: WARNING: braces {} are not necessary for single statement blocks
arch/parisc/lib/io.c:208: ERROR: trailing whitespace
arch/parisc/lib/io.c:214: ERROR: trailing whitespace
arch/parisc/lib/io.c:218: ERROR: that open brace { should be on the previous line
arch/parisc/lib/io.c:238: WARNING: space prohibited between function name and open parenthesis '('
arch/parisc/lib/io.c:244: ERROR: trailing whitespace
arch/parisc/lib/io.c:247: ERROR: trailing whitespace
arch/parisc/lib/io.c:248: ERROR: that open brace { should be on the previous line
arch/parisc/lib/io.c:251: ERROR: that open brace { should be on the previous line
arch/parisc/lib/io.c:257: ERROR: trailing whitespace
arch/parisc/lib/io.c:260: ERROR: trailing whitespace
arch/parisc/lib/io.c:264: ERROR: trailing whitespace
arch/parisc/lib/io.c:265: ERROR: that open brace { should be on the previous line
arch/parisc/lib/io.c:276: ERROR: trailing whitespace
arch/parisc/lib/io.c:282: ERROR: that open brace { should be on the previous line
arch/parisc/lib/io.c:293: ERROR: trailing whitespace
arch/parisc/lib/io.c:297: ERROR: that open brace { should be on the previous line
arch/parisc/lib/io.c:318: ERROR: "foo * bar" should be "foo *bar"
arch/parisc/lib/io.c:336: WARNING: space prohibited between function name and open parenthesis '('
arch/parisc/lib/io.c:342: ERROR: trailing whitespace
arch/parisc/lib/io.c:345: ERROR: trailing whitespace
arch/parisc/lib/io.c:346: ERROR: that open brace { should be on the previous line
arch/parisc/lib/io.c:349: ERROR: spaces required around that '>=' (ctx:VxV)
arch/parisc/lib/io.c:356: WARNING: braces {} are not necessary for single statement blocks
arch/parisc/lib/io.c:357: ERROR: "(foo*)" should be "(foo *)"
arch/parisc/lib/io.c:360: ERROR: trailing whitespace
arch/parisc/lib/io.c:362: ERROR: trailing whitespace
arch/parisc/lib/io.c:363: ERROR: "(foo*)" should be "(foo *)"
arch/parisc/lib/io.c:366: ERROR: trailing whitespace
arch/parisc/lib/io.c:367: ERROR: spaces required around that '>=' (ctx:VxV)
arch/parisc/lib/io.c:374: WARNING: braces {} are not necessary for single statement blocks
arch/parisc/lib/io.c:378: ERROR: trailing whitespace
arch/parisc/lib/io.c:379: ERROR: trailing whitespace
arch/parisc/lib/io.c:382: ERROR: trailing whitespace
arch/parisc/lib/io.c:386: ERROR: that open brace { should be on the previous line
arch/parisc/lib/io.c:392: ERROR: code indent should use tabs where possible
arch/parisc/lib/io.c:395: WARNING: space prohibited between function name and open parenthesis '('
arch/parisc/lib/io.c:397: ERROR: trailing whitespace
arch/parisc/lib/io.c:408: WARNING: space prohibited between function name and open parenthesis '('
arch/parisc/lib/io.c:414: ERROR: trailing whitespace
arch/parisc/lib/io.c:417: ERROR: trailing whitespace
arch/parisc/lib/io.c:418: ERROR: that open brace { should be on the previous line
arch/parisc/lib/io.c:421: ERROR: that open brace { should be on the previous line
arch/parisc/lib/io.c:427: ERROR: trailing whitespace
arch/parisc/lib/io.c:430: ERROR: trailing whitespace
arch/parisc/lib/io.c:433: ERROR: trailing whitespace
arch/parisc/lib/io.c:434: ERROR: that open brace { should be on the previous line
arch/parisc/lib/io.c:438: WARNING: space prohibited between function name and open parenthesis '('
arch/parisc/lib/io.c:442: WARNING: space prohibited between function name and open parenthesis '('
arch/parisc/lib/io.c:452: ERROR: that open brace { should be on the previous line
arch/parisc/lib/io.c:456: WARNING: space prohibited between function name and open parenthesis '('
arch/parisc/lib/io.c:460: WARNING: space prohibited between function name and open parenthesis '('
arch/parisc/lib/io.c:464: ERROR: trailing whitespace
arch/parisc/lib/io.c:468: ERROR: that open brace { should be on the previous line
arch/parisc/lib/io.c:472: WARNING: space prohibited between function name and open parenthesis '('
arch/parisc/lib/io.c:478: WARNING: space prohibited between function name and open parenthesis '('

Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
---
 arch/parisc/lib/io.c |  155 ++++++++++++++++++++++----------------------------
 1 files changed, 68 insertions(+), 87 deletions(-)

diff --git a/arch/parisc/lib/io.c b/arch/parisc/lib/io.c
index 7c1406f..c6c9ddb 100644
--- a/arch/parisc/lib/io.c
+++ b/arch/parisc/lib/io.c
@@ -52,7 +52,7 @@ void memcpy_toio(volatile void __iomem *dst, const void *src, int count)
 */
 void memcpy_fromio(void *dst, const volatile void __iomem *src, int count)
 {
-	/* first compare alignment of src/dst */ 
+	/* first compare alignment of src/dst */
 	if ( (((unsigned long)dst ^ (unsigned long)src) & 1) || (count < 2) )
 		goto bytecopy;
 
@@ -114,16 +114,15 @@ void memset_io(volatile void __iomem *addr, unsigned char val, int count)
 		addr += 4;
 		count -= 4;
 	}
-	while (count--) {
+	while (count--)
 		writeb(val, addr++);
-	}
 }
 
 /*
  * Read COUNT 8-bit bytes from port PORT into memory starting at
  * SRC.
  */
-void insb (unsigned long port, void *dst, unsigned long count)
+void insb(unsigned long port, void *dst, unsigned long count)
 {
 	unsigned char *p;
 
@@ -163,60 +162,56 @@ void insb (unsigned long port, void *dst, unsigned long count)
  * the interfaces seems to be slow: just using the inlined version
  * of the inw() breaks things.
  */
-void insw (unsigned long port, void *dst, unsigned long count)
+void insw(unsigned long port, void *dst, unsigned long count)
 {
 	unsigned int l = 0, l2;
 	unsigned char *p;
 
 	p = (unsigned char *)dst;
-	
+
 	if (!count)
 		return;
-	
-	switch (((unsigned long)p) & 0x3)
-	{
+
+	switch (((unsigned long)p) & 0x3) {
 	 case 0x00:			/* Buffer 32-bit aligned */
-		while (count>=2) {
-			
+		while (count >= 2) {
+
 			count -= 2;
 			l = cpu_to_le16(inw(port)) << 16;
 			l |= cpu_to_le16(inw(port));
 			*(unsigned int *)p = l;
 			p += 4;
 		}
-		if (count) {
+		if (count)
 			*(unsigned short *)p = cpu_to_le16(inw(port));
-		}
 		break;
-	
+
 	 case 0x02:			/* Buffer 16-bit aligned */
 		*(unsigned short *)p = cpu_to_le16(inw(port));
 		p += 2;
 		count--;
 		while (count>=2) {
-			
+
 			count -= 2;
 			l = cpu_to_le16(inw(port)) << 16;
 			l |= cpu_to_le16(inw(port));
 			*(unsigned int *)p = l;
 			p += 4;
 		}
-		if (count) {
+		if (count)
 			*(unsigned short *)p = cpu_to_le16(inw(port));
-		}
 		break;
-		
+
 	 case 0x01:			/* Buffer 8-bit aligned */
 	 case 0x03:
 		/* I don't bother with 32bit transfers
 		 * in this case, 16bit will have to do -- DE */
 		--count;
-		
+
 		l = cpu_to_le16(inw(port));
 		*p = l >> 8;
 		p++;
-		while (count--)
-		{
+		while (count--) {
 			l2 = cpu_to_le16(inw(port));
 			*(unsigned short *)p = (l & 0xff) << 8 | (l2 >> 8);
 			p += 2;
@@ -235,35 +230,32 @@ void insw (unsigned long port, void *dst, unsigned long count)
  * but the interfaces seems to be slow: just using the inlined version
  * of the inl() breaks things.
  */
-void insl (unsigned long port, void *dst, unsigned long count)
+void insl(unsigned long port, void *dst, unsigned long count)
 {
 	unsigned int l = 0, l2;
 	unsigned char *p;
 
 	p = (unsigned char *)dst;
-	
+
 	if (!count)
 		return;
-	
-	switch (((unsigned long) dst) & 0x3)
-	{
+
+	switch (((unsigned long) dst) & 0x3) {
 	 case 0x00:			/* Buffer 32-bit aligned */
-		while (count--)
-		{
+		while (count--) {
 			*(unsigned int *)p = cpu_to_le32(inl(port));
 			p += 4;
 		}
 		break;
-	
+
 	 case 0x02:			/* Buffer 16-bit aligned */
 		--count;
-		
+
 		l = cpu_to_le32(inl(port));
 		*(unsigned short *)p = l >> 16;
 		p += 2;
-		
-		while (count--)
-		{
+
+		while (count--) {
 			l2 = cpu_to_le32(inl(port));
 			*(unsigned int *)p = (l & 0xffff) << 16 | (l2 >> 16);
 			p += 4;
@@ -273,14 +265,13 @@ void insl (unsigned long port, void *dst, unsigned long count)
 		break;
 	 case 0x01:			/* Buffer 8-bit aligned */
 		--count;
-		
+
 		l = cpu_to_le32(inl(port));
 		*(unsigned char *)p = l >> 24;
 		p++;
 		*(unsigned short *)p = (l >> 8) & 0xffff;
 		p += 2;
-		while (count--)
-		{
+		while (count--) {
 			l2 = cpu_to_le32(inl(port));
 			*(unsigned int *)p = (l & 0xff) << 24 | (l2 >> 8);
 			p += 4;
@@ -290,12 +281,11 @@ void insl (unsigned long port, void *dst, unsigned long count)
 		break;
 	 case 0x03:			/* Buffer 8-bit aligned */
 		--count;
-		
+
 		l = cpu_to_le32(inl(port));
 		*p = l >> 24;
 		p++;
-		while (count--)
-		{
+		while (count--) {
 			l2 = cpu_to_le32(inl(port));
 			*(unsigned int *)p = (l & 0xffffff) << 8 | l2 >> 24;
 			p += 4;
@@ -315,7 +305,7 @@ void insl (unsigned long port, void *dst, unsigned long count)
  * doing byte reads the "slow" way isn't nearly as slow as
  * doing byte writes the slow way (no r-m-w cycle).
  */
-void outsb(unsigned long port, const void * src, unsigned long count)
+void outsb(unsigned long port, const void *src, unsigned long count)
 {
 	const unsigned char *p;
 
@@ -333,68 +323,64 @@ void outsb(unsigned long port, const void * src, unsigned long count)
  * interfaces seems to be slow: just using the inlined version of the
  * outw() breaks things.
  */
-void outsw (unsigned long port, const void *src, unsigned long count)
+void outsw(unsigned long port, const void *src, unsigned long count)
 {
 	unsigned int l = 0, l2;
 	const unsigned char *p;
 
 	p = (const unsigned char *)src;
-	
+
 	if (!count)
 		return;
-	
-	switch (((unsigned long)p) & 0x3)
-	{
+
+	switch (((unsigned long)p) & 0x3) {
 	 case 0x00:			/* Buffer 32-bit aligned */
-		while (count>=2) {
+		while (count >= 2) {
 			count -= 2;
 			l = *(unsigned int *)p;
 			p += 4;
 			outw(le16_to_cpu(l >> 16), port);
 			outw(le16_to_cpu(l & 0xffff), port);
 		}
-		if (count) {
-			outw(le16_to_cpu(*(unsigned short*)p), port);
-		}
+		if (count)
+			outw(le16_to_cpu(*(unsigned short *)p), port);
 		break;
-	
+
 	 case 0x02:			/* Buffer 16-bit aligned */
-		
-		outw(le16_to_cpu(*(unsigned short*)p), port);
+
+		outw(le16_to_cpu(*(unsigned short *)p), port);
 		p += 2;
 		count--;
-		
-		while (count>=2) {
+
+		while (count >= 2) {
 			count -= 2;
 			l = *(unsigned int *)p;
 			p += 4;
 			outw(le16_to_cpu(l >> 16), port);
 			outw(le16_to_cpu(l & 0xffff), port);
 		}
-		if (count) {
+		if (count)
 			outw(le16_to_cpu(*(unsigned short *)p), port);
-		}
 		break;
-		
-	 case 0x01:			/* Buffer 8-bit aligned */	
+
+	 case 0x01:			/* Buffer 8-bit aligned */
 		/* I don't bother with 32bit transfers
 		 * in this case, 16bit will have to do -- DE */
-		
+
 		l  = *p << 8;
 		p++;
 		count--;
-		while (count)
-		{
+		while (count) {
 			count--;
 			l2 = *(unsigned short *)p;
 			p += 2;
 			outw(le16_to_cpu(l | l2 >> 8), port);
-		        l = l2 << 8;
+			l = l2 << 8;
 		}
 		l2 = *(unsigned char *)p;
-		outw (le16_to_cpu(l | l2>>8), port);
+		outw(le16_to_cpu(l | l2>>8), port);
 		break;
-	
+
 	}
 }
 
@@ -405,41 +391,38 @@ void outsw (unsigned long port, const void *src, unsigned long count)
  *  Performance is important, but the interfaces seems to be slow:
  * just using the inlined version of the outl() breaks things.
  */
-void outsl (unsigned long port, const void *src, unsigned long count)
+void outsl(unsigned long port, const void *src, unsigned long count)
 {
 	unsigned int l = 0, l2;
 	const unsigned char *p;
 
 	p = (const unsigned char *)src;
-	
+
 	if (!count)
 		return;
-	
-	switch (((unsigned long)p) & 0x3)
-	{
+
+	switch (((unsigned long)p) & 0x3) {
 	 case 0x00:			/* Buffer 32-bit aligned */
-		while (count--)
-		{
+		while (count--) {
 			outl(le32_to_cpu(*(unsigned int *)p), port);
 			p += 4;
 		}
 		break;
-	
+
 	 case 0x02:			/* Buffer 16-bit aligned */
 		--count;
-		
+
 		l = *(unsigned short *)p;
 		p += 2;
-		
-		while (count--)
-		{
+
+		while (count--) {
 			l2 = *(unsigned int *)p;
 			p += 4;
-			outl (le32_to_cpu(l << 16 | l2 >> 16), port);
+			outl(le32_to_cpu(l << 16 | l2 >> 16), port);
 			l = l2;
 		}
 		l2 = *(unsigned short *)p;
-		outl (le32_to_cpu(l << 16 | l2), port);
+		outl(le32_to_cpu(l << 16 | l2), port);
 		break;
 	 case 0x01:			/* Buffer 8-bit aligned */
 		--count;
@@ -449,33 +432,31 @@ void outsl (unsigned long port, const void *src, unsigned long count)
 		l |= *(unsigned short *)p << 8;
 		p += 2;
 
-		while (count--)
-		{
+		while (count--) {
 			l2 = *(unsigned int *)p;
 			p += 4;
-			outl (le32_to_cpu(l | l2 >> 24), port);
+			outl(le32_to_cpu(l | l2 >> 24), port);
 			l = l2 << 8;
 		}
 		l2 = *p;
-		outl (le32_to_cpu(l | l2), port);
+		outl(le32_to_cpu(l | l2), port);
 		break;
 	 case 0x03:			/* Buffer 8-bit aligned */
 		--count;
-		
+
 		l = *p << 24;
 		p++;
 
-		while (count--)
-		{
+		while (count--) {
 			l2 = *(unsigned int *)p;
 			p += 4;
-			outl (le32_to_cpu(l | l2 >> 8), port);
+			outl(le32_to_cpu(l | l2 >> 8), port);
 			l = l2 << 24;
 		}
 		l2 = *(unsigned short *)p << 16;
 		p += 2;
 		l2 |= *p;
-		outl (le32_to_cpu(l | l2), port);
+		outl(le32_to_cpu(l | l2), port);
 		break;
 	}
 }
-- 
1.7.1.251.gf80a2


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

* [PATCH 175/199] arch/parisc/lib/memset.c: Checkpatch cleanup
  2010-05-23 20:01 [PATCH 149/199] arch/ia64/hp/sim/boot/ssc.h: Checkpatch cleanup Andrea Gelmini
                   ` (4 preceding siblings ...)
  2010-05-23 20:02 ` [PATCH 174/199] arch/parisc/lib/io.c: " Andrea Gelmini
@ 2010-05-23 20:02 ` Andrea Gelmini
  2010-05-23 20:02 ` [PATCH 176/199] arch/parisc/math-emu/driver.c: " Andrea Gelmini
                   ` (3 subsequent siblings)
  9 siblings, 0 replies; 15+ messages in thread
From: Andrea Gelmini @ 2010-05-23 20:02 UTC (permalink / raw)
  To: andrea.gelmini
  Cc: Kyle McMartin, Helge Deller, James E.J. Bottomley, linux-parisc

arch/parisc/lib/memset.c:28: WARNING: space prohibited between function name and open parenthesis '('
arch/parisc/lib/memset.c:33: ERROR: that open brace { should be on the previous line
arch/parisc/lib/memset.c:33: WARNING: suspect code indent for conditional statements (2, 6)
arch/parisc/lib/memset.c:47: ERROR: that open brace { should be on the previous line
arch/parisc/lib/memset.c:47: WARNING: suspect code indent for conditional statements (6, 10)
arch/parisc/lib/memset.c:56: ERROR: that open brace { should be on the previous line
arch/parisc/lib/memset.c:56: WARNING: suspect code indent for conditional statements (6, 10)
arch/parisc/lib/memset.c:73: ERROR: that open brace { should be on the previous line
arch/parisc/lib/memset.c:73: WARNING: suspect code indent for conditional statements (6, 10)
arch/parisc/lib/memset.c:83: ERROR: that open brace { should be on the previous line
arch/parisc/lib/memset.c:83: WARNING: suspect code indent for conditional statements (2, 6)

Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
---
 arch/parisc/lib/memset.c |   23 +++++++++--------------
 1 files changed, 9 insertions(+), 14 deletions(-)

diff --git a/arch/parisc/lib/memset.c b/arch/parisc/lib/memset.c
index 1d7929b..66e518f 100644
--- a/arch/parisc/lib/memset.c
+++ b/arch/parisc/lib/memset.c
@@ -25,13 +25,12 @@
 typedef unsigned long op_t;
 
 void *
-memset (void *dstpp, int sc, size_t len)
+memset(void *dstpp, int sc, size_t len)
 {
   unsigned int c = sc;
   long int dstp = (long int) dstpp;
 
-  if (len >= 8)
-    {
+  if (len >= 8) {
       size_t xlen;
       op_t cccc;
 
@@ -44,17 +43,15 @@ memset (void *dstpp, int sc, size_t len)
 
       /* There are at least some bytes to set.
 	 No need to test for LEN == 0 in this alignment loop.  */
-      while (dstp % OPSIZ != 0)
-	{
-	  ((unsigned char *) dstp)[0] = c;
-	  dstp += 1;
-	  len -= 1;
+	while (dstp % OPSIZ != 0) {
+		((unsigned char *) dstp)[0] = c;
+		dstp += 1;
+		len -= 1;
 	}
 
       /* Write 8 `op_t' per iteration until less than 8 `op_t' remain.  */
       xlen = len / (OPSIZ * 8);
-      while (xlen > 0)
-	{
+      while (xlen > 0) {
 	  ((op_t *) dstp)[0] = cccc;
 	  ((op_t *) dstp)[1] = cccc;
 	  ((op_t *) dstp)[2] = cccc;
@@ -70,8 +67,7 @@ memset (void *dstpp, int sc, size_t len)
 
       /* Write 1 `op_t' per iteration until less than OPSIZ bytes remain.  */
       xlen = len / OPSIZ;
-      while (xlen > 0)
-	{
+      while (xlen > 0) {
 	  ((op_t *) dstp)[0] = cccc;
 	  dstp += OPSIZ;
 	  xlen -= 1;
@@ -80,8 +76,7 @@ memset (void *dstpp, int sc, size_t len)
     }
 
   /* Write the last few bytes.  */
-  while (len > 0)
-    {
+  while (len > 0) {
       ((unsigned char *) dstp)[0] = c;
       dstp += 1;
       len -= 1;
-- 
1.7.1.251.gf80a2


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

* [PATCH 176/199] arch/parisc/math-emu/driver.c: Checkpatch cleanup
  2010-05-23 20:01 [PATCH 149/199] arch/ia64/hp/sim/boot/ssc.h: Checkpatch cleanup Andrea Gelmini
                   ` (5 preceding siblings ...)
  2010-05-23 20:02 ` [PATCH 175/199] arch/parisc/lib/memset.c: " Andrea Gelmini
@ 2010-05-23 20:02 ` Andrea Gelmini
  2010-05-23 20:02 ` [PATCH 177/199] arch/parisc/math-emu/fpbits.h: " Andrea Gelmini
                   ` (2 subsequent siblings)
  9 siblings, 0 replies; 15+ messages in thread
From: Andrea Gelmini @ 2010-05-23 20:02 UTC (permalink / raw)
  To: andrea.gelmini
  Cc: Kyle McMartin, Helge Deller, James E.J. Bottomley, linux-parisc

arch/parisc/math-emu/driver.c:90: ERROR: trailing whitespace
arch/parisc/math-emu/driver.c:118: WARNING: suspect code indent for conditional statements (8, 12)

Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
---
 arch/parisc/math-emu/driver.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/parisc/math-emu/driver.c b/arch/parisc/math-emu/driver.c
index 09ef413..273006e 100644
--- a/arch/parisc/math-emu/driver.c
+++ b/arch/parisc/math-emu/driver.c
@@ -87,7 +87,7 @@ handle_fpe(struct pt_regs *regs)
 	 * code expects an artificial last entry which contains zero
 	 *
 	 * also, the passed in fr registers contain one word that defines
-	 * the fpu type. the fpu type information is constructed 
+	 * the fpu type. the fpu type information is constructed
 	 * inside the emulation code
 	 */
 	__u64 frcopy[36];
-- 
1.7.1.251.gf80a2


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

* [PATCH 177/199] arch/parisc/math-emu/fpbits.h: Checkpatch cleanup
  2010-05-23 20:01 [PATCH 149/199] arch/ia64/hp/sim/boot/ssc.h: Checkpatch cleanup Andrea Gelmini
                   ` (6 preceding siblings ...)
  2010-05-23 20:02 ` [PATCH 176/199] arch/parisc/math-emu/driver.c: " Andrea Gelmini
@ 2010-05-23 20:02 ` Andrea Gelmini
  2010-05-23 20:02 ` [PATCH 178/199] arch/s390/crypto/crypto_des.h: " Andrea Gelmini
  2010-05-23 20:02 ` [uml-devel] [PATCH 183/199] arch/um/os-Linux/drivers/Makefile: " Andrea Gelmini
  9 siblings, 0 replies; 15+ messages in thread
From: Andrea Gelmini @ 2010-05-23 20:02 UTC (permalink / raw)
  To: andrea.gelmini
  Cc: Kyle McMartin, Helge Deller, James E.J. Bottomley, linux-parisc

arch/parisc/math-emu/fpbits.h:51: ERROR: trailing whitespace

Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
---
 arch/parisc/math-emu/fpbits.h |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/parisc/math-emu/fpbits.h b/arch/parisc/math-emu/fpbits.h
index cefad06..431b571 100644
--- a/arch/parisc/math-emu/fpbits.h
+++ b/arch/parisc/math-emu/fpbits.h
@@ -48,7 +48,7 @@
  * function "DeclBitfR" except its use is restricted to occur within a larger
  * enclosing structure or union definition.  This declaration is an unnamed
  * structure with the argument, name, as the member name and the argument,
- * uname, as the element name. 
+ * uname, as the element name.
  *----------------------------------------------------------------------- */
 #define Bitfield_extract(start, length, object) 	\
     ((object) >> (HOSTWDSZ - (start) - (length)) & 	\
-- 
1.7.1.251.gf80a2


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

* [PATCH 178/199] arch/s390/crypto/crypto_des.h: Checkpatch cleanup
  2010-05-23 20:01 [PATCH 149/199] arch/ia64/hp/sim/boot/ssc.h: Checkpatch cleanup Andrea Gelmini
                   ` (7 preceding siblings ...)
  2010-05-23 20:02 ` [PATCH 177/199] arch/parisc/math-emu/fpbits.h: " Andrea Gelmini
@ 2010-05-23 20:02 ` Andrea Gelmini
  2010-05-26  0:41   ` Herbert Xu
  2010-05-23 20:02 ` [uml-devel] [PATCH 183/199] arch/um/os-Linux/drivers/Makefile: " Andrea Gelmini
  9 siblings, 1 reply; 15+ messages in thread
From: Andrea Gelmini @ 2010-05-23 20:02 UTC (permalink / raw)
  To: andrea.gelmini
  Cc: Herbert Xu, David S. Miller, Martin Schwidefsky, Heiko Carstens,
	linux390, linux-crypto, linux-s390

arch/s390/crypto/crypto_des.h:18: ERROR: do not use C99 // comments

Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
---
 arch/s390/crypto/crypto_des.h |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/s390/crypto/crypto_des.h b/arch/s390/crypto/crypto_des.h
index c964b64..6210457 100644
--- a/arch/s390/crypto/crypto_des.h
+++ b/arch/s390/crypto/crypto_des.h
@@ -15,4 +15,4 @@
 
 extern int crypto_des_check_key(const u8*, unsigned int, u32*);
 
-#endif //__CRYPTO_DES_H__
+#endif /*__CRYPTO_DES_H__*/
-- 
1.7.1.251.gf80a2

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

* [uml-devel] [PATCH 183/199] arch/um/os-Linux/drivers/Makefile: Checkpatch cleanup
  2010-05-23 20:01 [PATCH 149/199] arch/ia64/hp/sim/boot/ssc.h: Checkpatch cleanup Andrea Gelmini
                   ` (8 preceding siblings ...)
  2010-05-23 20:02 ` [PATCH 178/199] arch/s390/crypto/crypto_des.h: " Andrea Gelmini
@ 2010-05-23 20:02 ` Andrea Gelmini
  9 siblings, 0 replies; 15+ messages in thread
From: Andrea Gelmini @ 2010-05-23 20:02 UTC (permalink / raw)
  To: andrea.gelmini
  Cc: user-mode-linux-devel, Jeff Dike, user-mode-linux-user, vtun,
	Maxim Krasnyansky

arch/um/os-Linux/drivers/Makefile:1: ERROR: trailing whitespace
arch/um/os-Linux/drivers/Makefile:9: ERROR: trailing whitespace

Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
---
 arch/um/os-Linux/drivers/Makefile |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/um/os-Linux/drivers/Makefile b/arch/um/os-Linux/drivers/Makefile
index 6c546dc..7106d8f 100644
--- a/arch/um/os-Linux/drivers/Makefile
+++ b/arch/um/os-Linux/drivers/Makefile
@@ -1,4 +1,4 @@
-# 
+#
 # Copyright (C) 2000, 2002 Jeff Dike (jdike@karaya.com)
 # Licensed under the GPL
 #
@@ -6,7 +6,7 @@
 ethertap-objs := ethertap_kern.o ethertap_user.o
 tuntap-objs := tuntap_kern.o tuntap_user.o
 
-obj-y = 
+obj-y =
 obj-$(CONFIG_UML_NET_ETHERTAP) += ethertap.o
 obj-$(CONFIG_UML_NET_TUNTAP) += tuntap.o
 
-- 
1.7.1.251.gf80a2


------------------------------------------------------------------------------

_______________________________________________
User-mode-linux-devel mailing list
User-mode-linux-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel


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

* Re: [PATCH 178/199] arch/s390/crypto/crypto_des.h: Checkpatch cleanup
  2010-05-23 20:02 ` [PATCH 178/199] arch/s390/crypto/crypto_des.h: " Andrea Gelmini
@ 2010-05-26  0:41   ` Herbert Xu
  0 siblings, 0 replies; 15+ messages in thread
From: Herbert Xu @ 2010-05-26  0:41 UTC (permalink / raw)
  To: Andrea Gelmini
  Cc: David S. Miller, Martin Schwidefsky, Heiko Carstens, linux390,
	linux-crypto, linux-s390

On Sun, May 23, 2010 at 10:02:05PM +0200, Andrea Gelmini wrote:
> arch/s390/crypto/crypto_des.h:18: ERROR: do not use C99 // comments
> 
> Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>

Patch applied.  Thanks!
-- 
Visit Openswan at http://www.openswan.org/
Email: Herbert Xu ~{PmV>HI~} <herbert@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt

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

* Re: [PATCH 159/199] arch/m68k/hp300/hp300map.map: Checkpatch cleanup
  2010-05-23 20:01 ` [PATCH 159/199] arch/m68k/hp300/hp300map.map: " Andrea Gelmini
@ 2010-06-06 11:08   ` Geert Uytterhoeven
  2010-06-06 11:30   ` Andreas Schwab
  1 sibling, 0 replies; 15+ messages in thread
From: Geert Uytterhoeven @ 2010-06-06 11:08 UTC (permalink / raw)
  To: Andrea Gelmini; +Cc: Philip Blundell, Roman Zippel, linux-m68k

On Sun, May 23, 2010 at 22:01, Andrea Gelmini <andrea.gelmini@gelma.net> wrote:
> arch/m68k/hp300/hp300map.map:185: ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8

Thx, applied!

Redone using recode "ISO-8859-15..UTF-8" as git am fails to apply such patches.

Gr{oetje,eeting}s,

						Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
							    -- Linus Torvalds

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

* Re: [PATCH 159/199] arch/m68k/hp300/hp300map.map: Checkpatch cleanup
  2010-05-23 20:01 ` [PATCH 159/199] arch/m68k/hp300/hp300map.map: " Andrea Gelmini
  2010-06-06 11:08   ` Geert Uytterhoeven
@ 2010-06-06 11:30   ` Andreas Schwab
  2010-06-06 11:48     ` Geert Uytterhoeven
  1 sibling, 1 reply; 15+ messages in thread
From: Andreas Schwab @ 2010-06-06 11:30 UTC (permalink / raw)
  To: Andrea Gelmini
  Cc: Philip Blundell, Geert Uytterhoeven, Roman Zippel, linux-m68k

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset=us-ascii, Size: 808 bytes --]

Andrea Gelmini <andrea.gelmini@gelma.net> writes:

> diff --git a/arch/m68k/hp300/hp300map.map b/arch/m68k/hp300/hp300map.map
> index 6b45f0a..621c076 100644
> --- a/arch/m68k/hp300/hp300map.map
> +++ b/arch/m68k/hp300/hp300map.map
> @@ -182,71 +182,71 @@ string Prior = "\033[5~"
>  string Next = "\033[6~"
>  string Macro = "\033[M"
>  string Pause = "\033[P"
> -compose '`' 'A' to 'À'
[...]
> +compose '`' 'A' to 'À'

This is obviously wrong.  The composed character is supposed to be a
single latin-1 character (namely '\300').

A patch subject containing "checkpatch" should always ring the alarm
bells.

Andreas.

-- 
Andreas Schwab, schwab@linux-m68k.org
GPG Key fingerprint = 58CA 54C7 6D53 942B 1756  01D3 44D5 214B 8276 4ED5
"And now for something completely different."

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

* Re: [PATCH 159/199] arch/m68k/hp300/hp300map.map: Checkpatch cleanup
  2010-06-06 11:30   ` Andreas Schwab
@ 2010-06-06 11:48     ` Geert Uytterhoeven
  0 siblings, 0 replies; 15+ messages in thread
From: Geert Uytterhoeven @ 2010-06-06 11:48 UTC (permalink / raw)
  To: Andreas Schwab; +Cc: Andrea Gelmini, Philip Blundell, Roman Zippel, linux-m68k

On Sun, Jun 6, 2010 at 13:30, Andreas Schwab <schwab@linux-m68k.org> wrote:
> Andrea Gelmini <andrea.gelmini@gelma.net> writes:
>
>> diff --git a/arch/m68k/hp300/hp300map.map b/arch/m68k/hp300/hp300map.map
>> index 6b45f0a..621c076 100644
>> --- a/arch/m68k/hp300/hp300map.map
>> +++ b/arch/m68k/hp300/hp300map.map
>> @@ -182,71 +182,71 @@ string Prior = "\033[5~"
>>  string Next = "\033[6~"
>>  string Macro = "\033[M"
>>  string Pause = "\033[P"
>> -compose '`' 'A' to 'À'
> [...]
>> +compose '`' 'A' to 'À'
>
> This is obviously wrong.  The composed character is supposed to be a
> single latin-1 character (namely '\300').

Sorry, you're right. Loadkeys is indeed not yet aware of UTF-8.

> A patch subject containing "checkpatch" should always ring the alarm
> bells.

;-)

Gr{oetje,eeting}s,

						Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
							    -- Linus Torvalds

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

end of thread, other threads:[~2010-06-06 11:49 UTC | newest]

Thread overview: 15+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-05-23 20:01 [PATCH 149/199] arch/ia64/hp/sim/boot/ssc.h: Checkpatch cleanup Andrea Gelmini
2010-05-23 20:01 ` [PATCH 159/199] arch/m68k/hp300/hp300map.map: " Andrea Gelmini
2010-06-06 11:08   ` Geert Uytterhoeven
2010-06-06 11:30   ` Andreas Schwab
2010-06-06 11:48     ` Geert Uytterhoeven
2010-05-23 20:01 ` [PATCH 160/199] arch/m68k/sun3/leds.c: " Andrea Gelmini
2010-05-23 20:01 ` [PATCH 172/199] arch/parisc/kernel/pa7300lc.c: " Andrea Gelmini
2010-05-23 20:02 ` [PATCH 173/199] arch/parisc/kernel/sys32.h: " Andrea Gelmini
2010-05-23 20:02 ` [PATCH 174/199] arch/parisc/lib/io.c: " Andrea Gelmini
2010-05-23 20:02 ` [PATCH 175/199] arch/parisc/lib/memset.c: " Andrea Gelmini
2010-05-23 20:02 ` [PATCH 176/199] arch/parisc/math-emu/driver.c: " Andrea Gelmini
2010-05-23 20:02 ` [PATCH 177/199] arch/parisc/math-emu/fpbits.h: " Andrea Gelmini
2010-05-23 20:02 ` [PATCH 178/199] arch/s390/crypto/crypto_des.h: " Andrea Gelmini
2010-05-26  0:41   ` Herbert Xu
2010-05-23 20:02 ` [uml-devel] [PATCH 183/199] arch/um/os-Linux/drivers/Makefile: " Andrea Gelmini

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.