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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux