On 3/15/26 10:49 AM, Krzysztof Kozlowski wrote:
> On Sat, Mar 14, 2026 at 06:06:30PM +0200, Erikas Bitovtas wrote:
>> This commit adds supply properties for the sensor, I2C bus and IR LED
>
> Please do not use "This commit/patch/change", but imperative mood. See
> longer explanation here:
> https://elixir.bootlin.com/linux/v6.16/source/Documentation/process/submitting-patches.rst#L94
>
>> anode supplies [1], which can be found in vcnl4000 ambient light and
>
> So other devices do not have these supplies? Or have only some?
>
> You already received that question, so I assume you checked. Then check
> again because 4040 has VDD.
>
VLED is supposed to be the IR anode pin. Every model has one. It is pin
1 on vcnl4000, vcnl4010 and vcnl4020, pin 6 on vcnl4200 and pin 4 on
vcnl4040.
VDD pin is pin 7 on vcnl4000 and vcnl4010, pin 5 on vcnl4020 and pin 3
on vcnl4040 and vcnl4200.
VDDIO is an external supply to the I2C bus for the sensor.
> thus in
> that case
>
>> proximity sensors, to ensure the sensor is powered on before proceeding
>> with setup.
>>
>> [1] https://www.vishay.com/docs/84274/vcnl4040.pdf
>>
>> Signed-off-by: Erikas Bitovtas <[email protected]>
>> ---
>> Documentation/devicetree/bindings/iio/light/vishay,vcnl4000.yaml | 7 +++++++
>> 1 file changed, 7 insertions(+)
>>
>> diff --git
>> a/Documentation/devicetree/bindings/iio/light/vishay,vcnl4000.yaml
>> b/Documentation/devicetree/bindings/iio/light/vishay,vcnl4000.yaml
>> index 2ba4d5de4ec4..a1b4c02db246 100644
>> --- a/Documentation/devicetree/bindings/iio/light/vishay,vcnl4000.yaml
>> +++ b/Documentation/devicetree/bindings/iio/light/vishay,vcnl4000.yaml
>> @@ -33,6 +33,10 @@ properties:
>> interrupts:
>> maxItems: 1
>>
>> + vdd-supply: true
>> + vddio-supply: true
>
> VCNL4000 does not have VDDIO and VLED pins.
>
> Best regards,
> Krzysztof
>
