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
