diff mbox

[U-Boot,1/2] command, log: Coding Style cleanup

Message ID 1329294067-4472-1-git-send-email-hs@denx.de
State Accepted
Delegated to: Detlev Zundel
Headers show

Commit Message

Heiko Schocher Feb. 15, 2012, 8:21 a.m. UTC 
Signed-off-by: Heiko Schocher <hs@denx.de>

---
[hs@pollux u-boot]$ ./tools/checkpatch.pl 20120215/0001-command-log-Coding-Style-cleanup.patch
total: 0 errors, 0 warnings, 217 lines checked

NOTE: Ignored message types: COMPLEX_MACRO CONSIDER_KSTRTO MINMAX MULTISTATEMENT_MACRO_USE_DO_WHILE

20120215/0001-command-log-Coding-Style-cleanup.patch has no obvious style problems and is ready for submission.

 common/cmd_log.c |   98 ++++++++++++++++++++++++++++-------------------------
 1 files changed, 52 insertions(+), 46 deletions(-)

Comments

Detlev Zundel Feb. 17, 2012, 2:34 p.m. UTC | #1 
Hi f,

> Signed-off-by: Heiko Schocher <hs@denx.de>

Our wiki[1] explicitely encourages such white-space only changes to be
marked as "cosmetic: " to make review easier.

Apart from that:

Acked-by: Detlev Zundel <dzu@denx.de>

[1] http://www.denx.de/wiki/U-Boot/Patches
Heiko Schocher Feb. 20, 2012, 7:54 a.m. UTC | #2 
Hello Detlev,

Detlev Zundel schrieb:
> Hi f,
> 
>> Signed-off-by: Heiko Schocher <hs@denx.de>
> 
> Our wiki[1] explicitely encourages such white-space only changes to be
> marked as "cosmetic: " to make review easier.

Argh.. of course forget that. Make a v2.

> Apart from that:
> 
> Acked-by: Detlev Zundel <dzu@denx.de>
> 
> [1] http://www.denx.de/wiki/U-Boot/Patches

bye,
Heiko
diff mbox

Patch

diff --git a/common/cmd_log.c b/common/cmd_log.c
index 249614f..9f76913 100644
--- a/common/cmd_log.c
+++ b/common/cmd_log.c
@@ -49,8 +49,8 @@ 
 DECLARE_GLOBAL_DATA_PTR;
 
 /* Local prototypes */
-static void logbuff_putc (const char c);
-static void logbuff_puts (const char *s);
+static void logbuff_putc(const char c);
+static void logbuff_puts(const char *s);
 static int logbuff_printk(const char *line);
 
 static char buf[1024];
@@ -70,9 +70,10 @@  unsigned long __logbuffer_base(void)
 {
 	return CONFIG_SYS_SDRAM_BASE + gd->ram_size - LOGBUFF_LEN;
 }
-unsigned long logbuffer_base (void) __attribute__((weak, alias("__logbuffer_base")));
+unsigned long logbuffer_base(void)
+__attribute__((weak, alias("__logbuffer_base")));
 
