[U-Boot] [PATCH] cmd_mmc: fix arg parsing for setdsr subcmd

Markus Niebel niebelm at tqsc.de
Fri Feb 3 14:26:36 UTC 2017


From: Markus Niebel <Markus.Niebel at tq-group.com>

The handler do_setdsr receives only the dsr parameter,
the action is parsed before.

Error was introduced when restructuring the mmc command
implementation in commit 1fd93c6e7d8a1e4d6261058cefab11b875ded252.

Reported-by: Michael Krummsdorf <Michael.Krummsdorf at tq-group.com>
Signed-off-by: Markus Niebel <Markus.Niebel at tq-group.com>
---
 cmd/mmc.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/cmd/mmc.c b/cmd/mmc.c
index b8dcc26..f83032e 100644
--- a/cmd/mmc.c
+++ b/cmd/mmc.c
@@ -710,7 +710,7 @@ static int do_mmc_setdsr(cmd_tbl_t *cmdtp, int flag,
 
 	if (argc != 2)
 		return CMD_RET_USAGE;
-	val = simple_strtoul(argv[2], NULL, 16);
+	val = simple_strtoul(argv[1], NULL, 16);
 
 	mmc = find_mmc_device(curr_device);
 	if (!mmc) {
-- 
1.9.1



More information about the U-Boot mailing list