[PATCH 0/3] arm: dts: DRA7: Add IVA and DSPEVE thermal domain data

OMAP5 has 3 thermal zones cpu, core and multimedia.
On the other hand DRA7 has 5 thermal zones cpu, gpu, core, dspeve
and iva. Currently cpu, core and multimedia are being added via dt
and the other 2 are getting added via kernel. Add the missing thermal
domains in dt so as give the correct trip numbers, type and temperatures.

Tested on DRA7-EVM.
 
Keerthy (3):
  arm: dts: DRA7: Add DSPEVE thermal data
  arm: dts: DRA7: Add IVA thermal data
  arm: dts: DRA7: Add missing IVA and DSPEVE thermal domain data

 arch/arm/boot/dts/dra7-dspeve-thermal.dtsi | 27 +++++++++++++++++++++++++++
 arch/arm/boot/dts/dra7-iva-thermal.dtsi    | 27 +++++++++++++++++++++++++++
 arch/arm/boot/dts/dra7.dtsi                |  2 ++
 3 files changed, 56 insertions(+)
 create mode 100644 arch/arm/boot/dts/dra7-dspeve-thermal.dtsi
 create mode 100644 arch/arm/boot/dts/dra7-iva-thermal.dtsi

-- 
1.9.1

OMAP5 has 3 thermal zones cpu, core and multimedia.
On the other hand DRA7 has 5 thermal zones cpu, gpu, core, dspeve
and iva. Currently cpu, core and multimedia are being added via device tree
and the other 2 are getting added via kernel. Add the missing thermal
domains in device tree so we can create the zones with the appropriate
trip numbers, type and temperatures.

Signed-off-by: Keerthy <j-keerthy@ti.com>
---
 arch/arm/boot/dts/dra7.dtsi | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/arch/arm/boot/dts/dra7.dtsi b/arch/arm/boot/dts/dra7.dtsi
index c4d9175..9f65d0d 100644
--- a/arch/arm/boot/dts/dra7.dtsi
+++ b/arch/arm/boot/dts/dra7.dtsi

@@ -1603,6 +1603,8 @@
 		#include "omap4-cpu-thermal.dtsi"
 		#include "omap5-gpu-thermal.dtsi"
 		#include "omap5-core-thermal.dtsi"
+		#include "dra7-dspeve-thermal.dtsi"
+		#include "dra7-iva-thermal.dtsi"
 	};
 
 };

-- 
1.9.1

This patch changes a dtsi file to contain the thermal data
for DSPEVE domain. This data will enable a thermal shutdown at 125C.

Signed-off-by: Keerthy <j-keerthy@ti.com>
---
 arch/arm/boot/dts/dra7-dspeve-thermal.dtsi | 27 +++++++++++++++++++++++++++
 1 file changed, 27 insertions(+)
 create mode 100644 arch/arm/boot/dts/dra7-dspeve-thermal.dtsi

diff --git a/arch/arm/boot/dts/dra7-dspeve-thermal.dtsi b/arch/arm/boot/dts/dra7-dspeve-thermal.dtsi
new file mode 100644
index 0000000..1c39a84
--- /dev/null
+++ b/arch/arm/boot/dts/dra7-dspeve-thermal.dtsi

@@ -0,0 +1,27 @@
+/*
+ * Device Tree Source for DRA7x SoC DSPEVE thermal
+ *
+ * Copyright (C) 2016 Texas Instruments Incorporated - http://www.ti.com/
+ *
+ * This file is licensed under the terms of the GNU General Public License
+ * version 2.  This program is licensed "as is" without any warranty of any
+ * kind, whether express or implied.
+ */
+
+#include <dt-bindings/thermal/thermal.h>
+
+dspeve_thermal: dspeve_thermal {
+	polling-delay-passive = <250>; /* milliseconds */
+	polling-delay = <500>; /* milliseconds */
+
+			/* sensor       ID */
+	thermal-sensors = <&bandgap     3>;
+
+	trips {
+		dspeve_crit: dspeve_crit {
+			temperature = <125000>; /* milliCelsius */
+			hysteresis = <2000>; /* milliCelsius */
+			type = "critical";
+		};
+	};
+};

-- 
1.9.1

This patch changes a dtsi file to contain the thermal data
for IVA domain. This data will enable a thermal shutdown at 125C.

