* [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.