Re: [RESEND PATCH] thermal: samsung: Add TMU support for Exynos3250 SoC

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

 




On Tue, Jul 22, 2014 at 11:41:56AM +0530, amit daniel kachhap wrote:
> On Mon, Jul 21, 2014 at 11:34 AM, Zhang Rui <rui.zhang@xxxxxxxxx> wrote:
> > On Fri, 2014-07-18 at 09:00 +0900, Chanwoo Choi wrote:
> >> This patch adds the registers, bit fields and compatible strings
> >> required to support for the 1 TMU channels on Exynos3250.
> >>
> >> Signed-off-by: Chanwoo Choi <cw00.choi@xxxxxxxxxxx>
> >> [Add MUX address setting bits by Jonghwa Lee]
> >> Signed-off-by: Jonghwa Lee <jonghwa3.lee@xxxxxxxxxxx>
> >> Acked-by: Kyungmin Park <kyungmin.park@xxxxxxxxxxx>
> >> Reviewed-by: Amit Daniel Kachhap<amit.daniel@xxxxxxxxxxx>
> >
> > Eduardo and Amit,
> >
> > do you have any comments on this?
> The changes looks fine to me. I already reviewed these patches earlier.


Bartlomiej, can you please rebase your work on top of this one?

> 
> thanks,
> Amit
> >
> > thanks,
> > rui
> >
> >> ---
> >>  .../devicetree/bindings/thermal/exynos-thermal.txt |  1 +
> >>  drivers/thermal/samsung/exynos_tmu.c               |  7 +-
> >>  drivers/thermal/samsung/exynos_tmu.h               |  3 +-
> >>  drivers/thermal/samsung/exynos_tmu_data.c          | 89 ++++++++++++++++++++++
> >>  drivers/thermal/samsung/exynos_tmu_data.h          |  7 ++
> >>  5 files changed, 105 insertions(+), 2 deletions(-)
> >>
> >> diff --git a/Documentation/devicetree/bindings/thermal/exynos-thermal.txt b/Documentation/devicetree/bindings/thermal/exynos-thermal.txt
> >> index c949092..ae738f5 100644
> >> --- a/Documentation/devicetree/bindings/thermal/exynos-thermal.txt
> >> +++ b/Documentation/devicetree/bindings/thermal/exynos-thermal.txt
> >> @@ -3,6 +3,7 @@
> >>  ** Required properties:
> >>
> >>  - compatible : One of the following:
> >> +            "samsung,exynos3250-tmu"
> >>              "samsung,exynos4412-tmu"
> >>              "samsung,exynos4210-tmu"
> >>              "samsung,exynos5250-tmu"
> >> diff --git a/drivers/thermal/samsung/exynos_tmu.c b/drivers/thermal/samsung/exynos_tmu.c
> >> index d7ca9f4..a2a08ea 100644
> >> --- a/drivers/thermal/samsung/exynos_tmu.c
> >> +++ b/drivers/thermal/samsung/exynos_tmu.c
> >> @@ -505,6 +505,10 @@ static irqreturn_t exynos_tmu_irq(int irq, void *id)
> >>
> >>  static const struct of_device_id exynos_tmu_match[] = {
> >>       {
> >> +             .compatible = "samsung,exynos3250-tmu",
> >> +             .data = (void *)EXYNOS3250_TMU_DRV_DATA,
> >> +     },
> >> +     {
> >>               .compatible = "samsung,exynos4210-tmu",
> >>               .data = (void *)EXYNOS4210_TMU_DRV_DATA,
> >>       },
> >> @@ -677,7 +681,8 @@ static int exynos_tmu_probe(struct platform_device *pdev)
> >>               goto err_clk_sec;
> >>       }
> >>
> >> -     if (pdata->type == SOC_ARCH_EXYNOS4210 ||
> >> +     if (pdata->type == SOC_ARCH_EXYNOS3250 ||
> >> +         pdata->type == SOC_ARCH_EXYNOS4210 ||
> >>           pdata->type == SOC_ARCH_EXYNOS4412 ||
> >>           pdata->type == SOC_ARCH_EXYNOS5250 ||
> >>           pdata->type == SOC_ARCH_EXYNOS5260 ||
> >> diff --git a/drivers/thermal/samsung/exynos_tmu.h b/drivers/thermal/samsung/exynos_tmu.h
> >> index edd08cf..1b4a644 100644
> >> --- a/drivers/thermal/samsung/exynos_tmu.h
> >> +++ b/drivers/thermal/samsung/exynos_tmu.h
> >> @@ -40,7 +40,8 @@ enum calibration_mode {
> >>  };
> >>
> >>  enum soc_type {
> >> -     SOC_ARCH_EXYNOS4210 = 1,
> >> +     SOC_ARCH_EXYNOS3250 = 1,
> >> +     SOC_ARCH_EXYNOS4210,
> >>       SOC_ARCH_EXYNOS4412,
> >>       SOC_ARCH_EXYNOS5250,
> >>       SOC_ARCH_EXYNOS5260,
> >> diff --git a/drivers/thermal/samsung/exynos_tmu_data.c b/drivers/thermal/samsung/exynos_tmu_data.c
> >> index c1d81dc..aa8e0de 100644
> >> --- a/drivers/thermal/samsung/exynos_tmu_data.c
> >> +++ b/drivers/thermal/samsung/exynos_tmu_data.c
> >> @@ -90,6 +90,95 @@ struct exynos_tmu_init_data const exynos4210_default_tmu_data = {
> >>  };
> >>  #endif
> >>
> >> +#if defined(CONFIG_SOC_EXYNOS3250)
> >> +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,
> >> +     .tmu_ctrl = EXYNOS_TMU_REG_CONTROL,
> >> +     .test_mux_addr_shift = EXYNOS4412_MUX_ADDR_SHIFT,
> >> +     .buf_vref_sel_shift = EXYNOS_TMU_REF_VOLTAGE_SHIFT,
> >> +     .buf_vref_sel_mask = EXYNOS_TMU_REF_VOLTAGE_MASK,
> >> +     .therm_trip_mode_shift = EXYNOS_TMU_TRIP_MODE_SHIFT,
> >> +     .therm_trip_mode_mask = EXYNOS_TMU_TRIP_MODE_MASK,
> >> +     .therm_trip_en_shift = EXYNOS_TMU_THERM_TRIP_EN_SHIFT,
> >> +     .buf_slope_sel_shift = EXYNOS_TMU_BUF_SLOPE_SEL_SHIFT,
> >> +     .buf_slope_sel_mask = EXYNOS_TMU_BUF_SLOPE_SEL_MASK,
> >> +     .core_en_shift = EXYNOS_TMU_CORE_EN_SHIFT,
> >> +     .tmu_status = EXYNOS_TMU_REG_STATUS,
> >> +     .tmu_cur_temp = EXYNOS_TMU_REG_CURRENT_TEMP,
> >> +     .threshold_th0 = EXYNOS_THD_TEMP_RISE,
> >> +     .threshold_th1 = EXYNOS_THD_TEMP_FALL,
> >> +     .tmu_inten = EXYNOS_TMU_REG_INTEN,
> >> +     .inten_rise0_shift = EXYNOS_TMU_INTEN_RISE0_SHIFT,
> >> +     .inten_rise1_shift = EXYNOS_TMU_INTEN_RISE1_SHIFT,
> >> +     .inten_rise2_shift = EXYNOS_TMU_INTEN_RISE2_SHIFT,
> >> +     .inten_fall0_shift = EXYNOS_TMU_INTEN_FALL0_SHIFT,
> >> +     .tmu_intstat = EXYNOS_TMU_REG_INTSTAT,
> >> +     .tmu_intclear = EXYNOS_TMU_REG_INTCLEAR,
> >> +     .intclr_fall_shift = EXYNOS_TMU_CLEAR_FALL_INT_SHIFT,
> >> +     .intclr_rise_shift = EXYNOS_TMU_RISE_INT_SHIFT,
> >> +     .intclr_rise_mask = EXYNOS_TMU_RISE_INT_MASK,
> >> +     .intclr_fall_mask = EXYNOS_TMU_FALL_INT_MASK,
> >> +     .emul_con = EXYNOS_EMUL_CON,
> >> +     .emul_temp_shift = EXYNOS_EMUL_DATA_SHIFT,
> >> +     .emul_time_shift = EXYNOS_EMUL_TIME_SHIFT,
> >> +     .emul_time_mask = EXYNOS_EMUL_TIME_MASK,
> >> +};
> >> +
> >> +#define EXYNOS3250_TMU_DATA \
> >> +     .threshold_falling = 10, \
> >> +     .trigger_levels[0] = 70, \
> >> +     .trigger_levels[1] = 95, \
> >> +     .trigger_levels[2] = 110, \
> >> +     .trigger_levels[3] = 120, \
> >> +     .trigger_enable[0] = true, \
> >> +     .trigger_enable[1] = true, \
> >> +     .trigger_enable[2] = true, \
> >> +     .trigger_enable[3] = false, \
> >> +     .trigger_type[0] = THROTTLE_ACTIVE, \
> >> +     .trigger_type[1] = THROTTLE_ACTIVE, \
> >> +     .trigger_type[2] = SW_TRIP, \
> >> +     .trigger_type[3] = HW_TRIP, \
> >> +     .max_trigger_level = 4, \
> >> +     .gain = 8, \
> >> +     .reference_voltage = 16, \
> >> +     .noise_cancel_mode = 4, \
> >> +     .cal_type = TYPE_TWO_POINT_TRIMMING, \
> >> +     .efuse_value = 55, \
> >> +     .min_efuse_value = 40, \
> >> +     .max_efuse_value = 100, \
> >> +     .first_point_trim = 25, \
> >> +     .second_point_trim = 85, \
> >> +     .default_temp_offset = 50, \
> >> +     .freq_tab[0] = { \
> >> +             .freq_clip_max = 800 * 1000, \
> >> +             .temp_level = 70, \
> >> +     }, \
> >> +     .freq_tab[1] = { \
> >> +             .freq_clip_max = 400 * 1000, \
> >> +             .temp_level = 95, \
> >> +     }, \
> >> +     .freq_tab_count = 2, \
> >> +     .registers = &exynos3250_tmu_registers, \
> >> +     .features = (TMU_SUPPORT_EMULATION | \
> >> +                     TMU_SUPPORT_FALLING_TRIP | TMU_SUPPORT_READY_STATUS | \
> >> +                     TMU_SUPPORT_EMUL_TIME)
> >> +#endif
> >> +
> >> +#if defined(CONFIG_SOC_EXYNOS3250)
> >> +struct exynos_tmu_init_data const exynos3250_default_tmu_data = {
> >> +     .tmu_data = {
> >> +             {
> >> +                     EXYNOS3250_TMU_DATA,
> >> +                     .type = SOC_ARCH_EXYNOS3250,
> >> +                     .test_mux = EXYNOS4412_MUX_ADDR_VALUE,
> >> +             },
> >> +     },
> >> +     .tmu_count = 1,
> >> +};
> >> +#endif
> >> +
> >>  #if defined(CONFIG_SOC_EXYNOS4412) || defined(CONFIG_SOC_EXYNOS5250)
> >>  static const struct exynos_tmu_registers exynos4412_tmu_registers = {
> >>       .triminfo_data = EXYNOS_TMU_REG_TRIMINFO,
> >> diff --git a/drivers/thermal/samsung/exynos_tmu_data.h b/drivers/thermal/samsung/exynos_tmu_data.h
> >> index d268981..f0979e5 100644
> >> --- a/drivers/thermal/samsung/exynos_tmu_data.h
> >> +++ b/drivers/thermal/samsung/exynos_tmu_data.h
> >> @@ -148,6 +148,13 @@
> >>  #define EXYNOS5440_TMU_TH_RISE4_SHIFT                24
> >>  #define EXYNOS5440_EFUSE_SWAP_OFFSET         8
> >>
> >> +#if defined(CONFIG_SOC_EXYNOS3250)
> >> +extern struct exynos_tmu_init_data const exynos3250_default_tmu_data;
> >> +#define EXYNOS3250_TMU_DRV_DATA (&exynos3250_default_tmu_data)
> >> +#else
> >> +#define EXYNOS3250_TMU_DRV_DATA (NULL)
> >> +#endif
> >> +
> >>  #if defined(CONFIG_CPU_EXYNOS4210)
> >>  extern struct exynos_tmu_init_data const exynos4210_default_tmu_data;
> >>  #define EXYNOS4210_TMU_DRV_DATA (&exynos4210_default_tmu_data)
> >
> >
> >
> > _______________________________________________
> > linux-arm-kernel mailing list
> > linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
> > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
> --
> To unsubscribe from this list: send the line "unsubscribe linux-pm" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux