Re: [PATCH v6 07/12] usb: chipidea: add a usb2 driver for ci13xxx
From: Peter Chen
Date: Wed Sep 24 2014 - 07:29:26 EST
On Wed, Sep 24, 2014 at 10:30:41AM +0200, Arnd Bergmann wrote:
> On Wednesday 24 September 2014 09:44:19 Arnd Bergmann wrote:
> >
> > We can also gradually move in some of the other glue drivers into
> > the main driver if the differences are small enough.
> >
>
> FWIW, I've just looked at the other glue drivers that already
> exist:
>
> - zevio can just get merged into the common driver, all that seems
> to be needed for that is the additional compatible string, and
> keying off the ci_default_zevio_platdata on the .data field of
> the of_device_id table.
>
> - msm has a custom notifier, which justifies leaving it in a separate
> driver, but it's also small enough that it wouldn't hurt to have
> that merged into the main driver too.
>
> - imx requires a lot of other things, in particular the dependency
> on the usbmisc driver means we don't want to have that in the
> core anyway, so we can't really merge that in.
>
> - the proposed ar933x driver again looks almost trivial, so no reason
> for a separate glue driver for that.
>
> Arnd
Thanks, Arnd.
So, it is IP CORE LIB (you suggest) vs IP CORE Platform Driver
(dwc3, musb, chipidea) you are talking about, right? Except for
creating another platform driver as well as related DT node (optional),
are there any advantages compared to current IP core driver structure?
In this thread, we are talking about creating common platform driver for glue
layer, its design purpose (adapt it for as many as platforms) should be the
same, no matter the IP core part is a LIB or platform driver, am I missing
something?
--
Best Regards,
Peter Chen
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/