From aa93ac59dcf6c7a7593cd7d67b8f8a5b01353efb Mon Sep 17 00:00:00 2001
From: Michal Simek <michal.simek@xilinx.com>
Date: Mon, 4 Jun 2018 13:57:19 +0200
Subject: [PATCH] cmd: fpga: Remove parameter checking from fpga loadfs command
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 | 7 -------
1 file changed, 7 deletions(-)
diff --git a/cmd/fpga.c b/cmd/fpga.c
index 48902286f1..b03dd9dc0a 100644
--- a/cmd/fpga.c
+++ b/cmd/fpga.c
@@ -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