thermal: intel: int340x: soc_slider: Simplify slider_def_*_set()

Return errors directly to simplify slider_def_balance_set() and
slider_def_offset_set().

Signed-off-by: Thorsten Blum <blum@kernel.org>
Link: https://patch.msgid.link/20260921143644.69381-2-blum@kernel.org
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
This commit is contained in:
Thorsten Blum 2026-09-21 16:36:41 +02:00 committed by Rafael J. Wysocki
parent d89c557bb0
commit f10f236b08

View File

@ -66,15 +66,16 @@ static int slider_def_balance_set(const char *arg, const struct kernel_param *kp
guard(mutex)(&slider_param_lock);
ret = kstrtou8(arg, 16, &slider_val);
if (!ret) {
if (slider_val <= slider_values[SOC_POWER_SLIDER_PERFORMANCE] ||
slider_val >= slider_values[SOC_POWER_SLIDER_POWERSAVE])
return -EINVAL;
if (ret)
return ret;
slider_balanced_param = slider_val;
}
if (slider_val <= slider_values[SOC_POWER_SLIDER_PERFORMANCE] ||
slider_val >= slider_values[SOC_POWER_SLIDER_POWERSAVE])
return -EINVAL;
return ret;
slider_balanced_param = slider_val;
return 0;
}
static int slider_def_balance_get(char *buf, const struct kernel_param *kp)
@ -101,14 +102,15 @@ static int slider_def_offset_set(const char *arg, const struct kernel_param *kp)
guard(mutex)(&slider_param_lock);
ret = kstrtou8(arg, 16, &offset);
if (!ret) {
if (offset > SOC_SLIDER_VALUE_MAXIMUM)
return -EINVAL;
if (ret)
return ret;
slider_offset = offset;
}
if (offset > SOC_SLIDER_VALUE_MAXIMUM)
return -EINVAL;
return ret;
slider_offset = offset;
return 0;
}
static int slider_def_offset_get(char *buf, const struct kernel_param *kp)