Re: [PATCH 2/3] iio: light: Update DT bindings for GP2AP002

["chr[]" <chris@rudorff.com>]

Hi Linus,

Am 16.10.26 um 23:01 schrieb Linus Walleij:
> Hi Christoph,
> 
> thanks for your patch!
> 
> On Fri, Oct 16, 2026 at 4:43 PM Christoph Rudorff <chris@rudorff.com> wrote:
> 
>> +  vled-supply:
>> +    description: LED power supply a phandle to a regulator
> 
> What datasheet is this from?
> 
> My datasheet for the GP2AP002S00F clearly has only
> VIO and VCC, enjoy:
> https://dflund.se/~triad/krad/GP2AP002S00F.pdf

That's the paper I'm looking at. Pin LEDA wants power!
See page 8 "Recommended external circuit"

> 
> There is however two *outputs* from the GP2AP002S00F
> called LEDA and LEDC.
> 
> These read "LED Anode" and "LED Cathode" and is where
> the IR LED is supposed to be connected.

The LED is build in. And it wants power:
see page 4 "Absolute Maximum Ratings", 3x "supply"

Vcc and Vled want the same voltage so they may have the same supply.
In the downstream kernel they #ifdef that case having an optional third 
supply and #ifdef two options for that.

> 
> I'm just puzzled by this patch.
> 

In the end it makes it work on my device.

> Yours,
> Linus Walleij

@all,

thanks for your reviews, there will be version 2, soon.

Regards,

Christoph Rudorff