Re: [PATCH] iio: light: bh1750: Add hardware reset support via GPIO

[Krzysztof Kozlowski <krzk@kernel.org>]

On 17/03/2025 12:58, Jonathan Cameron wrote:
>> +  reset-gpios:
>> +    description: GPIO connected to the sensor's reset line (active low)
>> +    maxItems: 1
>>  
>>  required:
>>    - compatible
>> @@ -41,5 +48,16 @@ examples:
>>          reg = <0x23>;
>>        };
>>      };
>> +  - |
>> +    i2c {
>> +      #address-cells = <1>;
>> +      #size-cells = <0>;
>> +
>> +      light-sensor@23 {
>> +        compatible = "rohm,bh1750";
>> +        reg = <0x23>;
>> +        reset-gpios = <&gpio2 17 GPIO_ACTIVE_HIGH>;
> Add the GPIO to the existing example rather than having a new one.
... and it is weird to see you claim here is ACTIVE_HIGH but in
description you say active low. It's possible, but unlikely. More likely
this is just wrong, so use proper flag.

Best regards,
Krzysztof