diff for duplicates of <cover.1794633179.git.rosenp@gmail.com> diff --git a/a/1.txt b/N1/1.txt index 24e4331..1e44d84 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,6 +1,7 @@ -Two input validation fixes for dell-smm-hwmon: a huge value written to -pwmN overflowed the scaling and turned the fan off, and an out-of-range -fan_max module parameter could wrap an allocation size to zero. +Three fixes for dell-laptop: keyboard backlight timeouts that overflow +during unit conversion were silently accepted, an unread keyboard state +was used at init when the SMBIOS call failed, and the charge end +threshold was parsed as unsigned into an int. These patches were developed with the help of an AI coding assistant (Claude Opus 5.5): it reviewed the drivers, found the issues and drafted @@ -11,17 +12,20 @@ All patches build cleanly with W=1. Tested on a Dell Precision M4800 running v7.2.15, using the equivalent backports built as out-of-tree modules: -fan_max=256 and fan_max=4294967295 are rejected with the new message, -fan_max=3 and the default still load. The pwm clamp could not be -observed: this machine's BIOS ignores manual pwm writes. + - writing 4424214d / 18989323d to stop_timeout set 4m / 2m before + patch 1 and is rejected with -EINVAL after it; 5m still works + - charge_control_end_threshold rejects -1 and 101, accepts 85 +The kbd_get_state() failure path of patch 2 was not triggered. -Rosen Penev (2): - hwmon: (dell-smm) Clamp the pwm value before scaling it - hwmon: (dell-smm) Reject a fan_max that the BIOS cannot take +Rosen Penev (3): + platform/x86: dell-laptop: Reject keyboard backlight timeouts that + overflow + platform/x86: dell-laptop: Don't use an unread keyboard state at init + platform/x86: dell-laptop: Parse the charge end threshold as an int - drivers/hwmon/dell-smm-hwmon.c | 7 +++++++ - 1 file changed, 7 insertions(+) + drivers/platform/x86/dell/dell-laptop.c | 30 ++++++++++++++++++------- + 1 file changed, 22 insertions(+), 8 deletions(-) base-commit: 2800868f63227c8237d70c41e70d441c919620d6 diff --git a/a/content_digest b/N1/content_digest index 6e47159..70e04cb 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,17 +1,20 @@ "From\0Rosen Penev <rosenp@gmail.com>\0" - "Subject\0[PATCH 0/2] hwmon: (dell-smm) Validate pwm and fan_max values\0" - "Date\0Fri, 13 Nov 2026 22:20:21 -0800\0" - "To\0Pali Roh\303\241r <pali@kernel.org>" - Guenter Roeck <linux@roeck-us.net> - Arnd Bergmann <arnd@arndb.de> - " Greg Kroah-Hartman <gregkh@linuxfoundation.org>\0" - "Cc\0linux-hwmon@vger.kernel.org" + "Subject\0[PATCH 0/3] platform/x86: dell-laptop: Fix keyboard backlight and charge threshold handling\0" + "Date\0Fri, 13 Nov 2026 22:29:24 -0800\0" + "To\0Matthew Garrett <mjg59@srcf.ucam.org>" + " Pali Roh\303\241r <pali@kernel.org>" + Hans de Goede <hansg@kernel.org> + " Ilpo J\303\244rvinen <ilpo.jarvinen@linux.intel.com>" + Darren Hart <dvhart@linux.intel.com> + " Gabriele Mazzotta <gabriele.mzt@gmail.com>\0" + "Cc\0platform-driver-x86@vger.kernel.org" " linux-kernel@vger.kernel.org\0" "\00:1\0" "b\0" - "Two input validation fixes for dell-smm-hwmon: a huge value written to\n" - "pwmN overflowed the scaling and turned the fan off, and an out-of-range\n" - "fan_max module parameter could wrap an allocation size to zero.\n" + "Three fixes for dell-laptop: keyboard backlight timeouts that overflow\n" + "during unit conversion were silently accepted, an unread keyboard state\n" + "was used at init when the SMBIOS call failed, and the charge end\n" + "threshold was parsed as unsigned into an int.\n" "\n" "These patches were developed with the help of an AI coding assistant\n" "(Claude Opus 5.5): it reviewed the drivers, found the issues and drafted\n" @@ -22,21 +25,24 @@ "\n" "Tested on a Dell Precision M4800 running v7.2.15, using the equivalent\n" "backports built as out-of-tree modules:\n" - "fan_max=256 and fan_max=4294967295 are rejected with the new message,\n" - "fan_max=3 and the default still load. The pwm clamp could not be\n" - "observed: this machine's BIOS ignores manual pwm writes.\n" + " - writing 4424214d / 18989323d to stop_timeout set 4m / 2m before\n" + " patch 1 and is rejected with -EINVAL after it; 5m still works\n" + " - charge_control_end_threshold rejects -1 and 101, accepts 85\n" + "The kbd_get_state() failure path of patch 2 was not triggered.\n" "\n" "\n" - "Rosen Penev (2):\n" - " hwmon: (dell-smm) Clamp the pwm value before scaling it\n" - " hwmon: (dell-smm) Reject a fan_max that the BIOS cannot take\n" + "Rosen Penev (3):\n" + " platform/x86: dell-laptop: Reject keyboard backlight timeouts that\n" + " overflow\n" + " platform/x86: dell-laptop: Don't use an unread keyboard state at init\n" + " platform/x86: dell-laptop: Parse the charge end threshold as an int\n" "\n" - " drivers/hwmon/dell-smm-hwmon.c | 7 +++++++\n" - " 1 file changed, 7 insertions(+)\n" + " drivers/platform/x86/dell/dell-laptop.c | 30 ++++++++++++++++++-------\n" + " 1 file changed, 22 insertions(+), 8 deletions(-)\n" "\n" "\n" "base-commit: 2800868f63227c8237d70c41e70d441c919620d6\n" "-- \n" 2.56.0 -a761d48282dc58c14ca913da47086d448f895255b86bd8d40daad812911f6a39 +535c5f34ab061c57c63ec3987784eee85854771ecdc06ed4df31c49edd700544
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.