usb: gadget: udc: net2280: Remove use_dma module parameter

use_dma parameter was designed to enable the dma on the chip. It was
enabled by default.

It comes from the time when the dma was not reliable. Now it has been
working ok in production.

This patch removes this parameter.

Signed-off-by: Ricardo Ribalda Delgado <ricardo.ribalda@gmail.com>
Signed-off-by: Felipe Balbi <balbi@ti.com>

diff --git a/drivers/usb/gadget/udc/net2280.c b/drivers/usb/gadget/udc/net2280.c
index fc5c0a2258149ff6b1a1dde2e719d8c881fdfc26..230b8d45f3e6a12bec3bc5044651f13d96cddc02 100644 (file)
--- a/drivers/usb/gadget/udc/net2280.c
+++ b/drivers/usb/gadget/udc/net2280.c
@@ -80,18 +80,6 @@ static const char *const ep_name[] = {
        "ep-e", "ep-f", "ep-g", "ep-h",
 };
 
-/* use_dma -- general goodness, fewer interrupts, less cpu load (vs PIO)
- *
- * The net2280 DMA engines are not tightly integrated with their FIFOs;
- * not all cases are (yet) handled well in this driver or the silicon.
- * Some gadget drivers work better with the dma support here than others.
- * These two parameters let you use PIO or more aggressive DMA.
- */
-static bool use_dma = true;
-
-/* "modprobe net2280 use_dma=n" etc */
-module_param(use_dma, bool, 0444);
-
 /* mode 0 == ep-{a,b,c,d} 1K fifo each
  * mode 1 == ep-{a,b} 2K fifo each, ep-{c,d} unavailable
  * mode 2 == ep-a 2K fifo, ep-{b,c} 1K each, ep-d unavailable
@@ -455,7 +443,7 @@ static int net2280_disable(struct usb_ep *_ep)
        /* synch memory views with the device */
        (void)readl(&ep->cfg->ep_cfg);
 
-       if (use_dma && !ep->dma && ep->num >= 1 && ep->num <= 4)
+       if (!ep->dma && ep->num >= 1 && ep->num <= 4)
                ep->dma = &ep->dev->dma[ep->num - 1];
 
        spin_unlock_irqrestore(&ep->dev->lock, flags);
@@ -1508,12 +1496,11 @@ static ssize_t registers_show(struct device *_dev,
 
        /* Main Control Registers */
        t = scnprintf(next, size, "%s version " DRIVER_VERSION
-                       ", chiprev %04x, dma %s\n\n"
+                       ", chiprev %04x\n\n"
                        "devinit %03x fifoctl %08x gadget '%s'\n"
                        "pci irqenb0 %02x irqenb1 %08x "
                        "irqstat0 %04x irqstat1 %08x\n",
                        driver_name, dev->chiprev,
-                       use_dma ? "enabled" : "disabled",
                        readl(&dev->regs->devinit),
                        readl(&dev->regs->fifoctl),
                        s,
@@ -1995,10 +1982,6 @@ static void usb_reset(struct net2280 *dev)
 static void usb_reinit_228x(struct net2280 *dev)
 {
        u32     tmp;
-       int     init_dma;
-
-       /* use_dma changes are ignored till next device re-init */
-       init_dma = use_dma;
 
        /* basic endpoint init */
        for (tmp = 0; tmp < 7; tmp++) {
@@ -2010,8 +1993,7 @@ static void usb_reinit_228x(struct net2280 *dev)
 
                if (tmp > 0 && tmp <= 4) {
                        ep->fifo_size = 1024;
-                       if (init_dma)
-                               ep->dma = &dev->dma[tmp - 1];
+                       ep->dma = &dev->dma[tmp - 1];
                } else
                        ep->fifo_size = 64;
                ep->regs = &dev->epregs[tmp];
@@ -2035,7 +2017,6 @@ static void usb_reinit_228x(struct net2280 *dev)
 
 static void usb_reinit_338x(struct net2280 *dev)
 {
-       int init_dma;
        int i;
        u32 tmp, val;
        u32 fsmvalue;
@@ -2043,9 +2024,6 @@ static void usb_reinit_338x(struct net2280 *dev)
        static const u32 ep_reg_addr[9] = { 0x00, 0xC0, 0x00, 0xC0, 0x00,
                                                0x00, 0xC0, 0x00, 0xC0 };
 
-       /* use_dma changes are ignored till next device re-init */
-       init_dma = use_dma;
-
        /* basic endpoint init */
        for (i = 0; i < dev->n_ep; i++) {
                struct net2280_ep *ep = &dev->ep[i];
@@ -2054,7 +2032,7 @@ static void usb_reinit_338x(struct net2280 *dev)
                ep->dev = dev;
                ep->num = i;
 
-               if (i > 0 && i <= 4 && init_dma)
+               if (i > 0 && i <= 4)
                        ep->dma = &dev->dma[i - 1];
 
                if (dev->enhanced_mode) {
@@ -3606,8 +3584,7 @@ static int net2280_probe(struct pci_dev *pdev, const struct pci_device_id *id)
        ep_info(dev, "%s\n", driver_desc);
        ep_info(dev, "irq %d, pci mem %p, chip rev %04x\n",
                        pdev->irq, base, dev->chiprev);
-       ep_info(dev, "version: " DRIVER_VERSION "; dma %s %s\n",
-               use_dma ? "enabled" : "disabled",
+       ep_info(dev, "version: " DRIVER_VERSION "; %s\n",
                dev->enhanced_mode ? "enhanced mode" : "legacy mode");
        retval = device_create_file(&pdev->dev, &dev_attr_registers);
        if (retval)