Re: [PATCH V1 1/2] leds: leds-qti-rgb: Add LED driver for QTI TRI_LED module

2017-06-01 Thread Jacek Anaszewski
Hi Wu, On 06/01/2017 02:30 AM, Wu Fenglin wrote: > Thanks Pavel. > I will remove these two dts properties and define the default on/off time > values in C code. There are many things to sort out regarding your patch. I'll be able to give you a detailed feedback probably no sooner than at the week

RE: [PATCH V1 1/2] leds: leds-qti-rgb: Add LED driver for QTI TRI_LED module

2017-05-31 Thread Wu Fenglin
Thanks Pavel. I will remove these two dts properties and define the default on/off time values in C code. -- Qualcomm Technologies, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project.

Re: [PATCH V1 1/2] leds: leds-qti-rgb: Add LED driver for QTI TRI_LED module

2017-05-31 Thread Pavel Machek
Hi! > Thanks for the reviewing. > > For the order, the hardware register mapping has this order (blue/green/red) > from bit0/1/2, I can revert it to (red/green/blue) if there is a strong > concern. I'd do that. > For these two properties: qcom,off-ms/ qcom,on-ms, I am using them to assign > the

RE: [PATCH V1 1/2] leds: leds-qti-rgb: Add LED driver for QTI TRI_LED module

2017-05-31 Thread Wu Fenglin
el.org; linux-kernel@vger.kernel.org; Richard Purdie ; Jacek Anaszewski ; Rob Herring ; Mark Rutland ; linux-l...@vger.kernel.org; devicet...@vger.kernel.org; subba...@quicinc.com; agha...@qti.qualcomm.com; wr...@quicinc.com; kgu...@qti.qualcomm.com Subject: Re: [PATCH V1 1/2] leds: leds-qti-rgb: Add L

Re: [PATCH V1 1/2] leds: leds-qti-rgb: Add LED driver for QTI TRI_LED module

2017-05-31 Thread Pavel Machek
Hi! > +- qcom,support-blink: > + Usage: optional > + Value type: > + Definition: An array of integer values to indicate if "blue", "green", > "red" > + LEDs support blink control. The values are listed as the fixed > + order for "blue", "green", "red" LEDs. N