diff options
| author | Joe Hershberger <joe.hershberger@ni.com> | 2012-10-03 10:56:16 +0000 | 
|---|---|---|
| committer | Tom Rini <trini@ti.com> | 2012-10-03 16:01:22 -0700 | 
| commit | e9455fcc4526bb354150655876b8badab8a8c2ea (patch) | |
| tree | 3c5b973c30a7b243da2ecd5cb591ed7daa98f595 /common/cmd_cache.c | |
| parent | 8cab08e8045e1ec3799d9d3ad875f5755818069c (diff) | |
| download | olio-uboot-2014.01-e9455fcc4526bb354150655876b8badab8a8c2ea.tar.xz olio-uboot-2014.01-e9455fcc4526bb354150655876b8badab8a8c2ea.zip | |
Fix checkpatch.pl complaints in cmd_cache.c
Old code that is not compliant.
Signed-off-by: Joe Hershberger <joe.hershberger@ni.com>
Diffstat (limited to 'common/cmd_cache.c')
| -rw-r--r-- | common/cmd_cache.c | 47 | 
1 files changed, 26 insertions, 21 deletions
| diff --git a/common/cmd_cache.c b/common/cmd_cache.c index 9c228e219..13cc3d81b 100644 --- a/common/cmd_cache.c +++ b/common/cmd_cache.c @@ -36,21 +36,24 @@ void __weak invalidate_icache_all(void)  	puts("No arch specific invalidate_icache_all available!\n");  } -int do_icache ( cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) +int do_icache(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])  {  	switch (argc) {  	case 2:			/* on / off	*/  		switch (parse_argv(argv[1])) { -		case 0:	icache_disable(); +		case 0: +			icache_disable();  			break; -		case 1:	icache_enable (); +		case 1: +			icache_enable();  			break; -		case 2: invalidate_icache_all(); +		case 2: +			invalidate_icache_all();  			break;  		}  		/* FALL TROUGH */  	case 1:			/* get status */ -		printf ("Instruction Cache is %s\n", +		printf("Instruction Cache is %s\n",  			icache_status() ? "ON" : "OFF");  		return 0;  	default: @@ -65,40 +68,42 @@ void __weak flush_dcache_all(void)  	/* please define arch specific flush_dcache_all */  } -int do_dcache ( cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) +int do_dcache(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])  {  	switch (argc) { -	case 2:			/* on / off	*/ +	case 2:			/* on / off */  		switch (parse_argv(argv[1])) { -		case 0:	dcache_disable(); +		case 0: +			dcache_disable();  			break; -		case 1:	dcache_enable (); +		case 1: +			dcache_enable();  			break; -		case 2: flush_dcache_all(); +		case 2: +			flush_dcache_all();  			break;  		} -		/* FALL TROUGH */ +		break;  	case 1:			/* get status */ -		printf ("Data (writethrough) Cache is %s\n", +		printf("Data (writethrough) Cache is %s\n",  			dcache_status() ? "ON" : "OFF");  		return 0;  	default:  		return CMD_RET_USAGE;  	}  	return 0; -  }  static int parse_argv(const char *s)  { -	if (strcmp(s, "flush") == 0) { -		return (2); -	} else if (strcmp(s, "on") == 0) { -		return (1); -	} else if (strcmp(s, "off") == 0) { -		return (0); -	} -	return (-1); +	if (strcmp(s, "flush") == 0) +		return 2; +	else if (strcmp(s, "on") == 0) +		return 1; +	else if (strcmp(s, "off") == 0) +		return 0; + +	return -1;  } |