drivers:usb:fsl:Make fsl ehci drv an independent driver module
authorRamneek Mehresh <ramneek.mehresh@freescale.com>
Thu, 14 May 2015 13:34:46 +0000 (19:04 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sun, 24 May 2015 16:29:36 +0000 (09:29 -0700)
Make Freescale EHCI driver an independent entity from ehci-hcd.c.
This involves
- using module_init/module_exit functions
- using overrides structure
- some necessary code cleanup

Signed-off-by: Ramneek Mehresh <ramneek.mehresh@freescale.com>
Acked-by: Alan Stern <stern@rowland.harvard.edu>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/usb/host/Kconfig
drivers/usb/host/Makefile
drivers/usb/host/ehci-fsl.c
drivers/usb/host/ehci-hcd.c

index 197a6a3e613b29dc8c736ea42f9c6c191da327b1..547cee83400b8b28560c88736a5dac530d1fece0 100644 (file)
@@ -137,7 +137,7 @@ config XPS_USB_HCD_XILINX
                devices only.
 
 config USB_EHCI_FSL
-       bool "Support for Freescale PPC on-chip EHCI USB controller"
+       tristate "Support for Freescale PPC on-chip EHCI USB controller"
        depends on FSL_SOC
        select USB_EHCI_ROOT_HUB_TT
        select USB_FSL_MPH_DR_OF if OF
index 972a74a6f428faa816d2c2cac587a4f468733c3d..754efaa8ccf8fbebf2e2c591ec4c77ae83893952 100644 (file)
@@ -72,6 +72,7 @@ obj-$(CONFIG_USB_R8A66597_HCD)        += r8a66597-hcd.o
 obj-$(CONFIG_USB_HWA_HCD)      += hwa-hc.o
 obj-$(CONFIG_USB_IMX21_HCD)    += imx21-hcd.o
 obj-$(CONFIG_USB_FSL_MPH_DR_OF)        += fsl-mph-dr-of.o
+obj-$(CONFIG_USB_EHCI_FSL)     += ehci-fsl.o
 obj-$(CONFIG_USB_HCD_BCMA)     += bcma-hcd.o
 obj-$(CONFIG_USB_HCD_SSB)      += ssb-hcd.o
 obj-$(CONFIG_USB_FUSBH200_HCD) += fusbh200-hcd.o
index ab4eee3df97acd655040f1d6015696bb88a2c53f..5352e74b92e2112c273c4e0a2c43387a8bd0a105 100644 (file)
@@ -1,6 +1,6 @@
 /*
  * Copyright 2005-2009 MontaVista Software, Inc.
- * Copyright 2008,2012      Freescale Semiconductor, Inc.
+ * Copyright 2008,2012,2015      Freescale Semiconductor, Inc.
  *
  * This program is free software; you can redistribute it and/or modify it
  * under the terms of the GNU General Public License as published by the
  */
 
 #include <linux/kernel.h>
+#include <linux/module.h>
 #include <linux/types.h>
 #include <linux/delay.h>
 #include <linux/pm.h>
 #include <linux/err.h>
+#include <linux/usb.h>
+#include <linux/usb/ehci_def.h>
+#include <linux/usb/hcd.h>
+#include <linux/usb/otg.h>
 #include <linux/platform_device.h>
 #include <linux/fsl_devices.h>
 
+#include "ehci.h"
 #include "ehci-fsl.h"
 
+#define DRIVER_DESC "Freescale EHCI Host controller driver"
+#define DRV_NAME "ehci-fsl"
+
+static struct hc_driver __read_mostly fsl_ehci_hc_driver;
+
 /* configure so an HC device and id are always provided */
 /* always called with process context; sleeping is OK */
 
-/**
- * usb_hcd_fsl_probe - initialize FSL-based HCDs
- * @drvier: Driver to be used for this HCD
+/*
+ * fsl_ehci_drv_probe - initialize FSL-based HCDs
  * @pdev: USB Host Controller being probed
  * Context: !in_interrupt()
  *
  * Allocates basic resources for this USB host controller.
  *
  */
-static int usb_hcd_fsl_probe(const struct hc_driver *driver,
-                            struct platform_device *pdev)
+static int fsl_ehci_drv_probe(struct platform_device *pdev)
 {
        struct fsl_usb2_platform_data *pdata;
        struct usb_hcd *hcd;
@@ -86,7 +95,8 @@ static int usb_hcd_fsl_probe(const struct hc_driver *driver,
        }
        irq = res->start;
 
-       hcd = usb_create_hcd(driver, &pdev->dev, dev_name(&pdev->dev));
+       hcd = usb_create_hcd(&fsl_ehci_hc_driver, &pdev->dev,
+                               dev_name(&pdev->dev));
        if (!hcd) {
                retval = -ENOMEM;
                goto err1;
@@ -159,38 +169,6 @@ static int usb_hcd_fsl_probe(const struct hc_driver *driver,
        return retval;
 }
 
-/* may be called without controller electrically present */
-/* may be called with controller, bus, and devices active */
-
-/**
- * usb_hcd_fsl_remove - shutdown processing for FSL-based HCDs
- * @dev: USB Host Controller being removed
- * Context: !in_interrupt()
- *
- * Reverses the effect of usb_hcd_fsl_probe().
- *
- */
-static void usb_hcd_fsl_remove(struct usb_hcd *hcd,
-                              struct platform_device *pdev)
-{
-       struct fsl_usb2_platform_data *pdata = dev_get_platdata(&pdev->dev);
-
-       if (!IS_ERR_OR_NULL(hcd->usb_phy)) {
-               otg_set_host(hcd->usb_phy->otg, NULL);
-               usb_put_phy(hcd->usb_phy);
-       }
-
-       usb_remove_hcd(hcd);
-
-       /*
-        * do platform specific un-initialization:
-        * release iomux pins, disable clock, etc.
-        */
-       if (pdata->exit)
-               pdata->exit(pdev);
-       usb_put_hcd(hcd);
-}
-
 static int ehci_fsl_setup_phy(struct usb_hcd *hcd,
                               enum fsl_usb2_phy_modes phy_mode,
                               unsigned int port_offset)
@@ -636,79 +614,77 @@ static int ehci_start_port_reset(struct usb_hcd *hcd, unsigned port)
 #define ehci_start_port_reset  NULL
 #endif /* CONFIG_USB_OTG */
 
+static struct ehci_driver_overrides ehci_fsl_overrides __initdata = {
+       .extra_priv_size = sizeof(struct ehci_fsl),
+       .reset = ehci_fsl_setup,
+};
 
-static const struct hc_driver ehci_fsl_hc_driver = {
-       .description = hcd_name,
-       .product_desc = "Freescale On-Chip EHCI Host Controller",
-       .hcd_priv_size = sizeof(struct ehci_fsl),
+/**
+ * fsl_ehci_drv_remove - shutdown processing for FSL-based HCDs
+ * @dev: USB Host Controller being removed
+ * Context: !in_interrupt()
+ *
+ * Reverses the effect of usb_hcd_fsl_probe().
+ *
+ */
 
-       /*
-        * generic hardware linkage
-        */
-       .irq = ehci_irq,
-       .flags = HCD_USB2 | HCD_MEMORY | HCD_BH,
+static int fsl_ehci_drv_remove(struct platform_device *pdev)
+{
+       struct fsl_usb2_platform_data *pdata = dev_get_platdata(&pdev->dev);
+       struct usb_hcd *hcd = platform_get_drvdata(pdev);
 
-       /*
-        * basic lifecycle operations
-        */
-       .reset = ehci_fsl_setup,
-       .start = ehci_run,
-       .stop = ehci_stop,
-       .shutdown = ehci_shutdown,
+       if (!IS_ERR_OR_NULL(hcd->usb_phy)) {
+               otg_set_host(hcd->usb_phy->otg, NULL);
+               usb_put_phy(hcd->usb_phy);
+       }
 
-       /*
-        * managing i/o requests and associated device resources
-        */
-       .urb_enqueue = ehci_urb_enqueue,
-       .urb_dequeue = ehci_urb_dequeue,
-       .endpoint_disable = ehci_endpoint_disable,
-       .endpoint_reset = ehci_endpoint_reset,
+       usb_remove_hcd(hcd);
 
        /*
-        * scheduling support
+        * do platform specific un-initialization:
+        * release iomux pins, disable clock, etc.
         */
-       .get_frame_number = ehci_get_frame,
+       if (pdata->exit)
+               pdata->exit(pdev);
+       usb_put_hcd(hcd);
 
-       /*
-        * root hub support
-        */
-       .hub_status_data = ehci_hub_status_data,
-       .hub_control = ehci_hub_control,
-       .bus_suspend = ehci_bus_suspend,
-       .bus_resume = ehci_bus_resume,
-       .start_port_reset = ehci_start_port_reset,
-       .relinquish_port = ehci_relinquish_port,
-       .port_handed_over = ehci_port_handed_over,
-
-       .clear_tt_buffer_complete = ehci_clear_tt_buffer_complete,
+       return 0;
+}
+
+static struct platform_driver ehci_fsl_driver = {
+       .probe = fsl_ehci_drv_probe,
+       .remove = fsl_ehci_drv_remove,
+       .shutdown = usb_hcd_platform_shutdown,
+       .driver = {
+               .name = "fsl-ehci",
+               .pm = EHCI_FSL_PM_OPS,
+       },
 };
 
-static int ehci_fsl_drv_probe(struct platform_device *pdev)
+static int __init ehci_fsl_init(void)
 {
        if (usb_disabled())
                return -ENODEV;
 
-       /* FIXME we only want one one probe() not two */
-       return usb_hcd_fsl_probe(&ehci_fsl_hc_driver, pdev);
-}
+       pr_info(DRV_NAME ": " DRIVER_DESC "\n");
 
-static int ehci_fsl_drv_remove(struct platform_device *pdev)
-{
-       struct usb_hcd *hcd = platform_get_drvdata(pdev);
+       ehci_init_driver(&fsl_ehci_hc_driver, &ehci_fsl_overrides);
 
-       /* FIXME we only want one one remove() not two */
-       usb_hcd_fsl_remove(hcd, pdev);
-       return 0;
+       fsl_ehci_hc_driver.product_desc =
+                       "Freescale On-Chip EHCI Host Controller";
+       fsl_ehci_hc_driver.start_port_reset = ehci_start_port_reset;
+
+
+       return platform_driver_register(&ehci_fsl_driver);
 }
+module_init(ehci_fsl_init);
 
-MODULE_ALIAS("platform:fsl-ehci");
+static void __exit ehci_fsl_cleanup(void)
+{
+       platform_driver_unregister(&ehci_fsl_driver);
+}
+module_exit(ehci_fsl_cleanup);
 
-static struct platform_driver ehci_fsl_driver = {
-       .probe = ehci_fsl_drv_probe,
-       .remove = ehci_fsl_drv_remove,
-       .shutdown = usb_hcd_platform_shutdown,
-       .driver = {
-               .name = "fsl-ehci",
-               .pm = EHCI_FSL_PM_OPS,
-       },
-};
+MODULE_DESCRIPTION(DRIVER_DESC);
+MODULE_LICENSE("GPL");
+MODULE_ALIAS("platform:" DRV_NAME);
index f4d88dfb26a7e5ca0200620e39210b1080a3f263..9dd161ce02a2c784b92d83be90e6f30d5c0f55c2 100644 (file)
@@ -1250,11 +1250,6 @@ MODULE_DESCRIPTION(DRIVER_DESC);
 MODULE_AUTHOR (DRIVER_AUTHOR);
 MODULE_LICENSE ("GPL");
 
-#ifdef CONFIG_USB_EHCI_FSL
-#include "ehci-fsl.c"
-#define        PLATFORM_DRIVER         ehci_fsl_driver
-#endif
-
 #ifdef CONFIG_USB_EHCI_SH
 #include "ehci-sh.c"
 #define PLATFORM_DRIVER                ehci_hcd_sh_driver