diff options
author | Paul Zimmerman <Paul.Zimmerman@synopsys.com> | 2013-03-25 17:03:35 -0700 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-03-25 17:10:35 -0700 |
commit | e62662c7f3079e02a1e59229901648758a49464a (patch) | |
tree | cd7549cf973733ca3f437257500784885a77d918 /drivers | |
parent | 8509f2f43a2b245ade076d1568daa2cf15446732 (diff) | |
download | linux-e62662c7f3079e02a1e59229901648758a49464a.tar.gz linux-e62662c7f3079e02a1e59229901648758a49464a.tar.bz2 linux-e62662c7f3079e02a1e59229901648758a49464a.zip |
staging: dwc2: remove unneeded arguments from two functions
Remove the unneeded struct device *dev argument from dwc2_hcd_init()
and dwc2_hcd_remove(), and pass in the value through the hsotg->dev
member instead
Signed-off-by: Paul Zimmerman <paulz@synopsys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/staging/dwc2/hcd.c | 19 | ||||
-rw-r--r-- | drivers/staging/dwc2/hcd.h | 6 | ||||
-rw-r--r-- | drivers/staging/dwc2/pci.c | 7 |
3 files changed, 16 insertions, 16 deletions
diff --git a/drivers/staging/dwc2/hcd.c b/drivers/staging/dwc2/hcd.c index 01dbdd85c725..af1a15c1a0a2 100644 --- a/drivers/staging/dwc2/hcd.c +++ b/drivers/staging/dwc2/hcd.c @@ -2682,7 +2682,7 @@ static void dwc2_set_uninitialized(int *p, int size) * USB bus with the core and calls the hc_driver->start() function. It returns * a negative error on failure. */ -int dwc2_hcd_init(struct device *dev, struct dwc2_hsotg *hsotg, int irq, +int dwc2_hcd_init(struct dwc2_hsotg *hsotg, int irq, struct dwc2_core_params *params) { struct usb_hcd *hcd; @@ -2691,7 +2691,7 @@ int dwc2_hcd_init(struct device *dev, struct dwc2_hsotg *hsotg, int irq, int i, num_channels; int retval = -ENOMEM; - dev_dbg(dev, "DWC OTG HCD INIT\n"); + dev_dbg(hsotg->dev, "DWC OTG HCD INIT\n"); /* * Attempt to ensure this device is really a DWC_otg Controller. @@ -2702,12 +2702,12 @@ int dwc2_hcd_init(struct device *dev, struct dwc2_hsotg *hsotg, int irq, snpsid = readl(hsotg->regs + GSNPSID); if ((snpsid & 0xfffff000) != 0x4f542000 && (snpsid & 0xfffff000) != 0x4f543000) { - dev_err(dev, "Bad value for GSNPSID: 0x%08x\n", snpsid); + dev_err(hsotg->dev, "Bad value for GSNPSID: 0x%08x\n", snpsid); retval = -ENODEV; goto error1; } - hcd = usb_create_hcd(&dwc2_hc_driver, dev, dev_name(dev)); + hcd = usb_create_hcd(&dwc2_hc_driver, hsotg->dev, dev_name(hsotg->dev)); if (!hcd) goto error1; @@ -2716,7 +2716,6 @@ int dwc2_hcd_init(struct device *dev, struct dwc2_hsotg *hsotg, int irq, spin_lock_init(&hsotg->lock); ((struct wrapper_priv_data *) &hcd->hcd_priv)->hsotg = hsotg; hsotg->priv = hcd; - hsotg->dev = dev; /* * Store the contents of the hardware configuration registers here for @@ -2913,7 +2912,7 @@ error2: usb_put_hcd(hcd); error1: - dev_err(dev, "%s() FAILED, returning %d\n", __func__, retval); + dev_err(hsotg->dev, "%s() FAILED, returning %d\n", __func__, retval); return retval; } EXPORT_SYMBOL_GPL(dwc2_hcd_init); @@ -2922,17 +2921,17 @@ EXPORT_SYMBOL_GPL(dwc2_hcd_init); * Removes the HCD. * Frees memory and resources associated with the HCD and deregisters the bus. */ -void dwc2_hcd_remove(struct device *dev, struct dwc2_hsotg *hsotg) +void dwc2_hcd_remove(struct dwc2_hsotg *hsotg) { struct usb_hcd *hcd; - dev_dbg(dev, "DWC OTG HCD REMOVE\n"); + dev_dbg(hsotg->dev, "DWC OTG HCD REMOVE\n"); hcd = dwc2_hsotg_to_hcd(hsotg); - dev_dbg(dev, "hsotg->hcd = %p\n", hcd); + dev_dbg(hsotg->dev, "hsotg->hcd = %p\n", hcd); if (!hcd) { - dev_dbg(dev, "%s: dwc2_hsotg_to_hcd(hsotg) NULL!\n", + dev_dbg(hsotg->dev, "%s: dwc2_hsotg_to_hcd(hsotg) NULL!\n", __func__); return; } diff --git a/drivers/staging/dwc2/hcd.h b/drivers/staging/dwc2/hcd.h index 775337e92785..8a60dcf886eb 100644 --- a/drivers/staging/dwc2/hcd.h +++ b/drivers/staging/dwc2/hcd.h @@ -447,9 +447,9 @@ static inline u8 dwc2_hcd_is_pipe_out(struct dwc2_hcd_pipe_info *pipe) return !dwc2_hcd_is_pipe_in(pipe); } -extern int dwc2_hcd_init(struct device *dev, struct dwc2_hsotg *hsotg, - int irq, struct dwc2_core_params *params); -extern void dwc2_hcd_remove(struct device *dev, struct dwc2_hsotg *hsotg); +extern int dwc2_hcd_init(struct dwc2_hsotg *hsotg, int irq, + struct dwc2_core_params *params); +extern void dwc2_hcd_remove(struct dwc2_hsotg *hsotg); extern int dwc2_set_parameters(struct dwc2_hsotg *hsotg, struct dwc2_core_params *params); diff --git a/drivers/staging/dwc2/pci.c b/drivers/staging/dwc2/pci.c index 117d3ce404dd..539f8591dd68 100644 --- a/drivers/staging/dwc2/pci.c +++ b/drivers/staging/dwc2/pci.c @@ -103,7 +103,7 @@ static void dwc2_driver_remove(struct pci_dev *dev) dev_dbg(&dev->dev, "%s(%p)\n", __func__, dev); - dwc2_hcd_remove(&dev->dev, hsotg); + dwc2_hcd_remove(hsotg); pci_disable_device(dev); } @@ -133,6 +133,7 @@ static int dwc2_driver_probe(struct pci_dev *dev, pci_set_power_state(dev, PCI_D0); + hsotg->dev = &dev->dev; hsotg->regs = devm_request_and_ioremap(&dev->dev, &dev->resource[0]); if (!hsotg->regs) return -ENOMEM; @@ -157,7 +158,7 @@ static int dwc2_driver_probe(struct pci_dev *dev, pci_set_consistent_dma_mask(dev, 0); } - retval = dwc2_hcd_init(&dev->dev, hsotg, dev->irq, &dwc2_module_params); + retval = dwc2_hcd_init(hsotg, dev->irq, &dwc2_module_params); if (retval) { pci_disable_device(dev); return retval; @@ -171,7 +172,7 @@ static int dwc2_driver_probe(struct pci_dev *dev, IRQF_SHARED | IRQ_LEVEL, dev_name(&dev->dev), hsotg); if (retval) - dwc2_hcd_remove(&dev->dev, hsotg); + dwc2_hcd_remove(hsotg); return retval; } |