[PATCH v9 3/9] dt-bindings: power: supply: max17042: remove reg from required

Dzmitry Sankouski posted 9 patches 1 year ago
There is a newer version of this series
[PATCH v9 3/9] dt-bindings: power: supply: max17042: remove reg from required
Posted by Dzmitry Sankouski 1 year ago
Remove 'reg' property from required list, because in platform
variant it should not be present, because i2c client is passed
from MFD driver.

Signed-off-by: Dzmitry Sankouski <dsankouski@gmail.com>
---
 Documentation/devicetree/bindings/power/supply/maxim,max17042.yaml | 1 -
 1 file changed, 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/power/supply/maxim,max17042.yaml b/Documentation/devicetree/bindings/power/supply/maxim,max17042.yaml
index f929e7e2b82a..7a26036a8890 100644
--- a/Documentation/devicetree/bindings/power/supply/maxim,max17042.yaml
+++ b/Documentation/devicetree/bindings/power/supply/maxim,max17042.yaml
@@ -64,7 +64,6 @@ properties:
 
 required:
   - compatible
-  - reg
 
 additionalProperties: false
 

-- 
2.39.5
Re: [PATCH v9 3/9] dt-bindings: power: supply: max17042: remove reg from required
Posted by Krzysztof Kozlowski 1 year ago
On 02/12/2024 10:47, Dzmitry Sankouski wrote:
> Remove 'reg' property from required list, because in platform
> variant it should not be present, because i2c client is passed
> from MFD driver.
> 
> Signed-off-by: Dzmitry Sankouski <dsankouski@gmail.com>

Sorry, not really. Device needs I2C address. I don't get what is
"platform variant" in terms of hardware, maybe you refer to some driver
changes. But anyway this needs proper hardware description, e.g. that
device does not have its own I2C address. If that's the case, of course.

Best regards,
Krzysztof