Re: [PATCH v2 1/4] thermal/core: Rename passive_delay and polling_delay with units

From: Daniel Lezcano
Date: Thu Dec 10 2020 - 10:35:14 EST



Cc'ing missing persons.

Sorry for the inconvenience, just a head up with those trivial changes
(1/4 and 4/4).


On 07/12/2020 20:08, Daniel Lezcano wrote:
> The input unit used by the thermal framework is the msec but it uses
> the jiffies to set the timers.
>
> As it is stored in the thermal zone device structure, everytime the
> timer is setup at each polling interval, the msecs to jiffies
> conversion happens. The jiffies is the unit the thermal framework is
> using, so keeping it under the jiffies instead of msecs will save some
> pointless conversion.
>
> Set the scene to directly store the delays under their jiffies
> form by adding to the variable name the 'ms' suffix.
>
> Signed-off-by: Daniel Lezcano <daniel.lezcano@xxxxxxxxxx>
> ---
> drivers/platform/x86/acerhdf.c | 2 +-
> drivers/thermal/da9062-thermal.c | 4 ++--
> drivers/thermal/gov_power_allocator.c | 2 +-
> drivers/thermal/thermal_core.c | 10 +++++-----
> drivers/thermal/thermal_of.c | 16 ++++++++--------
> drivers/thermal/thermal_sysfs.c | 6 +++---
> .../thermal/ti-soc-thermal/ti-thermal-common.c | 2 +-
> include/linux/thermal.h | 8 ++++----
> 8 files changed, 25 insertions(+), 25 deletions(-)
>
> diff --git a/drivers/platform/x86/acerhdf.c b/drivers/platform/x86/acerhdf.c
> index b6aa6e5514f4..7b26f601b407 100644
> --- a/drivers/platform/x86/acerhdf.c
> +++ b/drivers/platform/x86/acerhdf.c
> @@ -336,7 +336,7 @@ static void acerhdf_check_param(struct thermal_zone_device *thermal)
> pr_notice("interval changed to: %d\n", interval);
>
> if (thermal)
> - thermal->polling_delay = interval*1000;
> + thermal->polling_delay_ms = interval*1000;
>
> prev_interval = interval;
> }
> diff --git a/drivers/thermal/da9062-thermal.c b/drivers/thermal/da9062-thermal.c
> index 4d74994f160a..ebb3d0b4a5be 100644
> --- a/drivers/thermal/da9062-thermal.c
> +++ b/drivers/thermal/da9062-thermal.c
> @@ -95,7 +95,7 @@ static void da9062_thermal_poll_on(struct work_struct *work)
> thermal_zone_device_update(thermal->zone,
> THERMAL_EVENT_UNSPECIFIED);
>
> - delay = msecs_to_jiffies(thermal->zone->passive_delay);
> + delay = msecs_to_jiffies(thermal->zone->passive_delay_ms);
> queue_delayed_work(system_freezable_wq, &thermal->work, delay);
> return;
> }
> @@ -245,7 +245,7 @@ static int da9062_thermal_probe(struct platform_device *pdev)
>
> dev_dbg(&pdev->dev,
> "TJUNC temperature polling period set at %d ms\n",
> - thermal->zone->passive_delay);
> + thermal->zone->passive_delay_ms);
>
> ret = platform_get_irq_byname(pdev, "THERMAL");
> if (ret < 0) {
> diff --git a/drivers/thermal/gov_power_allocator.c b/drivers/thermal/gov_power_allocator.c
> index 7a4170a0b51f..f7a663f698d4 100644
> --- a/drivers/thermal/gov_power_allocator.c
> +++ b/drivers/thermal/gov_power_allocator.c
> @@ -258,7 +258,7 @@ static u32 pid_controller(struct thermal_zone_device *tz,
> * power being applied, slowing down the controller)
> */
> d = mul_frac(tz->tzp->k_d, err - params->prev_err);
> - d = div_frac(d, tz->passive_delay);
> + d = div_frac(d, tz->passive_delay_ms);
> params->prev_err = err;
>
> power_range = p + i + d;
> diff --git a/drivers/thermal/thermal_core.c b/drivers/thermal/thermal_core.c
> index 1bd23ff2247b..5b500d72aab4 100644
> --- a/drivers/thermal/thermal_core.c
> +++ b/drivers/thermal/thermal_core.c
> @@ -317,9 +317,9 @@ static void monitor_thermal_zone(struct thermal_zone_device *tz)
> mutex_lock(&tz->lock);
>
> if (!stop && tz->passive)
> - thermal_zone_device_set_polling(tz, tz->passive_delay);
> - else if (!stop && tz->polling_delay)
> - thermal_zone_device_set_polling(tz, tz->polling_delay);
> + thermal_zone_device_set_polling(tz, tz->passive_delay_ms);
> + else if (!stop && tz->polling_delay_ms)
> + thermal_zone_device_set_polling(tz, tz->polling_delay_ms);
> else
> thermal_zone_device_set_polling(tz, 0);
>
> @@ -1340,8 +1340,8 @@ thermal_zone_device_register(const char *type, int trips, int mask,
> tz->device.class = &thermal_class;
> tz->devdata = devdata;
> tz->trips = trips;
> - tz->passive_delay = passive_delay;
> - tz->polling_delay = polling_delay;
> + tz->passive_delay_ms = passive_delay;
> + tz->polling_delay_ms = polling_delay;
>
> /* sys I/F */
> /* Add nodes that are always present via .groups */
> diff --git a/drivers/thermal/thermal_of.c b/drivers/thermal/thermal_of.c
> index 69ef12f852b7..c01248e48c09 100644
> --- a/drivers/thermal/thermal_of.c
> +++ b/drivers/thermal/thermal_of.c
> @@ -51,8 +51,8 @@ struct __thermal_bind_params {
>
> /**
> * struct __thermal_zone - internal representation of a thermal zone
> - * @passive_delay: polling interval while passive cooling is activated
> - * @polling_delay: zone polling interval
> + * @passive_delay_ms: polling interval while passive cooling is activated
> + * @polling_delay_ms: zone polling interval
> * @slope: slope of the temperature adjustment curve
> * @offset: offset of the temperature adjustment curve
> * @ntrips: number of trip points
> @@ -64,8 +64,8 @@ struct __thermal_bind_params {
> */
>
> struct __thermal_zone {
> - int passive_delay;
> - int polling_delay;
> + int passive_delay_ms;
> + int polling_delay_ms;
> int slope;
> int offset;
>
> @@ -865,14 +865,14 @@ __init *thermal_of_build_thermal_zone(struct device_node *np)
> pr_err("%pOFn: missing polling-delay-passive property\n", np);
> goto free_tz;
> }
> - tz->passive_delay = prop;
> + tz->passive_delay_ms = prop;
>
> ret = of_property_read_u32(np, "polling-delay", &prop);
> if (ret < 0) {
> pr_err("%pOFn: missing polling-delay property\n", np);
> goto free_tz;
> }
> - tz->polling_delay = prop;
> + tz->polling_delay_ms = prop;
>
> /*
> * REVIST: for now, the thermal framework supports only
> @@ -1085,8 +1085,8 @@ int __init of_parse_thermal_zones(void)
> zone = thermal_zone_device_register(child->name, tz->ntrips,
> mask, tz,
> ops, tzp,
> - tz->passive_delay,
> - tz->polling_delay);
> + tz->passive_delay_ms,
> + tz->polling_delay_ms);
> if (IS_ERR(zone)) {
> pr_err("Failed to build %pOFn zone %ld\n", child,
> PTR_ERR(zone));
> diff --git a/drivers/thermal/thermal_sysfs.c b/drivers/thermal/thermal_sysfs.c
> index 0866e949339b..f465462d8aa1 100644
> --- a/drivers/thermal/thermal_sysfs.c
> +++ b/drivers/thermal/thermal_sysfs.c
> @@ -233,12 +233,12 @@ passive_store(struct device *dev, struct device_attribute *attr,
> return -EINVAL;
>
> if (state && !tz->forced_passive) {
> - if (!tz->passive_delay)
> - tz->passive_delay = 1000;
> + if (!tz->passive_delay_ms)
> + tz->passive_delay_ms = 1000;
> thermal_zone_device_rebind_exception(tz, "Processor",
> sizeof("Processor"));
> } else if (!state && tz->forced_passive) {
> - tz->passive_delay = 0;
> + tz->passive_delay_ms = 0;
> thermal_zone_device_unbind_exception(tz, "Processor",
> sizeof("Processor"));
> }
> diff --git a/drivers/thermal/ti-soc-thermal/ti-thermal-common.c b/drivers/thermal/ti-soc-thermal/ti-thermal-common.c
> index 2ce4b19f312a..4baff19e1142 100644
> --- a/drivers/thermal/ti-soc-thermal/ti-thermal-common.c
> +++ b/drivers/thermal/ti-soc-thermal/ti-thermal-common.c
> @@ -185,7 +185,7 @@ int ti_thermal_expose_sensor(struct ti_bandgap *bgp, int id,
>
> ti_bandgap_set_sensor_data(bgp, id, data);
> ti_bandgap_write_update_interval(bgp, data->sensor_id,
> - data->ti_thermal->polling_delay);
> + data->ti_thermal->polling_delay_ms);
>
> return 0;
> }
> diff --git a/include/linux/thermal.h b/include/linux/thermal.h
> index d07ea27e72a9..230d451bf335 100644
> --- a/include/linux/thermal.h
> +++ b/include/linux/thermal.h
> @@ -116,9 +116,9 @@ struct thermal_cooling_device {
> * @devdata: private pointer for device private data
> * @trips: number of trip points the thermal zone supports
> * @trips_disabled; bitmap for disabled trips
> - * @passive_delay: number of milliseconds to wait between polls when
> + * @passive_delay_ms: number of milliseconds to wait between polls when
> * performing passive cooling.
> - * @polling_delay: number of milliseconds to wait between polls when
> + * @polling_delay_ms: number of milliseconds to wait between polls when
> * checking whether trip points have been crossed (0 for
> * interrupt driven systems)
> * @temperature: current temperature. This is only for core code,
> @@ -159,8 +159,8 @@ struct thermal_zone_device {
> void *devdata;
> int trips;
> unsigned long trips_disabled; /* bitmap for disabled trips */
> - int passive_delay;
> - int polling_delay;
> + int passive_delay_ms;
> + int polling_delay_ms;
> int temperature;
> int last_temperature;
> int emul_temperature;
>


--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog