<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=koi8-r">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<style type="text/css" style="display:none;"><!-- P {margin-top:0;margin-bottom:0;} --></style>
<div id="divtagdefaultwrapper" style="font-size:12pt;color:#000000;font-family:Calibri,Arial,Helvetica,sans-serif;">
<p></p>
<div>
<p> Hi all!<br>
</p>
<p> There are some ideas for setting led color in DBus. Led nodes in DBus are created by phosphor-led-sysfs.
<span>phosphor-led-sysfs is started by udev rule for the sysfs led. Suggestion is to add enviroment file (e.x. /etc/default/obmc/leds/led.conf), which contains colors for all system leds. The conf file could be look like:</span></p>
<p><span></span></p>
<div><br>
</div>
<div>COLOR_alarm_red = "Red"<br>
COLOR_alarm_yel = "Yellow"<br>
COLOR_heartbeat = "Green"<br>
COLOR_id_blue = "Blue"<br>
COLOR_power_red = "Red"</div>
<p><br>
</p>
<span>phosphor-led-sysfs</span> looks for led color in the enviroment variables and sets led color.</div>
<br>
<p></p>
<p> Regards,</p>
<p> Alexnader Soldatov.<br>
</p>
</div>
<hr style="display:inline-block;width:98%" tabindex="-1">
<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" style="font-size:11pt" color="#000000"><b>От:</b> Alexander Amelkin<br>
<b>Отправлено:</b> 6 ноября 2018 г. 16:41:57<br>
<b>Кому:</b> openbmc@lists.ozlabs.org<br>
<b>Копия:</b> Alexander Soldatov; Vishwanatha Subbanna<br>
<b>Тема:</b> LED colors in d-bus</font>
<div> </div>
</div>
<div>
<p>Hi all!<br>
</p>
<p>Since commit a1e70a7af by <span><span role="listitem" data-id="1000014" title="vishwa@linux.vnet.ibm.com Votable:
Code-Review, Verified, Ok-To-Test" class="com-google-gerrit-client-change-ChangeScreen_BinderImpl_GenCss_style-label_user">Vishwanatha
Subbanna</span></span> to phosphor-dbus-interfaces (see <a class="moz-txt-link-freetext" href="https://gerrit.openbmc-project.xyz/#/c/openbmc/phosphor-dbus-interfaces/+/1136/">
https://gerrit.openbmc-project.xyz/#/c/openbmc/phosphor-dbus-interfaces/+/1136/</a> ), there is a property 'Color' for leds.</p>
<p>However, it looks like nothing is setting that property for any of the supported platforms. So any led always has it in d-bus set to Unknown.</p>
<p>It is unclear to us what was the intention for adding that property. It has not been explained in the commit log for the commit. It is also unclear what could be the correct way to set that property. phosphor-led-manager operates on groups. A single led
can be a part of multiple groups, so 'led.yaml' in phosphor-led-manager looks like a wrong place to attach a color property to leds. Each leds used in led.yaml is called by the name taken from Linux devicetree. However, the bindings for leds in Linux devicetree
do not imply color information as well.</p>
<p>Are there any hints on how to set that property the right way? Can you imagine any real use cases for the color property? Maybe we should just ditch it completely? It's been 2 years since it has been introduced and nobody seems to have cared...<br>
</p>
<p>Thanks.</p>
<p>Alexander.<br>
</p>
</div>
</body>
</html>