lkml.org 
[lkml]   [2017]   [May]   [31]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRE: [PATCH V1 1/2] leds: leds-qti-rgb: Add LED driver for QTI TRI_LED module
Date
Hi Pavel,

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.

For these two properties: qcom,off-ms/ qcom,on-ms, I am using them to assign
the default blinking on/off time, then the LEDs would have a default
blinking pattern if you do "echo 1 > /sys/class/leds/red/blink"

Fenglin Wu

-----Original Message-----
From: Pavel Machek [mailto:pavel@ucw.cz]
Sent: Wednesday, May 31, 2017 3:56 PM
To: fenglinw@codeaurora.org
Cc: linux-arm-msm@vger.kernel.org; linux-kernel@vger.kernel.org; Richard
Purdie <rpurdie@rpsys.net>; Jacek Anaszewski <jacek.anaszewski@gmail.com>;
Rob Herring <robh+dt@kernel.org>; Mark Rutland <mark.rutland@arm.com>;
linux-leds@vger.kernel.org; devicetree@vger.kernel.org;
subbaram@quicinc.com; aghayal@qti.qualcomm.com; wruan@quicinc.com;
kgunda@qti.qualcomm.com
Subject: Re: [PATCH V1 1/2] leds: leds-qti-rgb: Add LED driver for QTI
TRI_LED module

Hi!

> +- qcom,support-blink:
> + Usage: optional
> + Value type: <prop-encoded-array>
> + 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.

Normal order is RGB, and no need for the "s.

> +- qcom,on-ms:
> + Usage: optional
> + Value type: <prop-encoded-array>
> + Definition: An array of time values (milli-seconds) to represent the
> + on duration for "blue", "green", "red" LEDs. The values are
> + listed as the fixed order for "blue", "green", "red" LEDs.
> + This property has to be defined if "qcom,support-blink" is
> + present.
> +
> +- qcom,off-ms:
> + Usage: optional
> + Value type: <prop-encoded-array>
> + Definition: An array of time values (milli-seconds) to represent the
> + off duration for "blue", "green", "red" LEDs. The values are
> + listed as the fixed order for "blue", "green", "red" LEDs.
> + This property has to be defined if "qcom,support-blink" is
> + present.

I don't get it; why is this needed?

--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures)
http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

\
 
 \ /
  Last update: 2017-05-31 10:24    [W:0.618 / U:0.504 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site