Unverified Commit dce77641 authored by Jelle van der Waa's avatar Jelle van der Waa Committed by Ilpo Järvinen
Browse files

platform/x86: fujitsu: clamp charge_control_end_threshold values to 50 



Follow the sysfs ABI documentation that drivers should round written
values to the nearest supported value instead of returning an error.

Tested on a Fujitsu Lifebook U7720.

Signed-off-by: default avatarJelle van der Waa <jvanderwaa@redhat.com>
Reviewed-by: default avatarHans de Goede <hansg@kernel.org>
Acked-by: default avatarJonathan Woithe <jwoithe@just42.net>
Link: https://lore.kernel.org/r/20250609102115.36936-3-jvanderwaa@redhat.com


Reviewed-by: default avatarIlpo Järvinen <ilpo.jarvinen@linux.intel.com>
Signed-off-by: default avatarIlpo Järvinen <ilpo.jarvinen@linux.intel.com>
parent d9926f09

drivers/platform/x86/fujitsu-laptop.c

+4 −1
Original line number Diff line number Diff line
@@ -187,9 +187,12 @@ static ssize_t charge_control_end_threshold_store(struct device *dev, 
	if (ret)

		return ret;



	if (value < 50 || value > 100)

	if (value > 100)

		return -EINVAL;



	if (value < 50)

		value = 50;



	cc_end_value = value * 0x100 + 0x20;

	s006_cc_return = call_fext_func(fext, FUNC_S006_METHOD,

					CHARGE_CONTROL_RW, cc_end_value, 0x0);