Re: [PATCH 2/2] thermal: TI: always register sensor driver with corethermal framework

From: Eduardo Valentin
Date: Fri Oct 04 2013 - 14:25:25 EST


On 04-10-2013 08:40, Bartlomiej Zolnierkiewicz wrote:
> There is little sense in having separate config option for
> registering the sensor driver with the core thermal framework.
> Fix it by integrating TI_THERMAL config option with TI_SOC_THERMAL
> one.

I am a bit confused of what is exactly wrong or needs fixing. The code
organization is so that the file ti-bandgap.c adds the support for TI
Bandgap IP. It is not aware of thermal framework. And the
ti-thermal-common.c file is expected to talk to thermal framework. Can
you please elaborate a bit more why this split is hurting you?

>
> Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@xxxxxxxxxxx>
> Signed-off-by: Kyungmin Park <kyungmin.park@xxxxxxxxxxx>
> ---
> drivers/thermal/ti-soc-thermal/Kconfig | 15 +++-----------
> drivers/thermal/ti-soc-thermal/Makefile | 2 +-
> drivers/thermal/ti-soc-thermal/ti-thermal.h | 31 -----------------------------
> 3 files changed, 4 insertions(+), 44 deletions(-)
>
> diff --git a/drivers/thermal/ti-soc-thermal/Kconfig b/drivers/thermal/ti-soc-thermal/Kconfig
> index bf5d6e4..c64e78b 100644
> --- a/drivers/thermal/ti-soc-thermal/Kconfig
> +++ b/drivers/thermal/ti-soc-thermal/Kconfig
> @@ -7,18 +7,9 @@ config TI_SOC_THERMAL
> OMAP4460+ on die bandgap temperature sensor support. The register
> set is part of system control module.
>
> - This includes alert interrupts generation and also the TSHUT
> - support.
> -
> -config TI_THERMAL
> - bool "Texas Instruments SoCs thermal framework support"
> - depends on TI_SOC_THERMAL
> - help
> - If you say yes here you want to get support for generic thermal
> - framework for the Texas Instruments on die bandgap temperature sensor.
> -
> - This includes trip points definitions, extrapolation rules and
> - CPU cooling device bindings.
> + This includes alert interrupts generation, the TSHUT support,
> + trip points definitions, extrapolation rules and CPU cooling
> + device bindings.
>
> config OMAP4_THERMAL
> bool "Texas Instruments OMAP4 thermal support"
> diff --git a/drivers/thermal/ti-soc-thermal/Makefile b/drivers/thermal/ti-soc-thermal/Makefile
> index 1226b24..c75cc91 100644
> --- a/drivers/thermal/ti-soc-thermal/Makefile
> +++ b/drivers/thermal/ti-soc-thermal/Makefile
> @@ -1,6 +1,6 @@
> obj-$(CONFIG_TI_SOC_THERMAL) += ti-soc-thermal.o
> ti-soc-thermal-y := ti-bandgap.o
> -ti-soc-thermal-$(CONFIG_TI_THERMAL) += ti-thermal-common.o
> +ti-soc-thermal-y += ti-thermal-common.o
> ti-soc-thermal-$(CONFIG_DRA752_THERMAL) += dra752-thermal-data.o
> ti-soc-thermal-$(CONFIG_OMAP4_THERMAL) += omap4-thermal-data.o
> ti-soc-thermal-$(CONFIG_OMAP5_THERMAL) += omap5-thermal-data.o
> diff --git a/drivers/thermal/ti-soc-thermal/ti-thermal.h b/drivers/thermal/ti-soc-thermal/ti-thermal.h
> index f8b7ffe..5fb427c 100644
> --- a/drivers/thermal/ti-soc-thermal/ti-thermal.h
> +++ b/drivers/thermal/ti-soc-thermal/ti-thermal.h
> @@ -83,41 +83,10 @@
> #define ti_thermal_is_valid_trip(trip) \
> ((trip) >= 0 && (trip) < OMAP_TRIP_NUMBER)
>
> -#ifdef CONFIG_TI_THERMAL
> int ti_thermal_expose_sensor(struct ti_bandgap *bgp, int id, char *domain);
> int ti_thermal_remove_sensor(struct ti_bandgap *bgp, int id);
> int ti_thermal_report_sensor_temperature(struct ti_bandgap *bgp, int id);
> int ti_thermal_register_cpu_cooling(struct ti_bandgap *bgp, int id);
> int ti_thermal_unregister_cpu_cooling(struct ti_bandgap *bgp, int id);
> -#else
> -static inline
> -int ti_thermal_expose_sensor(struct ti_bandgap *bgp, int id, char *domain)
> -{
> - return 0;
> -}
>
> -static inline
> -int ti_thermal_remove_sensor(struct ti_bandgap *bgp, int id)
> -{
> - return 0;
> -}
> -
> -static inline
> -int ti_thermal_report_sensor_temperature(struct ti_bandgap *bgp, int id)
> -{
> - return 0;
> -}
> -
> -static inline
> -int ti_thermal_register_cpu_cooling(struct ti_bandgap *bgp, int id)
> -{
> - return 0;
> -}
> -
> -static inline
> -int ti_thermal_unregister_cpu_cooling(struct ti_bandgap *bgp, int id)
> -{
> - return 0;
> -}
> -#endif
> #endif
>


--
You have got to be excited about what you are doing. (L. Lamport)

Eduardo Valentin

Attachment: signature.asc
Description: OpenPGP digital signature