[PATCH] dt-bindings: hwmon: ti,tmp401: add vcc-supply and #thermal-sensor-cells
Vincent Jardin <[email protected]>
| Newsgroups | org.kernel.feeds.b4-sent,org.kernel.vger.linux-devicetree,org.kernel.vger.linux-hwmon,org.kernel.vger.linux-kernel |
|---|---|
| Message-ID | <20260824-for-upstream-dt-tmp401-vcc-tsc-v1-1-ecb5ef7898bd@free.fr> |
Both properties are used by the driver that backs this binding, and
neither is described, so a board that supplies either fails dtbs_check:
tmp411@4c (ti,tmp411): '#thermal-sensor-cells', 'vcc-supply' do not
match any of the regexes: '^pinctrl-[0-9]+$'
vcc-supply is fetched in lm90_probe():
drivers/hwmon/lm90.c:2777
err = devm_regulator_get_enable(dev, "vcc");
so a board with the sensor behind a switched rail has no other way to
describe it, and omitting it means the chip is probed with its supply
possibly off.
Signed-off-by: Vincent Jardin <[email protected]>
---
Documentation/devicetree/bindings/hwmon/ti,tmp401.yaml | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/Documentation/devicetree/bindings/hwmon/ti,tmp401.yaml b/Documentation/devicetree/bindings/hwmon/ti,tmp401.yaml
index 0e8ddf0ad7890..69f77aec01a33 100644
--- a/Documentation/devicetree/bindings/hwmon/ti,tmp401.yaml
+++ b/Documentation/devicetree/bindings/hwmon/ti,tmp401.yaml
@@ -50,6 +50,11 @@ properties:
minimum: 0
maximum: 15
+ vcc-supply: true
+
+ "#thermal-sensor-cells":
+ const: 1
+
allOf:
- if:
properties:
---
base-commit: 0a0d1d55dad570724bf8c7ea83409639cfb4be9b
change-id: 20260824-for-upstream-dt-tmp401-vcc-tsc-a16bec8352e4
Best regards,
--
Vincent Jardin <[email protected]>