Re: [PATCH v2 1/3] dt-bindings: thermal: samsung: Add tmu-name and sensor-index-ranges properties
From: Krzysztof Kozlowski <krzk@kernel.org>
Date: 2025-09-04 07:59:39
Also in:
linux-devicetree, linux-pm, linux-samsung-soc, lkml
On Wed, Sep 03, 2025 at 04:36:32PM +0900, Shin Son wrote:
The exynosautov920 TMU requires per-sensor interrupt enablement for its critical trip points. Add a DT property to the Samsung thermal bindings to support this requirement:
That's pretty redundant sentence.
- **samsung,hw-sensor-indices**: Defines the sensors currently
mapped to the TMU hardware.
Indices not listed are absent or fused offDon't write here any code, but concise prose dxescribing hardware. If sensors are fused out, you certainly can read their status from efuse, no? This is really vague description of hardware. I don't understand why you are changing sensor-cells, why older variants of tmu gets now cells=1 (missing constraints?). Why older variants also get that property for sensors? It does not make sense there, because they have one-to-one mapping between TMU and sensor.
quoted hunk ↗ jump to hunk
Additionally, add myself to the bindings' maintainers list, as I plan to actively work on the exynosautov920 TMU support and handle further updates in this area. Signed-off-by: Shin Son <redacted> --- .../bindings/thermal/samsung,exynos-thermal.yaml | 16 +++++++++++++++- 1 file changed, 15 insertions(+), 1 deletion(-)diff --git a/Documentation/devicetree/bindings/thermal/samsung,exynos-thermal.yaml b/Documentation/devicetree/bindings/thermal/samsung,exynos-thermal.yaml index 29a08b0729ee..abd89902d33a 100644 --- a/Documentation/devicetree/bindings/thermal/samsung,exynos-thermal.yaml +++ b/Documentation/devicetree/bindings/thermal/samsung,exynos-thermal.yaml@@ -8,6 +8,7 @@ title: Samsung Exynos SoC Thermal Management Unit (TMU) maintainers: - Krzysztof Kozlowski <krzk@kernel.org> + - Shin Son <shin.son@samsung.com> description: | For multi-instance tmu each instance should have an alias correctly numbered@@ -27,6 +28,7 @@ properties: - samsung,exynos5420-tmu-ext-triminfo - samsung,exynos5433-tmu - samsung,exynos7-tmu + - samsung,exynosautov920-tmu clocks: minItems: 1@@ -62,11 +64,22 @@ properties: minItems: 1 '#thermal-sensor-cells': - const: 0 + enum: + - 0 + - 1 vtmu-supply: description: The regulator node supplying voltage to TMU. + samsung,hw-sensor-indices: + description: |
Drop |
+ List of hardware sensor indices that are physically present and usable + in this TMU instance. Indices not listed are either unmapped or unused. + $ref: /schemas/types.yaml#/definitions/uint32-array + minItems: 1 + maxItems: 16 + uniqueItems: true
Best regards, Krzysztof