lkml.org 
[lkml]   [2019]   [Oct]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH V6 3/8] backlight: qcom-wled: Add new properties for PMI8998
On 2019-10-01 20:42, Dan Murphy wrote:
> Kiran
>
> On 9/30/19 1:39 AM, Kiran Gunda wrote:
>> Update the bindings with the new properties used for
>> PMI8998.
>>
>> Signed-off-by: Kiran Gunda <kgunda@codeaurora.org>
>> Reviewed-by: Bjorn Andersson <bjorn.andersson@linaro.org>
>> Reviewed-by: Rob Herring <robh@kernel.org>
>> Acked-by: Daniel Thompson <daniel.thompson@linaro.org>
>> ---
>> .../bindings/leds/backlight/qcom-wled.txt | 76
>> ++++++++++++++++++----
>> 1 file changed, 62 insertions(+), 14 deletions(-)
>>
>> diff --git
>> a/Documentation/devicetree/bindings/leds/backlight/qcom-wled.txt
>> b/Documentation/devicetree/bindings/leds/backlight/qcom-wled.txt
>> index 14f28f2..9d840d5 100644
>> --- a/Documentation/devicetree/bindings/leds/backlight/qcom-wled.txt
>> +++ b/Documentation/devicetree/bindings/leds/backlight/qcom-wled.txt
>> @@ -20,8 +20,7 @@ platforms. The PMIC is connected to the host
>> processor via SPMI bus.
>> - default-brightness
>> Usage: optional
>> Value type: <u32>
>> - Definition: brightness value on boot, value from: 0-4095
>> - Default: 2048
>> + Definition: brightness value on boot, value from: 0-4095.
>> - label
>> Usage: required
>> @@ -48,20 +47,24 @@ platforms. The PMIC is connected to the host
>> processor via SPMI bus.
>> - qcom,current-limit
>> Usage: optional
>> Value type: <u32>
>> - Definition: mA; per-string current limit
>> - value: For pm8941: from 0 to 25 with 5 mA step
>> - Default 20 mA.
>> - For pmi8998: from 0 to 30 with 5 mA step
>> - Default 25 mA.
>> + Definition: mA; per-string current limit; value from 0 to 25 with
>> + 1 mA step.
>> + This property is supported only for pm8941.
>> +
>> +- qcom,current-limit-microamp
>> + Usage: optional
>> + Value type: <u32>
>> + Definition: uA; per-string current limit; value from 0 to 30000
>> with
>> + 2500 uA step.
>> - qcom,current-boost-limit
>> Usage: optional
>> Value type: <u32>
>> Definition: mA; boost current limit.
>> For pm8941: one of: 105, 385, 525, 805, 980, 1260, 1400,
>> - 1680. Default: 805 mA
>> + 1680.
>> For pmi8998: one of: 105, 280, 450, 620, 970, 1150, 1300,
>> - 1500. Default: 970 mA
>> + 1500.
>> - qcom,switching-freq
>> Usage: optional
>> @@ -69,22 +72,66 @@ platforms. The PMIC is connected to the host
>> processor via SPMI bus.
>> Definition: kHz; switching frequency; one of: 600, 640, 685,
>> 738,
>> 800, 872, 960, 1066, 1200, 1371, 1600, 1920, 2400, 3200,
>> 4800, 9600.
>> - Default: for pm8941: 1600 kHz
>> - for pmi8998: 800 kHz
>> - qcom,ovp
>> Usage: optional
>> Value type: <u32>
>> Definition: V; Over-voltage protection limit; one of:
>> - 27, 29, 32, 35. default: 29V
>> + 27, 29, 32, 35.
>> This property is supported only for PM8941.
>> +- qcom,ovp-millivolt
>> + Usage: optional
>> + Value type: <u32>
>> + Definition: mV; Over-voltage protection limit;
>> + For pmi8998: one of 18100, 19600, 29600, 31100
>> + If this property is not specified for PM8941, it
>> + falls back to "qcom,ovp" property.
>> +
>> - qcom,num-strings
>> Usage: optional
>> Value type: <u32>
>> Definition: #; number of led strings attached;
>> - value from 1 to 3. default: 2
>> - This property is supported only for PM8941.
>> + value: For PM8941 from 1 to 3.
>> + For PMI8998 from 1 to 4.
>
> We probably don't need this since we define 1 led node per output. 
> And if you need to define
>
> multiple strings per node then you use led-sources.
>
> Then you will use fwnode_property_count_u32(child, "led-sources"); to
> get the number of outputs
>
>
I have kept this property as is to have the backward compatibility with
pm8941-wled.
The backward compatibility is broken if this property is removed.

>> +
>> +- interrupts
>> + Usage: optional
>> + Value type: <prop encoded array>
>> + Definition: Interrupts associated with WLED. This should be
>> + "short" and "ovp" interrupts. Interrupts can be
>> + specified as per the encoding listed under
>> + Documentation/devicetree/bindings/spmi/
>> + qcom,spmi-pmic-arb.txt.
>> +
>> +- interrupt-names
>> + Usage: optional
>> + Value type: <string>
>> + Definition: Interrupt names associated with the interrupts.
>> + Must be "short" and "ovp". The short circuit detection
>> + is not supported for PM8941.
>> +
>> +- qcom,enabled-strings
>> + Usage: optional
>> + Value tyoe: <u32 array>
>> + Definition: Array of the WLED strings numbered from 0 to 3. Each
>> + string of leds are operated individually. Specify the
>> + list of strings used by the device. Any combination of
>> + led strings can be used.
>
> We usually use the reg property per led node to denote what output is
> associated with which
>
> property node.  And if you want to define multiple outputs per node
> then you need to use
>
> led-sources
>
> See leds-lm3697.txt for an example
>
> Dan
As per the Qualcomm hardware recommendation , all the strings (leds)
properties like
OVP, FSC and brightness etc .. should have the same values. That's why
all the strings (leds)
and it's properties are mentioned in a single node.


\
 
 \ /
  Last update: 2019-10-15 10:55    [W:0.077 / U:0.776 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site