* [PATCH] dt-bindings: hwmon: jedec,jc42: add #thermal-sensor-cells
@ 2026-09-30 13:44 ` Vincent Jardin
0 siblings, 0 replies; 4+ messages in thread
From: Vincent Jardin via B4 Relay @ 2026-09-30 13:44 UTC (permalink / raw)
To: Guenter Roeck, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Jean Delvare
Cc: linux-hwmon, devicetree, linux-kernel, Vincent Jardin
From: Vincent Jardin <vjardin@free.fr>
Since the commit
032c1623c9de ("hwmon: (jc42) add HWMON_C_TZ_REGISTER")
the driver sets HWMON_C_REGISTER_TZ, so the hwmon core calls
devm_thermal_of_zone_register() for its temperature channel and the
device can be referenced as a thermal sensor.
The chip has a single temperature channel, registered as sensor 0, so
no cell is needed.
Signed-off-by: Vincent Jardin <vjardin@free.fr>
---
Documentation/devicetree/bindings/hwmon/jedec,jc42.yaml | 3 +++
1 file changed, 3 insertions(+)
diff --git a/Documentation/devicetree/bindings/hwmon/jedec,jc42.yaml b/Documentation/devicetree/bindings/hwmon/jedec,jc42.yaml
index bf3332153ad8..d835664a926b 100644
--- a/Documentation/devicetree/bindings/hwmon/jedec,jc42.yaml
+++ b/Documentation/devicetree/bindings/hwmon/jedec,jc42.yaml
@@ -59,6 +59,9 @@ properties:
supported on all chips.
type: boolean
+ "#thermal-sensor-cells":
+ const: 0
+
required:
- compatible
- reg
---
base-commit: 4781ca52761e666cf18b591e6bb0478396c90320
change-id: 20260930-for-upstream-dt-jc42-thermal-sensor-cells-ad1ce051661c
Best regards,
--
Vincent Jardin <vjardin@free.fr>
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH] dt-bindings: hwmon: jedec,jc42: add #thermal-sensor-cells
@ 2026-09-30 13:44 ` Vincent Jardin
0 siblings, 0 replies; 4+ messages in thread
From: Vincent Jardin @ 2026-09-30 13:44 UTC (permalink / raw)
To: Guenter Roeck, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Jean Delvare
Cc: linux-hwmon, devicetree, linux-kernel, Vincent Jardin
Since the commit
032c1623c9de ("hwmon: (jc42) add HWMON_C_TZ_REGISTER")
the driver sets HWMON_C_REGISTER_TZ, so the hwmon core calls
devm_thermal_of_zone_register() for its temperature channel and the
device can be referenced as a thermal sensor.
The chip has a single temperature channel, registered as sensor 0, so
no cell is needed.
Signed-off-by: Vincent Jardin <vjardin@free.fr>
---
Documentation/devicetree/bindings/hwmon/jedec,jc42.yaml | 3 +++
1 file changed, 3 insertions(+)
diff --git a/Documentation/devicetree/bindings/hwmon/jedec,jc42.yaml b/Documentation/devicetree/bindings/hwmon/jedec,jc42.yaml
index bf3332153ad8..d835664a926b 100644
--- a/Documentation/devicetree/bindings/hwmon/jedec,jc42.yaml
+++ b/Documentation/devicetree/bindings/hwmon/jedec,jc42.yaml
@@ -59,6 +59,9 @@ properties:
supported on all chips.
type: boolean
+ "#thermal-sensor-cells":
+ const: 0
+
required:
- compatible
- reg
---
base-commit: 4781ca52761e666cf18b591e6bb0478396c90320
change-id: 20260930-for-upstream-dt-jc42-thermal-sensor-cells-ad1ce051661c
Best regards,
--
Vincent Jardin <vjardin@free.fr>
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] dt-bindings: hwmon: jedec,jc42: add #thermal-sensor-cells
2026-09-30 13:44 ` Vincent Jardin
(?)
@ 2026-09-30 13:47 ` sashiko-bot
-1 siblings, 0 replies; 4+ messages in thread
From: sashiko-bot @ 2026-09-30 13:47 UTC (permalink / raw)
To: Vincent Jardin; +Cc: conor+dt, robh, devicetree, linux-hwmon
> From: Vincent Jardin <vjardin@free.fr>
>
> Since the commit
> 032c1623c9de ("hwmon: (jc42) add HWMON_C_TZ_REGISTER")
> the driver sets HWMON_C_REGISTER_TZ, so the hwmon core calls
> devm_thermal_of_zone_register() for its temperature channel and the
> device can be referenced as a thermal sensor.
>
> The chip has a single temperature channel, registered as sensor 0, so
> no cell is needed.
>
> Signed-off-by: Vincent Jardin <vjardin@free.fr>
Sashiko has reviewed this patch and found no issues. It looks great!
--
Sashiko AI review · https://sashiko.dev/#/patchset/20260930-for-upstream-dt-jc42-thermal-sensor-cells-v1-1-ef6365cae23e@free.fr?part=1
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] dt-bindings: hwmon: jedec,jc42: add #thermal-sensor-cells
2026-09-30 13:44 ` Vincent Jardin
(?)
(?)
@ 2026-10-01 8:36 ` Krzysztof Kozlowski
-1 siblings, 0 replies; 4+ messages in thread
From: Krzysztof Kozlowski @ 2026-10-01 8:36 UTC (permalink / raw)
To: Vincent Jardin
Cc: Guenter Roeck, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Jean Delvare, linux-hwmon, devicetree, linux-kernel
On Wed, Sep 30, 2026 at 03:44:51PM +0200, Vincent Jardin wrote:
> Since the commit
> 032c1623c9de ("hwmon: (jc42) add HWMON_C_TZ_REGISTER")
Odd formatting. This is one sentence.
> the driver sets HWMON_C_REGISTER_TZ, so the hwmon core calls
> devm_thermal_of_zone_register() for its temperature channel and the
> device can be referenced as a thermal sensor.
>
> The chip has a single temperature channel, registered as sensor 0, so
> no cell is needed.
>
> Signed-off-by: Vincent Jardin <vjardin@free.fr>
> ---
> Documentation/devicetree/bindings/hwmon/jedec,jc42.yaml | 3 +++
> 1 file changed, 3 insertions(+)
>
With commit msg fixed:
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2026-10-01 8:36 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-09-30 13:44 [PATCH] dt-bindings: hwmon: jedec,jc42: add #thermal-sensor-cells Vincent Jardin via B4 Relay
2026-09-30 13:44 ` Vincent Jardin
2026-09-30 13:47 ` sashiko-bot
2026-10-01 8:36 ` Krzysztof Kozlowski
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.