[PATCH V4 0/6] USB: OHCI: more bus glues as separate modules

[manjunath]

On Monday 22 July 2013 02:49 PM, Manjunath Goudar wrote:
> These patches are for separating the SOC On-Chip ohci host controller
> from ohci-hcd host code into its own driver module.
> This work is part of enabling multi-platform kernels on ARM;
> it would be nice to have in 3.12.
>
> V2:
>    In patch 5/6 and 6/6:
>    -Set non-standard fields in hc_driver manually, rather than
>     relying on an expanded struct ohci_driver_overrides.
>    -Save orig_ohci_hub_control and orig_ohci_hub_status_data rather than
>     relying on ohci_hub_control and hub_status_data being exported.
>
>    In patch 1/6 to 4/6
>    -ohci_setup() has been removed because it is called in .reset member
>     of the ohci_hc_driver structure.
>
> V3:
>   In patch 5/6 and 6/6:
>   -ohci_setup() has been removed because it is called in .reset member
>    of the ohci_hc_driver structure.
>
>   In patch 5/6:
>   -The ohci_restart() function is not required in  current scenario,
>    only discarding connection state of integrated transceivers is sufficient,
>    for this directly handling ohci->hc_control.
>
>   In patch 2/6 :
>   -rewritten if (config->otg || config->rwc) block statements into
>    two separate 'if blocks' to handle below scenarios
>    1. config->otg set scenario.
>    2. if any of these (config->otg, config->rwc) are set, this
>       scenario should be handled only after ohci_setup()
>
>   In patch 1/6 and 4/6:
>    No change.
>
> V4:
>   In patch 1/6 and 4/6:
>    No change.
>   
>   In patch 2/6 :
>   -usb_remove_hcd() function is required a valid clock that is what
>    omap_ohci_clock_power(0) is called after hcd shutdown.
>
>   In patch 3/6 :
>   -V3 modification revert back, only ohci->regs setting write()
>    function has been removed because ohci->regs doesn't get set until
>    usb_add_hcd.
>
>   In patch 5/6 :
>   - Removed extra space after "tristate".
>   - Removed extra space between function name  and '(' characters.
>   - MODULE_ALIAS line moved to last statement of ohci-at91 file.
>
>   In patch 6/6 :
>   - Removed extra space before the '='.
>   - Moved  /* forward definitions */ line before the declarations of functions.
>
> Manjunath Goudar (6):
>    USB: OHCI: make ohci-exynos a separate driver
>    USB: OHCI: make ohci-omap a separate driver
>    USB: OHCI: make ohci-omap3 a separate driver
>    USB: OHCI: make ohci-spear a separate driver
>    USB: OHCI: make ohci-at91 a separate driver
>    USB: OHCI: make ohci-s3c2410 a separate driver
>
>   drivers/usb/host/Kconfig        |   30 ++++++-
>   drivers/usb/host/Makefile       |    6 ++
>   drivers/usb/host/ohci-at91.c    |  153 ++++++++++++++++-------------------
>   drivers/usb/host/ohci-exynos.c  |  167 ++++++++++++++++-----------------------
>   drivers/usb/host/ohci-hcd.c     |  108 -------------------------
>   drivers/usb/host/ohci-omap.c    |  156 +++++++++++++-----------------------
>   drivers/usb/host/ohci-omap3.c   |  118 +++++++++------------------
>   drivers/usb/host/ohci-s3c2410.c |  128 +++++++++++++-----------------
>   drivers/usb/host/ohci-spear.c   |  140 +++++++++++++-------------------
>   9 files changed, 374 insertions(+), 632 deletions(-)
>
I am very sorry, this is patch V4 series.

Thanks
Manjunath Goudar