treewide: Tidy up semicolon after command macros

The U_BOOT_CMD_COMPLETE() macro has a semicolon at the end, perhaps
inadvertently. Some code has taken advantage of this.

Tidy this up by dropping the semicolon from the macro and adding it to
macro invocations as required.

Signed-off-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Tom Rini <trini@konsulko.com>
Reviewed-by: Ilias Apalodimas <ilias.apalodimas@linaro.org>

diff --git a/board/freescale/common/vid.c b/board/freescale/common/vid.c
index 5ec3f2a76b1904e46ed3ce5edb10d1805558d430..fc5d400cfe18d0e49a444c8fc15513a869a4c160 100644 (file)
--- a/board/freescale/common/vid.c
+++ b/board/freescale/common/vid.c
@@ -793,4 +793,4 @@ U_BOOT_CMD(
        vdd_read, 1, 0, do_vdd_read,
        "read VDD",
        " - Read the voltage specified in mV"
-)
+);

diff --git a/board/xilinx/common/fru.c b/board/xilinx/common/fru.c
index c916c3d6b4c8d647d9075bc66d862ecb96d3306f..12b21317496a834b3ed0d7758c1c0ba4c4743357 100644 (file)
--- a/board/xilinx/common/fru.c
+++ b/board/xilinx/common/fru.c
@@ -85,4 +85,4 @@ U_BOOT_CMD(
        fru, 8, 1, do_fru,
        "FRU table info",
        fru_help_text
-)
+);

diff --git a/board/xilinx/versal/cmds.c b/board/xilinx/versal/cmds.c
index 9cc2cdcebf1c4c95e1677b8279fa4323cb9fc8bc..2a74e49aedec27389f465b778ede60a79275fdd3 100644 (file)
--- a/board/xilinx/versal/cmds.c
+++ b/board/xilinx/versal/cmds.c
@@ -98,4 +98,4 @@ U_BOOT_LONGHELP(versal,
 U_BOOT_CMD(versal, 4, 1, do_versal,
           "versal sub-system",
           versal_help_text
-)
+);

diff --git a/board/xilinx/zynqmp/cmds.c b/board/xilinx/zynqmp/cmds.c
index f1f3eff501e1c2973c125dfb1697021b89d9d85e..9524688f27d9c1d7e42c8f5fb7c0679fe20717ab 100644 (file)
--- a/board/xilinx/zynqmp/cmds.c
+++ b/board/xilinx/zynqmp/cmds.c
@@ -427,4 +427,4 @@ U_BOOT_CMD(
        zynqmp, 9, 1, do_zynqmp,
        "ZynqMP sub-system",
        zynqmp_help_text
-)
+);

diff --git a/cmd/btrfs.c b/cmd/btrfs.c
index 98daea99e9edd1ac2bf28b560ef6a39fd38185ee..2843835d08b80710f1de6820c3402c3e15be90f9 100644 (file)
--- a/cmd/btrfs.c
+++ b/cmd/btrfs.c
@@ -24,4 +24,4 @@ U_BOOT_CMD(btrsubvol, 3, 1, do_btrsubvol,
        "list subvolumes of a BTRFS filesystem",
        "<interface> <dev[:part]>\n"
        "     - List subvolumes of a BTRFS filesystem."
-)
+);

diff --git a/cmd/eeprom.c b/cmd/eeprom.c
index 0b6ca8c505fb3cadf660517e7c17ebcd5ad130aa..322765ad02a013b1c5dff318f1f4285207f9714a 100644 (file)
--- a/cmd/eeprom.c
+++ b/cmd/eeprom.c
@@ -435,4 +435,4 @@ U_BOOT_CMD(
        "The values which can be provided with the -l option are:\n"
        CONFIG_EEPROM_LAYOUT_HELP_STRING"\n"
 #endif
-)
+);

diff --git a/cmd/ext2.c b/cmd/ext2.c
index 57a99516a6ac15844b69027c01c9d3458f504cef..a0ce0cf5796b9dd091c004ddbbcb4ee36bbfd590 100644 (file)
--- a/cmd/ext2.c
+++ b/cmd/ext2.c
@@ -42,7 +42,7 @@ U_BOOT_CMD(
        "list files in a directory (default /)",
        "<interface> <dev[:part]> [directory]\n"
        "    - list files from 'dev' on 'interface' in a 'directory'"
-)
+);
 
 U_BOOT_CMD(
        ext2load,       6,      0,      do_ext2load,
@@ -50,4 +50,4 @@ U_BOOT_CMD(
        "<interface> [<dev[:part]> [addr [filename [bytes [pos]]]]]\n"
        "    - load binary file 'filename' from 'dev' on 'interface'\n"
        "      to address 'addr' from ext2 filesystem."
-)
+);

