In the entire cmd/bcb.c file, the return value of strcmp() is not
directly compared to 0. Therefore, it would be better to maintain this
style in the new do_bcb_ab_select() function as well.

Reviewed-by: Mattijs Korpershoek <mkorpersh...@baylibre.com>
Reviewed-by: Simon Glass <s...@chromium.org>
Signed-off-by: Dmitry Rokosov <ddroko...@salutedevices.com>
---
 cmd/bcb.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/cmd/bcb.c b/cmd/bcb.c
index 
4f3b8a1538a1f05b8b4a1a51c8962c422981166a..970c58e56cba0e1bdff21b7cd099f69151f0c5b8
 100644
--- a/cmd/bcb.c
+++ b/cmd/bcb.c
@@ -432,7 +432,7 @@ static int do_bcb_ab_select(struct cmd_tbl *cmdtp, int 
flag, int argc,
        bool dec_tries = true;
 
        for (int i = 4; i < argc; i++) {
-               if (strcmp(argv[i], "--no-dec") == 0)
+               if (!strcmp(argv[i], "--no-dec"))
                        dec_tries = false;
                else
                        return CMD_RET_USAGE;

-- 
2.43.0

Reply via email to