u-boot.git/common/cmd_led.c, branch v2012.04.01

Convert cmd_usage() calls in common to use a return value

2012-03-06T20:09:46+00:00

Change all files in common/ to use CMD_RET_USAGE instead of calling
cmd_usage() directly. I'm not completely sure about this patch since
the code since impact is small (100 byte or so on ARM) and it might
need splitting into smaller patches. But for now here it is.

Signed-off-by: Simon Glass

led: remove camel casing of led identifiers globally

2011-09-13T06:30:52+00:00

Result of running the following command to address Wolfgang's
comment about camel case:

for file in `find . | grep '\.[chS]$'`; do perl -i -pe
's/(green|yellow|red|blue)_LED_(on|off)/$1_led_$2/g' $file; done

Discussion:
http://patchwork.ozlabs.org/patch/84988/

Signed-off-by: Jason Kridner 
Signed-off-by: Joel A Fernandes 
Signed-off-by: Sandeep Paulraj

led: Remove state-saving of led for toggle functionality and add toggle option to led command

2011-09-04T09:36:22+00:00

* Read the led output state from GPIO instead saving state in memory when it is [re]set
* Added a toggle option to the led command

Previous discussion:
http://lists.denx.de/pipermail/u-boot/2011-May/093068.html

Changes since v1:
Fixed checkpatch errors

Signed-off-by: Joel A Fernandes 
Signed-off-by: Jason Kridner 
Signed-off-by: Sandeep Paulraj

led: Fixed setting of STATUS_LED_BIT1 when led_name is 'all'

2011-09-04T09:36:22+00:00

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 
Signed-off-by: Jason Kridner 
Cc: Sergei Shtylyov 
Signed-off-by: Sandeep Paulraj

led: correct off/on locations in structure

2011-09-04T09:36:22+00:00

Although the initialization should probably be done with names, the
existing implementation has these structures filled in the opposite
order.

Signed-off-by: Jason Kridner 
Signed-off-by: Koen Kooi 
Signed-off-by: Joel A Fernandes 
Signed-off-by: Sandeep Paulraj

Corrected LED name match finding avoiding extraneous Usage printouts

2011-09-04T09:36:21+00:00

Signed-off-by: Jason Kridner 
Signed-off-by: Koen Kooi 
Signed-off-by: Joel A Fernandes 
Signed-off-by: Sandeep Paulraj

Minor coding style cleanup.

2011-05-19T20:22:44+00:00

Signed-off-by: Wolfgang Denk

Add 'led' command

2011-04-27T17:38:05+00:00

This patch allows any board implementing the coloured LED API
to control the LEDs from the console.

led [green | yellow | red | all ]  [ on | off ]

or

led [ 1 | 2 | 3 | all ]  [ on | off ]

Adds configuration item CONFIG_CMD_LED enabling the command.

Partially based on patch from Ulf Samuelsson:
http://www.mail-archive.com/u-boot@lists.denx.de/msg09593.html.

Updated based on feedback:
http://www.mail-archive.com/u-boot@lists.denx.de/msg41847.html
https://groups.google.com/d/topic/beagleboard/8Wf1HiK_QBo/discussion
* Fixed a handful of style issues.
* Significantly reduced the number of #ifdefs and redundant code
* Converted redundant code into loops test against a structure
* Made use of cmd_usage()
* Introduced a str_onoff() function, but haven't yet put it in common
* Eliminated trailing newline

Signed-off-by: Jason Kridner 
Signed-off-by: Sandeep Paulraj