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.
We have basically four ways:
- module parameter (difficult tracking of changes through sysfs)
- sysfs file
- current implementation
- create new control with different name (like "HDA Beep Enable Switch")
3 and 4 have the same concern, so I'd vote for 1 or 2.
For hackers, of course :-) But it might not be preferred way for all users.
Jaroslav
----- Jaroslav Kysela perex@perex.cz Linux Kernel Sound Maintainer ALSA Project, Red Hat, Inc.