On 27.07.20 14:02, Pavel Machek wrote:
> Hi!
> 
>> +    pwmleds {
>> +            compatible = "pwm-leds";
>> +
>> +            blue {
>> +                    label = "blue:status";
>> +                    max-brightness = <248>;
>> +                    pwms = <&pwm2 0 50000>;
>> +            };
>> +
>> +            green {
>> +                    label = "green:status";
>> +                    max-brightness = <248>;
>> +                    pwms = <&pwm4 0 50000>;
>> +            };
>> +
>> +            red {
>> +                    label = "red:status";
>> +                    max-brightness = <248>;
>> +                    pwms = <&pwm3 0 50000>;
>> +            };
>> +    };
> 
> Can we postpone this?

I guess we can. It's indeed easier to add that interface once we know we
can use a stable one.

> 
> 1) green:status is not really name used anywhere else
> 
> 2) this should really use multicolor framework, as it is single LED,
> not three of them.
> 
> Fixing that in future will be hard :-(.
>                                                               Pavel
>                                                               
> 

thanks,
                              martin

Reply via email to