Re: [PATCH] thermal: kconfig: select THERMAL_WRITABLE_TRIPS for x86 thermal

[Eduardo Valentin <edubezval@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 2559 bytes --]

Srinivas,

On Sat, Aug 01, 2015 at 10:46:41PM -0700, Srinivas Pandruvada wrote:
> After the commit "thermal: core: Add Kconfig option to enable writable trips",
> user space tools like thermal daemon and Chrome OS DPTF thermal controller
> can no longer receive async events for thermal thresholds. Since we need to
> enable the new config introduced by above commit to allow writable trips.
> Selecting CONFIG_THERMAL_WRITABLE_TRIPS for x86 thermal drivers.

I am fine with enabling the config on x86 drivers. However the
description is somewhat confusing. Are you sure you are not receiving
async events or you are not able to write on trip point files?

Because that's all this option does (quoting thermal_core.c):
                if (IS_ENABLED(CONFIG_THERMAL_WRITABLE_TRIPS) &&
                    mask & (1 << indx)) {
                        tz->trip_temp_attrs[indx].attr.attr.mode |= S_IWUSR;
                        tz->trip_temp_attrs[indx].attr.store =
                                                        trip_point_temp_store;
                }


I am not really sure if this change is going to give your events back.


can you please confirm what you are trying to achieve here?

> 
> Signed-off-by: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
> ---
>  drivers/thermal/Kconfig | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
> index 118938e..0e3b576 100644
> --- a/drivers/thermal/Kconfig
> +++ b/drivers/thermal/Kconfig
> @@ -275,6 +275,7 @@ config X86_PKG_TEMP_THERMAL
>  	tristate "X86 package temperature thermal driver"
>  	depends on X86_THERMAL_VECTOR
>  	select THERMAL_GOV_USER_SPACE
> +	select THERMAL_WRITABLE_TRIPS
>  	default m
>  	help
>  	  Enable this to register CPU digital sensor for package temperature as
> @@ -296,6 +297,7 @@ config INTEL_SOC_DTS_THERMAL
>  	tristate "Intel SoCs DTS thermal driver"
>  	depends on X86
>  	select INTEL_SOC_DTS_IOSF_CORE
> +	select THERMAL_WRITABLE_TRIPS
>  	help
>  	  Enable this to register Intel SoCs (e.g. Bay Trail) platform digital
>  	  temperature sensor (DTS). These SoCs have two additional DTSs in
> @@ -322,6 +324,7 @@ config INT340X_THERMAL
>  	select ACPI_THERMAL_REL
>  	select ACPI_FAN
>  	select INTEL_SOC_DTS_IOSF_CORE
> +	select THERMAL_WRITABLE_TRIPS
>  	help
>  	  Newer laptops and tablets that use ACPI may have thermal sensors and
>  	  other devices with thermal control capabilities outside the core
> -- 
> 2.4.3
> 

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 490 bytes --]