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@xxxxxx] Sent: Wednesday, May 31, 2017 3:56 PM To: fenglinw@xxxxxxxxxxxxxx Cc: linux-arm-msm@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; Richard Purdie <rpurdie@xxxxxxxxx>; Jacek Anaszewski <jacek.anaszewski@xxxxxxxxx>; Rob Herring <robh+dt@xxxxxxxxxx>; Mark Rutland <mark.rutland@xxxxxxx>; linux-leds@xxxxxxxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx; subbaram@xxxxxxxxxxx; aghayal@xxxxxxxxxxxxxxxx; wruan@xxxxxxxxxxx; kgunda@xxxxxxxxxxxxxxxx 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 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