Signed-off-by: Keerthy <j-keerthy@ti.com>
---
 arch/arm/boot/dts/dra7-iva-thermal.dtsi | 27 +++++++++++++++++++++++++++
 1 file changed, 27 insertions(+)
 create mode 100644 arch/arm/boot/dts/dra7-iva-thermal.dtsi

diff --git a/arch/arm/boot/dts/dra7-iva-thermal.dtsi b/arch/arm/boot/dts/dra7-iva-thermal.dtsi
new file mode 100644
index 0000000..dd74a53
--- /dev/null
+++ b/arch/arm/boot/dts/dra7-iva-thermal.dtsi

@@ -0,0 +1,27 @@
+/*
+ * Device Tree Source for DRA7x SoC IVA thermal
+ *
+ * Copyright (C) 2016 Texas Instruments Incorporated - http://www.ti.com/
+ *
+ * This file is licensed under the terms of the GNU General Public License
+ * version 2.  This program is licensed "as is" without any warranty of any
+ * kind, whether express or implied.
+ */
+
+#include <dt-bindings/thermal/thermal.h>
+
+iva_thermal: iva_thermal {
+	polling-delay-passive = <250>; /* milliseconds */
+	polling-delay = <500>; /* milliseconds */
+
+			/* sensor       ID */
+	thermal-sensors = <&bandgap     4>;
+
+	trips {
+		iva_crit: iva_crit {
+			temperature = <125000>; /* milliCelsius */
+			hysteresis = <2000>; /* milliCelsius */
+			type = "critical";
+		};
+	};
+};

-- 
1.9.1

* Keerthy [off-list ref] [160208 01:19]:

OMAP5 has 3 thermal zones cpu, core and multimedia.
On the other hand DRA7 has 5 thermal zones cpu, gpu, core, dspeve
and iva. Currently cpu, core and multimedia are being added via dt
and the other 2 are getting added via kernel. Add the missing thermal
domains in dt so as give the correct trip numbers, type and temperatures.

Tested on DRA7-EVM.

Are these critical fixes to have for v4.5-rc cycle or can they
wait for v4.6 merge window?

Regards,

Tony

Keerthy (3):
  arm: dts: DRA7: Add DSPEVE thermal data
  arm: dts: DRA7: Add IVA thermal data
  arm: dts: DRA7: Add missing IVA and DSPEVE thermal domain data

 arch/arm/boot/dts/dra7-dspeve-thermal.dtsi | 27 +++++++++++++++++++++++++++
 arch/arm/boot/dts/dra7-iva-thermal.dtsi    | 27 +++++++++++++++++++++++++++
 arch/arm/boot/dts/dra7.dtsi                |  2 ++
 3 files changed, 56 insertions(+)
 create mode 100644 arch/arm/boot/dts/dra7-dspeve-thermal.dtsi
 create mode 100644 arch/arm/boot/dts/dra7-iva-thermal.dtsi

-- 
1.9.1

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

On Tuesday 09 February 2016 01:27 AM, Tony Lindgren wrote:

* Keerthy [off-list ref] [160208 01:19]:

quoted

OMAP5 has 3 thermal zones cpu, core and multimedia.
On the other hand DRA7 has 5 thermal zones cpu, gpu, core, dspeve
and iva. Currently cpu, core and multimedia are being added via dt
and the other 2 are getting added via kernel. Add the missing thermal
domains in dt so as give the correct trip numbers, type and temperatures.

Tested on DRA7-EVM.

Are these critical fixes to have for v4.5-rc cycle or can they
wait for v4.6 merge window?

Driver configures some default thresholds that are a bit higher but 
there are some thresholds already for dspeve and iva too. So guess these 
can wait if its already too late.

Regards,
Keerthy

Regards,

Tony

quoted

Keerthy (3):
   arm: dts: DRA7: Add DSPEVE thermal data
   arm: dts: DRA7: Add IVA thermal data
   arm: dts: DRA7: Add missing IVA and DSPEVE thermal domain data

  arch/arm/boot/dts/dra7-dspeve-thermal.dtsi | 27 +++++++++++++++++++++++++++
  arch/arm/boot/dts/dra7-iva-thermal.dtsi    | 27 +++++++++++++++++++++++++++
  arch/arm/boot/dts/dra7.dtsi                |  2 ++
  3 files changed, 56 insertions(+)
  create mode 100644 arch/arm/boot/dts/dra7-dspeve-thermal.dtsi
  create mode 100644 arch/arm/boot/dts/dra7-iva-thermal.dtsi

--
1.9.1

