This patch (as1630) cleans up a few minor items resulting from the
split-up of the ehci-hcd driver:
Remove the product_desc string from the ehci_driver_overrides
structure. All drivers will use the generic "EHCI Host
Controller" string. (This was requested by Felipe Balbi.)
Allow drivers to pass a NULL pointer to ehci_init_driver()
if they don't have to override any settings.
Remove a #define symbol that is no longer used from the
ChipIdea host driver.
Rename overrides to pci_overrides in ehci-pci.c, for
consistency with ehci-platform.c.
Mark the *_overrides structures as __initdata.
Signed-off-by: Alan Stern <stern@xxxxxxxxxxxxxxxxxxx>
CC: Felipe Balbi <balbi@xxxxxx>
---
drivers/usb/chipidea/host.c | 7 +------
drivers/usb/host/ehci-hcd.c | 9 +++++----
drivers/usb/host/ehci-pci.c | 5 ++---
drivers/usb/host/ehci-platform.c | 3 +--
drivers/usb/host/ehci.h | 1 -
5 files changed, 9 insertions(+), 16 deletions(-)
Index: usb-3.7/drivers/usb/host/ehci.h
===================================================================
--- usb-3.7.orig/drivers/usb/host/ehci.h
+++ usb-3.7/drivers/usb/host/ehci.h
@@ -785,7 +785,6 @@ static inline u32 hc32_to_cpup (const st
/* Declarations of things exported for use by ehci platform drivers */
struct ehci_driver_overrides {
- const char *product_desc;
size_t extra_priv_size;
int (*reset)(struct usb_hcd *hcd);
};
Index: usb-3.7/drivers/usb/host/ehci-hcd.c
===================================================================
--- usb-3.7.orig/drivers/usb/host/ehci-hcd.c
+++ usb-3.7/drivers/usb/host/ehci-hcd.c
@@ -1231,10 +1231,11 @@ void ehci_init_driver(struct hc_driver *
/* Copy the generic table to drv and then apply the overrides */
*drv = ehci_hc_driver;
- drv->product_desc = over->product_desc;
- drv->hcd_priv_size += over->extra_priv_size;
- if (over->reset)
- drv->reset = over->reset;
+ if (over) {
+ drv->hcd_priv_size += over->extra_priv_size;
+ if (over->reset)
+ drv->reset = over->reset;
+ }
}
EXPORT_SYMBOL_GPL(ehci_init_driver);
Index: usb-3.7/drivers/usb/chipidea/host.c
===================================================================
--- usb-3.7.orig/drivers/usb/chipidea/host.c
+++ usb-3.7/drivers/usb/chipidea/host.c
@@ -25,17 +25,12 @@
#include <linux/usb/hcd.h>
#include <linux/usb/chipidea.h>
-#define CHIPIDEA_EHCI
#include "../host/ehci.h"
#include "ci.h"
#include "bits.h"
#include "host.h"
-static const struct ehci_driver_overrides ci_overrides = {
- .product_desc = "ChipIdea HDRC EHCI host controller",
-};
-
static struct hc_driver __read_mostly ci_ehci_hc_driver;
static irqreturn_t host_irq(struct ci13xxx *ci)
@@ -103,7 +98,7 @@ int ci_hdrc_host_init(struct ci13xxx *ci
rdrv->name = "host";
ci->roles[CI_ROLE_HOST] = rdrv;
- ehci_init_driver(&ci_ehci_hc_driver, &ci_overrides);
+ ehci_init_driver(&ci_ehci_hc_driver, NULL);
return 0;
}
Index: usb-3.7/drivers/usb/host/ehci-pci.c
===================================================================
--- usb-3.7.orig/drivers/usb/host/ehci-pci.c
+++ usb-3.7/drivers/usb/host/ehci-pci.c
@@ -383,8 +383,7 @@ static int ehci_pci_resume(struct usb_hc
static struct hc_driver __read_mostly ehci_pci_hc_driver;
-static const struct ehci_driver_overrides overrides = {
- .product_desc = "EHCI PCI host controller",
+static const struct ehci_driver_overrides pci_overrides __initdata = {
.reset = ehci_pci_setup,
};
@@ -426,7 +425,7 @@ static int __init ehci_pci_init(void)
pr_info("%s: " DRIVER_DESC "\n", hcd_name);
- ehci_init_driver(&ehci_pci_hc_driver, &overrides);
+ ehci_init_driver(&ehci_pci_hc_driver, &pci_overrides);
/* Entries for the PCI suspend/resume callbacks are special */
ehci_pci_hc_driver.pci_suspend = ehci_suspend;
Index: usb-3.7/drivers/usb/host/ehci-platform.c
===================================================================
--- usb-3.7.orig/drivers/usb/host/ehci-platform.c
+++ usb-3.7/drivers/usb/host/ehci-platform.c
@@ -57,8 +57,7 @@ static int ehci_platform_reset(struct us
static struct hc_driver __read_mostly ehci_platform_hc_driver;
-static const struct ehci_driver_overrides platform_overrides = {
- .product_desc = "Generic Platform EHCI controller",
+static const struct ehci_driver_overrides platform_overrides __initdata = {
.reset = ehci_platform_reset,
};
--
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
