cmd: fpga: Move fpga_get_op to avoid local function declaration
authorMichal Simek <michal.simek@xilinx.com>
Wed, 30 May 2018 08:00:40 +0000 (10:00 +0200)
committerMichal Simek <michal.simek@xilinx.com>
Tue, 11 Sep 2018 08:58:41 +0000 (10:58 +0200)
Move fpga_get_op() to top of file to remove local function declaration
and also remove useless retyping.

Signed-off-by: Michal Simek <michal.simek@xilinx.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
cmd/fpga.c

index abe6837202854536f6c426508f3ffe1df1c65fce..de8505e9d4c83db527f07fdebb4df282076769ef 100644 (file)
@@ -13,9 +13,6 @@
 #include <fs.h>
 #include <malloc.h>
 
-/* Local functions */
-static int fpga_get_op(char *opstr);
-
 /* Local defines */
 enum {
        FPGA_NONE = -1,
@@ -30,6 +27,46 @@ enum {
        FPGA_LOADS,
 };
 
+/*
+ * Map op to supported operations.  We don't use a table since we
+ * would just have to relocate it from flash anyway.
+ */
+static int fpga_get_op(char *opstr)
+{
+       int op = FPGA_NONE;
+
+       if (!strcmp("info", opstr))
+               op = FPGA_INFO;
+       else if (!strcmp("loadb", opstr))
+               op = FPGA_LOADB;
+       else if (!strcmp("load", opstr))
+               op = FPGA_LOAD;
+#if defined(CONFIG_CMD_FPGA_LOADP)
+       else if (!strcmp("loadp", opstr))
+               op = FPGA_LOADP;
+#endif
+#if defined(CONFIG_CMD_FPGA_LOADBP)
+       else if (!strcmp("loadbp", opstr))
+               op = FPGA_LOADBP;
+#endif
+#if defined(CONFIG_CMD_FPGA_LOADFS)
+       else if (!strcmp("loadfs", opstr))
+               op = FPGA_LOADFS;
+#endif
+#if defined(CONFIG_CMD_FPGA_LOADMK)
+       else if (!strcmp("loadmk", opstr))
+               op = FPGA_LOADMK;
+#endif
+       else if (!strcmp("dump", opstr))
+               op = FPGA_DUMP;
+#if defined(CONFIG_CMD_FPGA_LOAD_SECURE)
+       else if (!strcmp("loads", opstr))
+               op = FPGA_LOADS;
+#endif
+
+       return op;
+}
+
 /* ------------------------------------------------------------------------- */
 /* command form:
  *   fpga <op> <device number> <data addr> <datasize>
@@ -71,7 +108,7 @@ int do_fpga(cmd_tbl_t *cmdtp, int flag, int argc, char *const argv[])
                return CMD_RET_USAGE;
        }
 
-       op = (int)fpga_get_op(argv[1]);
+       op = fpga_get_op(argv[1]);
 
        switch (op) {
        case FPGA_NONE:
@@ -326,46 +363,6 @@ int do_fpga(cmd_tbl_t *cmdtp, int flag, int argc, char *const argv[])
        return rc;
 }
 
-/*
- * Map op to supported operations.  We don't use a table since we
- * would just have to relocate it from flash anyway.
- */
-static int fpga_get_op(char *opstr)
-{
-       int op = FPGA_NONE;
-
-       if (!strcmp("info", opstr))
-               op = FPGA_INFO;
-       else if (!strcmp("loadb", opstr))
-               op = FPGA_LOADB;
-       else if (!strcmp("load", opstr))
-               op = FPGA_LOAD;
-#if defined(CONFIG_CMD_FPGA_LOADP)
-       else if (!strcmp("loadp", opstr))
-               op = FPGA_LOADP;
-#endif
-#if defined(CONFIG_CMD_FPGA_LOADBP)
-       else if (!strcmp("loadbp", opstr))
-               op = FPGA_LOADBP;
-#endif
-#if defined(CONFIG_CMD_FPGA_LOADFS)
-       else if (!strcmp("loadfs", opstr))
-               op = FPGA_LOADFS;
-#endif
-#if defined(CONFIG_CMD_FPGA_LOADMK)
-       else if (!strcmp("loadmk", opstr))
-               op = FPGA_LOADMK;
-#endif
-       else if (!strcmp("dump", opstr))
-               op = FPGA_DUMP;
-#if defined(CONFIG_CMD_FPGA_LOAD_SECURE)
-       else if (!strcmp("loads", opstr))
-               op = FPGA_LOADS;
-#endif
-
-       return op;
-}
-
 #if defined(CONFIG_CMD_FPGA_LOADFS) || defined(CONFIG_CMD_FPGA_LOAD_SECURE)
 U_BOOT_CMD(fpga, 9, 1, do_fpga,
 #else