Re: [PATCH 4/7] USB: EHCI: Add a generic platform device driver

[Greg KH <greg@kroah.com>]

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