From 23b16b517814c367e593be96e477818ec5b2abff Mon Sep 17 00:00:00 2001 From: Thorsten Blum Date: Wed, 16 Sep 2026 00:15:48 +0200 Subject: [PATCH] thermal: intel: powerclamp: Simplify duration_get() Use guard(mutex) and return the result of sysfs_emit() directly to simplify the code. Also use %u to format the unsigned duration. Signed-off-by: Thorsten Blum Link: https://patch.msgid.link/20260915221548.104852-3-blum@kernel.org Signed-off-by: Rafael J. Wysocki --- drivers/thermal/intel/intel_powerclamp.c | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/drivers/thermal/intel/intel_powerclamp.c b/drivers/thermal/intel/intel_powerclamp.c index 044b185f748b..ee1f2fa8d20f 100644 --- a/drivers/thermal/intel/intel_powerclamp.c +++ b/drivers/thermal/intel/intel_powerclamp.c @@ -103,13 +103,9 @@ static int duration_set(const char *arg, const struct kernel_param *kp) static int duration_get(char *buf, const struct kernel_param *kp) { - int ret; + guard(mutex)(&powerclamp_lock); - mutex_lock(&powerclamp_lock); - ret = sysfs_emit(buf, "%d\n", duration / 1000); - mutex_unlock(&powerclamp_lock); - - return ret; + return sysfs_emit(buf, "%u\n", duration / 1000); } static const struct kernel_param_ops duration_ops = {