diff --git a/cmd/fs.c b/cmd/fs.c
index 6044f73af5b4173151eb47d8f56c833d3989aac3..46cb43dcdb5bfafe4ad273a37b437ae1450398b2 100644 (file)
--- a/cmd/fs.c
+++ b/cmd/fs.c
@@ -39,7 +39,7 @@ U_BOOT_CMD(
        "      If 'bytes' is 0 or omitted, the file is read until the end.\n"
        "      'pos' gives the file byte position to start reading from.\n"
        "      If 'pos' is 0 or omitted, the file is read from the start."
-)
+);
 
 static int do_save_wrapper(struct cmd_tbl *cmdtp, int flag, int argc,
                           char *const argv[])
@@ -56,7 +56,7 @@ U_BOOT_CMD(
        "      'bytes' gives the size to save in bytes and is mandatory.\n"
        "      'pos' gives the file byte position to start writing to.\n"
        "      If 'pos' is 0 or omitted, the file is written from the start."
-)
+);
 
 static int do_ls_wrapper(struct cmd_tbl *cmdtp, int flag, int argc,
                         char *const argv[])
@@ -70,7 +70,7 @@ U_BOOT_CMD(
        "<interface> [<dev[:part]> [directory]]\n"
        "    - List files in directory 'directory' of partition 'part' on\n"
        "      device type 'interface' instance 'dev'."
-)
+);
 
 static int do_ln_wrapper(struct cmd_tbl *cmdtp, int flag, int argc,
                         char *const argv[])
@@ -84,7 +84,7 @@ U_BOOT_CMD(
        "<interface> <dev[:part]> target linkname\n"
        "    - create a symbolic link to 'target' with the name 'linkname' on\n"
        "      device type 'interface' instance 'dev'."
-)
+);
 
 static int do_fstype_wrapper(struct cmd_tbl *cmdtp, int flag, int argc,
                             char *const argv[])

diff --git a/cmd/pinmux.c b/cmd/pinmux.c
index f17cf4110d9fdf1f0191164e193dd0eee9ea26ef..105f01eaafffce364e673af7817d122d27f9547d 100644 (file)
--- a/cmd/pinmux.c
+++ b/cmd/pinmux.c
@@ -178,4 +178,4 @@ U_BOOT_CMD(pinmux, CONFIG_SYS_MAXARGS, 1, do_pinmux,
           "list                     - list UCLASS_PINCTRL devices\n"
           "pinmux dev [pincontroller-name] - select pin-controller device\n"
           "pinmux status [-a | pin-name]   - print pin-controller muxing [for all | for pin-name]\n"
-)
+);

diff --git a/cmd/qfw.c b/cmd/qfw.c
index d6ecfa60d5a71a7414b3741e84cff569a6e93df0..1b8c775ebf5ab10cffac06998f1a043dd3ff2b84 100644 (file)
--- a/cmd/qfw.c
+++ b/cmd/qfw.c
@@ -121,4 +121,4 @@ U_BOOT_CMD(
        "    - list                             : print firmware(s) currently loaded\n"
        "    - cpus                             : print online cpu number\n"
        "    - load <kernel addr> <initrd addr> : load kernel and initrd (if any), and setup for zboot\n"
-)
+);

diff --git a/include/command.h b/include/command.h
index 6262365e128fe9d063002ed26050fbaf299c21b5..5bd3ecbe8f919ebb80c2fdf27d55ff523f4b6825 100644 (file)
--- a/include/command.h
+++ b/include/command.h
@@ -390,7 +390,7 @@ int cmd_source_script(ulong addr, const char *fit_uname, const char *confname);
 #define U_BOOT_CMD_COMPLETE(_name, _maxargs, _rep, _cmd, _usage, _help, _comp) \
        ll_entry_declare(struct cmd_tbl, _name, cmd) =                  \
                U_BOOT_CMD_MKENT_COMPLETE(_name, _maxargs, _rep, _cmd,  \
-                                               _usage, _help, _comp);
+                                               _usage, _help, _comp)
 
 #define U_BOOT_CMDREP_COMPLETE(_name, _maxargs, _cmd_rep, _usage,      \
                               _help, _comp)                            \