Re: [PATCH 1/2] dt-bindings: Add vendor prefix and docs for CR0014114

[Jacek Anaszewski <jacek.anaszewski@gmail.com>]

On 03/28/2018 08:36 AM, Oleh Kravchenko wrote:
> Hello Jacek,
> 
> On 27.03.18 23:58, Jacek Anaszewski wrote:
>>> +Example
>>> +-------
>>> +
>>> +led-controller@0 {
>>> +	compatible = "crane,cr0014114";
>>> +	reg = <0>;
>>> +	spi-max-frequency = <50000>;
>>> +	#address-cells = <1>;
>>> +	#size-cells = <0>;
>>> +
>>> +	led@0 {
>>> +		reg = <0>;
>>> +		label = "cr0:red:coin";
>>> +	};
>>> +	led@1 {
>>> +		reg = <1>;
>>> +		label = "cr0:green:coin";
>>> +	};
>>> +	led@2 {
>>> +		reg = <2>;
>>> +		label = "cr0:blue:coin";
>>> +	};
>>> +	led@3 {
>>> +		reg = <3>;
>>> +		label = "cr1:red:bill";
>>> +	};
>>> +	led@4 {
>>> +		reg = <4>;
>>> +		label = "cr1:green:bill";
>>> +	};
>>> +	led@5 {
>>> +		reg = <5>;
>>> +		label = "cr1:blue:bill";
>>> +	};
>>
>> Why cr0 and cr1? It should be cr0014114 to stick to the
>> current LED naming pattern <devicename:colour:function>.
>>
>> Nonetheless, we lately came to the conclusion that devicename
>> segment is redundant in LED class device name, so until we change
>> LED naming convention officially, let's remove devicename segment
>> at least from DT and prepend the label with "cr0014114" in the driver.
>>
>> Please compare how it is approached in [0] (not merged yet).
>>
> 
> It's just example.
> But anyway our applications works with LEDs by numbers.
> 
> Using function names instead numbers will increase code complexity,
> so we use numbers :)

This is LED class device naming convention in mainline and examples
also must stick to it to keep the things consistent.

Please switch labels so that they matched the pattern

label = "color:function";

Regardless of how LEDs are named in your port of kernel,
they for sure have some functions for the vending machine
user. Just use those names in DT :-)

Best regards,
Jacek Anaszewski

>>> +};
>>> diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt
>>> index ae850d6c0ad3..f17949c365f5 100644
>>> --- a/Documentation/devicetree/bindings/vendor-prefixes.txt
>>> +++ b/Documentation/devicetree/bindings/vendor-prefixes.txt
>>> @@ -75,6 +75,7 @@ cnxt	Conexant Systems, Inc.
>>>  compulab	CompuLab Ltd.
>>>  cortina	Cortina Systems, Inc.
>>>  cosmic	Cosmic Circuits
>>> +crane	Crane Connectivity Solutions
>>>  creative	Creative Technology Ltd
>>>  crystalfontz	Crystalfontz America, Inc.
>>>  cubietech	Cubietech, Ltd.
>>>
>>
>> [0] https://www.spinics.net/lists/devicetree/msg220561.html
>>
>