On Mon, 03 May 2010, Matthew Garrett wrote: > Not all Thinkpads generate events for volume hotkeys, so hook into the > CMOS update and generate events from there without polling. This should > let Pulseaudio do something sensible with the mute state. The idea is sound, but it would be probably cleaner to do it only on those thinkpads which need it. I don't like the idea of handling duplicate notifications, and the logic depends entirely on nothing messing with the mixer state behind our backs, I'd prefer to rely on it just when strictly required. Also, UCMS is called for a number of reasons. It would be best to filter based on the UCMS parameter before doing an expensive PC CMOS RAM read or EC read. We will be dealing with a finite number of thinkpads anyway. On the same note, we need to be called *after* the UCMS method completes, otherwise, we will have to schedule delayed work. I didn't study your ACPI patch enough to know whether this happens, but that detail is not clear from the description. -- "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 -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
