* [PATCH] thermal: exynos: Add support for TRIM_RELOAD feature at Exynos3250 @ 2014-08-19 11:57 Chanwoo Choi 2014-08-19 12:07 ` [PATCHv2] " Chanwoo Choi 0 siblings, 1 reply; 14+ messages in thread From: Chanwoo Choi @ 2014-08-19 11:57 UTC (permalink / raw) To: linux-arm-kernel This patch add support for TRIM_RELOAD feature at Exynos3250. The TMU of Exynos3250 has two TRIMINFO_CON register. Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com> Acked-by: Kyungmin Park <kyungmin.park@samsung.com> Cc: Zhang Rui <rui.zhang@intel.com> Cc: Eduardo Valentin <edubezval@gmail.com> --- drivers/thermal/samsung/exynos_tmu.c | 7 +++++-- drivers/thermal/samsung/exynos_tmu.h | 5 +++-- drivers/thermal/samsung/exynos_tmu_data.c | 9 ++++++++- drivers/thermal/samsung/exynos_tmu_data.h | 7 +++++-- 4 files changed, 21 insertions(+), 7 deletions(-) diff --git a/drivers/thermal/samsung/exynos_tmu.c b/drivers/thermal/samsung/exynos_tmu.c index acbff14..ed01606 100644 --- a/drivers/thermal/samsung/exynos_tmu.c +++ b/drivers/thermal/samsung/exynos_tmu.c @@ -164,8 +164,11 @@ static int exynos_tmu_initialize(struct platform_device *pdev) } } - if (TMU_SUPPORTS(pdata, TRIM_RELOAD)) - __raw_writel(1, data->base + reg->triminfo_ctrl); + if (TMU_SUPPORTS(pdata, TRIM_RELOAD)) { + for (i = 0; i < pdata->triminfo_reload_count; i++) + __raw_writel(pdata->triminfo_reload[i], + data->base + reg->triminfo_ctrl[i]); + } if (pdata->cal_mode == HW_MODE) goto skip_calib_data; diff --git a/drivers/thermal/samsung/exynos_tmu.h b/drivers/thermal/samsung/exynos_tmu.h index 1b4a644..72cb54e 100644 --- a/drivers/thermal/samsung/exynos_tmu.h +++ b/drivers/thermal/samsung/exynos_tmu.h @@ -151,8 +151,7 @@ struct exynos_tmu_registers { u32 triminfo_25_shift; u32 triminfo_85_shift; - u32 triminfo_ctrl; - u32 triminfo_ctrl1; + u32 triminfo_ctrl[2]; u32 triminfo_reload_shift; u32 tmu_ctrl; @@ -295,6 +294,8 @@ struct exynos_tmu_platform_data { u8 second_point_trim; u8 default_temp_offset; u8 test_mux; + u8 triminfo_reload[2]; + u8 triminfo_reload_count; enum calibration_type cal_type; enum calibration_mode cal_mode; diff --git a/drivers/thermal/samsung/exynos_tmu_data.c b/drivers/thermal/samsung/exynos_tmu_data.c index aa8e0de..25f9890 100644 --- a/drivers/thermal/samsung/exynos_tmu_data.c +++ b/drivers/thermal/samsung/exynos_tmu_data.c @@ -95,6 +95,8 @@ static const struct exynos_tmu_registers exynos3250_tmu_registers = { .triminfo_data = EXYNOS_TMU_REG_TRIMINFO, .triminfo_25_shift = EXYNOS_TRIMINFO_25_SHIFT, .triminfo_85_shift = EXYNOS_TRIMINFO_85_SHIFT, + .triminfo_ctrl[0] = EXYNOS_TMU_TRIMINFO_CON1, + .triminfo_ctrl[1] = EXYNOS_TMU_TRIMINFO_CON2, .tmu_ctrl = EXYNOS_TMU_REG_CONTROL, .test_mux_addr_shift = EXYNOS4412_MUX_ADDR_SHIFT, .buf_vref_sel_shift = EXYNOS_TMU_REF_VOLTAGE_SHIFT, @@ -160,6 +162,9 @@ static const struct exynos_tmu_registers exynos3250_tmu_registers = { .temp_level = 95, \ }, \ .freq_tab_count = 2, \ + .triminfo_reload[0] = 0x1, \ + .triminfo_reload[1] = 0x11, \ + .triminfo_reload_count = 2, \ .registers = &exynos3250_tmu_registers, \ .features = (TMU_SUPPORT_EMULATION | \ TMU_SUPPORT_FALLING_TRIP | TMU_SUPPORT_READY_STATUS | \ @@ -184,7 +189,7 @@ static const struct exynos_tmu_registers exynos4412_tmu_registers = { .triminfo_data = EXYNOS_TMU_REG_TRIMINFO, .triminfo_25_shift = EXYNOS_TRIMINFO_25_SHIFT, .triminfo_85_shift = EXYNOS_TRIMINFO_85_SHIFT, - .triminfo_ctrl = EXYNOS_TMU_TRIMINFO_CON, + .triminfo_ctrl[0] = EXYNOS_TMU_TRIMINFO_CON2, .triminfo_reload_shift = EXYNOS_TRIMINFO_RELOAD_SHIFT, .tmu_ctrl = EXYNOS_TMU_REG_CONTROL, .test_mux_addr_shift = EXYNOS4412_MUX_ADDR_SHIFT, @@ -252,6 +257,8 @@ static const struct exynos_tmu_registers exynos4412_tmu_registers = { .temp_level = 95, \ }, \ .freq_tab_count = 2, \ + .triminfo_reload[0] = 0x1, \ + .triminfo_reload_count = 1, \ .registers = &exynos4412_tmu_registers, \ .features = (TMU_SUPPORT_EMULATION | TMU_SUPPORT_TRIM_RELOAD | \ TMU_SUPPORT_FALLING_TRIP | TMU_SUPPORT_READY_STATUS | \ diff --git a/drivers/thermal/samsung/exynos_tmu_data.h b/drivers/thermal/samsung/exynos_tmu_data.h index f0979e5..e0536c3 100644 --- a/drivers/thermal/samsung/exynos_tmu_data.h +++ b/drivers/thermal/samsung/exynos_tmu_data.h @@ -57,8 +57,11 @@ #define EXYNOS4210_TMU_TRIG_LEVEL_MASK 0x1111 #define EXYNOS4210_TMU_INTCLEAR_VAL 0x1111 -/* Exynos5250 and Exynos4412 specific registers */ -#define EXYNOS_TMU_TRIMINFO_CON 0x14 +/* Exynos3250 specific registers */ +#define EXYNOS_TMU_TRIMINFO_CON1 0x10 + +/* Exynos5250, Exynos4412 and Exynos3250 specific registers */ +#define EXYNOS_TMU_TRIMINFO_CON2 0x14 #define EXYNOS_THD_TEMP_RISE 0x50 #define EXYNOS_THD_TEMP_FALL 0x54 #define EXYNOS_EMUL_CON 0x80 -- 1.8.0 ^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCHv2] thermal: exynos: Add support for TRIM_RELOAD feature at Exynos3250 2014-08-19 11:57 [PATCH] thermal: exynos: Add support for TRIM_RELOAD feature at Exynos3250 Chanwoo Choi @ 2014-08-19 12:07 ` Chanwoo Choi 2014-08-19 12:28 ` edubezval at gmail.com ` (2 more replies) 0 siblings, 3 replies; 14+ messages in thread From: Chanwoo Choi @ 2014-08-19 12:07 UTC (permalink / raw) To: linux-arm-kernel This patch add support for TRIM_RELOAD feature at Exynos3250. The TMU of Exynos3250 has two TRIMINFO_CON register. Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com> Acked-by: Kyungmin Park <kyungmin.park@samsung.com> Cc: Zhang Rui <rui.zhang@intel.com> Cc: Eduardo Valentin <edubezval@gmail.com> --- Changes from v1: - Add missing 'TMU_SUPPORT_TRIM_RELOAD' features drivers/thermal/samsung/exynos_tmu.c | 7 +++++-- drivers/thermal/samsung/exynos_tmu.h | 5 +++-- drivers/thermal/samsung/exynos_tmu_data.c | 11 +++++++++-- drivers/thermal/samsung/exynos_tmu_data.h | 7 +++++-- 4 files changed, 22 insertions(+), 8 deletions(-) diff --git a/drivers/thermal/samsung/exynos_tmu.c b/drivers/thermal/samsung/exynos_tmu.c index acbff14..ed01606 100644 --- a/drivers/thermal/samsung/exynos_tmu.c +++ b/drivers/thermal/samsung/exynos_tmu.c @@ -164,8 +164,11 @@ static int exynos_tmu_initialize(struct platform_device *pdev) } } - if (TMU_SUPPORTS(pdata, TRIM_RELOAD)) - __raw_writel(1, data->base + reg->triminfo_ctrl); + if (TMU_SUPPORTS(pdata, TRIM_RELOAD)) { + for (i = 0; i < pdata->triminfo_reload_count; i++) + __raw_writel(pdata->triminfo_reload[i], + data->base + reg->triminfo_ctrl[i]); + } if (pdata->cal_mode == HW_MODE) goto skip_calib_data; diff --git a/drivers/thermal/samsung/exynos_tmu.h b/drivers/thermal/samsung/exynos_tmu.h index 1b4a644..72cb54e 100644 --- a/drivers/thermal/samsung/exynos_tmu.h +++ b/drivers/thermal/samsung/exynos_tmu.h @@ -151,8 +151,7 @@ struct exynos_tmu_registers { u32 triminfo_25_shift; u32 triminfo_85_shift; - u32 triminfo_ctrl; - u32 triminfo_ctrl1; + u32 triminfo_ctrl[2]; u32 triminfo_reload_shift; u32 tmu_ctrl; @@ -295,6 +294,8 @@ struct exynos_tmu_platform_data { u8 second_point_trim; u8 default_temp_offset; u8 test_mux; + u8 triminfo_reload[2]; + u8 triminfo_reload_count; enum calibration_type cal_type; enum calibration_mode cal_mode; diff --git a/drivers/thermal/samsung/exynos_tmu_data.c b/drivers/thermal/samsung/exynos_tmu_data.c index aa8e0de..b45aa90 100644 --- a/drivers/thermal/samsung/exynos_tmu_data.c +++ b/drivers/thermal/samsung/exynos_tmu_data.c @@ -95,6 +95,8 @@ static const struct exynos_tmu_registers exynos3250_tmu_registers = { .triminfo_data = EXYNOS_TMU_REG_TRIMINFO, .triminfo_25_shift = EXYNOS_TRIMINFO_25_SHIFT, .triminfo_85_shift = EXYNOS_TRIMINFO_85_SHIFT, + .triminfo_ctrl[0] = EXYNOS_TMU_TRIMINFO_CON1, + .triminfo_ctrl[1] = EXYNOS_TMU_TRIMINFO_CON2, .tmu_ctrl = EXYNOS_TMU_REG_CONTROL, .test_mux_addr_shift = EXYNOS4412_MUX_ADDR_SHIFT, .buf_vref_sel_shift = EXYNOS_TMU_REF_VOLTAGE_SHIFT, @@ -160,8 +162,11 @@ static const struct exynos_tmu_registers exynos3250_tmu_registers = { .temp_level = 95, \ }, \ .freq_tab_count = 2, \ + .triminfo_reload[0] = 0x1, \ + .triminfo_reload[1] = 0x11, \ + .triminfo_reload_count = 2, \ .registers = &exynos3250_tmu_registers, \ - .features = (TMU_SUPPORT_EMULATION | \ + .features = (TMU_SUPPORT_EMULATION | TMU_SUPPORT_TRIM_RELOAD\ TMU_SUPPORT_FALLING_TRIP | TMU_SUPPORT_READY_STATUS | \ TMU_SUPPORT_EMUL_TIME) #endif @@ -184,7 +189,7 @@ static const struct exynos_tmu_registers exynos4412_tmu_registers = { .triminfo_data = EXYNOS_TMU_REG_TRIMINFO, .triminfo_25_shift = EXYNOS_TRIMINFO_25_SHIFT, .triminfo_85_shift = EXYNOS_TRIMINFO_85_SHIFT, - .triminfo_ctrl = EXYNOS_TMU_TRIMINFO_CON, + .triminfo_ctrl[0] = EXYNOS_TMU_TRIMINFO_CON2, .triminfo_reload_shift = EXYNOS_TRIMINFO_RELOAD_SHIFT, .tmu_ctrl = EXYNOS_TMU_REG_CONTROL, .test_mux_addr_shift = EXYNOS4412_MUX_ADDR_SHIFT, @@ -252,6 +257,8 @@ static const struct exynos_tmu_registers exynos4412_tmu_registers = { .temp_level = 95, \ }, \ .freq_tab_count = 2, \ + .triminfo_reload[0] = 0x1, \ + .triminfo_reload_count = 1, \ .registers = &exynos4412_tmu_registers, \ .features = (TMU_SUPPORT_EMULATION | TMU_SUPPORT_TRIM_RELOAD | \ TMU_SUPPORT_FALLING_TRIP | TMU_SUPPORT_READY_STATUS | \ diff --git a/drivers/thermal/samsung/exynos_tmu_data.h b/drivers/thermal/samsung/exynos_tmu_data.h index f0979e5..e0536c3 100644 --- a/drivers/thermal/samsung/exynos_tmu_data.h +++ b/drivers/thermal/samsung/exynos_tmu_data.h @@ -57,8 +57,11 @@ #define EXYNOS4210_TMU_TRIG_LEVEL_MASK 0x1111 #define EXYNOS4210_TMU_INTCLEAR_VAL 0x1111 -/* Exynos5250 and Exynos4412 specific registers */ -#define EXYNOS_TMU_TRIMINFO_CON 0x14 +/* Exynos3250 specific registers */ +#define EXYNOS_TMU_TRIMINFO_CON1 0x10 + +/* Exynos5250, Exynos4412 and Exynos3250 specific registers */ +#define EXYNOS_TMU_TRIMINFO_CON2 0x14 #define EXYNOS_THD_TEMP_RISE 0x50 #define EXYNOS_THD_TEMP_FALL 0x54 #define EXYNOS_EMUL_CON 0x80 -- 1.8.0 ^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCHv2] thermal: exynos: Add support for TRIM_RELOAD feature at Exynos3250 2014-08-19 12:07 ` [PATCHv2] " Chanwoo Choi @ 2014-08-19 12:28 ` edubezval at gmail.com 2014-08-19 23:47 ` Chanwoo Choi 2014-08-19 23:52 ` [PATCHv3] " Chanwoo Choi 2014-08-20 4:24 ` [PATCH] thermal: exynos: Remove duplicate code when reading triminfo register of Exynos5440 Chanwoo Choi 2 siblings, 1 reply; 14+ messages in thread From: edubezval at gmail.com @ 2014-08-19 12:28 UTC (permalink / raw) To: linux-arm-kernel Amit, On Tue, Aug 19, 2014 at 8:07 AM, Chanwoo Choi <cw00.choi@samsung.com> wrote: > This patch add support for TRIM_RELOAD feature at Exynos3250. The TMU of > Exynos3250 has two TRIMINFO_CON register. > > Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com> > Acked-by: Kyungmin Park <kyungmin.park@samsung.com> > Cc: Zhang Rui <rui.zhang@intel.com> > Cc: Eduardo Valentin <edubezval@gmail.com> What do you think about this patch? > --- > Changes from v1: > - Add missing 'TMU_SUPPORT_TRIM_RELOAD' features > > drivers/thermal/samsung/exynos_tmu.c | 7 +++++-- > drivers/thermal/samsung/exynos_tmu.h | 5 +++-- > drivers/thermal/samsung/exynos_tmu_data.c | 11 +++++++++-- > drivers/thermal/samsung/exynos_tmu_data.h | 7 +++++-- > 4 files changed, 22 insertions(+), 8 deletions(-) > > diff --git a/drivers/thermal/samsung/exynos_tmu.c b/drivers/thermal/samsung/exynos_tmu.c > index acbff14..ed01606 100644 > --- a/drivers/thermal/samsung/exynos_tmu.c > +++ b/drivers/thermal/samsung/exynos_tmu.c > @@ -164,8 +164,11 @@ static int exynos_tmu_initialize(struct platform_device *pdev) > } > } > > - if (TMU_SUPPORTS(pdata, TRIM_RELOAD)) > - __raw_writel(1, data->base + reg->triminfo_ctrl); > + if (TMU_SUPPORTS(pdata, TRIM_RELOAD)) { > + for (i = 0; i < pdata->triminfo_reload_count; i++) > + __raw_writel(pdata->triminfo_reload[i], > + data->base + reg->triminfo_ctrl[i]); > + } > > if (pdata->cal_mode == HW_MODE) > goto skip_calib_data; > diff --git a/drivers/thermal/samsung/exynos_tmu.h b/drivers/thermal/samsung/exynos_tmu.h > index 1b4a644..72cb54e 100644 > --- a/drivers/thermal/samsung/exynos_tmu.h > +++ b/drivers/thermal/samsung/exynos_tmu.h > @@ -151,8 +151,7 @@ struct exynos_tmu_registers { > u32 triminfo_25_shift; > u32 triminfo_85_shift; > > - u32 triminfo_ctrl; > - u32 triminfo_ctrl1; > + u32 triminfo_ctrl[2]; > u32 triminfo_reload_shift; > > u32 tmu_ctrl; > @@ -295,6 +294,8 @@ struct exynos_tmu_platform_data { > u8 second_point_trim; > u8 default_temp_offset; > u8 test_mux; > + u8 triminfo_reload[2]; > + u8 triminfo_reload_count; > > enum calibration_type cal_type; > enum calibration_mode cal_mode; > diff --git a/drivers/thermal/samsung/exynos_tmu_data.c b/drivers/thermal/samsung/exynos_tmu_data.c > index aa8e0de..b45aa90 100644 > --- a/drivers/thermal/samsung/exynos_tmu_data.c > +++ b/drivers/thermal/samsung/exynos_tmu_data.c > @@ -95,6 +95,8 @@ static const struct exynos_tmu_registers exynos3250_tmu_registers = { > .triminfo_data = EXYNOS_TMU_REG_TRIMINFO, > .triminfo_25_shift = EXYNOS_TRIMINFO_25_SHIFT, > .triminfo_85_shift = EXYNOS_TRIMINFO_85_SHIFT, > + .triminfo_ctrl[0] = EXYNOS_TMU_TRIMINFO_CON1, > + .triminfo_ctrl[1] = EXYNOS_TMU_TRIMINFO_CON2, > .tmu_ctrl = EXYNOS_TMU_REG_CONTROL, > .test_mux_addr_shift = EXYNOS4412_MUX_ADDR_SHIFT, > .buf_vref_sel_shift = EXYNOS_TMU_REF_VOLTAGE_SHIFT, > @@ -160,8 +162,11 @@ static const struct exynos_tmu_registers exynos3250_tmu_registers = { > .temp_level = 95, \ > }, \ > .freq_tab_count = 2, \ > + .triminfo_reload[0] = 0x1, \ > + .triminfo_reload[1] = 0x11, \ > + .triminfo_reload_count = 2, \ > .registers = &exynos3250_tmu_registers, \ > - .features = (TMU_SUPPORT_EMULATION | \ > + .features = (TMU_SUPPORT_EMULATION | TMU_SUPPORT_TRIM_RELOAD\ > TMU_SUPPORT_FALLING_TRIP | TMU_SUPPORT_READY_STATUS | \ > TMU_SUPPORT_EMUL_TIME) > #endif > @@ -184,7 +189,7 @@ static const struct exynos_tmu_registers exynos4412_tmu_registers = { > .triminfo_data = EXYNOS_TMU_REG_TRIMINFO, > .triminfo_25_shift = EXYNOS_TRIMINFO_25_SHIFT, > .triminfo_85_shift = EXYNOS_TRIMINFO_85_SHIFT, > - .triminfo_ctrl = EXYNOS_TMU_TRIMINFO_CON, > + .triminfo_ctrl[0] = EXYNOS_TMU_TRIMINFO_CON2, > .triminfo_reload_shift = EXYNOS_TRIMINFO_RELOAD_SHIFT, > .tmu_ctrl = EXYNOS_TMU_REG_CONTROL, > .test_mux_addr_shift = EXYNOS4412_MUX_ADDR_SHIFT, > @@ -252,6 +257,8 @@ static const struct exynos_tmu_registers exynos4412_tmu_registers = { > .temp_level = 95, \ > }, \ > .freq_tab_count = 2, \ > + .triminfo_reload[0] = 0x1, \ > + .triminfo_reload_count = 1, \ > .registers = &exynos4412_tmu_registers, \ > .features = (TMU_SUPPORT_EMULATION | TMU_SUPPORT_TRIM_RELOAD | \ > TMU_SUPPORT_FALLING_TRIP | TMU_SUPPORT_READY_STATUS | \ > diff --git a/drivers/thermal/samsung/exynos_tmu_data.h b/drivers/thermal/samsung/exynos_tmu_data.h > index f0979e5..e0536c3 100644 > --- a/drivers/thermal/samsung/exynos_tmu_data.h > +++ b/drivers/thermal/samsung/exynos_tmu_data.h > @@ -57,8 +57,11 @@ > #define EXYNOS4210_TMU_TRIG_LEVEL_MASK 0x1111 > #define EXYNOS4210_TMU_INTCLEAR_VAL 0x1111 > > -/* Exynos5250 and Exynos4412 specific registers */ > -#define EXYNOS_TMU_TRIMINFO_CON 0x14 > +/* Exynos3250 specific registers */ > +#define EXYNOS_TMU_TRIMINFO_CON1 0x10 > + > +/* Exynos5250, Exynos4412 and Exynos3250 specific registers */ > +#define EXYNOS_TMU_TRIMINFO_CON2 0x14 > #define EXYNOS_THD_TEMP_RISE 0x50 > #define EXYNOS_THD_TEMP_FALL 0x54 > #define EXYNOS_EMUL_CON 0x80 > -- > 1.8.0 > -- Eduardo Bezerra Valentin ^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCHv2] thermal: exynos: Add support for TRIM_RELOAD feature at Exynos3250 2014-08-19 12:28 ` edubezval at gmail.com @ 2014-08-19 23:47 ` Chanwoo Choi 0 siblings, 0 replies; 14+ messages in thread From: Chanwoo Choi @ 2014-08-19 23:47 UTC (permalink / raw) To: linux-arm-kernel Dear Eduardo, I'm so sorry. This patch has build break as following: It is my mistake about missing 'or' operation when adding TMU_SUPPORT_TRIM_RELOAD. + .features = (TMU_SUPPORT_EMULATION | TMU_SUPPORT_TRIM_RELOAD\ I'll send new patch to fix it right now. Best Regards, Chanwoo Choi On 08/19/2014 09:28 PM, edubezval at gmail.com wrote: > Amit, > > On Tue, Aug 19, 2014 at 8:07 AM, Chanwoo Choi <cw00.choi@samsung.com> wrote: >> This patch add support for TRIM_RELOAD feature at Exynos3250. The TMU of >> Exynos3250 has two TRIMINFO_CON register. >> >> Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com> >> Acked-by: Kyungmin Park <kyungmin.park@samsung.com> >> Cc: Zhang Rui <rui.zhang@intel.com> >> Cc: Eduardo Valentin <edubezval@gmail.com> > > What do you think about this patch? > >> --- >> Changes from v1: >> - Add missing 'TMU_SUPPORT_TRIM_RELOAD' features >> >> drivers/thermal/samsung/exynos_tmu.c | 7 +++++-- >> drivers/thermal/samsung/exynos_tmu.h | 5 +++-- >> drivers/thermal/samsung/exynos_tmu_data.c | 11 +++++++++-- >> drivers/thermal/samsung/exynos_tmu_data.h | 7 +++++-- >> 4 files changed, 22 insertions(+), 8 deletions(-) >> >> diff --git a/drivers/thermal/samsung/exynos_tmu.c b/drivers/thermal/samsung/exynos_tmu.c >> index acbff14..ed01606 100644 >> --- a/drivers/thermal/samsung/exynos_tmu.c >> +++ b/drivers/thermal/samsung/exynos_tmu.c >> @@ -164,8 +164,11 @@ static int exynos_tmu_initialize(struct platform_device *pdev) >> } >> } >> >> - if (TMU_SUPPORTS(pdata, TRIM_RELOAD)) >> - __raw_writel(1, data->base + reg->triminfo_ctrl); >> + if (TMU_SUPPORTS(pdata, TRIM_RELOAD)) { >> + for (i = 0; i < pdata->triminfo_reload_count; i++) >> + __raw_writel(pdata->triminfo_reload[i], >> + data->base + reg->triminfo_ctrl[i]); >> + } >> >> if (pdata->cal_mode == HW_MODE) >> goto skip_calib_data; >> diff --git a/drivers/thermal/samsung/exynos_tmu.h b/drivers/thermal/samsung/exynos_tmu.h >> index 1b4a644..72cb54e 100644 >> --- a/drivers/thermal/samsung/exynos_tmu.h >> +++ b/drivers/thermal/samsung/exynos_tmu.h >> @@ -151,8 +151,7 @@ struct exynos_tmu_registers { >> u32 triminfo_25_shift; >> u32 triminfo_85_shift; >> >> - u32 triminfo_ctrl; >> - u32 triminfo_ctrl1; >> + u32 triminfo_ctrl[2]; >> u32 triminfo_reload_shift; >> >> u32 tmu_ctrl; >> @@ -295,6 +294,8 @@ struct exynos_tmu_platform_data { >> u8 second_point_trim; >> u8 default_temp_offset; >> u8 test_mux; >> + u8 triminfo_reload[2]; >> + u8 triminfo_reload_count; >> >> enum calibration_type cal_type; >> enum calibration_mode cal_mode; >> diff --git a/drivers/thermal/samsung/exynos_tmu_data.c b/drivers/thermal/samsung/exynos_tmu_data.c >> index aa8e0de..b45aa90 100644 >> --- a/drivers/thermal/samsung/exynos_tmu_data.c >> +++ b/drivers/thermal/samsung/exynos_tmu_data.c >> @@ -95,6 +95,8 @@ static const struct exynos_tmu_registers exynos3250_tmu_registers = { >> .triminfo_data = EXYNOS_TMU_REG_TRIMINFO, >> .triminfo_25_shift = EXYNOS_TRIMINFO_25_SHIFT, >> .triminfo_85_shift = EXYNOS_TRIMINFO_85_SHIFT, >> + .triminfo_ctrl[0] = EXYNOS_TMU_TRIMINFO_CON1, >> + .triminfo_ctrl[1] = EXYNOS_TMU_TRIMINFO_CON2, >> .tmu_ctrl = EXYNOS_TMU_REG_CONTROL, >> .test_mux_addr_shift = EXYNOS4412_MUX_ADDR_SHIFT, >> .buf_vref_sel_shift = EXYNOS_TMU_REF_VOLTAGE_SHIFT, >> @@ -160,8 +162,11 @@ static const struct exynos_tmu_registers exynos3250_tmu_registers = { >> .temp_level = 95, \ >> }, \ >> .freq_tab_count = 2, \ >> + .triminfo_reload[0] = 0x1, \ >> + .triminfo_reload[1] = 0x11, \ >> + .triminfo_reload_count = 2, \ >> .registers = &exynos3250_tmu_registers, \ >> - .features = (TMU_SUPPORT_EMULATION | \ >> + .features = (TMU_SUPPORT_EMULATION | TMU_SUPPORT_TRIM_RELOAD\ >> TMU_SUPPORT_FALLING_TRIP | TMU_SUPPORT_READY_STATUS | \ >> TMU_SUPPORT_EMUL_TIME) >> #endif >> @@ -184,7 +189,7 @@ static const struct exynos_tmu_registers exynos4412_tmu_registers = { >> .triminfo_data = EXYNOS_TMU_REG_TRIMINFO, >> .triminfo_25_shift = EXYNOS_TRIMINFO_25_SHIFT, >> .triminfo_85_shift = EXYNOS_TRIMINFO_85_SHIFT, >> - .triminfo_ctrl = EXYNOS_TMU_TRIMINFO_CON, >> + .triminfo_ctrl[0] = EXYNOS_TMU_TRIMINFO_CON2, >> .triminfo_reload_shift = EXYNOS_TRIMINFO_RELOAD_SHIFT, >> .tmu_ctrl = EXYNOS_TMU_REG_CONTROL, >> .test_mux_addr_shift = EXYNOS4412_MUX_ADDR_SHIFT, >> @@ -252,6 +257,8 @@ static const struct exynos_tmu_registers exynos4412_tmu_registers = { >> .temp_level = 95, \ >> }, \ >> .freq_tab_count = 2, \ >> + .triminfo_reload[0] = 0x1, \ >> + .triminfo_reload_count = 1, \ >> .registers = &exynos4412_tmu_registers, \ >> .features = (TMU_SUPPORT_EMULATION | TMU_SUPPORT_TRIM_RELOAD | \ >> TMU_SUPPORT_FALLING_TRIP | TMU_SUPPORT_READY_STATUS | \ >> diff --git a/drivers/thermal/samsung/exynos_tmu_data.h b/drivers/thermal/samsung/exynos_tmu_data.h >> index f0979e5..e0536c3 100644 >> --- a/drivers/thermal/samsung/exynos_tmu_data.h >> +++ b/drivers/thermal/samsung/exynos_tmu_data.h >> @@ -57,8 +57,11 @@ >> #define EXYNOS4210_TMU_TRIG_LEVEL_MASK 0x1111 >> #define EXYNOS4210_TMU_INTCLEAR_VAL 0x1111 >> >> -/* Exynos5250 and Exynos4412 specific registers */ >> -#define EXYNOS_TMU_TRIMINFO_CON 0x14 >> +/* Exynos3250 specific registers */ >> +#define EXYNOS_TMU_TRIMINFO_CON1 0x10 >> + >> +/* Exynos5250, Exynos4412 and Exynos3250 specific registers */ >> +#define EXYNOS_TMU_TRIMINFO_CON2 0x14 >> #define EXYNOS_THD_TEMP_RISE 0x50 >> #define EXYNOS_THD_TEMP_FALL 0x54 >> #define EXYNOS_EMUL_CON 0x80 >> -- >> 1.8.0 >> > > > ^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCHv3] thermal: exynos: Add support for TRIM_RELOAD feature at Exynos3250 2014-08-19 12:07 ` [PATCHv2] " Chanwoo Choi 2014-08-19 12:28 ` edubezval at gmail.com @ 2014-08-19 23:52 ` Chanwoo Choi 2014-08-20 4:51 ` amit daniel kachhap ` (2 more replies) 2014-08-20 4:24 ` [PATCH] thermal: exynos: Remove duplicate code when reading triminfo register of Exynos5440 Chanwoo Choi 2 siblings, 3 replies; 14+ messages in thread From: Chanwoo Choi @ 2014-08-19 23:52 UTC (permalink / raw) To: linux-arm-kernel This patch add support for TRIM_RELOAD feature at Exynos3250. The TMU of Exynos3250 has two TRIMINFO_CON register. Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com> Acked-by: Kyungmin Park <kyungmin.park@samsung.com> Cc: Zhang Rui <rui.zhang@intel.com> Cc: Eduardo Valentin <edubezval@gmail.com> Cc: Amit Daniel Kachhap <amit.daniel@samsung.com> --- Changes from v2: - Fix build break because of missing 'or' operation. Changes from v1: - Add missing 'TMU_SUPPORT_TRIM_RELOAD' feature drivers/thermal/samsung/exynos_tmu.c | 7 +++++-- drivers/thermal/samsung/exynos_tmu.h | 5 +++-- drivers/thermal/samsung/exynos_tmu_data.c | 11 +++++++++-- drivers/thermal/samsung/exynos_tmu_data.h | 7 +++++-- 4 files changed, 22 insertions(+), 8 deletions(-) diff --git a/drivers/thermal/samsung/exynos_tmu.c b/drivers/thermal/samsung/exynos_tmu.c index acbff14..ed01606 100644 --- a/drivers/thermal/samsung/exynos_tmu.c +++ b/drivers/thermal/samsung/exynos_tmu.c @@ -164,8 +164,11 @@ static int exynos_tmu_initialize(struct platform_device *pdev) } } - if (TMU_SUPPORTS(pdata, TRIM_RELOAD)) - __raw_writel(1, data->base + reg->triminfo_ctrl); + if (TMU_SUPPORTS(pdata, TRIM_RELOAD)) { + for (i = 0; i < pdata->triminfo_reload_count; i++) + __raw_writel(pdata->triminfo_reload[i], + data->base + reg->triminfo_ctrl[i]); + } if (pdata->cal_mode == HW_MODE) goto skip_calib_data; diff --git a/drivers/thermal/samsung/exynos_tmu.h b/drivers/thermal/samsung/exynos_tmu.h index 1b4a644..72cb54e 100644 --- a/drivers/thermal/samsung/exynos_tmu.h +++ b/drivers/thermal/samsung/exynos_tmu.h @@ -151,8 +151,7 @@ struct exynos_tmu_registers { u32 triminfo_25_shift; u32 triminfo_85_shift; - u32 triminfo_ctrl; - u32 triminfo_ctrl1; + u32 triminfo_ctrl[2]; u32 triminfo_reload_shift; u32 tmu_ctrl; @@ -295,6 +294,8 @@ struct exynos_tmu_platform_data { u8 second_point_trim; u8 default_temp_offset; u8 test_mux; + u8 triminfo_reload[2]; + u8 triminfo_reload_count; enum calibration_type cal_type; enum calibration_mode cal_mode; diff --git a/drivers/thermal/samsung/exynos_tmu_data.c b/drivers/thermal/samsung/exynos_tmu_data.c index aa8e0de..8cd609c 100644 --- a/drivers/thermal/samsung/exynos_tmu_data.c +++ b/drivers/thermal/samsung/exynos_tmu_data.c @@ -95,6 +95,8 @@ static const struct exynos_tmu_registers exynos3250_tmu_registers = { .triminfo_data = EXYNOS_TMU_REG_TRIMINFO, .triminfo_25_shift = EXYNOS_TRIMINFO_25_SHIFT, .triminfo_85_shift = EXYNOS_TRIMINFO_85_SHIFT, + .triminfo_ctrl[0] = EXYNOS_TMU_TRIMINFO_CON1, + .triminfo_ctrl[1] = EXYNOS_TMU_TRIMINFO_CON2, .tmu_ctrl = EXYNOS_TMU_REG_CONTROL, .test_mux_addr_shift = EXYNOS4412_MUX_ADDR_SHIFT, .buf_vref_sel_shift = EXYNOS_TMU_REF_VOLTAGE_SHIFT, @@ -160,8 +162,11 @@ static const struct exynos_tmu_registers exynos3250_tmu_registers = { .temp_level = 95, \ }, \ .freq_tab_count = 2, \ + .triminfo_reload[0] = 0x1, \ + .triminfo_reload[1] = 0x11, \ + .triminfo_reload_count = 2, \ .registers = &exynos3250_tmu_registers, \ - .features = (TMU_SUPPORT_EMULATION | \ + .features = (TMU_SUPPORT_EMULATION | TMU_SUPPORT_TRIM_RELOAD | \ TMU_SUPPORT_FALLING_TRIP | TMU_SUPPORT_READY_STATUS | \ TMU_SUPPORT_EMUL_TIME) #endif @@ -184,7 +189,7 @@ static const struct exynos_tmu_registers exynos4412_tmu_registers = { .triminfo_data = EXYNOS_TMU_REG_TRIMINFO, .triminfo_25_shift = EXYNOS_TRIMINFO_25_SHIFT, .triminfo_85_shift = EXYNOS_TRIMINFO_85_SHIFT, - .triminfo_ctrl = EXYNOS_TMU_TRIMINFO_CON, + .triminfo_ctrl[0] = EXYNOS_TMU_TRIMINFO_CON2, .triminfo_reload_shift = EXYNOS_TRIMINFO_RELOAD_SHIFT, .tmu_ctrl = EXYNOS_TMU_REG_CONTROL, .test_mux_addr_shift = EXYNOS4412_MUX_ADDR_SHIFT, @@ -252,6 +257,8 @@ static const struct exynos_tmu_registers exynos4412_tmu_registers = { .temp_level = 95, \ }, \ .freq_tab_count = 2, \ + .triminfo_reload[0] = 0x1, \ + .triminfo_reload_count = 1, \ .registers = &exynos4412_tmu_registers, \ .features = (TMU_SUPPORT_EMULATION | TMU_SUPPORT_TRIM_RELOAD | \ TMU_SUPPORT_FALLING_TRIP | TMU_SUPPORT_READY_STATUS | \ diff --git a/drivers/thermal/samsung/exynos_tmu_data.h b/drivers/thermal/samsung/exynos_tmu_data.h index f0979e5..e0536c3 100644 --- a/drivers/thermal/samsung/exynos_tmu_data.h +++ b/drivers/thermal/samsung/exynos_tmu_data.h @@ -57,8 +57,11 @@ #define EXYNOS4210_TMU_TRIG_LEVEL_MASK 0x1111 #define EXYNOS4210_TMU_INTCLEAR_VAL 0x1111 -/* Exynos5250 and Exynos4412 specific registers */ -#define EXYNOS_TMU_TRIMINFO_CON 0x14 +/* Exynos3250 specific registers */ +#define EXYNOS_TMU_TRIMINFO_CON1 0x10 + +/* Exynos5250, Exynos4412 and Exynos3250 specific registers */ +#define EXYNOS_TMU_TRIMINFO_CON2 0x14 #define EXYNOS_THD_TEMP_RISE 0x50 #define EXYNOS_THD_TEMP_FALL 0x54 #define EXYNOS_EMUL_CON 0x80 -- 1.8.0 ^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCHv3] thermal: exynos: Add support for TRIM_RELOAD feature at Exynos3250 2014-08-19 23:52 ` [PATCHv3] " Chanwoo Choi @ 2014-08-20 4:51 ` amit daniel kachhap 2014-08-20 4:52 ` Chanwoo Choi 2014-08-20 12:14 ` Bartlomiej Zolnierkiewicz 2014-08-20 13:38 ` edubezval at gmail.com 2 siblings, 1 reply; 14+ messages in thread From: amit daniel kachhap @ 2014-08-20 4:51 UTC (permalink / raw) To: linux-arm-kernel On Wed, Aug 20, 2014 at 5:22 AM, Chanwoo Choi <cw00.choi@samsung.com> wrote: > This patch add support for TRIM_RELOAD feature at Exynos3250. The TMU of > Exynos3250 has two TRIMINFO_CON register. > > Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com> > Acked-by: Kyungmin Park <kyungmin.park@samsung.com> > Cc: Zhang Rui <rui.zhang@intel.com> > Cc: Eduardo Valentin <edubezval@gmail.com> > Cc: Amit Daniel Kachhap <amit.daniel@samsung.com> The changes looks fine, Reviewed-by: Amit Daniel Kachhap <amit.daniel@samsung.com> Thanks, Amit > --- > Changes from v2: > - Fix build break because of missing 'or' operation. > Changes from v1: > - Add missing 'TMU_SUPPORT_TRIM_RELOAD' feature > > drivers/thermal/samsung/exynos_tmu.c | 7 +++++-- > drivers/thermal/samsung/exynos_tmu.h | 5 +++-- > drivers/thermal/samsung/exynos_tmu_data.c | 11 +++++++++-- > drivers/thermal/samsung/exynos_tmu_data.h | 7 +++++-- > 4 files changed, 22 insertions(+), 8 deletions(-) > > diff --git a/drivers/thermal/samsung/exynos_tmu.c b/drivers/thermal/samsung/exynos_tmu.c > index acbff14..ed01606 100644 > --- a/drivers/thermal/samsung/exynos_tmu.c > +++ b/drivers/thermal/samsung/exynos_tmu.c > @@ -164,8 +164,11 @@ static int exynos_tmu_initialize(struct platform_device *pdev) > } > } > > - if (TMU_SUPPORTS(pdata, TRIM_RELOAD)) > - __raw_writel(1, data->base + reg->triminfo_ctrl); > + if (TMU_SUPPORTS(pdata, TRIM_RELOAD)) { > + for (i = 0; i < pdata->triminfo_reload_count; i++) > + __raw_writel(pdata->triminfo_reload[i], > + data->base + reg->triminfo_ctrl[i]); > + } > > if (pdata->cal_mode == HW_MODE) > goto skip_calib_data; > diff --git a/drivers/thermal/samsung/exynos_tmu.h b/drivers/thermal/samsung/exynos_tmu.h > index 1b4a644..72cb54e 100644 > --- a/drivers/thermal/samsung/exynos_tmu.h > +++ b/drivers/thermal/samsung/exynos_tmu.h > @@ -151,8 +151,7 @@ struct exynos_tmu_registers { > u32 triminfo_25_shift; > u32 triminfo_85_shift; > > - u32 triminfo_ctrl; > - u32 triminfo_ctrl1; > + u32 triminfo_ctrl[2]; > u32 triminfo_reload_shift; > > u32 tmu_ctrl; > @@ -295,6 +294,8 @@ struct exynos_tmu_platform_data { > u8 second_point_trim; > u8 default_temp_offset; > u8 test_mux; > + u8 triminfo_reload[2]; > + u8 triminfo_reload_count; > > enum calibration_type cal_type; > enum calibration_mode cal_mode; > diff --git a/drivers/thermal/samsung/exynos_tmu_data.c b/drivers/thermal/samsung/exynos_tmu_data.c > index aa8e0de..8cd609c 100644 > --- a/drivers/thermal/samsung/exynos_tmu_data.c > +++ b/drivers/thermal/samsung/exynos_tmu_data.c > @@ -95,6 +95,8 @@ static const struct exynos_tmu_registers exynos3250_tmu_registers = { > .triminfo_data = EXYNOS_TMU_REG_TRIMINFO, > .triminfo_25_shift = EXYNOS_TRIMINFO_25_SHIFT, > .triminfo_85_shift = EXYNOS_TRIMINFO_85_SHIFT, > + .triminfo_ctrl[0] = EXYNOS_TMU_TRIMINFO_CON1, > + .triminfo_ctrl[1] = EXYNOS_TMU_TRIMINFO_CON2, > .tmu_ctrl = EXYNOS_TMU_REG_CONTROL, > .test_mux_addr_shift = EXYNOS4412_MUX_ADDR_SHIFT, > .buf_vref_sel_shift = EXYNOS_TMU_REF_VOLTAGE_SHIFT, > @@ -160,8 +162,11 @@ static const struct exynos_tmu_registers exynos3250_tmu_registers = { > .temp_level = 95, \ > }, \ > .freq_tab_count = 2, \ > + .triminfo_reload[0] = 0x1, \ > + .triminfo_reload[1] = 0x11, \ > + .triminfo_reload_count = 2, \ > .registers = &exynos3250_tmu_registers, \ > - .features = (TMU_SUPPORT_EMULATION | \ > + .features = (TMU_SUPPORT_EMULATION | TMU_SUPPORT_TRIM_RELOAD | \ > TMU_SUPPORT_FALLING_TRIP | TMU_SUPPORT_READY_STATUS | \ > TMU_SUPPORT_EMUL_TIME) > #endif > @@ -184,7 +189,7 @@ static const struct exynos_tmu_registers exynos4412_tmu_registers = { > .triminfo_data = EXYNOS_TMU_REG_TRIMINFO, > .triminfo_25_shift = EXYNOS_TRIMINFO_25_SHIFT, > .triminfo_85_shift = EXYNOS_TRIMINFO_85_SHIFT, > - .triminfo_ctrl = EXYNOS_TMU_TRIMINFO_CON, > + .triminfo_ctrl[0] = EXYNOS_TMU_TRIMINFO_CON2, > .triminfo_reload_shift = EXYNOS_TRIMINFO_RELOAD_SHIFT, > .tmu_ctrl = EXYNOS_TMU_REG_CONTROL, > .test_mux_addr_shift = EXYNOS4412_MUX_ADDR_SHIFT, > @@ -252,6 +257,8 @@ static const struct exynos_tmu_registers exynos4412_tmu_registers = { > .temp_level = 95, \ > }, \ > .freq_tab_count = 2, \ > + .triminfo_reload[0] = 0x1, \ > + .triminfo_reload_count = 1, \ > .registers = &exynos4412_tmu_registers, \ > .features = (TMU_SUPPORT_EMULATION | TMU_SUPPORT_TRIM_RELOAD | \ > TMU_SUPPORT_FALLING_TRIP | TMU_SUPPORT_READY_STATUS | \ > diff --git a/drivers/thermal/samsung/exynos_tmu_data.h b/drivers/thermal/samsung/exynos_tmu_data.h > index f0979e5..e0536c3 100644 > --- a/drivers/thermal/samsung/exynos_tmu_data.h > +++ b/drivers/thermal/samsung/exynos_tmu_data.h > @@ -57,8 +57,11 @@ > #define EXYNOS4210_TMU_TRIG_LEVEL_MASK 0x1111 > #define EXYNOS4210_TMU_INTCLEAR_VAL 0x1111 > > -/* Exynos5250 and Exynos4412 specific registers */ > -#define EXYNOS_TMU_TRIMINFO_CON 0x14 > +/* Exynos3250 specific registers */ > +#define EXYNOS_TMU_TRIMINFO_CON1 0x10 > + > +/* Exynos5250, Exynos4412 and Exynos3250 specific registers */ > +#define EXYNOS_TMU_TRIMINFO_CON2 0x14 > #define EXYNOS_THD_TEMP_RISE 0x50 > #define EXYNOS_THD_TEMP_FALL 0x54 > #define EXYNOS_EMUL_CON 0x80 > -- > 1.8.0 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-pm" in > the body of a message to majordomo at vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCHv3] thermal: exynos: Add support for TRIM_RELOAD feature at Exynos3250 2014-08-20 4:51 ` amit daniel kachhap @ 2014-08-20 4:52 ` Chanwoo Choi 0 siblings, 0 replies; 14+ messages in thread From: Chanwoo Choi @ 2014-08-20 4:52 UTC (permalink / raw) To: linux-arm-kernel Dear Amit, On 08/20/2014 01:51 PM, amit daniel kachhap wrote: > On Wed, Aug 20, 2014 at 5:22 AM, Chanwoo Choi <cw00.choi@samsung.com> wrote: >> This patch add support for TRIM_RELOAD feature at Exynos3250. The TMU of >> Exynos3250 has two TRIMINFO_CON register. >> >> Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com> >> Acked-by: Kyungmin Park <kyungmin.park@samsung.com> >> Cc: Zhang Rui <rui.zhang@intel.com> >> Cc: Eduardo Valentin <edubezval@gmail.com> >> Cc: Amit Daniel Kachhap <amit.daniel@samsung.com> > The changes looks fine, > Reviewed-by: Amit Daniel Kachhap <amit.daniel@samsung.com> > Thanks for your review. Best Regards, Chanwoo Choi ^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCHv3] thermal: exynos: Add support for TRIM_RELOAD feature at Exynos3250 2014-08-19 23:52 ` [PATCHv3] " Chanwoo Choi 2014-08-20 4:51 ` amit daniel kachhap @ 2014-08-20 12:14 ` Bartlomiej Zolnierkiewicz 2014-08-20 13:38 ` edubezval at gmail.com 2 siblings, 0 replies; 14+ messages in thread From: Bartlomiej Zolnierkiewicz @ 2014-08-20 12:14 UTC (permalink / raw) To: linux-arm-kernel Hi, On Wednesday, August 20, 2014 08:52:44 AM Chanwoo Choi wrote: > This patch add support for TRIM_RELOAD feature at Exynos3250. The TMU of > Exynos3250 has two TRIMINFO_CON register. This patch also silently fixes wrong behavior of the Exynos thermal driver and TRIM_RELOAD feature for Exynos5260 and Exynos5420. Currently these SoCs claim TRIM_RELOAD support but don't have triminfo_ctrl register address defined in their struct exynos_tmu_registers entries. This causes write of value "1" to data->base + 0x00 address (which happens to be TRIMINFO register). Your patch doesn't define triminfo_reload_count on all SoCs that use TRIM_RELOAD flag so wrong write to TRIMINFO register no longer happens but I think that it would be the best to fix it explicitly by removing TRIM_RELOAD flag for Exynos5260 and Exynos5420 in a pre-patch to your patch, please see: https://lkml.org/lkml/2014/8/20/334 One other issue with your patch is that it modifies struct exynos_tmu_registers and struct exynos_tmu_platform_data without updating corresponding documentation. Please fix it. Otherwise it looks fine to me. Best regards, -- Bartlomiej Zolnierkiewicz Samsung R&D Institute Poland Samsung Electronics > Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com> > Acked-by: Kyungmin Park <kyungmin.park@samsung.com> > Cc: Zhang Rui <rui.zhang@intel.com> > Cc: Eduardo Valentin <edubezval@gmail.com> > Cc: Amit Daniel Kachhap <amit.daniel@samsung.com> > --- > Changes from v2: > - Fix build break because of missing 'or' operation. > Changes from v1: > - Add missing 'TMU_SUPPORT_TRIM_RELOAD' feature > > drivers/thermal/samsung/exynos_tmu.c | 7 +++++-- > drivers/thermal/samsung/exynos_tmu.h | 5 +++-- > drivers/thermal/samsung/exynos_tmu_data.c | 11 +++++++++-- > drivers/thermal/samsung/exynos_tmu_data.h | 7 +++++-- > 4 files changed, 22 insertions(+), 8 deletions(-) > > diff --git a/drivers/thermal/samsung/exynos_tmu.c b/drivers/thermal/samsung/exynos_tmu.c > index acbff14..ed01606 100644 > --- a/drivers/thermal/samsung/exynos_tmu.c > +++ b/drivers/thermal/samsung/exynos_tmu.c > @@ -164,8 +164,11 @@ static int exynos_tmu_initialize(struct platform_device *pdev) > } > } > > - if (TMU_SUPPORTS(pdata, TRIM_RELOAD)) > - __raw_writel(1, data->base + reg->triminfo_ctrl); > + if (TMU_SUPPORTS(pdata, TRIM_RELOAD)) { > + for (i = 0; i < pdata->triminfo_reload_count; i++) > + __raw_writel(pdata->triminfo_reload[i], > + data->base + reg->triminfo_ctrl[i]); > + } > > if (pdata->cal_mode == HW_MODE) > goto skip_calib_data; > diff --git a/drivers/thermal/samsung/exynos_tmu.h b/drivers/thermal/samsung/exynos_tmu.h > index 1b4a644..72cb54e 100644 > --- a/drivers/thermal/samsung/exynos_tmu.h > +++ b/drivers/thermal/samsung/exynos_tmu.h > @@ -151,8 +151,7 @@ struct exynos_tmu_registers { > u32 triminfo_25_shift; > u32 triminfo_85_shift; > > - u32 triminfo_ctrl; > - u32 triminfo_ctrl1; > + u32 triminfo_ctrl[2]; > u32 triminfo_reload_shift; > > u32 tmu_ctrl; > @@ -295,6 +294,8 @@ struct exynos_tmu_platform_data { > u8 second_point_trim; > u8 default_temp_offset; > u8 test_mux; > + u8 triminfo_reload[2]; > + u8 triminfo_reload_count; > > enum calibration_type cal_type; > enum calibration_mode cal_mode; > diff --git a/drivers/thermal/samsung/exynos_tmu_data.c b/drivers/thermal/samsung/exynos_tmu_data.c > index aa8e0de..8cd609c 100644 > --- a/drivers/thermal/samsung/exynos_tmu_data.c > +++ b/drivers/thermal/samsung/exynos_tmu_data.c > @@ -95,6 +95,8 @@ static const struct exynos_tmu_registers exynos3250_tmu_registers = { > .triminfo_data = EXYNOS_TMU_REG_TRIMINFO, > .triminfo_25_shift = EXYNOS_TRIMINFO_25_SHIFT, > .triminfo_85_shift = EXYNOS_TRIMINFO_85_SHIFT, > + .triminfo_ctrl[0] = EXYNOS_TMU_TRIMINFO_CON1, > + .triminfo_ctrl[1] = EXYNOS_TMU_TRIMINFO_CON2, > .tmu_ctrl = EXYNOS_TMU_REG_CONTROL, > .test_mux_addr_shift = EXYNOS4412_MUX_ADDR_SHIFT, > .buf_vref_sel_shift = EXYNOS_TMU_REF_VOLTAGE_SHIFT, > @@ -160,8 +162,11 @@ static const struct exynos_tmu_registers exynos3250_tmu_registers = { > .temp_level = 95, \ > }, \ > .freq_tab_count = 2, \ > + .triminfo_reload[0] = 0x1, \ > + .triminfo_reload[1] = 0x11, \ > + .triminfo_reload_count = 2, \ > .registers = &exynos3250_tmu_registers, \ > - .features = (TMU_SUPPORT_EMULATION | \ > + .features = (TMU_SUPPORT_EMULATION | TMU_SUPPORT_TRIM_RELOAD | \ > TMU_SUPPORT_FALLING_TRIP | TMU_SUPPORT_READY_STATUS | \ > TMU_SUPPORT_EMUL_TIME) > #endif > @@ -184,7 +189,7 @@ static const struct exynos_tmu_registers exynos4412_tmu_registers = { > .triminfo_data = EXYNOS_TMU_REG_TRIMINFO, > .triminfo_25_shift = EXYNOS_TRIMINFO_25_SHIFT, > .triminfo_85_shift = EXYNOS_TRIMINFO_85_SHIFT, > - .triminfo_ctrl = EXYNOS_TMU_TRIMINFO_CON, > + .triminfo_ctrl[0] = EXYNOS_TMU_TRIMINFO_CON2, > .triminfo_reload_shift = EXYNOS_TRIMINFO_RELOAD_SHIFT, > .tmu_ctrl = EXYNOS_TMU_REG_CONTROL, > .test_mux_addr_shift = EXYNOS4412_MUX_ADDR_SHIFT, > @@ -252,6 +257,8 @@ static const struct exynos_tmu_registers exynos4412_tmu_registers = { > .temp_level = 95, \ > }, \ > .freq_tab_count = 2, \ > + .triminfo_reload[0] = 0x1, \ > + .triminfo_reload_count = 1, \ > .registers = &exynos4412_tmu_registers, \ > .features = (TMU_SUPPORT_EMULATION | TMU_SUPPORT_TRIM_RELOAD | \ > TMU_SUPPORT_FALLING_TRIP | TMU_SUPPORT_READY_STATUS | \ > diff --git a/drivers/thermal/samsung/exynos_tmu_data.h b/drivers/thermal/samsung/exynos_tmu_data.h > index f0979e5..e0536c3 100644 > --- a/drivers/thermal/samsung/exynos_tmu_data.h > +++ b/drivers/thermal/samsung/exynos_tmu_data.h > @@ -57,8 +57,11 @@ > #define EXYNOS4210_TMU_TRIG_LEVEL_MASK 0x1111 > #define EXYNOS4210_TMU_INTCLEAR_VAL 0x1111 > > -/* Exynos5250 and Exynos4412 specific registers */ > -#define EXYNOS_TMU_TRIMINFO_CON 0x14 > +/* Exynos3250 specific registers */ > +#define EXYNOS_TMU_TRIMINFO_CON1 0x10 > + > +/* Exynos5250, Exynos4412 and Exynos3250 specific registers */ > +#define EXYNOS_TMU_TRIMINFO_CON2 0x14 > #define EXYNOS_THD_TEMP_RISE 0x50 > #define EXYNOS_THD_TEMP_FALL 0x54 > #define EXYNOS_EMUL_CON 0x80 ^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCHv3] thermal: exynos: Add support for TRIM_RELOAD feature at Exynos3250 2014-08-19 23:52 ` [PATCHv3] " Chanwoo Choi 2014-08-20 4:51 ` amit daniel kachhap 2014-08-20 12:14 ` Bartlomiej Zolnierkiewicz @ 2014-08-20 13:38 ` edubezval at gmail.com 2014-08-21 1:38 ` Chanwoo Choi 2 siblings, 1 reply; 14+ messages in thread From: edubezval at gmail.com @ 2014-08-20 13:38 UTC (permalink / raw) To: linux-arm-kernel Hello Chanwoo, On Tue, Aug 19, 2014 at 7:52 PM, Chanwoo Choi <cw00.choi@samsung.com> wrote: > This patch add support for TRIM_RELOAD feature at Exynos3250. The TMU of > Exynos3250 has two TRIMINFO_CON register. Can you please split the two changes above into two patches? Meaning, one that adds TRIMINFO_CON2 and another that adds TRIM_RELOAD? > > Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com> > Acked-by: Kyungmin Park <kyungmin.park@samsung.com> > Cc: Zhang Rui <rui.zhang@intel.com> > Cc: Eduardo Valentin <edubezval@gmail.com> > Cc: Amit Daniel Kachhap <amit.daniel@samsung.com> > --- > Changes from v2: > - Fix build break because of missing 'or' operation. > Changes from v1: > - Add missing 'TMU_SUPPORT_TRIM_RELOAD' feature > > drivers/thermal/samsung/exynos_tmu.c | 7 +++++-- > drivers/thermal/samsung/exynos_tmu.h | 5 +++-- > drivers/thermal/samsung/exynos_tmu_data.c | 11 +++++++++-- > drivers/thermal/samsung/exynos_tmu_data.h | 7 +++++-- > 4 files changed, 22 insertions(+), 8 deletions(-) > > diff --git a/drivers/thermal/samsung/exynos_tmu.c b/drivers/thermal/samsung/exynos_tmu.c > index acbff14..ed01606 100644 > --- a/drivers/thermal/samsung/exynos_tmu.c > +++ b/drivers/thermal/samsung/exynos_tmu.c > @@ -164,8 +164,11 @@ static int exynos_tmu_initialize(struct platform_device *pdev) > } > } > > - if (TMU_SUPPORTS(pdata, TRIM_RELOAD)) > - __raw_writel(1, data->base + reg->triminfo_ctrl); > + if (TMU_SUPPORTS(pdata, TRIM_RELOAD)) { > + for (i = 0; i < pdata->triminfo_reload_count; i++) > + __raw_writel(pdata->triminfo_reload[i], > + data->base + reg->triminfo_ctrl[i]); > + } What is the logic behind the trim reload feature? Which SoCs support it? > > if (pdata->cal_mode == HW_MODE) > goto skip_calib_data; > diff --git a/drivers/thermal/samsung/exynos_tmu.h b/drivers/thermal/samsung/exynos_tmu.h > index 1b4a644..72cb54e 100644 > --- a/drivers/thermal/samsung/exynos_tmu.h > +++ b/drivers/thermal/samsung/exynos_tmu.h > @@ -151,8 +151,7 @@ struct exynos_tmu_registers { > u32 triminfo_25_shift; > u32 triminfo_85_shift; > > - u32 triminfo_ctrl; > - u32 triminfo_ctrl1; > + u32 triminfo_ctrl[2]; The above change needs to be documented. > u32 triminfo_reload_shift; > > u32 tmu_ctrl; > @@ -295,6 +294,8 @@ struct exynos_tmu_platform_data { > u8 second_point_trim; > u8 default_temp_offset; > u8 test_mux; > + u8 triminfo_reload[2]; > + u8 triminfo_reload_count; > The above addition needs to be documented too. > enum calibration_type cal_type; > enum calibration_mode cal_mode; > diff --git a/drivers/thermal/samsung/exynos_tmu_data.c b/drivers/thermal/samsung/exynos_tmu_data.c > index aa8e0de..8cd609c 100644 > --- a/drivers/thermal/samsung/exynos_tmu_data.c > +++ b/drivers/thermal/samsung/exynos_tmu_data.c > @@ -95,6 +95,8 @@ static const struct exynos_tmu_registers exynos3250_tmu_registers = { > .triminfo_data = EXYNOS_TMU_REG_TRIMINFO, > .triminfo_25_shift = EXYNOS_TRIMINFO_25_SHIFT, > .triminfo_85_shift = EXYNOS_TRIMINFO_85_SHIFT, > + .triminfo_ctrl[0] = EXYNOS_TMU_TRIMINFO_CON1, > + .triminfo_ctrl[1] = EXYNOS_TMU_TRIMINFO_CON2, > .tmu_ctrl = EXYNOS_TMU_REG_CONTROL, > .test_mux_addr_shift = EXYNOS4412_MUX_ADDR_SHIFT, > .buf_vref_sel_shift = EXYNOS_TMU_REF_VOLTAGE_SHIFT, > @@ -160,8 +162,11 @@ static const struct exynos_tmu_registers exynos3250_tmu_registers = { > .temp_level = 95, \ > }, \ > .freq_tab_count = 2, \ > + .triminfo_reload[0] = 0x1, \ > + .triminfo_reload[1] = 0x11, \ What does 0x1 mean? How about 0x11? > + .triminfo_reload_count = 2, \ What is count? > .registers = &exynos3250_tmu_registers, \ > - .features = (TMU_SUPPORT_EMULATION | \ > + .features = (TMU_SUPPORT_EMULATION | TMU_SUPPORT_TRIM_RELOAD | \ > TMU_SUPPORT_FALLING_TRIP | TMU_SUPPORT_READY_STATUS | \ > TMU_SUPPORT_EMUL_TIME) > #endif > @@ -184,7 +189,7 @@ static const struct exynos_tmu_registers exynos4412_tmu_registers = { > .triminfo_data = EXYNOS_TMU_REG_TRIMINFO, > .triminfo_25_shift = EXYNOS_TRIMINFO_25_SHIFT, > .triminfo_85_shift = EXYNOS_TRIMINFO_85_SHIFT, > - .triminfo_ctrl = EXYNOS_TMU_TRIMINFO_CON, > + .triminfo_ctrl[0] = EXYNOS_TMU_TRIMINFO_CON2, > .triminfo_reload_shift = EXYNOS_TRIMINFO_RELOAD_SHIFT, > .tmu_ctrl = EXYNOS_TMU_REG_CONTROL, > .test_mux_addr_shift = EXYNOS4412_MUX_ADDR_SHIFT, > @@ -252,6 +257,8 @@ static const struct exynos_tmu_registers exynos4412_tmu_registers = { > .temp_level = 95, \ > }, \ > .freq_tab_count = 2, \ > + .triminfo_reload[0] = 0x1, \ > + .triminfo_reload_count = 1, \ Your patch description says nothing about TRIM_RELOAD on 4412, does it? What about the other SoCs? > .registers = &exynos4412_tmu_registers, \ > .features = (TMU_SUPPORT_EMULATION | TMU_SUPPORT_TRIM_RELOAD | \ > TMU_SUPPORT_FALLING_TRIP | TMU_SUPPORT_READY_STATUS | \ > diff --git a/drivers/thermal/samsung/exynos_tmu_data.h b/drivers/thermal/samsung/exynos_tmu_data.h > index f0979e5..e0536c3 100644 > --- a/drivers/thermal/samsung/exynos_tmu_data.h > +++ b/drivers/thermal/samsung/exynos_tmu_data.h > @@ -57,8 +57,11 @@ > #define EXYNOS4210_TMU_TRIG_LEVEL_MASK 0x1111 > #define EXYNOS4210_TMU_INTCLEAR_VAL 0x1111 > > -/* Exynos5250 and Exynos4412 specific registers */ > -#define EXYNOS_TMU_TRIMINFO_CON 0x14 > +/* Exynos3250 specific registers */ > +#define EXYNOS_TMU_TRIMINFO_CON1 0x10 > + > +/* Exynos5250, Exynos4412 and Exynos3250 specific registers */ > +#define EXYNOS_TMU_TRIMINFO_CON2 0x14 > #define EXYNOS_THD_TEMP_RISE 0x50 > #define EXYNOS_THD_TEMP_FALL 0x54 > #define EXYNOS_EMUL_CON 0x80 > -- > 1.8.0 > -- Eduardo Bezerra Valentin ^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCHv3] thermal: exynos: Add support for TRIM_RELOAD feature at Exynos3250 2014-08-20 13:38 ` edubezval at gmail.com @ 2014-08-21 1:38 ` Chanwoo Choi 2014-08-21 10:32 ` Andreas Färber 0 siblings, 1 reply; 14+ messages in thread From: Chanwoo Choi @ 2014-08-21 1:38 UTC (permalink / raw) To: linux-arm-kernel Dear Eduardo, On 08/20/2014 10:38 PM, edubezval at gmail.com wrote: > Hello Chanwoo, > > On Tue, Aug 19, 2014 at 7:52 PM, Chanwoo Choi <cw00.choi@samsung.com> wrote: >> This patch add support for TRIM_RELOAD feature at Exynos3250. The TMU of >> Exynos3250 has two TRIMINFO_CON register. > > Can you please split the two changes above into two patches? Meaning, > one that adds TRIMINFO_CON2 and another that adds TRIM_RELOAD? OK, I'll split this patch as two patches. > >> >> Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com> >> Acked-by: Kyungmin Park <kyungmin.park@samsung.com> >> Cc: Zhang Rui <rui.zhang@intel.com> >> Cc: Eduardo Valentin <edubezval@gmail.com> >> Cc: Amit Daniel Kachhap <amit.daniel@samsung.com> >> --- >> Changes from v2: >> - Fix build break because of missing 'or' operation. >> Changes from v1: >> - Add missing 'TMU_SUPPORT_TRIM_RELOAD' feature >> >> drivers/thermal/samsung/exynos_tmu.c | 7 +++++-- >> drivers/thermal/samsung/exynos_tmu.h | 5 +++-- >> drivers/thermal/samsung/exynos_tmu_data.c | 11 +++++++++-- >> drivers/thermal/samsung/exynos_tmu_data.h | 7 +++++-- >> 4 files changed, 22 insertions(+), 8 deletions(-) >> >> diff --git a/drivers/thermal/samsung/exynos_tmu.c b/drivers/thermal/samsung/exynos_tmu.c >> index acbff14..ed01606 100644 >> --- a/drivers/thermal/samsung/exynos_tmu.c >> +++ b/drivers/thermal/samsung/exynos_tmu.c >> @@ -164,8 +164,11 @@ static int exynos_tmu_initialize(struct platform_device *pdev) >> } >> } >> >> - if (TMU_SUPPORTS(pdata, TRIM_RELOAD)) >> - __raw_writel(1, data->base + reg->triminfo_ctrl); >> + if (TMU_SUPPORTS(pdata, TRIM_RELOAD)) { >> + for (i = 0; i < pdata->triminfo_reload_count; i++) >> + __raw_writel(pdata->triminfo_reload[i], >> + data->base + reg->triminfo_ctrl[i]); >> + } > > What is the logic behind the trim reload feature? Which SoCs support it? TRIMINFO_CONTROL register has 'RELOAD' field. TMU of Exynos SOC have to set 'RELOAD' field of TRIMINFO_CONTROL register before reading TRIMINFO register. As I know, Exynos4412/Exynos4212 and Exynos3250 SoC need RELOAD feature. > >> >> if (pdata->cal_mode == HW_MODE) >> goto skip_calib_data; >> diff --git a/drivers/thermal/samsung/exynos_tmu.h b/drivers/thermal/samsung/exynos_tmu.h >> index 1b4a644..72cb54e 100644 >> --- a/drivers/thermal/samsung/exynos_tmu.h >> +++ b/drivers/thermal/samsung/exynos_tmu.h >> @@ -151,8 +151,7 @@ struct exynos_tmu_registers { >> u32 triminfo_25_shift; >> u32 triminfo_85_shift; >> >> - u32 triminfo_ctrl; >> - u32 triminfo_ctrl1; >> + u32 triminfo_ctrl[2]; > > > The above change needs to be documented. OK, I'll add it. > >> u32 triminfo_reload_shift; >> >> u32 tmu_ctrl; >> @@ -295,6 +294,8 @@ struct exynos_tmu_platform_data { >> u8 second_point_trim; >> u8 default_temp_offset; >> u8 test_mux; >> + u8 triminfo_reload[2]; >> + u8 triminfo_reload_count; >> > > The above addition needs to be documented too. OK, I'll add it. > >> enum calibration_type cal_type; >> enum calibration_mode cal_mode; >> diff --git a/drivers/thermal/samsung/exynos_tmu_data.c b/drivers/thermal/samsung/exynos_tmu_data.c >> index aa8e0de..8cd609c 100644 >> --- a/drivers/thermal/samsung/exynos_tmu_data.c >> +++ b/drivers/thermal/samsung/exynos_tmu_data.c >> @@ -95,6 +95,8 @@ static const struct exynos_tmu_registers exynos3250_tmu_registers = { >> .triminfo_data = EXYNOS_TMU_REG_TRIMINFO, >> .triminfo_25_shift = EXYNOS_TRIMINFO_25_SHIFT, >> .triminfo_85_shift = EXYNOS_TRIMINFO_85_SHIFT, >> + .triminfo_ctrl[0] = EXYNOS_TMU_TRIMINFO_CON1, >> + .triminfo_ctrl[1] = EXYNOS_TMU_TRIMINFO_CON2, >> .tmu_ctrl = EXYNOS_TMU_REG_CONTROL, >> .test_mux_addr_shift = EXYNOS4412_MUX_ADDR_SHIFT, >> .buf_vref_sel_shift = EXYNOS_TMU_REF_VOLTAGE_SHIFT, >> @@ -160,8 +162,11 @@ static const struct exynos_tmu_registers exynos3250_tmu_registers = { >> .temp_level = 95, \ >> }, \ >> .freq_tab_count = 2, \ >> + .triminfo_reload[0] = 0x1, \ >> + .triminfo_reload[1] = 0x11, \ > > What does 0x1 mean? How about 0x11? The bit of 'RELOAD' field in TRIMINFO_CONTROL register is [0]. and The bit of 'AC Time' field in TRIMINFO_CONTROL register is [5:4]. 0x1 means that set RELOAD field. 0x11 means that set RELOAD field and ACTIME field. > >> + .triminfo_reload_count = 2, \ > > What is count? Just, the number of TRIMINFO_CONTROL registers. Exynos4412/4212 has only one TRIMINFO_CONTROL register and Exynos3250 has two TRIMINFO_CONTROL register. > >> .registers = &exynos3250_tmu_registers, \ >> - .features = (TMU_SUPPORT_EMULATION | \ >> + .features = (TMU_SUPPORT_EMULATION | TMU_SUPPORT_TRIM_RELOAD | \ >> TMU_SUPPORT_FALLING_TRIP | TMU_SUPPORT_READY_STATUS | \ >> TMU_SUPPORT_EMUL_TIME) >> #endif >> @@ -184,7 +189,7 @@ static const struct exynos_tmu_registers exynos4412_tmu_registers = { >> .triminfo_data = EXYNOS_TMU_REG_TRIMINFO, >> .triminfo_25_shift = EXYNOS_TRIMINFO_25_SHIFT, >> .triminfo_85_shift = EXYNOS_TRIMINFO_85_SHIFT, >> - .triminfo_ctrl = EXYNOS_TMU_TRIMINFO_CON, >> + .triminfo_ctrl[0] = EXYNOS_TMU_TRIMINFO_CON2, >> .triminfo_reload_shift = EXYNOS_TRIMINFO_RELOAD_SHIFT, >> .tmu_ctrl = EXYNOS_TMU_REG_CONTROL, >> .test_mux_addr_shift = EXYNOS4412_MUX_ADDR_SHIFT, >> @@ -252,6 +257,8 @@ static const struct exynos_tmu_registers exynos4412_tmu_registers = { >> .temp_level = 95, \ >> }, \ >> .freq_tab_count = 2, \ >> + .triminfo_reload[0] = 0x1, \ >> + .triminfo_reload_count = 1, \ > > > Your patch description says nothing about TRIM_RELOAD on 4412, does it? Exynos3250 has two triminfo_control register, existing exynos_tmu_platform_data don't support Exynos3250 RELOAD feature because the existing exynos_tmu_platform_data has only one triminfo_control register. > > What about the other SoCs? I has only boards based on Exynos4 and Exynos3250 SoC. I can't test Exynos5 SoC series. Best Regards, Chanwoo Choi > >> .registers = &exynos4412_tmu_registers, \ >> .features = (TMU_SUPPORT_EMULATION | TMU_SUPPORT_TRIM_RELOAD | \ >> TMU_SUPPORT_FALLING_TRIP | TMU_SUPPORT_READY_STATUS | \ >> diff --git a/drivers/thermal/samsung/exynos_tmu_data.h b/drivers/thermal/samsung/exynos_tmu_data.h >> index f0979e5..e0536c3 100644 >> --- a/drivers/thermal/samsung/exynos_tmu_data.h >> +++ b/drivers/thermal/samsung/exynos_tmu_data.h >> @@ -57,8 +57,11 @@ >> #define EXYNOS4210_TMU_TRIG_LEVEL_MASK 0x1111 >> #define EXYNOS4210_TMU_INTCLEAR_VAL 0x1111 >> >> -/* Exynos5250 and Exynos4412 specific registers */ >> -#define EXYNOS_TMU_TRIMINFO_CON 0x14 >> +/* Exynos3250 specific registers */ >> +#define EXYNOS_TMU_TRIMINFO_CON1 0x10 >> + >> +/* Exynos5250, Exynos4412 and Exynos3250 specific registers */ >> +#define EXYNOS_TMU_TRIMINFO_CON2 0x14 >> #define EXYNOS_THD_TEMP_RISE 0x50 >> #define EXYNOS_THD_TEMP_FALL 0x54 >> #define EXYNOS_EMUL_CON 0x80 >> -- >> 1.8.0 >> > > > ^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCHv3] thermal: exynos: Add support for TRIM_RELOAD feature at Exynos3250 2014-08-21 1:38 ` Chanwoo Choi @ 2014-08-21 10:32 ` Andreas Färber 2014-08-21 16:24 ` Chanwoo Choi 0 siblings, 1 reply; 14+ messages in thread From: Andreas Färber @ 2014-08-21 10:32 UTC (permalink / raw) To: linux-arm-kernel Hello, Am 21.08.2014 03:38, schrieb Chanwoo Choi: > On 08/20/2014 10:38 PM, edubezval at gmail.com wrote: >> On Tue, Aug 19, 2014 at 7:52 PM, Chanwoo Choi <cw00.choi@samsung.com> wrote: >>> diff --git a/drivers/thermal/samsung/exynos_tmu_data.c b/drivers/thermal/samsung/exynos_tmu_data.c >>> index aa8e0de..8cd609c 100644 >>> --- a/drivers/thermal/samsung/exynos_tmu_data.c >>> +++ b/drivers/thermal/samsung/exynos_tmu_data.c >>> @@ -95,6 +95,8 @@ static const struct exynos_tmu_registers exynos3250_tmu_registers = { >>> .triminfo_data = EXYNOS_TMU_REG_TRIMINFO, >>> .triminfo_25_shift = EXYNOS_TRIMINFO_25_SHIFT, >>> .triminfo_85_shift = EXYNOS_TRIMINFO_85_SHIFT, >>> + .triminfo_ctrl[0] = EXYNOS_TMU_TRIMINFO_CON1, >>> + .triminfo_ctrl[1] = EXYNOS_TMU_TRIMINFO_CON2, >>> .tmu_ctrl = EXYNOS_TMU_REG_CONTROL, >>> .test_mux_addr_shift = EXYNOS4412_MUX_ADDR_SHIFT, >>> .buf_vref_sel_shift = EXYNOS_TMU_REF_VOLTAGE_SHIFT, >>> @@ -160,8 +162,11 @@ static const struct exynos_tmu_registers exynos3250_tmu_registers = { >>> .temp_level = 95, \ >>> }, \ >>> .freq_tab_count = 2, \ >>> + .triminfo_reload[0] = 0x1, \ >>> + .triminfo_reload[1] = 0x11, \ >> >> What does 0x1 mean? How about 0x11? > > The bit of 'RELOAD' field in TRIMINFO_CONTROL register is [0]. > and The bit of 'AC Time' field in TRIMINFO_CONTROL register is [5:4]. > > 0x1 means that set RELOAD field. > 0x11 means that set RELOAD field and ACTIME field. Might be a good idea to use #defines for 0x1 and 0x10 then? Regards, Andreas -- SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 N?rnberg, Germany GF: Jeff Hawn, Jennifer Guild, Felix Imend?rffer; HRB 16746 AG N?rnberg ^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCHv3] thermal: exynos: Add support for TRIM_RELOAD feature at Exynos3250 2014-08-21 10:32 ` Andreas Färber @ 2014-08-21 16:24 ` Chanwoo Choi 0 siblings, 0 replies; 14+ messages in thread From: Chanwoo Choi @ 2014-08-21 16:24 UTC (permalink / raw) To: linux-arm-kernel Dear Andreas, On Thu, Aug 21, 2014 at 7:32 PM, Andreas F?rber <afaerber@suse.de> wrote: > Hello, > > Am 21.08.2014 03:38, schrieb Chanwoo Choi: >> On 08/20/2014 10:38 PM, edubezval at gmail.com wrote: >>> On Tue, Aug 19, 2014 at 7:52 PM, Chanwoo Choi <cw00.choi@samsung.com> wrote: >>>> diff --git a/drivers/thermal/samsung/exynos_tmu_data.c b/drivers/thermal/samsung/exynos_tmu_data.c >>>> index aa8e0de..8cd609c 100644 >>>> --- a/drivers/thermal/samsung/exynos_tmu_data.c >>>> +++ b/drivers/thermal/samsung/exynos_tmu_data.c >>>> @@ -95,6 +95,8 @@ static const struct exynos_tmu_registers exynos3250_tmu_registers = { >>>> .triminfo_data = EXYNOS_TMU_REG_TRIMINFO, >>>> .triminfo_25_shift = EXYNOS_TRIMINFO_25_SHIFT, >>>> .triminfo_85_shift = EXYNOS_TRIMINFO_85_SHIFT, >>>> + .triminfo_ctrl[0] = EXYNOS_TMU_TRIMINFO_CON1, >>>> + .triminfo_ctrl[1] = EXYNOS_TMU_TRIMINFO_CON2, >>>> .tmu_ctrl = EXYNOS_TMU_REG_CONTROL, >>>> .test_mux_addr_shift = EXYNOS4412_MUX_ADDR_SHIFT, >>>> .buf_vref_sel_shift = EXYNOS_TMU_REF_VOLTAGE_SHIFT, >>>> @@ -160,8 +162,11 @@ static const struct exynos_tmu_registers exynos3250_tmu_registers = { >>>> .temp_level = 95, \ >>>> }, \ >>>> .freq_tab_count = 2, \ >>>> + .triminfo_reload[0] = 0x1, \ >>>> + .triminfo_reload[1] = 0x11, \ >>> >>> What does 0x1 mean? How about 0x11? >> >> The bit of 'RELOAD' field in TRIMINFO_CONTROL register is [0]. >> and The bit of 'AC Time' field in TRIMINFO_CONTROL register is [5:4]. >> >> 0x1 means that set RELOAD field. >> 0x11 means that set RELOAD field and ACTIME field. > > Might be a good idea to use #defines for 0x1 and 0x10 then? I agree. I'm implementing it to use 'define' keyword. Thanks, Chanwoo Choi ^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH] thermal: exynos: Remove duplicate code when reading triminfo register of Exynos5440 2014-08-19 12:07 ` [PATCHv2] " Chanwoo Choi 2014-08-19 12:28 ` edubezval at gmail.com 2014-08-19 23:52 ` [PATCHv3] " Chanwoo Choi @ 2014-08-20 4:24 ` Chanwoo Choi 2014-08-20 5:03 ` amit daniel kachhap 2 siblings, 1 reply; 14+ messages in thread From: Chanwoo Choi @ 2014-08-20 4:24 UTC (permalink / raw) To: linux-arm-kernel This patch remove simply duplicate code when reading triminfo register of Exynos5440. Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com> Acked-by: Kyungmin Park <kyungmin.park@samsung.com> Cc: Zhang Rui <rui.zhang@intel.com> Cc: Eduardo Valentin <edubezval@gmail.com> Cc: Amit Daniel Kachhap <amit.daniel@samsung.com> --- drivers/thermal/samsung/exynos_tmu.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/thermal/samsung/exynos_tmu.c b/drivers/thermal/samsung/exynos_tmu.c index ed01606..13c2e01 100644 --- a/drivers/thermal/samsung/exynos_tmu.c +++ b/drivers/thermal/samsung/exynos_tmu.c @@ -181,15 +181,13 @@ static int exynos_tmu_initialize(struct platform_device *pdev) */ switch (data->id) { case 0: + case 2: trim_info = readl(data->base + EXYNOS5440_EFUSE_SWAP_OFFSET + reg->triminfo_data); break; case 1: trim_info = readl(data->base + reg->triminfo_data); break; - case 2: - trim_info = readl(data->base - - EXYNOS5440_EFUSE_SWAP_OFFSET + reg->triminfo_data); } } else { /* On exynos5420 the triminfo register is in the shared space */ -- 1.8.0 ^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH] thermal: exynos: Remove duplicate code when reading triminfo register of Exynos5440 2014-08-20 4:24 ` [PATCH] thermal: exynos: Remove duplicate code when reading triminfo register of Exynos5440 Chanwoo Choi @ 2014-08-20 5:03 ` amit daniel kachhap 0 siblings, 0 replies; 14+ messages in thread From: amit daniel kachhap @ 2014-08-20 5:03 UTC (permalink / raw) To: linux-arm-kernel On Wed, Aug 20, 2014 at 9:54 AM, Chanwoo Choi <cw00.choi@samsung.com> wrote: > This patch remove simply duplicate code when reading triminfo register of Exynos5440. > > Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com> > Acked-by: Kyungmin Park <kyungmin.park@samsung.com> > Cc: Zhang Rui <rui.zhang@intel.com> > Cc: Eduardo Valentin <edubezval@gmail.com> > Cc: Amit Daniel Kachhap <amit.daniel@samsung.com> Reviewed-by: Amit Daniel Kachhap <amit.daniel@samsung.com> Thanks, Amit > --- > drivers/thermal/samsung/exynos_tmu.c | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > > diff --git a/drivers/thermal/samsung/exynos_tmu.c b/drivers/thermal/samsung/exynos_tmu.c > index ed01606..13c2e01 100644 > --- a/drivers/thermal/samsung/exynos_tmu.c > +++ b/drivers/thermal/samsung/exynos_tmu.c > @@ -181,15 +181,13 @@ static int exynos_tmu_initialize(struct platform_device *pdev) > */ > switch (data->id) { > case 0: > + case 2: > trim_info = readl(data->base + > EXYNOS5440_EFUSE_SWAP_OFFSET + reg->triminfo_data); > break; > case 1: > trim_info = readl(data->base + reg->triminfo_data); > break; > - case 2: > - trim_info = readl(data->base - > - EXYNOS5440_EFUSE_SWAP_OFFSET + reg->triminfo_data); > } > } else { > /* On exynos5420 the triminfo register is in the shared space */ > -- > 1.8.0 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-pm" in > the body of a message to majordomo at vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2014-08-21 16:24 UTC | newest] Thread overview: 14+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2014-08-19 11:57 [PATCH] thermal: exynos: Add support for TRIM_RELOAD feature at Exynos3250 Chanwoo Choi 2014-08-19 12:07 ` [PATCHv2] " Chanwoo Choi 2014-08-19 12:28 ` edubezval at gmail.com 2014-08-19 23:47 ` Chanwoo Choi 2014-08-19 23:52 ` [PATCHv3] " Chanwoo Choi 2014-08-20 4:51 ` amit daniel kachhap 2014-08-20 4:52 ` Chanwoo Choi 2014-08-20 12:14 ` Bartlomiej Zolnierkiewicz 2014-08-20 13:38 ` edubezval at gmail.com 2014-08-21 1:38 ` Chanwoo Choi 2014-08-21 10:32 ` Andreas Färber 2014-08-21 16:24 ` Chanwoo Choi 2014-08-20 4:24 ` [PATCH] thermal: exynos: Remove duplicate code when reading triminfo register of Exynos5440 Chanwoo Choi 2014-08-20 5:03 ` amit daniel kachhap
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).