From: Eugeniu Rosca <roscaeugeniu@gmail.com>
Date: Fri, 19 Jul 2019 21:26:12 +0000 (+0200)
Subject: cmd: bcb: Fix duplicated handling in two case-branches
X-Git-Tag: v2019.10-rc1~15^2~19
X-Git-Url: https://git.librecmc.org/?a=commitdiff_plain;h=e47b73b821f7f6078c0d11b453545507e25727cc;p=oweals%2Fu-boot.git

cmd: bcb: Fix duplicated handling in two case-branches

Fix warning V1037 reported by PVS-Studio Static Analyzer:
Two or more case-branches perform the same actions. Check lines: 49, 53

Fixes: db7b7a05b267 ("cmd: Add 'bcb' command to read/modify/write BCB fields")
Signed-off-by: Eugeniu Rosca <erosca@de.adit-jv.com>
Reviewed-by: Igor Opaniuk <igor.opaniuk@gmail.com>
Reviewed-by: Sam Protsenko <semen.protsenko@linaro.org>
---

diff --git a/cmd/bcb.c b/cmd/bcb.c
index 2bd5a744de..3b1c7434e2 100644
--- a/cmd/bcb.c
+++ b/cmd/bcb.c
@@ -46,9 +46,6 @@ static int bcb_is_misused(int argc, char *const argv[])
 
 	switch (cmd) {
 	case BCB_CMD_LOAD:
-		if (argc != 3)
-			goto err;
-		break;
 	case BCB_CMD_FIELD_SET:
 		if (argc != 3)
 			goto err;