On Thu, 12 Nov 2009, Takashi Iwai wrote:
At Thu, 12 Nov 2009 12:24:27 +0100 (CET), Jaroslav Kysela wrote:
On Thu, 12 Nov 2009, Takashi Iwai wrote:
At Thu, 12 Nov 2009 12:06:57 +0100 (CET), Jaroslav Kysela wrote:
On Thu, 12 Nov 2009, Takashi Iwai wrote:
IOW, I'm fine with an additional implementation for the dynamic beep on/off. But, the mixer interface doesn't look like the best interface to me.
I would prefer something which is comfortable for user. The module parameter or sysfs file is not a good way to explain users how they can change the beep behaviour.
A module parameter would be easy enough for most users who really care things like beep :)
Nope. You expect that all users knows how to change module parameters. I'm sorry, but I'm lazy to explain in bugzilla many times, how to change the kernel module parameter (it's enough to explain model= for HDA). But 'turn off "Beep" in alsamixer or any ALSA-aware mixer app' is quite shorther and intuitive.
But because of this "easy-to-use", the switch can be soo easily changed unintentionally. So, we are speaking roughly the same thing of both good and bad sides.
And here I'm more concerned about the bad side than the good side.
Ok, take another look:
If you have an USB input device (mouse for example), how you can force the user to not plug in / plug out the device to USB port? With my latest change postponing the input device unregistration, the code is quite robust and even keyboard repeat should not make much hassle, because the input device will not be quickly unregistered in this case.
I'm sorry, but we cannot take responsibility about this. If our code does not crash and works as expected, everything is ok.
Jaroslav
----- Jaroslav Kysela perex@perex.cz Linux Kernel Sound Maintainer ALSA Project, Red Hat, Inc.