From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1761509AbYD2PUS (ORCPT ); Tue, 29 Apr 2008 11:20:18 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1754234AbYD2PUE (ORCPT ); Tue, 29 Apr 2008 11:20:04 -0400 Received: from ns2.suse.de ([195.135.220.15]:36030 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752280AbYD2PUC (ORCPT ); Tue, 29 Apr 2008 11:20:02 -0400 Date: Tue, 29 Apr 2008 08:14:36 -0700 From: Greg KH To: Takashi Iwai Cc: Stas Sergeev , 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 Message-ID: <20080429151436.GB10587@suse.de> References: <12094266793898@kroah.org> <4816A888.9000802@aknet.ru> <20080429045849.GA6348@suse.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.16 (2007-06-09) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Apr 29, 2008 at 12:41:56PM +0200, Takashi Iwai wrote: > At Mon, 28 Apr 2008 21:58:49 -0700, > Greg KH wrote: > > > > On Tue, Apr 29, 2008 at 08:48:08AM +0400, Stas Sergeev wrote: > > > Hello. > > > > > > gregkh@suse.de wrote: > > > > This is a note to let you know that I've just added the patch titled > > > > Subject: driver core: warn about duplicate driver names on the same bus > > > > to my gregkh-2.6 tree. Its filename is > > > Thanks. > > > > > > Takashi, do you think this fixes the > > > pcspkr/snd-pcsp conflict sufficienly > > > well? I think the Kconfig protection > > > is no longer needed. For the better > > > convinience the modprobe rule can be > > > created that will remove one driver > > > before inserting another. But this is > > > optional. > > > > No, don't rely on the driver core warning and catching code errors like > > this, that's not a good idea at all. I like the warning and your patch, > > but don't assume it's always going to be there. > > But multiple drivers for the same device are allowed on other buses > like PCI. I think this is no reason to prohibit the multiple platform > drivers for the same platform device. No, the driver core doesn't allow that at all right now (I'm working on fixing that though...) What this patch did is just not allow you to have the same "name" for the driver on the same bus, which is reasonable as without the check, sysfs complains about trying to create a duplicate name in the same directory. Different names is fine, and I'll continue to work on the ability to allow multiple drivers for the same device at the same time... thanks, greg k-h