Patchwork [U-Boot,2/3] led: The gpio_led.c code expects that LED state is from the enum

login
register
mail settings
Submitter Holger Freyther
Date Jan. 18, 2013, 3:19 p.m.
Message ID <1358522390-10161-2-git-send-email-holger@freyther.de>
Download mbox | patch
Permalink /patch/213646/
State Changes Requested
Delegated to: Tom Rini
Headers show

Comments

Holger Freyther - Jan. 18, 2013, 3:19 p.m.
u-boot is not consistent if state should be 0|1 or of the enum, the
GPIO driver expects this to be one of the enum values. Update the
caller.
---
 common/cmd_led.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Patch

diff --git a/common/cmd_led.c b/common/cmd_led.c
index 7f5ab43..c725f95 100644
--- a/common/cmd_led.c
+++ b/common/cmd_led.c
@@ -110,13 +110,13 @@  int do_led (cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
 				if (led_commands[i].on)
 					led_commands[i].on();
 				else
-					__led_set(led_commands[i].mask, 1);
+					__led_set(led_commands[i].mask, STATUS_LED_ON);
 				break;
 			case LED_OFF:
 				if (led_commands[i].off)
 					led_commands[i].off();
 				else
-					__led_set(led_commands[i].mask, 0);
+					__led_set(led_commands[i].mask, STATUS_LED_OFF);
 				break;
 			case LED_TOGGLE:
 				if (led_commands[i].toggle)