Re: [patch 1/1] pc-speaker: add SND_SILENT

On Sun, Mar 26, 2006 at 01:52:50PM +0400, Stas Sergeev wrote:
Bodo Eggert wrote:
The problem is that the snd-pcsp doesn't replace pcspkr.

If that's the problem, create a minimal input driver that will signal the
snd-pcsp to beep, and change the original pcspkr to include
"(Non-ALSA)".

Yes, making snd-pcsp to produce the console beeps and
making it mutually exclusive with pcspkr is possible.
But I think it is undesireable. People that don't like
the console beeps (myself included) simply do not load
the pcspkr module right now. If snd-pcsp is to produce
the beeps, then not loading pcspkr will not get the desired
effect any more, and the only possibility would be to,
probably, add the separate mixer control for the beeps.
I find this counter-intuitive: some people will be able to
disable the beeps by simply not loading pcspkr, while for
others this will suddenly magically stop working. This may
lead to a few unnecessary bug reports and confusions.

My laptop already has such a mixer control (and it works too,
though it does depend on pcspkr being loaded). Have there been bug
reports about PC speakers that don't work because the PC speaker mixer
control was not unmuted? Well, probably, but it's not a new
situation.

It doesn't seem unreasonable to me to expect users to configure
their PC speaker beep preference through the ALSA mixer when
configuring the PC speaker to be an audio device.

I would think the ideal situation would be to make every ALSA
device capable of acting as the console bell (defaulting to muted,
like every other ALSA mixer control). Then only pcspkr would be the
odd case (though maybe a common one).

I dunno if there's a reasonably easy way to do that (without
changing every ALSA driver) though.
--
Joseph Fannin
jfannin@xxxxxxxxx

"That's all I have to say about that." -- Forrest Gump.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

Relevant Pages