Re: [PATCH v11 01/16] dt: bindings: Add multicolor class dt bindings documention

[Jacek Anaszewski]

Dan,

On 10/10/19 2:49 AM, Dan Murphy wrote:
> Jacek
> 
> On 10/9/19 5:07 PM, Jacek Anaszewski wrote:
>> Dan,
>>
>> On 10/8/19 10:47 PM, Dan Murphy wrote:
>>> Add DT bindings for the LEDs multicolor class framework.
>>>
>>> Signed-off-by: Dan Murphy <dmurphy@xxxxxx>
>>> ---
>>> Â .../bindings/leds/leds-class-multicolor.txtÂÂ | 98 +++++++++++++++++++
>>> Â 1 file changed, 98 insertions(+)
>>> Â create mode 100644
>>> Documentation/devicetree/bindings/leds/leds-class-multicolor.txt
>>>
>>> diff --git
>>> a/Documentation/devicetree/bindings/leds/leds-class-multicolor.txt
>>> b/Documentation/devicetree/bindings/leds/leds-class-multicolor.txt
>>> new file mode 100644
>>> index 000000000000..8619c9bf1811
>>> --- /dev/null
>>> +++ b/Documentation/devicetree/bindings/leds/leds-class-multicolor.txt
>>> @@ -0,0 +1,98 @@
>>> +* Multicolor LED properties
>>> +
>>> +Bindings for multi color LEDs show how to describe current outputs of
>>> +either integrated multi-color LED elements (like RGB, RGBW, RGBWA-UV
>>> +etc.) or standalone LEDs, to achieve logically grouped multi-color LED
>>> +modules. This is achieved by adding multi-led nodes layer to the
>>> +monochrome LED bindings.
>>> +
>>> +The nodes and properties defined in this document are unique to the
>>> multicolor
>>> +LED class. Common LED nodes and properties are inherited from the
>>> common.txt
>>> +within this documentation directory.
>>> +
>>> +Required LED Child properties:
>> s/Child/child/
> 
> Ack
> 
> 
>>
>>> +ÂÂÂ - color : For multicolor LED support this property should be
>>> defined as
>>> +ÂÂÂÂÂÂÂÂÂ LED_COLOR_ID_MULTI and further definition can be found in
>>> +ÂÂÂÂÂÂÂÂÂ include/linux/leds/common.h.
>>> +
>>> +led-controller@30 {
>>> +ÂÂÂ #address-cells = <1>;
>>> +ÂÂÂ #size-cells = <0>;
>>> +ÂÂÂ compatible = "ti,lp5024";
>>> +ÂÂÂ reg = <0x29>;
>>> +
>>> +ÂÂÂ multi-led@1 {
>>> +ÂÂÂÂÂÂÂ #address-cells = <1>;
>>> +ÂÂÂÂÂÂÂ #size-cells = <0>;
>>> +ÂÂÂÂÂÂÂ reg = <1>;
>>> +ÂÂÂÂÂÂÂ color = <LED_COLOR_ID_MULTI>;
>>> +ÂÂÂÂÂÂÂ function = LED_FUNCTION_STATUS;
>> Status is not too fancy function for multi color LED ;-)
>> I'd skip it entirely for this example if we don't have
>> anything suitable at the moment for our disposal.
> 
> Not sure I understand. Status is a good example as a RGB module can be
> used to present charging status

So, maybe LED_FUNCTION_CHARGING ? This implies that color can change
gradually during charging. However me might think of adding
LED_FUNCTION_VBAT or so.

-- 
Best regards,
Jacek Anaszewski