Messages in this thread | | | From | Bodo Eggert <> | Subject | Re: [patch 1/1] pc-speaker: add SND_SILENT | Date | Fri, 24 Mar 2006 23:43:48 +0100 |
| |
Stas Sergeev <stsp@aknet.ru> wrote:
> [ adding LKML to CC and removing akpm as this went into a > discussion phase apparently ] > > Dmitry Torokhov wrote:
>>> Well, the main reason behind that change, is that there is a PC-Speaker >>> PCM driver/emulator, snd-pcsp, pending in an ALSA CVS. I can't get it >>> included in kernel before there is a way to disable the pcspkr driver. >> Why can't you? We have ALSA and OSS together in the kernel just fine. > But the pcspkr driver is not an OSS, it is an "input" driver. > >> If you are concerned about both modules baing active at the same time >> do not let user compile pcspkr if snd_pcsp is selected for now. > 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)". Obviously both drivers will exclude each other, but that should be fine, and users who prefer "music" over beeps will be fine, too.
And as a bonus, you might upload a custom PC beep ... but if you do, a userspace notifyer combined with a beep-daemon might be preferable (uinput? I know it exists ...).
Off cause I might be suggesting exactly what you did, since I didn't see the patch. -- Ich danke GMX dafür, die Verwendung meiner Adressen mittels per SPF verbreiteten Lügen zu sabotieren. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |