Skip to content
Snippets Groups Projects
Commit e9455fcc authored by Joe Hershberger's avatar Joe Hershberger Committed by Tom Rini
Browse files

Fix checkpatch.pl complaints in cmd_cache.c


Old code that is not compliant.

Signed-off-by: default avatarJoe Hershberger <joe.hershberger@ni.com>
parent 8cab08e8
Branches
Tags
No related merge requests found
...@@ -41,11 +41,14 @@ int do_icache ( cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) ...@@ -41,11 +41,14 @@ int do_icache ( cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
switch (argc) { switch (argc) {
case 2: /* on / off */ case 2: /* on / off */
switch (parse_argv(argv[1])) { switch (parse_argv(argv[1])) {
case 0: icache_disable(); case 0:
icache_disable();
break; break;
case 1: icache_enable (); case 1:
icache_enable();
break; break;
case 2: invalidate_icache_all(); case 2:
invalidate_icache_all();
break; break;
} }
/* FALL TROUGH */ /* FALL TROUGH */
...@@ -70,14 +73,17 @@ int do_dcache ( cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) ...@@ -70,14 +73,17 @@ int do_dcache ( cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
switch (argc) { switch (argc) {
case 2: /* on / off */ case 2: /* on / off */
switch (parse_argv(argv[1])) { switch (parse_argv(argv[1])) {
case 0: dcache_disable(); case 0:
dcache_disable();
break; break;
case 1: dcache_enable (); case 1:
dcache_enable();
break; break;
case 2: flush_dcache_all(); case 2:
flush_dcache_all();
break; break;
} }
/* FALL TROUGH */ break;
case 1: /* get status */ case 1: /* get status */
printf("Data (writethrough) Cache is %s\n", printf("Data (writethrough) Cache is %s\n",
dcache_status() ? "ON" : "OFF"); dcache_status() ? "ON" : "OFF");
...@@ -86,19 +92,18 @@ int do_dcache ( cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) ...@@ -86,19 +92,18 @@ int do_dcache ( cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
return CMD_RET_USAGE; return CMD_RET_USAGE;
} }
return 0; return 0;
} }
static int parse_argv(const char *s) static int parse_argv(const char *s)
{ {
if (strcmp(s, "flush") == 0) { if (strcmp(s, "flush") == 0)
return (2); return 2;
} else if (strcmp(s, "on") == 0) { else if (strcmp(s, "on") == 0)
return (1); return 1;
} else if (strcmp(s, "off") == 0) { else if (strcmp(s, "off") == 0)
return (0); return 0;
}
return (-1); return -1;
} }
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment