Hi Daniel, On 12/16/20 11:03 PM, Daniel Lezcano wrote: > The code does no longer use the ms unit based fields to set the > delays as they are replaced by the jiffies. > > Remove them and replace their user to use the jiffies version instead. > > Cc: Thara Gopinath <[email protected]> > Signed-off-by: Daniel Lezcano <[email protected]> > Reviewed-by: Lukasz Luba <[email protected]>
I assume that you will merge this through the thermal tree, here is my ack for doing so for the pdx86 bits: Acked-by: Hans de Goede <[email protected]> Regards, Hans > --- > drivers/platform/x86/acerhdf.c | 3 ++- > drivers/thermal/da9062-thermal.c | 4 ++-- > drivers/thermal/gov_power_allocator.c | 2 +- > drivers/thermal/thermal_core.c | 4 +--- > drivers/thermal/ti-soc-thermal/ti-thermal-common.c | 6 ++++-- > include/linux/thermal.h | 7 ------- > 6 files changed, 10 insertions(+), 16 deletions(-) > > diff --git a/drivers/platform/x86/acerhdf.c b/drivers/platform/x86/acerhdf.c > index b6aa6e5514f4..6b8b3ab8db48 100644 > --- a/drivers/platform/x86/acerhdf.c > +++ b/drivers/platform/x86/acerhdf.c > @@ -336,7 +336,8 @@ 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_jiffies = > + round_jiffies(msecs_to_jiffies(interval * > 1000)); > > prev_interval = interval; > } > diff --git a/drivers/thermal/da9062-thermal.c > b/drivers/thermal/da9062-thermal.c > index 4d74994f160a..180edec34e07 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 = thermal->zone->passive_delay_jiffies; > 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); > + jiffies_to_msecs(thermal->zone->passive_delay_jiffies)); > > 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..f8c3d1e40b86 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, jiffies_to_msecs(tz->passive_delay_jiffies)); > params->prev_err = err; > > power_range = p + i + d; > diff --git a/drivers/thermal/thermal_core.c b/drivers/thermal/thermal_core.c > index d96c515af3cb..b2615449b18f 100644 > --- a/drivers/thermal/thermal_core.c > +++ b/drivers/thermal/thermal_core.c > @@ -313,7 +313,7 @@ static void monitor_thermal_zone(struct > thermal_zone_device *tz) > > if (!stop && tz->passive) > thermal_zone_device_set_polling(tz, tz->passive_delay_jiffies); > - else if (!stop && tz->polling_delay) > + else if (!stop && tz->polling_delay_jiffies) > thermal_zone_device_set_polling(tz, tz->polling_delay_jiffies); > else > thermal_zone_device_set_polling(tz, 0); > @@ -1307,8 +1307,6 @@ 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; > > thermal_set_delay_jiffies(&tz->passive_delay_jiffies, passive_delay); > thermal_set_delay_jiffies(&tz->polling_delay_jiffies, polling_delay); > diff --git a/drivers/thermal/ti-soc-thermal/ti-thermal-common.c > b/drivers/thermal/ti-soc-thermal/ti-thermal-common.c > index 2ce4b19f312a..f84375865c97 100644 > --- a/drivers/thermal/ti-soc-thermal/ti-thermal-common.c > +++ b/drivers/thermal/ti-soc-thermal/ti-thermal-common.c > @@ -166,6 +166,7 @@ int ti_thermal_expose_sensor(struct ti_bandgap *bgp, int > id, > char *domain) > { > struct ti_thermal_data *data; > + int interval; > > data = ti_bandgap_get_sensor_data(bgp, id); > > @@ -183,9 +184,10 @@ int ti_thermal_expose_sensor(struct ti_bandgap *bgp, int > id, > return PTR_ERR(data->ti_thermal); > } > > + interval = jiffies_to_msecs(data->ti_thermal->polling_delay_jiffies); > + > ti_bandgap_set_sensor_data(bgp, id, data); > - ti_bandgap_write_update_interval(bgp, data->sensor_id, > - data->ti_thermal->polling_delay); > + ti_bandgap_write_update_interval(bgp, data->sensor_id, interval); > > return 0; > } > diff --git a/include/linux/thermal.h b/include/linux/thermal.h > index d1b82c70de69..1e686404951b 100644 > --- a/include/linux/thermal.h > +++ b/include/linux/thermal.h > @@ -115,13 +115,8 @@ 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 > - * performing passive cooling. > * @passive_delay_jiffies: number of jiffies to wait between polls when > * performing passive cooling. > - * @polling_delay: number of milliseconds to wait between polls when > - * checking whether trip points have been crossed (0 for > - * interrupt driven systems) > * @polling_delay_jiffies: number of jiffies to wait between polls when > * checking whether trip points have been crossed (0 for > * interrupt driven systems) > @@ -162,8 +157,6 @@ struct thermal_zone_device { > unsigned long trips_disabled; /* bitmap for disabled trips */ > unsigned long passive_delay_jiffies; > unsigned long polling_delay_jiffies; > - int passive_delay; > - int polling_delay; > int temperature; > int last_temperature; > int emul_temperature; >
