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

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

[Keerthy]

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

[PATCH 1/3] arm: dts: DRA7: Add DSPEVE thermal data

[Keerthy]

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

[PATCH 2/3] arm: dts: DRA7: Add IVA thermal data

[Keerthy]

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

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

[Keerthy]

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

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

[Tony Lindgren]

* Keerthy <j-keerthy-l0cyMroinI0@public.gmane.org> [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

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

[Keerthy]

On Tuesday 09 February 2016 01:27 AM, Tony Lindgren wrote:
> * Keerthy <j-keerthy@ti.com> [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?

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

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

[Tony Lindgren]

* Keerthy <a0393675@ti.com> [160208 19:38]:
> 
> 
> On Tuesday 09 February 2016 01:27 AM, Tony Lindgren wrote:
> >* Keerthy <j-keerthy@ti.com> [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?
> 
> 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

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

[Eduardo Valentin]

[-- Attachment #1: Type: text/plain, Size: 644 bytes --]

On Tue, Feb 09, 2016 at 09:07:27AM +0530, Keerthy wrote:
> 
> 
> On Tuesday 09 February 2016 01:27 AM, Tony Lindgren wrote:
> >* Keerthy <j-keerthy@ti.com> [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.

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


[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 473 bytes --]

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

[Keerthy]

On Wednesday 10 February 2016 03:23 AM, Eduardo Valentin wrote:
> On Tue, Feb 09, 2016 at 09:07:27AM +0530, Keerthy wrote:
>>
>>
>> On Tuesday 09 February 2016 01:27 AM, Tony Lindgren wrote:
>>> * Keerthy <j-keerthy@ti.com> [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.
>
> 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.

>

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

[Eduardo Valentin]

[-- Attachment #1: Type: text/plain, Size: 1088 bytes --]

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
> 

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 473 bytes --]

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

[Tony Lindgren]

* Eduardo Valentin <edubezval-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> [160209 13:53]:
> 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-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>

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