thermal: intel: powerclamp: Simplify idle_inject_update()

Drop the local update variable and call idle_inject_set_duration()
directly instead. Also return !should_skip directly.

Signed-off-by: Thorsten Blum <blum@kernel.org>
Link: https://patch.msgid.link/20260828194530.43370-2-blum@kernel.org
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
This commit is contained in:
Thorsten Blum 2026-08-28 21:45:27 +02:00 committed by Rafael J. Wysocki
parent e56e2b49c6
commit bbc68b683d

View File

@ -534,23 +534,17 @@ static struct idle_inject_device *ii_dev;
*/
static bool idle_inject_update(void)
{
bool update = false;
/* We can't sleep in this callback */
if (!mutex_trylock(&powerclamp_lock))
return true;
if (!(powerclamp_data.count % powerclamp_data.window_size_now)) {
unsigned int runtime;
should_skip = powerclamp_adjust_controls(powerclamp_data.target_ratio,
powerclamp_data.guard,
powerclamp_data.window_size_now);
update = true;
}
if (update) {
unsigned int runtime = get_run_time();
runtime = get_run_time();
idle_inject_set_duration(ii_dev, runtime, duration);
}
@ -558,10 +552,7 @@ static bool idle_inject_update(void)
mutex_unlock(&powerclamp_lock);
if (should_skip)
return false;
return true;
return !should_skip;
}
/* This function starts idle injection by calling idle_inject_start() */