Re: [PATCH v1 1/2] dt-bindings: hwmon: ina2xx: add supply property
From: Krzysztof Kozlowski
Date: Wed Mar 08 2023 - 05:37:07 EST
On 08/03/2023 11:32, Svyatoslav Ryhel wrote:
> ср, 8 бер. 2023 р. о 12:27 Krzysztof Kozlowski
> <krzysztof.kozlowski@xxxxxxxxxx> пише:
>>
>> On 08/03/2023 10:40, Svyatoslav Ryhel wrote:
>>> Add supply property.
>>
>> You have entire commit msg to explain and give background, but instead
>> there is just sentence duplicating subject. And since you did not
>> explain anything, we have questions... like: INA238 does not have VDD,
>> so this does not look correct.
>>
>
> This is why a regulator is not mandatory. If ina238 does not have vdd
> then one who needs ina238 may omit this prop. How about looking from
> this perspective?
Not required means "optional", not "not existing" or "totally invalid".
Best regards,
Krzysztof