linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/8] usb: ehci: more bus glues as separate modules
@ 2013-02-10 18:58 manjunath.goudar at linaro.org
  2013-02-10 21:28 ` Jason Cooper
  2013-02-11 18:45 ` Ben Dooks
  0 siblings, 2 replies; 3+ messages in thread
From: manjunath.goudar at linaro.org @ 2013-02-10 18:58 UTC (permalink / raw)
  To: linux-arm-kernel

From: Manjunath Goudar <manjunath.goudar@linaro.org>

Separate the SOC On-Chip host controller driver from ehci-hcd 
host code into its own driver module

Manjunath Goudar (8):
  USB: EHCI: make ehci-spear a separate driver
  USB: EHCI: make ehci-atmel a separate driver
  USB: EHCI: make ehci-s5p a separate driver
  USB: EHCI: make ehci-mv a separate driver
  USB: EHCI: make ehci-vt8500 a separate driver
  USB: EHCI: make ehci-msm a separate driver
  USB: EHCI: make ehci-w90X900 a separate driver
  USB: EHCI: make ehci-orion a separate driver

 drivers/usb/host/Kconfig        |   42 +++++++++++--
 drivers/usb/host/Makefile       |    9 ++-
 drivers/usb/host/ehci-atmel.c   |   77 ++++++++++++-----------
 drivers/usb/host/ehci-hcd.c     |   52 +++-------------
 drivers/usb/host/ehci-msm.c     |   85 +++++++++++--------------
 drivers/usb/host/ehci-mv.c      |   80 +++++++++++-------------
 drivers/usb/host/ehci-orion.c   |   91 +++++++++++++--------------
 drivers/usb/host/ehci-s5p.c     |   69 ++++++++++++---------
 drivers/usb/host/ehci-spear.c   |  131 ++++++++++++++++++++-------------------
 drivers/usb/host/ehci-vt8500.c  |   73 ++++++++++------------
 drivers/usb/host/ehci-w90x900.c |   90 ++++++++++++---------------
 drivers/usb/host/ehci.h         |    2 +-
 12 files changed, 392 insertions(+), 409 deletions(-)

-- 
1.7.9.5

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [PATCH 0/8] usb: ehci: more bus glues as separate modules
  2013-02-10 18:58 [PATCH 0/8] usb: ehci: more bus glues as separate modules manjunath.goudar at linaro.org
@ 2013-02-10 21:28 ` Jason Cooper
  2013-02-11 18:45 ` Ben Dooks
  1 sibling, 0 replies; 3+ messages in thread
From: Jason Cooper @ 2013-02-10 21:28 UTC (permalink / raw)
  To: linux-arm-kernel

Manjunath,

I think this is the second version of this series.  Please indicate so
in the subject "[PATCH V2 X/Y]" and provide a changelog with each patch
so that we can easily confirm suggested changes from one version to the
next.

thx,

Jason.

