dmaengine: kill enum dma_state_client

DMA_NAK is now useless.  We can just use a bool instead.

Reviewed-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Dan Williams <dan.j.williams@intel.com>

3 files changed, 11 insertions, 17 deletions

diff --git a/drivers/dma/dmaengine.c b/drivers/dma/dmaengine.c
index b245c38dbec3..cdc8ecfc2c2c 100644
--- a/drivers/dma/dmaengine.c
+++ b/drivers/dma/dmaengine.c
@@ -440,7 +440,7 @@ struct dma_chan *__dma_request_channel(dma_cap_mask_t *mask, dma_filter_fn fn, v
 {
         struct dma_device *device, *_d;
         struct dma_chan *chan = NULL;
-        enum dma_state_client ack;
+        bool ack;
         int err;
 
         /* Find a channel */
@@ -453,9 +453,9 @@ struct dma_chan *__dma_request_channel(dma_cap_mask_t *mask, dma_filter_fn fn, v
                 if (fn)
                         ack = fn(chan, fn_param);
                 else
-                        ack = DMA_ACK;
+                        ack = true;
 
-                if (ack == DMA_ACK) {
+                if (ack) {
                         /* Found a suitable channel, try to grab, prep, and
                          * return it.  We first set DMA_PRIVATE to disable
                          * balance_ref_count as this channel will not be
@@ -473,15 +473,9 @@ struct dma_chan *__dma_request_channel(dma_cap_mask_t *mask, dma_filter_fn fn, v
                                        dev_name(&chan->dev), err);
                         else
                                 break;
-                } else if (ack == DMA_DUP) {
-                        pr_debug("%s: %s filter said DMA_DUP\n",
-                                 __func__, dev_name(&chan->dev));
-                } else if (ack == DMA_NAK) {
-                        pr_debug("%s: %s filter said DMA_NAK\n",
-                                 __func__, dev_name(&chan->dev));
-                        break;
                 } else
-                        WARN_ONCE(1, "filter_fn: unknown response?\n");
+                        pr_debug("%s: %s filter said false\n",
+                                 __func__, dev_name(&chan->dev));
                 chan = NULL;
         }
         mutex_unlock(&dma_list_mutex);
diff --git a/drivers/dma/dmatest.c b/drivers/dma/dmatest.c
index 1d6e48f9cd02..c77d47c4ec5b 100644
--- a/drivers/dma/dmatest.c
+++ b/drivers/dma/dmatest.c
@@ -363,12 +363,12 @@ static int dmatest_add_channel(struct dma_chan *chan)
         return 0;
 }
 
-static enum dma_state_client filter(struct dma_chan *chan, void *param)
+static bool filter(struct dma_chan *chan, void *param)
 {
         if (!dmatest_match_channel(chan) || !dmatest_match_device(chan->device))
-                return DMA_DUP;
+                return false;
         else
-                return DMA_ACK;
+                return true;
 }
 
 static int __init dmatest_init(void)
diff --git a/drivers/mmc/host/atmel-mci.c b/drivers/mmc/host/atmel-mci.c
index 4b567a0408e1..b0042d06eaf7 100644
--- a/drivers/mmc/host/atmel-mci.c
+++ b/drivers/mmc/host/atmel-mci.c
@@ -1544,14 +1544,14 @@ static void __exit atmci_cleanup_slot(struct atmel_mci_slot *slot,
 }
 
 #ifdef CONFIG_MMC_ATMELMCI_DMA
-static enum dma_state_client filter(struct dma_chan *chan, void *slave)
+static bool filter(struct dma_chan *chan, void *slave)
 {
         struct dw_dma_slave *dws = slave;
 
         if (dws->dma_dev == chan->device->dev)
-                return DMA_ACK;
+                return true;
         else
-                return DMA_DUP;
+                return false;
 }
 #endif