linux-kernel - Re: [PATCH 1/2] leds: add DT binding for Qualcomm PM8941 WLED block

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Tue, 16 Dec 2014 16:54:45 -0800
From:	Bryan Wu <cooloney@...il.com>
To:	Bjorn Andersson <bjorn.andersson@...ymobile.com>
Cc:	Rob Herring <robh+dt@...nel.org>, Pawel Moll <pawel.moll@....com>,
	Mark Rutland <mark.rutland@....com>,
	Ian Campbell <ijc+devicetree@...lion.org.uk>,
	Kumar Gala <galak@...eaurora.org>,
	Richard Purdie <rpurdie@...ys.net>,
	Grant Likely <grant.likely@...aro.org>,
	Courtney Cavin <courtney.cavin@...ymobile.com>,
	"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
	lkml <linux-kernel@...r.kernel.org>,
	Linux LED Subsystem <linux-leds@...r.kernel.org>,
	linux-arm-msm@...r.kernel.org
Subject: Re: [PATCH 1/2] leds: add DT binding for Qualcomm PM8941 WLED block

On Tue, Dec 9, 2014 at 5:19 PM, Bryan Wu <cooloney@...il.com> wrote:
> On Mon, Dec 8, 2014 at 4:22 PM, Bjorn Andersson
> <bjorn.andersson@...ymobile.com> wrote:
>> From: Courtney Cavin <courtney.cavin@...ymobile.com>
>>
>> This adds device tree binding documentation for the WLED ('White' LED)
>> block on Qualcomm's PM8941 PMICs.
>>
>
> Looks good to me, I need device tree reviewer's Ack to merge it.
>

Hi Rob, can you help to review it?

-Bryan

> Thanks,
> -Bryan
>
>> Signed-off-by: Courtney Cavin <courtney.cavin@...ymobile.com>
>> Signed-off-by: Bjorn Andersson <bjorn.andersson@...ymobile.com>
>> ---
>>  .../devicetree/bindings/leds/leds-pm8941-wled.txt  | 43 ++++++++++++++++++++++
>>  1 file changed, 43 insertions(+)
>>  create mode 100644 Documentation/devicetree/bindings/leds/leds-pm8941-wled.txt
>>
>> diff --git a/Documentation/devicetree/bindings/leds/leds-pm8941-wled.txt b/Documentation/devicetree/bindings/leds/leds-pm8941-wled.txt
>> new file mode 100644
>> index 0000000..a85a964
>> --- /dev/null
>> +++ b/Documentation/devicetree/bindings/leds/leds-pm8941-wled.txt
>> @@ -0,0 +1,43 @@
>> +Binding for Qualcomm PM8941 WLED driver
>> +
>> +Required properties:
>> +- compatible: should be "qcom,pm8941-wled"
>> +- reg: slave address
>> +
>> +Optional properties:
>> +- label: The label for this led
>> +  See Documentation/devicetree/bindings/leds/common.txt
>> +- linux,default-trigger: Default trigger assigned to the LED
>> +  See Documentation/devicetree/bindings/leds/common.txt
>> +- qcom,cs-out: bool; enable current sink output
>> +- qcom,cabc: bool; enable content adaptive backlight control
>> +- qcom,ext-gen: bool; use externally generated modulator signal to dim
>> +- qcom,current-limit: mA; per-string current limit; value from 0 to 25
>> +       default: 20mA
>> +- qcom,current-boost-limit: mA; boost current limit; one of:
>> +       105, 385, 525, 805, 980, 1260, 1400, 1680
>> +       default: 805mA
>> +- qcom,switching-freq: kHz; switching frequency; one of:
>> +       600, 640, 685, 738, 800, 872, 960, 1066, 1200, 1371,
>> +       1600, 1920, 2400, 3200, 4800, 9600,
>> +       default: 1600kHz
>> +- qcom,ovp: V; Over-voltage protection limit; one of:
>> +       27, 29, 32, 35
>> +       default: 29V
>> +- qcom,num-strings: #; number of led strings attached; value from 1 to 3
>> +       default: 2
>> +
>> +Example:
>> +
>> +pm8941-wled@...0 {
>> +       compatible = "qcom,pm8941-wled";
>> +       reg = <0xd800>;
>> +       label = "backlight";
>> +
>> +       qcom,cs-out;
>> +       qcom,current-limit = <20>;
>> +       qcom,current-boost-limit = <805>;
>> +       qcom,switching-freq = <1600>;
>> +       qcom,ovp = <29>;
>> +       qcom,num-strings = <2>;
>> +};
>> --
>> 1.8.2.2
>>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/