cmd: fpga: Extract fpga info command to separate function
authorMichal Simek <michal.simek@xilinx.com>
Mon, 4 Jun 2018 12:57:34 +0000 (14:57 +0200)
committerMichal Simek <michal.simek@xilinx.com>
Tue, 11 Sep 2018 08:58:42 +0000 (10:58 +0200)
Move fpga info to U_BOOT_CMD_MKENT subcommand.
Also use strtol instead of simple_strtoul. The reason is that if -1 is
passed (or fpga info without "fpga" variable) the list of all fpgas is
shown.
This functionality is in the fpga core but it couldn't be performed.

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

index ac12af2fa06df130c2363c85d41d3484d0c57b3c..039803870b024cfaa974d3925bcf7a62b76bf3b7 100644 (file)
 #include <fs.h>
 #include <malloc.h>
 
+static long do_fpga_get_device(char *arg)
+{
+       long dev = FPGA_INVALID_DEVICE;
+       char *devstr = env_get("fpga");
+
+       if (devstr)
+               /* Should be strtol to handle -1 cases */
+               dev = simple_strtol(devstr, NULL, 16);
+
+       if (arg)
+               dev = simple_strtol(arg, NULL, 16);
+
+       debug("%s: device = %ld\n", __func__, dev);
+
+       return dev;
+}
+
 /* Local defines */
 enum {
        FPGA_NONE = -1,
-       FPGA_INFO,
        FPGA_LOAD,
        FPGA_LOADB,
        FPGA_DUMP,
@@ -35,9 +51,7 @@ static int fpga_get_op(char *opstr)
 {
        int op = FPGA_NONE;
 
-       if (!strcmp("info", opstr))
-               op = FPGA_INFO;
-       else if (!strcmp("loadb", opstr))
+       if (!strcmp("loadb", opstr))
                op = FPGA_LOADB;
        else if (!strcmp("load", opstr))
                op = FPGA_LOAD;
@@ -194,10 +208,6 @@ int do_fpga(cmd_tbl_t *cmdtp, int flag, int argc, char *const argv[])
        }
 
        switch (op) {
-       case FPGA_INFO:
-               rc = fpga_info(dev);
-               break;
-
        case FPGA_LOAD:
                rc = fpga_load(dev, fpga_data, data_size, BIT_FULL);
                break;
@@ -331,7 +341,16 @@ int do_fpga(cmd_tbl_t *cmdtp, int flag, int argc, char *const argv[])
        return rc;
 }
 
+static int do_fpga_info(cmd_tbl_t *cmdtp, int flag, int argc,
+                       char * const argv[])
+{
+       long dev = do_fpga_get_device(argv[0]);
+
+       return fpga_info(dev);
+}
+
 static cmd_tbl_t fpga_commands[] = {
+       U_BOOT_CMD_MKENT(info, 1, 1, do_fpga_info, "", ""),
 };
 
 static int do_fpga_wrapper(cmd_tbl_t *cmdtp, int flag, int argc,