From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S935557AbYEBQAi (ORCPT ); Fri, 2 May 2008 12:00:38 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1763883AbYEBQA1 (ORCPT ); Fri, 2 May 2008 12:00:27 -0400 Received: from mx2.suse.de ([195.135.220.15]:57560 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1763159AbYEBQAZ (ORCPT ); Fri, 2 May 2008 12:00:25 -0400 Date: Fri, 02 May 2008 18:00:24 +0200 Message-ID: From: Takashi Iwai To: Stas Sergeev Cc: Linux kernel , Dmitry Torokhov Subject: Re: patch driver-core-warn-about-duplicate-driver-names-on-the-same-bus.patch added to gregkh-2.6 tree In-Reply-To: <481B355B.9040200@aknet.ru> References: <12094266793898@kroah.org> <4816A888.9000802@aknet.ru> <20080429045849.GA6348@suse.de> <481776EA.5010200@aknet.ru> <4818B057.4080803@aknet.ru> <481B355B.9040200@aknet.ru> User-Agent: Wanderlust/2.15.5 (Almost Unreal) SEMI/1.14.6 (Maruoka) FLIM/1.14.7 (=?ISO-8859-4?Q?Sanj=F2?=) APEL/10.6 MULE XEmacs/21.5 (beta28) (fuki) (+CVS-20070806) (i386-suse-linux) MIME-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org At Fri, 02 May 2008 19:38:03 +0400, Stas Sergeev wrote: > > Hello. > > Takashi Iwai wrote: > > And this won't work in most cases. People don't want to replace the > > existing pcspkr driver with snd-pcsp. They don't want to load the > > sound subsystem on their systems just because of beep. > Why should they? They may just stick with > pcspkr driver if they want. ... if it's not built-in. And you'd need extra to manage selectively loading the driver for the very same platform name. > > If you compare pcspkr.c and pcsp_input.c, it's found that the only > > essential difference is the additional check at the head of the event > > handler: > > if (atomic_read(&pcsp_chip.timer_active) || !pcsp_chip.pcspkr) > > return 0; > > If this can be added dynamically to input pcspkr.c, no big point to > > have duped codes. > Another point is PM callbacks. Somehow > snd-pcsp will have to register them with > pcspkr. Hmm, isn't platform PM hook enough? pcspkr.c just sends an event at suspend or shutdown to the event handler. So, if the event was shut up at the beginning in the handler, nothing would be needed. > > Distros usually make input-pcspkr as built-in, not as module. > > So, snd-pcsp is practically unusable on standard kernels of major > > distros as is, unfortunately... > Oh, that's really bad, I didn't know > they do. For what reason? And how then > people disable the beeps? Why real men need to disable beep? :) > Btw, could you please name a few? At > least Fedora has it as a module. Well, then I remember wrongly or an old information... Takashi