From: Hannes Schmelzer hannes.schmelzer@br-automation.com
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 hannes.schmelzer@br-automation.com
Signed-off-by: Hannes Schmelzer oe5hpm@oevsv.at ---
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();