Re: [PATCH V2 1/2] ARM: dts: DRA7: Add bandgap and related thermal nodes

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

 






On Friday 27 March 2015 11:32 AM, Keerthy wrote:


On Tuesday 24 March 2015 01:09 AM, Nishanth Menon wrote:
From: Keerthy <j-keerthy@xxxxxx>

Add bandgap and related thermal nodes. The patch adds 5 thermal
sensors. Only one cooling device for mpu as of now. The sensors are
the exact same on both dra72 and dra7. Introduce CPU, GPU, core nodes
for the moment as they are direct reuse of OMAP5 entities.

NOTE: OMAP4 has a finer counter granularity, which allows for a delay
of 1000ms in the thermal zone polling intervals. DRA7 have different
counter mechanism, which allows at maximum a 500ms timer. Adjust the
cpu thermal zone accordingly for DRA7.

Signed-off-by: Keerthy <j-keerthy@xxxxxx>
[t-kristo@xxxxxx: few reuse from OMAP5 entities]
Signed-off-by: Tero Kristo <t-kristo@xxxxxx>
Signed-off-by: Nishanth Menon <nm@xxxxxx>
---
Changes since V1:
    - omap5-cpu-thermal.dtsi dropped
    - approach similar to omap5
    - added core and gpu thermal dtsi(they can be reused)
    - Squashed patch #1 from v1 to patch #2 - this patch.
    - Not carrying forward ack due to change

Thanks for updating Nishanth. Looks good to me.

Tested on DRA7 EVM.

Read the temperatures from all the 5 sensors they were sane.
Loaded the CPU and saw the temperatures rise and fall with increase and decrease in the load.

Tested-by: Keerthy <j-keerthy@xxxxxx>

- Keerthy


V1: http://marc.info/?t=142688102700006&r=1&w=2

  arch/arm/boot/dts/dra7.dtsi   |   23 +++++++++++++++++++++++
  arch/arm/boot/dts/dra72x.dtsi |    5 +++++
  arch/arm/boot/dts/dra74x.dtsi |    5 +++++
  3 files changed, 33 insertions(+)

diff --git a/arch/arm/boot/dts/dra7.dtsi b/arch/arm/boot/dts/dra7.dtsi
index eea4a54d6cb3..2d6a7e3e001f 100644
--- a/arch/arm/boot/dts/dra7.dtsi
+++ b/arch/arm/boot/dts/dra7.dtsi
@@ -177,6 +177,18 @@
              };
          };

+        bandgap: bandgap@4a0021e0 {
+            reg = <0x4a0021e0 0xc
+                0x4a00232c 0xc
+                0x4a002380 0x2c
+                0x4a0023C0 0x3c
+                0x4a002564 0x8
+                0x4a002574 0x50>;
+                compatible = "ti,dra752-bandgap";
+                interrupts = <GIC_SPI 121 IRQ_TYPE_LEVEL_HIGH>;
+                #thermal-sensor-cells = <1>;
+        };
+
          cm_core_aon: cm_core_aon@4a005000 {
              compatible = "ti,dra7-cm-core-aon";
              reg = <0x4a005000 0x2000>;
@@ -1419,6 +1431,17 @@
              status = "disabled";
          };
      };
+
+    thermal_zones: thermal-zones {
+        #include "omap4-cpu-thermal.dtsi"
+        #include "omap5-gpu-thermal.dtsi"
+        #include "omap5-core-thermal.dtsi"
+    };
+
+};
+
+&cpu_thermal {
+    polling-delay = <500>; /* milliseconds */
  };

  /include/ "dra7xx-clocks.dtsi"
diff --git a/arch/arm/boot/dts/dra72x.dtsi
b/arch/arm/boot/dts/dra72x.dtsi
index e5a3d23a3df1..6ac8e3601499 100644
--- a/arch/arm/boot/dts/dra72x.dtsi
+++ b/arch/arm/boot/dts/dra72x.dtsi
@@ -20,6 +20,11 @@
              device_type = "cpu";
              compatible = "arm,cortex-a15";
              reg = <0>;
+
+            /* cooling options */
+            cooling-min-level = <0>;
+            cooling-max-level = <2>;
+            #cooling-cells = <2>; /* min followed by max */
          };
      };

diff --git a/arch/arm/boot/dts/dra74x.dtsi
b/arch/arm/boot/dts/dra74x.dtsi
index 10173fab1a15..eef981f4bcd5 100644
--- a/arch/arm/boot/dts/dra74x.dtsi
+++ b/arch/arm/boot/dts/dra74x.dtsi
@@ -31,6 +31,11 @@
              clock-names = "cpu";

              clock-latency = <300000>; /* From omap-cpufreq driver */
+
+            /* cooling options */
+            cooling-min-level = <0>;
+            cooling-max-level = <2>;
+            #cooling-cells = <2>; /* min followed by max */
          };
          cpu@1 {
              device_type = "cpu";

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[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