[alsa-devel] PC Beep or PC Speaker or just Beep? and HDA Beep code...

[Jaroslav Kysela]

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:
>>
>> 1) module parameter (difficult tracking of changes through sysfs)
>> 2) sysfs file
>> 3) current implementation
>> 4) 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 at perex.cz>
Linux Kernel Sound Maintainer
ALSA Project, Red Hat, Inc.