<feed xmlns='http://www.w3.org/2005/Atom'>
<title>u-boot.git/common/cmd_led.c, branch v2011.09-rc2</title>
<subtitle>Unnamed repository; edit this file 'description' to name the repository.</subtitle>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/ausil/public_git/u-boot.git/'/>
<entry>
<title>led: remove camel casing of led identifiers globally</title>
<updated>2011-09-13T06:30:52+00:00</updated>
<author>
<name>Jason Kridner</name>
<email>jkridner@beagleboard.org</email>
</author>
<published>2011-09-04T18:40:16+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/ausil/public_git/u-boot.git/commit/?id=2d3be7c456f9d0bcfd4d4b0515aecd32d6a06037'/>
<id>2d3be7c456f9d0bcfd4d4b0515aecd32d6a06037</id>
<content type='text'>
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 &lt;jkridner@beagleboard.org&gt;
Signed-off-by: Joel A Fernandes &lt;agnel.joel@gmail.com&gt;
Signed-off-by: Sandeep Paulraj &lt;s-paulraj@ti.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
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 &lt;jkridner@beagleboard.org&gt;
Signed-off-by: Joel A Fernandes &lt;agnel.joel@gmail.com&gt;
Signed-off-by: Sandeep Paulraj &lt;s-paulraj@ti.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>led: Remove state-saving of led for toggle functionality and add toggle option to led command</title>
<updated>2011-09-04T09:36:22+00:00</updated>
<author>
<name>Joel A Fernandes</name>
<email>agnel.joel@gmail.com</email>
</author>
<published>2011-08-12T04:16:53+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/ausil/public_git/u-boot.git/commit/?id=b8bc8973a1830bb92e7a9bf3356dc209afb2f4e8'/>
<id>b8bc8973a1830bb92e7a9bf3356dc209afb2f4e8</id>
<content type='text'>
* 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 &lt;agnel.joel@gmail.com&gt;
Signed-off-by: Jason Kridner &lt;jkridner@beagleboard.org&gt;
Signed-off-by: Sandeep Paulraj &lt;s-paulraj@ti.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
* 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 &lt;agnel.joel@gmail.com&gt;
Signed-off-by: Jason Kridner &lt;jkridner@beagleboard.org&gt;
Signed-off-by: Sandeep Paulraj &lt;s-paulraj@ti.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>led: Fixed setting of STATUS_LED_BIT1 when led_name is 'all'</title>
<updated>2011-09-04T09:36:22+00:00</updated>
<author>
<name>Joel A Fernandes</name>
<email>agnel.joel@gmail.com</email>
</author>
<published>2011-08-12T00:10:19+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/ausil/public_git/u-boot.git/commit/?id=d604cda34ab2190fb07a1134bb488105e1a2f9c3'/>
<id>d604cda34ab2190fb07a1134bb488105e1a2f9c3</id>
<content type='text'>
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 &lt;agnel.joel@gmail.com&gt;
Signed-off-by: Jason Kridner &lt;jkridner@beagleboard.org&gt;
Cc: Sergei Shtylyov &lt;sshtylyov@mvista.com&gt;
Signed-off-by: Sandeep Paulraj &lt;s-paulraj@ti.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
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 &lt;agnel.joel@gmail.com&gt;
Signed-off-by: Jason Kridner &lt;jkridner@beagleboard.org&gt;
Cc: Sergei Shtylyov &lt;sshtylyov@mvista.com&gt;
Signed-off-by: Sandeep Paulraj &lt;s-paulraj@ti.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>led: correct off/on locations in structure</title>
<updated>2011-09-04T09:36:22+00:00</updated>
<author>
<name>Jason Kridner</name>
<email>jkridner@beagleboard.org</email>
</author>
<published>2011-04-20T23:13:49+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/ausil/public_git/u-boot.git/commit/?id=4086b51cb7eac451f96130affbc17094a9e7b75a'/>
<id>4086b51cb7eac451f96130affbc17094a9e7b75a</id>
<content type='text'>
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 &lt;jkridner@beagleboard.org&gt;
Signed-off-by: Koen Kooi &lt;koen@dominion.thruhere.net&gt;
Signed-off-by: Joel A Fernandes &lt;agnel.joel@gmail.com&gt;
Signed-off-by: Sandeep Paulraj &lt;s-paulraj@ti.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
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 &lt;jkridner@beagleboard.org&gt;
Signed-off-by: Koen Kooi &lt;koen@dominion.thruhere.net&gt;
Signed-off-by: Joel A Fernandes &lt;agnel.joel@gmail.com&gt;
Signed-off-by: Sandeep Paulraj &lt;s-paulraj@ti.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Corrected LED name match finding avoiding extraneous Usage printouts</title>
<updated>2011-09-04T09:36:21+00:00</updated>
<author>
<name>Jason Kridner</name>
<email>jkridner@beagleboard.org</email>
</author>
<published>2011-03-09T02:42:01+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/ausil/public_git/u-boot.git/commit/?id=95492d784b6167a5de47f7aae691646e99f5b623'/>
<id>95492d784b6167a5de47f7aae691646e99f5b623</id>
<content type='text'>
Signed-off-by: Jason Kridner &lt;jkridner@beagleboard.org&gt;
Signed-off-by: Koen Kooi &lt;koen@dominion.thruhere.net&gt;
Signed-off-by: Joel A Fernandes &lt;agnel.joel@gmail.com&gt;
Signed-off-by: Sandeep Paulraj &lt;s-paulraj@ti.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Signed-off-by: Jason Kridner &lt;jkridner@beagleboard.org&gt;
Signed-off-by: Koen Kooi &lt;koen@dominion.thruhere.net&gt;
Signed-off-by: Joel A Fernandes &lt;agnel.joel@gmail.com&gt;
Signed-off-by: Sandeep Paulraj &lt;s-paulraj@ti.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Minor coding style cleanup.</title>
<updated>2011-05-19T20:22:44+00:00</updated>
<author>
<name>Wolfgang Denk</name>
<email>wd@denx.de</email>
</author>
<published>2011-05-19T20:21:41+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/ausil/public_git/u-boot.git/commit/?id=cd6881b519c94b15907d4f28149e5ed3fd0beea5'/>
<id>cd6881b519c94b15907d4f28149e5ed3fd0beea5</id>
<content type='text'>
Signed-off-by: Wolfgang Denk &lt;wd@denx.de&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Signed-off-by: Wolfgang Denk &lt;wd@denx.de&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Add 'led' command</title>
<updated>2011-04-27T17:38:05+00:00</updated>
<author>
<name>Jason Kridner</name>
<email>jkridner@beagleboard.org</email>
</author>
<published>2011-04-18T21:22:44+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/ausil/public_git/u-boot.git/commit/?id=b633f66f02ab3df6adc8fc65f13769e5fb9c328f'/>
<id>b633f66f02ab3df6adc8fc65f13769e5fb9c328f</id>
<content type='text'>
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 &lt;jkridner@beagleboard.org&gt;
Signed-off-by: Sandeep Paulraj &lt;s-paulraj@ti.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
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 &lt;jkridner@beagleboard.org&gt;
Signed-off-by: Sandeep Paulraj &lt;s-paulraj@ti.com&gt;
</pre>
</div>
</content>
</entry>
</feed>
