From b30c584d6a724d350b7b8f6a30cc4bbbf9dbac3d Mon Sep 17 00:00:00 2001 From: Werner Sembach Date: Tue, 30 Jan 2024 14:26:53 +0100 Subject: [PATCH] kbd backlight: Fix double brightness restore on LidOpen-resume This fixes the issue that on LidOpen-resume the brightness might get restored two times. Once to the correct vallue and once incorrectly to 0. To archive this it uses the same simple "don't restore to 0"-policy as is used in dimdisplay.cpp, dpms.cpp, and handlebuttonevent.cpp. Signed-off-by: Werner Sembach (cherry picked from commit aa2fa106) --- daemon/actions/bundled/keyboardbrightnesscontrol.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/daemon/actions/bundled/keyboardbrightnesscontrol.cpp b/daemon/actions/bundled/keyboardbrightnesscontrol.cpp index a7c5f6408..760e17ff5 100644 --- a/daemon/actions/bundled/keyboardbrightnesscontrol.cpp +++ b/daemon/actions/bundled/keyboardbrightnesscontrol.cpp @@ -75,7 +75,7 @@ KeyboardBrightnessControl::KeyboardBrightnessControl(QObject* parent, const QVar // My laptop sets the keyboard brightness to zero when I close the lid and it suspends // this makes sure the keyboard brightness is restored when we wake up :) connect(backend(), &PowerDevil::BackendInterface::resumeFromSuspend, this, [this] { - if (m_lastKeyboardBrightness > -1) { + if (m_lastKeyboardBrightness > 0) { setKeyboardBrightnessSilent(m_lastKeyboardBrightness); } }); -- GitLab