Re: [PATCH v2 03/13] dt-bindings: thermal: tegra: Add nvidia,thermal-zones property

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 12/10/2023 19:58, Thierry Reding wrote:
From: Thierry Reding <treding@xxxxxxxxxx>

The throttle configurations need to be associated with one or more
thermal zones before they can be enabled, so introduce a new property,
called nvidia,thermal-zones, that contains a list of phandles to the
thermal zones for which a throttle configuration is applicable.

Signed-off-by: Thierry Reding <treding@xxxxxxxxxx>
---
Changes in v2:
- new patch to hook up throttling with thermal zones

  .../bindings/thermal/nvidia,tegra124-soctherm.yaml           | 5 +++++
  1 file changed, 5 insertions(+)

diff --git a/Documentation/devicetree/bindings/thermal/nvidia,tegra124-soctherm.yaml b/Documentation/devicetree/bindings/thermal/nvidia,tegra124-soctherm.yaml
index 0eb6277082fe..359344f60a6e 100644
--- a/Documentation/devicetree/bindings/thermal/nvidia,tegra124-soctherm.yaml
+++ b/Documentation/devicetree/bindings/thermal/nvidia,tegra124-soctherm.yaml
@@ -161,6 +161,11 @@ properties:
                throttling is engaged after the OC event is deasserted.
              default: 0
+ nvidia,thermal-zones:
+            $ref: /schemas/types.yaml#/definitions/phandle
+            description: A list of phandles to the thermal zones that this
+              throttle configuration applies to.
+

From a DT perspective, I believe it is more correct to point to the devices the hardware throttling mechanism applies to instead of the thermal zones which is kind of software component

    # optional
    nvidia,thermtrips:
      $ref: /schemas/types.yaml#/definitions/uint32-matrix

--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog





[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux