https://bugzilla.kernel.org/show_bug.cgi?id=210269 Hans de Goede (jwrdegoede@xxxxxxxxxxxxxxxxx) changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jwrdegoede@xxxxxxxxxxxxxxxx | |g --- Comment #10 from Hans de Goede (jwrdegoede@xxxxxxxxxxxxxxxxx) --- (In reply to Benjamin Berg from comment #8) > I wonder if we should only do this if the switch actually changed. But I > suspect the input layer will already take care of that case. There is no need to check if the switch actually changed, as you already presumed the input layer will filter out the event if there is no change. (In reply to Julian from comment #9) > Your patch seems to fix the problem Benjamin, thanks. Awesome, thank you for testing. Benjamin, can you submit the fix for this upstream please ? Then I'll pick it up and merge it for the 5.11 cycle. Note please add the following tags to the commit msg: Fixes: dda3ec0aa631 ("platform/x86: thinkpad_acpi: Implement tablet mode using GMMS method") BugLink: https://bugzilla.kernel.org/show_bug.cgi?id=210269 -- You are receiving this mail because: You are watching the assignee of the bug.
