From: Hannes Schmelzer <[email protected]> In commit 0ffe6ab521f900bcc765be8f2f31d2c2ba3f0a7e the abbreviation for the subcommand 'status' was introduced and breaks the 'gpio set' command.
The change runs a test on the first character of the subcommand only, this method does not apply since status set are both beginning with the character 's'. So we have to test at least two characters to be matched, otherwise the condition for "status" is always true. Signed-off-by: Hannes Schmelzer <[email protected]> Signed-off-by: Hannes Schmelzer <[email protected]> --- cmd/gpio.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cmd/gpio.c b/cmd/gpio.c index 2b78b16..693998e 100644 --- a/cmd/gpio.c +++ b/cmd/gpio.c @@ -141,7 +141,7 @@ static int do_gpio(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) #endif if (argc > 0) str_gpio = *argv; - if (!strncmp(str_cmd, "status", 1)) { + if (!strncmp(str_cmd, "status", 2)) { /* Support deprecated gpio_status() */ #ifdef gpio_status gpio_status(); -- 1.9.1 _______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
