From c79a3bbda5972bf2a03405adb6cf0ba553d0d2ec Mon Sep 17 00:00:00 2001 From: Rick Peters Date: Fri, 25 Sep 2026 10:08:06 +0200 Subject: [PATCH] fix: No LED warnings when the kernel pin just replaced the running kernel --- CHANGELOG.md | 3 ++- lib/steam-machine.sh | 7 +++++++ 2 files changed, 9 insertions(+), 1 deletion(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index bace610..449e499 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -10,7 +10,8 @@ one per merged pull request. - **perf: Pinned kernel before the LED driver** - With the kernel pin ticked, Steam Machine support installs kernel 7.1.6-1 first, so DKMS builds the LED driver once, for that kernel, - instead of for the current kernel first and then again. + instead of for the current kernel first and then again. The LED driver + then loads after the restart, which it now says instead of warning. - **fix: steamos-manager starts after cecd** - It checks only at its start whether cecd runs; if it was first, Steam's HDMI-CEC settings were missing until it restarted. A drop-in diff --git a/lib/steam-machine.sh b/lib/steam-machine.sh index 331648a..1e7a25b 100644 --- a/lib/steam-machine.sh +++ b/lib/steam-machine.sh @@ -251,6 +251,13 @@ install_valve_led_driver() { # Load now, and on every boot. echo leds-valve | sudo tee /etc/modules-load.d/leds-valve.conf >/dev/null + # The running kernel was just replaced (the kernel pin): its modules are + # gone, so the driver can only load after the restart. + if [[ ! -d "/usr/lib/modules/$(uname -r)/kernel" ]]; then + ok "The LED driver loads after the restart (into the new kernel)." + return 0 + fi + info "Loading the leds-valve kernel module..." if sudo modprobe leds-valve; then ok "Module loaded."