diff mbox series

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

Message ID 20241202-starqltechn_integration_upstream-v9-3-a1adc3bae2b8@gmail.com (mailing list archive)
State New
Headers show
Series Add support for Maxim Integrated MAX77705 PMIC | expand

Commit Message

Dzmitry Sankouski Dec. 2, 2024, 9:47 a.m. UTC
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(-)

Comments

Krzysztof Kozlowski Dec. 2, 2024, 10:18 a.m. UTC | #1
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
diff mbox series

Patch

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