-void logbuff_init_ptrs (void)
+void logbuff_init_ptrs(void)
 {
 	unsigned long tag, post_word;
 	char *s;
@@ -81,13 +82,13 @@  void logbuff_init_ptrs (void)
 	log = (logbuff_t *)CONFIG_ALT_LH_ADDR;
 	lbuf = (char *)CONFIG_ALT_LB_ADDR;
 #else
-	log = (logbuff_t *)(logbuffer_base ()) - 1;
+	log = (logbuff_t *)(logbuffer_base()) - 1;
 	lbuf = (char *)log->buf;
 #endif
 
 	/* Set up log version */
 	if ((s = getenv ("logversion")) != NULL)
-		log_version = (int)simple_strtoul (s, NULL, 10);
+		log_version = (int)simple_strtoul(s, NULL, 10);
 
 	if (log_version == 2)
 		tag = log->v2.tag;
@@ -96,9 +97,8 @@  void logbuff_init_ptrs (void)
 	post_word = post_word_load();
 #ifdef CONFIG_POST
 	/* The post routines have setup the word so we can simply test it */
-	if (tag != LOGBUFF_MAGIC || (post_word & POST_COLDBOOT)) {
-		logbuff_reset ();
-	}
+	if (tag != LOGBUFF_MAGIC || (post_word & POST_COLDBOOT))
+		logbuff_reset();
 #else
 	/* No post routines, so we do our own checking                    */
 	if (tag != LOGBUFF_MAGIC || post_word != LOGBUFF_MAGIC) {
@@ -111,15 +111,15 @@  void logbuff_init_ptrs (void)
 
 	/* Initialize default loglevel if present */
 	if ((s = getenv ("loglevel")) != NULL)
-		console_loglevel = (int)simple_strtoul (s, NULL, 10);
+		console_loglevel = (int)simple_strtoul(s, NULL, 10);
 
 	gd->flags |= GD_FLG_LOGINIT;
 }
 
-void logbuff_reset (void)
+void logbuff_reset(void)
 {
 #ifndef CONFIG_ALT_LB_ADDR
-	memset (log, 0, sizeof (logbuff_t));
+	memset(log, 0, sizeof(logbuff_t));
 #endif
 	if (log_version == 2) {
 		log->v2.tag = LOGBUFF_MAGIC;
@@ -140,7 +140,7 @@  void logbuff_reset (void)
 	}
 }
 
-int drv_logbuff_init (void)
+int drv_logbuff_init(void)
 {
 	struct stdio_dev logdev;
 	int rc;
@@ -154,20 +154,20 @@  int drv_logbuff_init (void)
 	logdev.putc  = logbuff_putc;		/* 'putc' function */
 	logdev.puts  = logbuff_puts;		/* 'puts' function */
 
-	rc = stdio_register (&logdev);
+	rc = stdio_register(&logdev);
 
 	return (rc == 0) ? 1 : rc;
 }
 
-static void logbuff_putc (const char c)
+static void logbuff_putc(const char c)
 {
 	char buf[2];
 	buf[0] = c;
 	buf[1] = '\0';
-	logbuff_printk (buf);
+	logbuff_printk(buf);
 }
 
-static void logbuff_puts (const char *s)
+static void logbuff_puts(const char *s)
 {
 	logbuff_printk (s);
 }
@@ -175,10 +175,12 @@  static void logbuff_puts (const char *s)
 void logbuff_log(char *msg)
 {
 	if ((gd->flags & GD_FLG_LOGINIT)) {
-		logbuff_printk (msg);
+		logbuff_printk(msg);
 	} else {
-		/* Can happen only for pre-relocated errors as logging */
-		/* at that stage should be disabled                    */
+		/*
+		 * Can happen only for pre-relocated errors as logging
+		 * at that stage should be disabled
+		 */
 		puts (msg);
 	}
 }
@@ -193,16 +195,16 @@  void logbuff_log(char *msg)
  * Return:      None
  *
  */
-int do_log (cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
+int do_log(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
 {
 	char *s;
 	unsigned long i, start, size;
 
-	if (strcmp(argv[1],"append") == 0) {
+	if (strcmp(argv[1], "append") == 0) {
 		/* Log concatenation of all arguments separated by spaces */
-		for (i=2; i<argc; i++) {
-			logbuff_printk (argv[i]);
-			logbuff_putc ((i<argc-1) ? ' ' : '\n');
+		for (i = 2; i < argc; i++) {
+			logbuff_printk(argv[i]);
+			logbuff_putc((i < argc - 1) ? ' ' : '\n');
 		}
 		return 0;
 	}
@@ -210,34 +212,38 @@  int do_log (cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
 	switch (argc) {
 
 	case 2:
-		if (strcmp(argv[1],"show") == 0) {
+		if (strcmp(argv[1], "show") == 0) {
 			if (log_version == 2) {
 				start = log->v2.start;
 				size = log->v2.end - log->v2.start;
-			}
-			else {
+			} else {
 				start = log->v1.start;
 				size = log->v1.size;
 			}
-			for (i=0; i < (size&LOGBUFF_MASK); i++) {
-				s = lbuf+((start+i)&LOGBUFF_MASK);
-				putc (*s);
+			for (i = 0; i < (size & LOGBUFF_MASK); i++) {
+				s = lbuf + ((start + i) & LOGBUFF_MASK);
+				putc(*s);
 			}
 			return 0;
-		} else if (strcmp(argv[1],"reset") == 0) {
-			logbuff_reset ();
+		} else if (strcmp(argv[1], "reset") == 0) {
+			logbuff_reset();
 			return 0;
-		} else if (strcmp(argv[1],"info") == 0) {
-			printf ("Logbuffer   at  %08lx\n", (unsigned long)lbuf);
+		} else if (strcmp(argv[1], "info") == 0) {
+			printf("Logbuffer   at  %08lx\n", (unsigned long)lbuf);
 			if (log_version == 2) {
-				printf ("log_start    =  %08lx\n", log->v2.start);
-				printf ("log_end      =  %08lx\n", log->v2.end);
-				printf ("logged_chars =  %08lx\n", log->v2.chars);
+				printf("log_start    =  %08lx\n",
+					log->v2.start);
+				printf("log_end      =  %08lx\n", log->v2.end);
+				printf("logged_chars =  %08lx\n",
+					log->v2.chars);
 			}
 			else {
-				printf ("log_start    =  %08lx\n", log->v1.start);
-				printf ("log_size     =  %08lx\n", log->v1.size);
-				printf ("logged_chars =  %08lx\n", log->v1.chars);
+				printf("log_start    =  %08lx\n",
+					log->v1.start);
+				printf("log_size     =  %08lx\n",
+					log->v1.size);
+				printf("logged_chars =  %08lx\n",
+					log->v1.chars);
 			}
 			return 0;
 		}
@@ -264,8 +270,8 @@  static int logbuff_printk(const char *line)
 	int line_feed;
 	static signed char msg_level = -1;
 
-	strcpy (buf + 3, line);
-	i = strlen (line);
+	strcpy(buf + 3, line);
+	i = strlen(line);
 	buf_end = buf + 3 + i;
 	for (p = buf + 3; p < buf_end; p++) {
 		msg = p;
@@ -280,8 +286,9 @@  static int logbuff_printk(const char *line)
 				p[0] = '<';
 				p[1] = default_message_loglevel + '0';
 				p[2] = '>';
-			} else
+			} else {
 				msg += 3;
+			}
 			msg_level = p[1] - '0';
 		}
 		line_feed = 0;
@@ -292,8 +299,7 @@  static int logbuff_printk(const char *line)
 				if (log->v2.end - log->v2.start > LOGBUFF_LEN)
 					log->v2.start++;
 				log->v2.chars++;
-			}
-			else {
+			} else {
 				lbuf[(log->v1.start + log->v1.size) &
 					 LOGBUFF_MASK] = *p;
 				if (log->v1.size < LOGBUFF_LEN)