On Fri, 12 Sep 2014, Andrew Lutomirski wrote: > > This behavior is unfortunate, since modern userspace will also > > handle the hotkeys and change the other mixer. If the software > > mixer is muted and the hardware mixer is unmuted and you push mute, > > hilarity ensues as they both switch state. > > This patch seems to have fallen into oblivion. My X220s still has Mostly because I was out of time to review it due to its size, and ended up forgetting about it. I am deeply sorry about that. We can bring this patch back into the light, though. > screwy mute controls. (To reproduce in GNOME 3, get into an unmuted, > nonzero volume state, then press volume down a bunch of times until a > mute icon shows up, then press mute, then press volume up. You are > now have no sound until you fiddle with the hardware controls a bunch > and undo it.) > > I can rebase this to 3.17-whatever, but I'm also somewhat inclined to > get rid of all the compatibility bits and just modify the driver to > force all Thinkpads into nonmagical software-only mode. The latter > approach will be *much* simpler Changing the behaviour on the newer ones is okay, but not the old ones (where old == not broken). That would be a regression. Syncing the hardware and software mute gating is fine, though, as long as they are always kept in sync. If "toggle mode" is much easier to implement than the v5 version of your patch, we could try that. -- "One disk to rule them all, One disk to find them. One disk to bring them all and in the darkness grind them. In the Land of Redmond where the shadows lie." -- The Silicon Valley Tarot Henrique Holschuh ------------------------------------------------------------------------------ Want excitement? Manually upgrade your production database. When you want reliability, choose Perforce Perforce version control. Predictably reliable. http://pubads.g.doubleclick.net/gampad/clk?id=157508191&iu=/4140/ostg.clktrk _______________________________________________ ibm-acpi-devel mailing list ibm-acpi-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.sourceforge.net/lists/listinfo/ibm-acpi-devel
