On 3/17/26 9:20 AM, Krzysztof Kozlowski wrote:
> On Mon, Mar 16, 2026 at 07:19:45PM +0200, Erikas Bitovtas wrote:
>> These sensors can accept 2 supplies - one for the sensor and one for IR
>> LED [1]. Add supply properties for the sensor - 2 for the sensors and
>> one external, for their I2C bus, to ensure the sensor is powered on
>> before proceeding with setup.
>>
>> [1] https://www.vishay.com/docs/84274/vcnl4040.pdf
>>
>> Reviewed-by: David Lechner <[email protected]>
>> 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
>> + vled-supply: true
>
> Same comments as before, I do not see improvements.
>
> Best regards,
> Krzysztof
>
I explained why in v3. The message can be found here:
https://lore.kernel.org/linux-iio/[email protected]/T/#m6da277894cf5c2b473fbeebd4b7bafc7519ebc26