* [PATCH 2/2] therma: exynos: Supports thermal tripping
@ 2012-10-30 5:32 Jonghwan Choi
2012-10-31 5:06 ` jonghwa3.lee
0 siblings, 1 reply; 3+ messages in thread
From: Jonghwan Choi @ 2012-10-30 5:32 UTC (permalink / raw)
To: 'open list'
Cc: 'Amit Daniel Kachhap', 'Zhang Rui',
'Sachin Kamat'
TMU urgently sends active-high signal (thermal trip) to PMU,
and thermal tripping by hardware logic i.e PMU is performed.
Thermal tripping means that PMU cut off the whole power of SoC
by controlling external voltage regulator.
Signed-off-by: Jonghwan Choi <jhbird.choi@samsung.com>
---
drivers/thermal/exynos_thermal.c | 7 ++++++-
include/linux/platform_data/exynos_thermal.h | 4 ++++
2 files changed, 10 insertions(+), 1 deletions(-)
diff --git a/drivers/thermal/exynos_thermal.c
b/drivers/thermal/exynos_thermal.c
index 6ce6667..edac601 100644
--- a/drivers/thermal/exynos_thermal.c
+++ b/drivers/thermal/exynos_thermal.c
@@ -53,6 +53,7 @@
#define EXYNOS_TMU_TRIM_TEMP_MASK 0xff
#define EXYNOS_TMU_GAIN_SHIFT 8
#define EXYNOS_TMU_REF_VOLTAGE_SHIFT 24
+#define EXYNOS_TMU_TRIP_EN BIT(12)
#define EXYNOS_TMU_CORE_ON 1
#define EXYNOS_TMU_CORE_OFF 0
#define EXYNOS_TMU_DEF_CODE_TO_TEMP_OFFSET 50
@@ -656,6 +657,8 @@ static void exynos_tmu_control(struct platform_device
*pdev, bool on)
if (data->soc == SOC_ARCH_EXYNOS) {
con |= pdata->noise_cancel_mode <<
EXYNOS_TMU_TRIP_MODE_SHIFT;
con |= (EXYNOS_MUX_ADDR_VALUE << EXYNOS_MUX_ADDR_SHIFT);
+ if (pdata->trip_en)
+ con |= EXYNOS_THERMAL_TRIP_EN;
}
if (on) {
@@ -762,10 +765,12 @@ static struct exynos_tmu_platform_data const
exynos_default_tmu_data = {
.trigger_levels[0] = 85,
.trigger_levels[1] = 103,
.trigger_levels[2] = 110,
+ .trigger_levels[3] = 120,
.trigger_level0_en = 1,
.trigger_level1_en = 1,
.trigger_level2_en = 1,
- .trigger_level3_en = 0,
+ .trigger_level3_en = 1,
+ .trip_en = 1,
.gain = 8,
.reference_voltage = 16,
.noise_cancel_mode = 4,
diff --git a/include/linux/platform_data/exynos_thermal.h
b/include/linux/platform_data/exynos_thermal.h
index a7bdb2f..9e44aac 100644
--- a/include/linux/platform_data/exynos_thermal.h
+++ b/include/linux/platform_data/exynos_thermal.h
@@ -79,6 +79,9 @@ struct freq_clip_table {
* @trigger_level3_en:
* 1 = enable trigger_level3 interrupt,
* 0 = disable trigger_level3 interrupt
+ * @trip_en:
+ * 1 = enable thermal tripping
+ * 0 = disable thermal tripping
* @gain: gain of amplifier in the positive-TC generator block
* 0 <= gain <= 15
* @reference_voltage: reference voltage of amplifier
@@ -102,6 +105,7 @@ struct exynos_tmu_platform_data {
bool trigger_level1_en;
bool trigger_level2_en;
bool trigger_level3_en;
+ bool trip_en;
u8 gain;
u8 reference_voltage;
--
1.7.4.1
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH 2/2] therma: exynos: Supports thermal tripping
2012-10-30 5:32 [PATCH 2/2] therma: exynos: Supports thermal tripping Jonghwan Choi
@ 2012-10-31 5:06 ` jonghwa3.lee
2012-10-31 6:22 ` Jonghwan Choi
0 siblings, 1 reply; 3+ messages in thread
From: jonghwa3.lee @ 2012-10-31 5:06 UTC (permalink / raw)
To: Jonghwan Choi
Cc: 'open list', 'Amit Daniel Kachhap',
'Zhang Rui', 'Sachin Kamat'
On 2012년 10월 30일 14:32, Jonghwan Choi wrote:
> TMU urgently sends active-high signal (thermal trip) to PMU,
> and thermal tripping by hardware logic i.e PMU is performed.
> Thermal tripping means that PMU cut off the whole power of SoC
> by controlling external voltage regulator.
>
> Signed-off-by: Jonghwan Choi <jhbird.choi@samsung.com>
> ---
> drivers/thermal/exynos_thermal.c | 7 ++++++-
> include/linux/platform_data/exynos_thermal.h | 4 ++++
> 2 files changed, 10 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/thermal/exynos_thermal.c
> b/drivers/thermal/exynos_thermal.c
> index 6ce6667..edac601 100644
> --- a/drivers/thermal/exynos_thermal.c
> +++ b/drivers/thermal/exynos_thermal.c
> @@ -53,6 +53,7 @@
> #define EXYNOS_TMU_TRIM_TEMP_MASK 0xff
> #define EXYNOS_TMU_GAIN_SHIFT 8
> #define EXYNOS_TMU_REF_VOLTAGE_SHIFT 24
> +#define EXYNOS_TMU_TRIP_EN BIT(12)
> #define EXYNOS_TMU_CORE_ON 1
> #define EXYNOS_TMU_CORE_OFF 0
> #define EXYNOS_TMU_DEF_CODE_TO_TEMP_OFFSET 50
> @@ -656,6 +657,8 @@ static void exynos_tmu_control(struct platform_device
> *pdev, bool on)
> if (data->soc == SOC_ARCH_EXYNOS) {
> con |= pdata->noise_cancel_mode <<
> EXYNOS_TMU_TRIP_MODE_SHIFT;
> con |= (EXYNOS_MUX_ADDR_VALUE << EXYNOS_MUX_ADDR_SHIFT);
> + if (pdata->trip_en)
> + con |= EXYNOS_THERMAL_TRIP_EN;
> }
>
> if (on) {
> @@ -762,10 +765,12 @@ static struct exynos_tmu_platform_data const
> exynos_default_tmu_data = {
> .trigger_levels[0] = 85,
> .trigger_levels[1] = 103,
> .trigger_levels[2] = 110,
> + .trigger_levels[3] = 120,
> .trigger_level0_en = 1,
> .trigger_level1_en = 1,
> .trigger_level2_en = 1,
> - .trigger_level3_en = 0,
> + .trigger_level3_en = 1,
> + .trip_en = 1,
> .gain = 8,
> .reference_voltage = 16,
> .noise_cancel_mode = 4,
> diff --git a/include/linux/platform_data/exynos_thermal.h
> b/include/linux/platform_data/exynos_thermal.h
> index a7bdb2f..9e44aac 100644
> --- a/include/linux/platform_data/exynos_thermal.h
> +++ b/include/linux/platform_data/exynos_thermal.h
> @@ -79,6 +79,9 @@ struct freq_clip_table {
> * @trigger_level3_en:
> * 1 = enable trigger_level3 interrupt,
> * 0 = disable trigger_level3 interrupt
> + * @trip_en:
> + * 1 = enable thermal tripping
> + * 0 = disable thermal tripping
> * @gain: gain of amplifier in the positive-TC generator block
> * 0 <= gain <= 15
> * @reference_voltage: reference voltage of amplifier
> @@ -102,6 +105,7 @@ struct exynos_tmu_platform_data {
> bool trigger_level1_en;
> bool trigger_level2_en;
> bool trigger_level3_en;
> + bool trip_en;
I think this variable addition is not necessary. It's enough to use
trigger_level3_en variable
to determine hardware thermal tripping enable or not. Since exynos4210
has different register map from
other exynos SOC's, trigger level 3 enable is existed different region.
exynos4210 has it in
interrupt enable register and others has it in TMU core control register
(exynos4x12 and exynos5
doesn't have trigger_level3_en bit in interrupt enable register).
Thus it's better to use trigger_level3_en variable to enable hardware
thermal tripping support for compatibility.
Thanks,
Jonghwa Lee.
>
> u8 gain;
> u8 reference_voltage;
> --
> 1.7.4.1
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
>
^ permalink raw reply [flat|nested] 3+ messages in thread* RE: [PATCH 2/2] therma: exynos: Supports thermal tripping
2012-10-31 5:06 ` jonghwa3.lee
@ 2012-10-31 6:22 ` Jonghwan Choi
0 siblings, 0 replies; 3+ messages in thread
From: Jonghwan Choi @ 2012-10-31 6:22 UTC (permalink / raw)
To: jonghwa3.lee
Cc: 'open list', 'Amit Daniel Kachhap',
'Zhang Rui', 'Sachin Kamat'
Thanks.
Let me check.
> -----Original Message-----
> From: jonghwa3.lee@samsung.com [mailto:jonghwa3.lee@samsung.com]
> Sent: Wednesday, October 31, 2012 2:06 PM
> To: Jonghwan Choi
> Cc: 'open list'; 'Amit Daniel Kachhap'; 'Zhang Rui'; 'Sachin Kamat'
> Subject: Re: [PATCH 2/2] therma: exynos: Supports thermal tripping
>
> On 2012년 10월 30일 14:32, Jonghwan Choi wrote:
> > TMU urgently sends active-high signal (thermal trip) to PMU,
> > and thermal tripping by hardware logic i.e PMU is performed.
> > Thermal tripping means that PMU cut off the whole power of SoC
> > by controlling external voltage regulator.
> >
> > Signed-off-by: Jonghwan Choi <jhbird.choi@samsung.com>
> > ---
> > drivers/thermal/exynos_thermal.c | 7 ++++++-
> > include/linux/platform_data/exynos_thermal.h | 4 ++++
> > 2 files changed, 10 insertions(+), 1 deletions(-)
> >
> > diff --git a/drivers/thermal/exynos_thermal.c
> > b/drivers/thermal/exynos_thermal.c
> > index 6ce6667..edac601 100644
> > --- a/drivers/thermal/exynos_thermal.c
> > +++ b/drivers/thermal/exynos_thermal.c
> > @@ -53,6 +53,7 @@
> > #define EXYNOS_TMU_TRIM_TEMP_MASK 0xff
> > #define EXYNOS_TMU_GAIN_SHIFT 8
> > #define EXYNOS_TMU_REF_VOLTAGE_SHIFT 24
> > +#define EXYNOS_TMU_TRIP_EN BIT(12)
> > #define EXYNOS_TMU_CORE_ON 1
> > #define EXYNOS_TMU_CORE_OFF 0
> > #define EXYNOS_TMU_DEF_CODE_TO_TEMP_OFFSET 50
> > @@ -656,6 +657,8 @@ static void exynos_tmu_control(struct
> platform_device
> > *pdev, bool on)
> > if (data->soc == SOC_ARCH_EXYNOS) {
> > con |= pdata->noise_cancel_mode <<
> > EXYNOS_TMU_TRIP_MODE_SHIFT;
> > con |= (EXYNOS_MUX_ADDR_VALUE << EXYNOS_MUX_ADDR_SHIFT);
> > + if (pdata->trip_en)
> > + con |= EXYNOS_THERMAL_TRIP_EN;
> > }
> >
> > if (on) {
> > @@ -762,10 +765,12 @@ static struct exynos_tmu_platform_data const
> > exynos_default_tmu_data = {
> > .trigger_levels[0] = 85,
> > .trigger_levels[1] = 103,
> > .trigger_levels[2] = 110,
> > + .trigger_levels[3] = 120,
> > .trigger_level0_en = 1,
> > .trigger_level1_en = 1,
> > .trigger_level2_en = 1,
> > - .trigger_level3_en = 0,
> > + .trigger_level3_en = 1,
> > + .trip_en = 1,
> > .gain = 8,
> > .reference_voltage = 16,
> > .noise_cancel_mode = 4,
> > diff --git a/include/linux/platform_data/exynos_thermal.h
> > b/include/linux/platform_data/exynos_thermal.h
> > index a7bdb2f..9e44aac 100644
> > --- a/include/linux/platform_data/exynos_thermal.h
> > +++ b/include/linux/platform_data/exynos_thermal.h
> > @@ -79,6 +79,9 @@ struct freq_clip_table {
> > * @trigger_level3_en:
> > * 1 = enable trigger_level3 interrupt,
> > * 0 = disable trigger_level3 interrupt
> > + * @trip_en:
> > + * 1 = enable thermal tripping
> > + * 0 = disable thermal tripping
> > * @gain: gain of amplifier in the positive-TC generator block
> > * 0 <= gain <= 15
> > * @reference_voltage: reference voltage of amplifier
> > @@ -102,6 +105,7 @@ struct exynos_tmu_platform_data {
> > bool trigger_level1_en;
> > bool trigger_level2_en;
> > bool trigger_level3_en;
> > + bool trip_en;
> I think this variable addition is not necessary. It's enough to use
> trigger_level3_en variable
> to determine hardware thermal tripping enable or not. Since exynos4210
> has different register map from
> other exynos SOC's, trigger level 3 enable is existed different region.
> exynos4210 has it in
> interrupt enable register and others has it in TMU core control register
> (exynos4x12 and exynos5
> doesn't have trigger_level3_en bit in interrupt enable register).
> Thus it's better to use trigger_level3_en variable to enable hardware
> thermal tripping support for compatibility.
>
> Thanks,
> Jonghwa Lee.
> >
> > u8 gain;
> > u8 reference_voltage;
> > --
> > 1.7.4.1
> >
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-kernel"
> in
> > the body of a message to majordomo@vger.kernel.org
> > More majordomo info at http://vger.kernel.org/majordomo-info.html
> > Please read the FAQ at http://www.tux.org/lkml/
> >
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2012-10-31 6:22 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-10-30 5:32 [PATCH 2/2] therma: exynos: Supports thermal tripping Jonghwan Choi
2012-10-31 5:06 ` jonghwa3.lee
2012-10-31 6:22 ` Jonghwan Choi
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.