* Keerthy [off-list ref] [160208 19:38]:

On Tuesday 09 February 2016 01:27 AM, Tony Lindgren wrote:

quoted

* Keerthy [off-list ref] [160208 01:19]:

quoted

OMAP5 has 3 thermal zones cpu, core and multimedia.
On the other hand DRA7 has 5 thermal zones cpu, gpu, core, dspeve
and iva. Currently cpu, core and multimedia are being added via dt
and the other 2 are getting added via kernel. Add the missing thermal
domains in dt so as give the correct trip numbers, type and temperatures.

Tested on DRA7-EVM.

Are these critical fixes to have for v4.5-rc cycle or can they
wait for v4.6 merge window?

Driver configures some default thresholds that are a bit higher but there
are some thresholds already for dspeve and iva too. So guess these can wait
if its already too late.

OK let's rather wait then.

Regards,

Tony

On Tue, Feb 09, 2016 at 09:07:27AM +0530, Keerthy wrote:

On Tuesday 09 February 2016 01:27 AM, Tony Lindgren wrote:

quoted

* Keerthy [off-list ref] [160208 01:19]:

quoted

OMAP5 has 3 thermal zones cpu, core and multimedia.
On the other hand DRA7 has 5 thermal zones cpu, gpu, core, dspeve
and iva. Currently cpu, core and multimedia are being added via dt
and the other 2 are getting added via kernel. Add the missing thermal
domains in dt so as give the correct trip numbers, type and temperatures.

But the driver would add these default domains only if booted /loaded with
missing DT device entry, right?

On Wednesday 10 February 2016 03:23 AM, Eduardo Valentin wrote:

On Tue, Feb 09, 2016 at 09:07:27AM +0530, Keerthy wrote:

quoted

On Tuesday 09 February 2016 01:27 AM, Tony Lindgren wrote:

quoted

* Keerthy [off-list ref] [160208 01:19]:

quoted

OMAP5 has 3 thermal zones cpu, core and multimedia.
On the other hand DRA7 has 5 thermal zones cpu, gpu, core, dspeve
and iva. Currently cpu, core and multimedia are being added via dt
and the other 2 are getting added via kernel. Add the missing thermal
domains in dt so as give the correct trip numbers, type and temperatures.

But the driver would add these default domains only if booted /loaded with
missing DT device entry, right?

Yes Eduardo, but with default trip values not the absolute which is 
added with this patch series.

On Mon, Feb 08, 2016 at 02:46:27PM +0530, Keerthy wrote:

OMAP5 has 3 thermal zones cpu, core and multimedia.
On the other hand DRA7 has 5 thermal zones cpu, gpu, core, dspeve
and iva. Currently cpu, core and multimedia are being added via dt
and the other 2 are getting added via kernel. Add the missing thermal
domains in dt so as give the correct trip numbers, type and temperatures.

Tested on DRA7-EVM.

For the series:

Acked-by: Eduardo Valentin <edubezval@gmail.com>

 
Keerthy (3):
  arm: dts: DRA7: Add DSPEVE thermal data
  arm: dts: DRA7: Add IVA thermal data
  arm: dts: DRA7: Add missing IVA and DSPEVE thermal domain data

 arch/arm/boot/dts/dra7-dspeve-thermal.dtsi | 27 +++++++++++++++++++++++++++
 arch/arm/boot/dts/dra7-iva-thermal.dtsi    | 27 +++++++++++++++++++++++++++
 arch/arm/boot/dts/dra7.dtsi                |  2 ++
 3 files changed, 56 insertions(+)
 create mode 100644 arch/arm/boot/dts/dra7-dspeve-thermal.dtsi
 create mode 100644 arch/arm/boot/dts/dra7-iva-thermal.dtsi

-- 
1.9.1

* Eduardo Valentin [off-list ref] [160209 13:53]:

On Mon, Feb 08, 2016 at 02:46:27PM +0530, Keerthy wrote:

quoted

OMAP5 has 3 thermal zones cpu, core and multimedia.
On the other hand DRA7 has 5 thermal zones cpu, gpu, core, dspeve
and iva. Currently cpu, core and multimedia are being added via dt
and the other 2 are getting added via kernel. Add the missing thermal
domains in dt so as give the correct trip numbers, type and temperatures.

Tested on DRA7-EVM.

For the series:

Acked-by: Eduardo Valentin <redacted>

Applying all into omap-for-v4.6/dt thanks.

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