[PATCH 1/1] cmd: change the return value when argc error

Shenlin Liang liangshenlin at eswincomputing.com
Fri Dec 9 10:17:43 CET 2022


When the number of parameters is wrong, the return value should be processed in 
the same way as other cmds, return CMD_RET_USAGE so that it can print the information.

Signed-off-by: Shenlin Liang <liangshenlin at eswincomputing.com>
---
 cmd/mvebu/comphy_rx_training.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/cmd/mvebu/comphy_rx_training.c b/cmd/mvebu/comphy_rx_training.c
index 25a9e153de..4ee8f54ea9 100644
--- a/cmd/mvebu/comphy_rx_training.c
+++ b/cmd/mvebu/comphy_rx_training.c
@@ -22,7 +22,7 @@ int mvebu_comphy_rx_training_cmd(struct cmd_tbl *cmdtp, int flag, int argc,
 
 	if (argc != 3) {
 		printf("missing arguments\n");
-		return -1;
+		return CMD_RET_USAGE;
 	}
 
 	cp_index = hextoul(argv[1], NULL);
-- 
2.31.1.windows.1



More information about the U-Boot mailing list