Devicetree
 help / color / mirror / Atom feed
* [PATCH v2] dt-bindings: hwmon: jedec,jc42: add #thermal-sensor-cells
@ 2026-10-01 21:58 Vincent Jardin via B4 Relay
  2026-10-02  9:13 ` sashiko-bot
  0 siblings, 1 reply; 2+ messages in thread
From: Vincent Jardin via B4 Relay @ 2026-10-01 21:58 UTC (permalink / raw)
  To: Guenter Roeck, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	Jean Delvare
  Cc: linux-hwmon, devicetree, linux-kernel, Krzysztof Kozlowski,
	Vincent Jardin

From: Vincent Jardin <vjardin@free.fr>

Since 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.

Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
Signed-off-by: Vincent Jardin <vjardin@free.fr>
---
Changes in v2:
- fix sentence format (Krzysztof)
- record Krzysztof's Reviewed-by
- Link to v1: https://lore.kernel.org/r/20260930-for-upstream-dt-jc42-thermal-sensor-cells-v1-1-ef6365cae23e@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] 2+ messages in thread

* Re: [PATCH v2] dt-bindings: hwmon: jedec,jc42: add #thermal-sensor-cells
  2026-10-01 21:58 [PATCH v2] dt-bindings: hwmon: jedec,jc42: add #thermal-sensor-cells Vincent Jardin via B4 Relay
@ 2026-10-02  9:13 ` sashiko-bot
  0 siblings, 0 replies; 2+ messages in thread
From: sashiko-bot @ 2026-10-02  9:13 UTC (permalink / raw)
  To: Vincent Jardin; +Cc: robh, devicetree, conor+dt, linux-hwmon

> From: Vincent Jardin <vjardin@free.fr>
> 
> Since 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.
> 
> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
> 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/20261001-for-upstream-dt-jc42-thermal-sensor-cells-v2-1-6edd30b082f3@free.fr?part=1


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2026-10-02  9:13 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-10-01 21:58 [PATCH v2] dt-bindings: hwmon: jedec,jc42: add #thermal-sensor-cells Vincent Jardin via B4 Relay
2026-10-02  9:13 ` sashiko-bot

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox