cmd: fpga: Remove parameter checking from fpga loadfs command
authorMichal Simek <michal.simek@xilinx.com>
Mon, 4 Jun 2018 11:57:19 +0000 (13:57 +0200)
committerMichal Simek <michal.simek@xilinx.com>
Tue, 11 Sep 2018 08:58:42 +0000 (10:58 +0200)
Parameter checking is dead code because all the time there must be all
params assigned. If they are not assigned there is no 9th parameters
passed and checking before return CMD_RET_USAGE.

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

index 48902286f1d5966683940103e10d3403b5715794..b03dd9dc0aceb25eab7fb81e41e966b51f2328f3 100644 (file)
@@ -124,13 +124,6 @@ int do_fpga(cmd_tbl_t *cmdtp, int flag, int argc, char *const argv[])
                fpga_fsinfo.dev_part = argv[7];
                fpga_fsinfo.filename = argv[8];
 
-               /* Blocksize can be zero */
-               if (!fpga_fsinfo.interface || !fpga_fsinfo.dev_part ||
-                   !fpga_fsinfo.filename) {
-                       puts("ERR: Wrong interface, dev_part or filename\n");
-                       return CMD_RET_USAGE;
-               }
-
                argc = 5;
                break;
 #endif