Re: [PATCH 10/10] thermal: exynos: remove identical values from exynos*_tmu_registers structures
On 5/5/14, Bartlomiej Zolnierkiewicz b.zolnier...@samsung.com wrote: There is no need for abstracting configuration for registers that are identical on all SoC types. Changes look fine and also that shift and masks may not change in future socs also. Reviewed-by: Amit Daniel Kachhapamit.dan...@samsung.com There should be no functional changes caused by this patch. Signed-off-by: Bartlomiej Zolnierkiewicz b.zolnier...@samsung.com --- drivers/thermal/samsung/exynos_tmu.c | 12 ++-- drivers/thermal/samsung/exynos_tmu.h | 11 --- drivers/thermal/samsung/exynos_tmu_data.c | 15 --- 3 files changed, 6 insertions(+), 32 deletions(-) diff --git a/drivers/thermal/samsung/exynos_tmu.c b/drivers/thermal/samsung/exynos_tmu.c index 45d7c6f..d37e755 100644 --- a/drivers/thermal/samsung/exynos_tmu.c +++ b/drivers/thermal/samsung/exynos_tmu.c @@ -215,11 +215,11 @@ static void exynos_tmu_control(struct platform_device *pdev, bool on) if (pdata-test_mux) con |= (pdata-test_mux reg-test_mux_addr_shift); - con = ~(reg-buf_vref_sel_mask reg-buf_vref_sel_shift); - con |= pdata-reference_voltage reg-buf_vref_sel_shift; + con = ~(EXYNOS_TMU_REF_VOLTAGE_MASK EXYNOS_TMU_REF_VOLTAGE_SHIFT); + con |= pdata-reference_voltage EXYNOS_TMU_REF_VOLTAGE_SHIFT; - con = ~(reg-buf_slope_sel_mask reg-buf_slope_sel_shift); - con |= (pdata-gain reg-buf_slope_sel_shift); + con = ~(EXYNOS_TMU_BUF_SLOPE_SEL_MASK EXYNOS_TMU_BUF_SLOPE_SEL_SHIFT); + con |= (pdata-gain EXYNOS_TMU_BUF_SLOPE_SEL_SHIFT); if (pdata-noise_cancel_mode) { con = ~(reg-therm_trip_mode_mask @@ -228,7 +228,7 @@ static void exynos_tmu_control(struct platform_device *pdev, bool on) } if (on) { - con |= (1 reg-core_en_shift); + con |= (1 EXYNOS_TMU_CORE_EN_SHIFT); interrupt_en = pdata-trigger_enable[3] reg-inten_rise3_shift | pdata-trigger_enable[2] reg-inten_rise2_shift | @@ -238,7 +238,7 @@ static void exynos_tmu_control(struct platform_device *pdev, bool on) interrupt_en |= interrupt_en reg-inten_fall0_shift; } else { - con = ~(1 reg-core_en_shift); + con = ~(1 EXYNOS_TMU_CORE_EN_SHIFT); interrupt_en = 0; /* Disable all interrupts */ } writel(interrupt_en, data-base + reg-tmu_inten); diff --git a/drivers/thermal/samsung/exynos_tmu.h b/drivers/thermal/samsung/exynos_tmu.h index 4845171..5c25a4b 100644 --- a/drivers/thermal/samsung/exynos_tmu.h +++ b/drivers/thermal/samsung/exynos_tmu.h @@ -69,15 +69,9 @@ enum soc_type { * @triminfo_ctrl: trim info controller register. * @tmu_ctrl: TMU main controller register. * @test_mux_addr_shift: shift bits of test mux address. - * @buf_vref_sel_shift: shift bits of reference voltage in tmu_ctrl register. - * @buf_vref_sel_mask: mask bits of reference voltage in tmu_ctrl register. * @therm_trip_mode_shift: shift bits of tripping mode in tmu_ctrl register. * @therm_trip_mode_mask: mask bits of tripping mode in tmu_ctrl register. * @therm_trip_en_shift: shift bits of tripping enable in tmu_ctrl register. - * @buf_slope_sel_shift: shift bits of amplifier gain value in tmu_ctrl - register. - * @buf_slope_sel_mask: mask bits of amplifier gain value in tmu_ctrl register. - * @core_en_shift: shift bits of TMU core enable bit in tmu_ctrl register. * @tmu_status: register drescribing the TMU status. * @tmu_cur_temp: register containing the current temperature of the TMU. * @threshold_temp: register containing the base threshold level. @@ -111,14 +105,9 @@ struct exynos_tmu_registers { u32 tmu_ctrl; u32 test_mux_addr_shift; - u32 buf_vref_sel_shift; - u32 buf_vref_sel_mask; u32 therm_trip_mode_shift; u32 therm_trip_mode_mask; u32 therm_trip_en_shift; - u32 buf_slope_sel_shift; - u32 buf_slope_sel_mask; - u32 core_en_shift; u32 tmu_status; diff --git a/drivers/thermal/samsung/exynos_tmu_data.c b/drivers/thermal/samsung/exynos_tmu_data.c index ef7f186..32530dc 100644 --- a/drivers/thermal/samsung/exynos_tmu_data.c +++ b/drivers/thermal/samsung/exynos_tmu_data.c @@ -28,11 +28,6 @@ static const struct exynos_tmu_registers exynos4210_tmu_registers = { .triminfo_data = EXYNOS_TMU_REG_TRIMINFO, .tmu_ctrl = EXYNOS_TMU_REG_CONTROL, - .buf_vref_sel_shift = EXYNOS_TMU_REF_VOLTAGE_SHIFT, - .buf_vref_sel_mask = EXYNOS_TMU_REF_VOLTAGE_MASK, - .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 =
[PATCH 10/10] thermal: exynos: remove identical values from exynos*_tmu_registers structures
There is no need for abstracting configuration for registers that are identical on all SoC types. There should be no functional changes caused by this patch. Signed-off-by: Bartlomiej Zolnierkiewicz b.zolnier...@samsung.com --- drivers/thermal/samsung/exynos_tmu.c | 12 ++-- drivers/thermal/samsung/exynos_tmu.h | 11 --- drivers/thermal/samsung/exynos_tmu_data.c | 15 --- 3 files changed, 6 insertions(+), 32 deletions(-) diff --git a/drivers/thermal/samsung/exynos_tmu.c b/drivers/thermal/samsung/exynos_tmu.c index 45d7c6f..d37e755 100644 --- a/drivers/thermal/samsung/exynos_tmu.c +++ b/drivers/thermal/samsung/exynos_tmu.c @@ -215,11 +215,11 @@ static void exynos_tmu_control(struct platform_device *pdev, bool on) if (pdata-test_mux) con |= (pdata-test_mux reg-test_mux_addr_shift); - con = ~(reg-buf_vref_sel_mask reg-buf_vref_sel_shift); - con |= pdata-reference_voltage reg-buf_vref_sel_shift; + con = ~(EXYNOS_TMU_REF_VOLTAGE_MASK EXYNOS_TMU_REF_VOLTAGE_SHIFT); + con |= pdata-reference_voltage EXYNOS_TMU_REF_VOLTAGE_SHIFT; - con = ~(reg-buf_slope_sel_mask reg-buf_slope_sel_shift); - con |= (pdata-gain reg-buf_slope_sel_shift); + con = ~(EXYNOS_TMU_BUF_SLOPE_SEL_MASK EXYNOS_TMU_BUF_SLOPE_SEL_SHIFT); + con |= (pdata-gain EXYNOS_TMU_BUF_SLOPE_SEL_SHIFT); if (pdata-noise_cancel_mode) { con = ~(reg-therm_trip_mode_mask @@ -228,7 +228,7 @@ static void exynos_tmu_control(struct platform_device *pdev, bool on) } if (on) { - con |= (1 reg-core_en_shift); + con |= (1 EXYNOS_TMU_CORE_EN_SHIFT); interrupt_en = pdata-trigger_enable[3] reg-inten_rise3_shift | pdata-trigger_enable[2] reg-inten_rise2_shift | @@ -238,7 +238,7 @@ static void exynos_tmu_control(struct platform_device *pdev, bool on) interrupt_en |= interrupt_en reg-inten_fall0_shift; } else { - con = ~(1 reg-core_en_shift); + con = ~(1 EXYNOS_TMU_CORE_EN_SHIFT); interrupt_en = 0; /* Disable all interrupts */ } writel(interrupt_en, data-base + reg-tmu_inten); diff --git a/drivers/thermal/samsung/exynos_tmu.h b/drivers/thermal/samsung/exynos_tmu.h index 4845171..5c25a4b 100644 --- a/drivers/thermal/samsung/exynos_tmu.h +++ b/drivers/thermal/samsung/exynos_tmu.h @@ -69,15 +69,9 @@ enum soc_type { * @triminfo_ctrl: trim info controller register. * @tmu_ctrl: TMU main controller register. * @test_mux_addr_shift: shift bits of test mux address. - * @buf_vref_sel_shift: shift bits of reference voltage in tmu_ctrl register. - * @buf_vref_sel_mask: mask bits of reference voltage in tmu_ctrl register. * @therm_trip_mode_shift: shift bits of tripping mode in tmu_ctrl register. * @therm_trip_mode_mask: mask bits of tripping mode in tmu_ctrl register. * @therm_trip_en_shift: shift bits of tripping enable in tmu_ctrl register. - * @buf_slope_sel_shift: shift bits of amplifier gain value in tmu_ctrl - register. - * @buf_slope_sel_mask: mask bits of amplifier gain value in tmu_ctrl register. - * @core_en_shift: shift bits of TMU core enable bit in tmu_ctrl register. * @tmu_status: register drescribing the TMU status. * @tmu_cur_temp: register containing the current temperature of the TMU. * @threshold_temp: register containing the base threshold level. @@ -111,14 +105,9 @@ struct exynos_tmu_registers { u32 tmu_ctrl; u32 test_mux_addr_shift; - u32 buf_vref_sel_shift; - u32 buf_vref_sel_mask; u32 therm_trip_mode_shift; u32 therm_trip_mode_mask; u32 therm_trip_en_shift; - u32 buf_slope_sel_shift; - u32 buf_slope_sel_mask; - u32 core_en_shift; u32 tmu_status; diff --git a/drivers/thermal/samsung/exynos_tmu_data.c b/drivers/thermal/samsung/exynos_tmu_data.c index ef7f186..32530dc 100644 --- a/drivers/thermal/samsung/exynos_tmu_data.c +++ b/drivers/thermal/samsung/exynos_tmu_data.c @@ -28,11 +28,6 @@ static const struct exynos_tmu_registers exynos4210_tmu_registers = { .triminfo_data = EXYNOS_TMU_REG_TRIMINFO, .tmu_ctrl = EXYNOS_TMU_REG_CONTROL, - .buf_vref_sel_shift = EXYNOS_TMU_REF_VOLTAGE_SHIFT, - .buf_vref_sel_mask = EXYNOS_TMU_REF_VOLTAGE_MASK, - .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_temp = EXYNOS4210_TMU_REG_THRESHOLD_TEMP, @@ -92,14 +87,9 @@ static const struct exynos_tmu_registers exynos4412_tmu_registers = { .triminfo_ctrl =