Re: [PATCH v3 1/4] dt-bindings: backlight: Add max25014 bindingsy

[Maud Spierings <maudspierings@gocontroll.com>]

On 9/12/25 17:22, Frank Li wrote:
> On Fri, Sep 12, 2025 at 08:17:09AM +0200, Maud Spierings wrote:
>> Hi Frank,
>> Thanks for the review.
>>
>> On 9/11/25 17:33, Frank Li wrote:
>>> On Thu, Sep 11, 2025 at 09:53:18AM +0200, Maud Spierings via B4 Relay wrote:
>>>> From: Maud Spierings <maudspierings@gocontroll.com>
>>>>
>>>> The Maxim MAX25014 is a 4-channel automotive grade backlight driver IC
>>>> with integrated boost controller.
>>>>
>>>> Signed-off-by: Maud Spierings <maudspierings@gocontroll.com>
>>>> ---
>>>>    .../bindings/leds/backlight/maxim,max25014.yaml    | 81 ++++++++++++++++++++++
>>>>    MAINTAINERS                                        |  5 ++
>>>>    2 files changed, 86 insertions(+)
>>>>
>>>> diff --git a/Documentation/devicetree/bindings/leds/backlight/maxim,max25014.yaml b/Documentation/devicetree/bindings/leds/backlight/maxim,max25014.yaml
>>>> new file mode 100644
>>>> index 0000000000000000000000000000000000000000..e113a2ad16aa74f982b9c2ea80578aed2d9424fe
>>>> --- /dev/null
>>>> +++ b/Documentation/devicetree/bindings/leds/backlight/maxim,max25014.yaml
>>>> @@ -0,0 +1,81 @@
>>>> +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
>>>> +%YAML 1.2
>>>> +---
>>>> +$id: http://devicetree.org/schemas/leds/backlight/maxim,max25014.yaml#
>>>> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>>>> +
>>>> +title: Maxim max25014 backlight controller
>>>> +
>>>> +maintainers:
>>>> +  - Maud Spierings <maudspierings@gocontroll.com>
>>>> +
>>>> +allOf:
>>>> +  - $ref: common.yaml#
>>>> +
>>>> +properties:
>>>> +  compatible:
>>>> +    enum:
>>>> +      - maxim,max25014
>>>> +
>>>> +  reg:
>>>> +    maxItems: 1
>>>> +
>>>> +  enable-gpios:
>>>> +    maxItems: 1
>>>> +
>>>> +  interrupts:
>>>> +    maxItems: 1
>>>> +
>>>> +  power-supply:
>>>> +    description: Regulator which controls the boost converter input rail.
>>>> +
>>>> +  pwms:
>>>> +    maxItems: 1
>>>> +
>>>> +  maxim,iset:
>>>> +    $ref: /schemas/types.yaml#/definitions/uint32
>>>> +    maximum: 15
>>>> +    default: 11
>>>> +    description:
>>>> +      Value of the ISET register field. This controls the current scale of the
>>>> +      outputs, a higher number means more current.
>>>
>>> Use standard unit. Do not use register value directly.
>>
>> It is unfortunately not just a value in Amps, it depends on the hardware
>> design. There is a kind of "default" table with a 49.9K resistor, but
>> depending on that resistor the current is different.
> 
> You should calculate in your driver. if 49.9K is dependence, you should
> add xxx_ohm at dts.

I've tried to find the logic behind the Riref resistor and its 
values/effects, but there is no formula for it, there are example values 
for 49.9k and 40.2k, besides that all that is stated that the minimum 
allowed value is 27.5k and the maximum value is 83.5k.

Not sure how to continue after that, I cannot verify/approximate any 
relation with only two data points.
Kind regards,
Maud