diff options
| author | Joel A Fernandes <agnel.joel@gmail.com> | 2011-08-11 19:10:19 -0500 |
|---|---|---|
| committer | Albert ARIBAUD <albert.u.boot@aribaud.net> | 2011-09-04 11:36:22 +0200 |
| commit | d604cda34ab2190fb07a1134bb488105e1a2f9c3 (patch) | |
| tree | d154d0c48f92a13144055f212456384abd6f0fda /common/cmd_led.c | |
| parent | 4086b51cb7eac451f96130affbc17094a9e7b75a (diff) | |
led: Fixed setting of STATUS_LED_BIT1 when led_name is 'all'
Fix for only one led getting set or reset when the led_name is 'all'
Previous discussion:
http://lists.denx.de/pipermail/u-boot/2011-May/093068.html
Changes since v1:
Fixed checkpatch if statement error noticed by Sergei.
Signed-off-by: Joel A Fernandes <agnel.joel@gmail.com>
Signed-off-by: Jason Kridner <jkridner@beagleboard.org>
Cc: Sergei Shtylyov <sshtylyov@mvista.com>
Signed-off-by: Sandeep Paulraj <s-paulraj@ti.com>
Diffstat (limited to 'common/cmd_led.c')
| -rw-r--r-- | common/cmd_led.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/common/cmd_led.c b/common/cmd_led.c index d14dd5adab5..183dc0b42f7 100644 --- a/common/cmd_led.c +++ b/common/cmd_led.c @@ -108,7 +108,9 @@ int do_led (cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) } else { __led_set(led_commands[i].mask, state); } - break; + /* Need to set only 1 led if led_name wasn't 'all' */ + if (strcmp("all", argv[1]) != 0) + break; } } |