From mboxrd@z Thu Jan 1 00:00:00 1970 Received: with ECARTIS (v1.0.0; list linux-mips); Sun, 22 Jan 2012 20:27:24 +0100 (CET) Received: from out3-smtp.messagingengine.com ([66.111.4.27]:38273 "EHLO out3-smtp.messagingengine.com" rhost-flags-OK-OK-OK-OK) by eddie.linux-mips.org with ESMTP id S1901173Ab2AVT1R (ORCPT ); Sun, 22 Jan 2012 20:27:17 +0100 Received: from compute5.internal (compute5.nyi.mail.srv.osa [10.202.2.45]) by gateway1.nyi.mail.srv.osa (Postfix) with ESMTP id 77A1620368 for ; Sun, 22 Jan 2012 14:27:15 -0500 (EST) Received: from frontend2.nyi.mail.srv.osa ([10.202.2.161]) by compute5.internal (MEProxy); Sun, 22 Jan 2012 14:27:15 -0500 DKIM-Signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d= messagingengine.com; h=date:from:to:cc:subject:message-id :references:mime-version:content-type:in-reply-to; s=smtpout; bh=PcHVgK0lHnf4XsLrBKRE0t5cMvo=; b=VlSDcIYiFlx23z3cNysc+a1IUBXq /ZOMGkAHo0sFfhk+e+TY0Qow9RcmUKHwUKwSF47wiFjJnzMjGYBpRRCXdnnr+tX3 CHeooj20KAB1ORtQEIsfCIwp01pmp4f/hSFry/HwepWQqJ1xa3RAOdtxKKfzc567 nFwqCThPzYDvYKk= X-Sasl-enc: DPphdUu3uZbniG/kdPsy9QFCtkrUL5s6nW/Zl4igL+cH 1327260434 Received: from localhost (m1e0f36d0.tmodns.net [208.54.15.30]) by mail.messagingengine.com (Postfix) with ESMTPSA id 84562482488; Sun, 22 Jan 2012 14:27:14 -0500 (EST) Date: Sun, 22 Jan 2012 14:23:46 -0500 From: Greg KH To: Hauke Mehrtens Cc: Alan Stern , ralf@linux-mips.org, linux-mips@linux-mips.org, USB list , zajec5@gmail.com, linux-wireless@vger.kernel.org, m@bues.ch, george@znau.edu.ua Subject: Re: [PATCH 4/7] USB: EHCI: Add a generic platform device driver Message-ID: <20120122192346.GA1691@kroah.com> References: <4F1C24F5.2020506@hauke-m.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <4F1C24F5.2020506@hauke-m.de> User-Agent: Mutt/1.5.21 (2010-09-15) X-archive-position: 32310 X-ecartis-version: Ecartis v1.0.0 Sender: linux-mips-bounce@linux-mips.org Errors-to: linux-mips-bounce@linux-mips.org X-original-sender: greg@kroah.com Precedence: bulk X-list: linux-mips Return-Path: On Sun, Jan 22, 2012 at 04:02:13PM +0100, Hauke Mehrtens wrote: > On 01/22/2012 04:41 AM, Alan Stern wrote: > > On Sat, 21 Jan 2012, Hauke Mehrtens wrote: > > > >> This adds a generic driver for platform devices. It works like the PCI > >> driver and is based on it. This is for devices which do not have an own > >> bus but their EHCI controller works like a PCI controller. It will be > >> used for the Broadcom bcma and ssb USB EHCI controller. > > > > Before adding a generic platform driver for EHCI, you should give some > > to thought to how it might be generalized. There are a lot of EHCI > > platform drivers, all differing in various major or minor respects. > > It should be possible to replace a lot of them with the generic driver, > > but first it will need some way to cope with a few minor quirks. > > > > Please consider this, and think about which of the existing drivers > > could be replaced. > > For now I just build this for bcma and ssb based SoCs. Yes there are > some drivers which could be replaced with this one, but most (all ??) of > them do something special in the device probing and this have to be > moved to somewhere else e.g. where the platform device is created. > I could rename it so it would not be generic any more, but I think it is > the wrong approach. ;-) > I am not able and do not have the time to convert all EHCI platform > drivers, where it is possible to this generic platform driver, as I do > not have the devices to test this and time is limited. Time is not limited for us, sorry, this seems like the correct thing to do, and because of that, we (well I at least) will not accept this patch as-is. Please go rework it to be as Alan suggested. > If someone else wants to improve something on these "generic" platform > drivers to make them work with an other device I am totally fine with it. I think that someone just became you :) Yes, this isn't fair, but it's how Linux kernel development works, sorry. thanks, greg k-h