On Mon, Feb 11, 2013 at 12:28:09AM +0530, manjunath.goudar at linaro.org wrote:
> From: Manjunath Goudar <manjunath.goudar@linaro.org>
> 
> Separate the SOC On-Chip host controller driver from ehci-hcd 
> host code into its own driver module
> 
> Manjunath Goudar (8):
>   USB: EHCI: make ehci-spear a separate driver
>   USB: EHCI: make ehci-atmel a separate driver
>   USB: EHCI: make ehci-s5p a separate driver
>   USB: EHCI: make ehci-mv a separate driver
>   USB: EHCI: make ehci-vt8500 a separate driver
>   USB: EHCI: make ehci-msm a separate driver
>   USB: EHCI: make ehci-w90X900 a separate driver
>   USB: EHCI: make ehci-orion a separate driver
> 
>  drivers/usb/host/Kconfig        |   42 +++++++++++--
>  drivers/usb/host/Makefile       |    9 ++-
>  drivers/usb/host/ehci-atmel.c   |   77 ++++++++++++-----------
>  drivers/usb/host/ehci-hcd.c     |   52 +++-------------
>  drivers/usb/host/ehci-msm.c     |   85 +++++++++++--------------
>  drivers/usb/host/ehci-mv.c      |   80 +++++++++++-------------
>  drivers/usb/host/ehci-orion.c   |   91 +++++++++++++--------------
>  drivers/usb/host/ehci-s5p.c     |   69 ++++++++++++---------
>  drivers/usb/host/ehci-spear.c   |  131 ++++++++++++++++++++-------------------
>  drivers/usb/host/ehci-vt8500.c  |   73 ++++++++++------------
>  drivers/usb/host/ehci-w90x900.c |   90 ++++++++++++---------------
>  drivers/usb/host/ehci.h         |    2 +-
>  12 files changed, 392 insertions(+), 409 deletions(-)
> 
> -- 
> 1.7.9.5
> 
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [PATCH 0/8] usb: ehci: more bus glues as separate modules
  2013-02-10 18:58 [PATCH 0/8] usb: ehci: more bus glues as separate modules manjunath.goudar at linaro.org
  2013-02-10 21:28 ` Jason Cooper
@ 2013-02-11 18:45 ` Ben Dooks
  1 sibling, 0 replies; 3+ messages in thread
From: Ben Dooks @ 2013-02-11 18:45 UTC (permalink / raw)
  To: linux-arm-kernel

On 10/02/13 18:58, manjunath.goudar at linaro.org wrote:
> From: Manjunath Goudar<manjunath.goudar@linaro.org>
>
> Separate the SOC On-Chip host controller driver from ehci-hcd
> host code into its own driver module
>
> Manjunath Goudar (8):
>    USB: EHCI: make ehci-spear a separate driver
>    USB: EHCI: make ehci-atmel a separate driver
>    USB: EHCI: make ehci-s5p a separate driver
>    USB: EHCI: make ehci-mv a separate driver
>    USB: EHCI: make ehci-vt8500 a separate driver
>    USB: EHCI: make ehci-msm a separate driver
>    USB: EHCI: make ehci-w90X900 a separate driver
>    USB: EHCI: make ehci-orion a separate driver
>
>   drivers/usb/host/Kconfig        |   42 +++++++++++--
>   drivers/usb/host/Makefile       |    9 ++-
>   drivers/usb/host/ehci-atmel.c   |   77 ++++++++++++-----------
>   drivers/usb/host/ehci-hcd.c     |   52 +++-------------
>   drivers/usb/host/ehci-msm.c     |   85 +++++++++++--------------
>   drivers/usb/host/ehci-mv.c      |   80 +++++++++++-------------
>   drivers/usb/host/ehci-orion.c   |   91 +++++++++++++--------------
>   drivers/usb/host/ehci-s5p.c     |   69 ++++++++++++---------
>   drivers/usb/host/ehci-spear.c   |  131 ++++++++++++++++++++-------------------
>   drivers/usb/host/ehci-vt8500.c  |   73 ++++++++++------------
>   drivers/usb/host/ehci-w90x900.c |   90 ++++++++++++---------------
>   drivers/usb/host/ehci.h         |    2 +-
>   12 files changed, 392 insertions(+), 409 deletions(-)

Having looked at this, we will end up with a number of modules,
each having very similar code in them.#

How about we add drivers/usb/host/ehci-platform.c for all
the platform based drivers.

I think it is possible to remove all the specific names, as
we certainly don't need the driver advertising itself for each
system. Also, does anyone rely on the name field in the HC info,
or can it just be "Platform EHCI" ?

-- 
Ben Dooks				http://www.codethink.co.uk/
Senior Engineer				Codethink - Providing Genius

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2013-02-11 18:45 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-02-10 18:58 [PATCH 0/8] usb: ehci: more bus glues as separate modules manjunath.goudar at linaro.org
2013-02-10 21:28 ` Jason Cooper
2013-02-11 18:45 ` Ben Dooks

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).