dmaengine: cleanup with list_first_entry_or_null()
authorMasahiro Yamada <yamada.masahiro@socionext.com>
Mon, 12 Sep 2016 18:08:17 +0000 (03:08 +0900)
committerVinod Koul <vinod.koul@intel.com>
Wed, 14 Sep 2016 13:40:46 +0000 (19:10 +0530)
The combo of list_empty() check and return list_first_entry()
can be replaced with list_first_entry_or_null().

Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
Acked-by: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: Vinod Koul <vinod.koul@intel.com>
drivers/dma/coh901318.c
drivers/dma/ep93xx_dma.c
drivers/dma/ste_dma40.c
drivers/dma/virt-dma.h

index e4acd63e42aa37a9048b91a69f8351b444acc498..a373ecacfabad8cb31c947ef22a2acfb5db01856 100644 (file)
@@ -1553,15 +1553,8 @@ coh901318_desc_submit(struct coh901318_chan *cohc, struct coh901318_desc *desc)
 static struct coh901318_desc *
 coh901318_first_active_get(struct coh901318_chan *cohc)
 {
-       struct coh901318_desc *d;
-
-       if (list_empty(&cohc->active))
-               return NULL;
-
-       d = list_first_entry(&cohc->active,
-                            struct coh901318_desc,
-                            node);
-       return d;
+       return list_first_entry_or_null(&cohc->active, struct coh901318_desc,
+                                       node);
 }
 
 static void
@@ -1579,15 +1572,8 @@ coh901318_desc_queue(struct coh901318_chan *cohc, struct coh901318_desc *desc)
 static struct coh901318_desc *
 coh901318_first_queued(struct coh901318_chan *cohc)
 {
-       struct coh901318_desc *d;
-
-       if (list_empty(&cohc->queue))
-               return NULL;
-
-       d = list_first_entry(&cohc->queue,
-                            struct coh901318_desc,
-                            node);
-       return d;
+       return list_first_entry_or_null(&cohc->queue, struct coh901318_desc,
+                                       node);
 }
 
 static inline u32 coh901318_get_bytes_in_lli(struct coh901318_lli *in_lli)
index 21f08cc3352b97fbf047cd8661a2191e1988e0f6..2ffaca25267e8f88f59e55eab3ecb3b16d0ad008 100644 (file)
@@ -262,10 +262,8 @@ static void ep93xx_dma_set_active(struct ep93xx_dma_chan *edmac,
 static struct ep93xx_dma_desc *
 ep93xx_dma_get_active(struct ep93xx_dma_chan *edmac)
 {
-       if (list_empty(&edmac->active))
-               return NULL;
-
-       return list_first_entry(&edmac->active, struct ep93xx_dma_desc, node);
+       return list_first_entry_or_null(&edmac->active,
+                                       struct ep93xx_dma_desc, node);
 }
 
 /**
index 8b18e44a02d51438809216a6d1b16e002d60fbde..e43d2bbfd122dd2cc24ac7221aa492ae59241fbb 100644 (file)
@@ -941,15 +941,7 @@ static void d40_desc_load(struct d40_chan *d40c, struct d40_desc *d40d)
 
 static struct d40_desc *d40_first_active_get(struct d40_chan *d40c)
 {
-       struct d40_desc *d;
-
-       if (list_empty(&d40c->active))
-               return NULL;
-
-       d = list_first_entry(&d40c->active,
-                            struct d40_desc,
-                            node);
-       return d;
+       return list_first_entry_or_null(&d40c->active, struct d40_desc, node);
 }
 
 /* remove desc from current queue and add it to the pending_queue */
@@ -962,36 +954,18 @@ static void d40_desc_queue(struct d40_chan *d40c, struct d40_desc *desc)
 
 static struct d40_desc *d40_first_pending(struct d40_chan *d40c)
 {
-       struct d40_desc *d;
-
-       if (list_empty(&d40c->pending_queue))
-               return NULL;
-
-       d = list_first_entry(&d40c->pending_queue,
-                            struct d40_desc,
-                            node);
-       return d;
+       return list_first_entry_or_null(&d40c->pending_queue, struct d40_desc,
+                                       node);
 }
 
 static struct d40_desc *d40_first_queued(struct d40_chan *d40c)
 {
-       struct d40_desc *d;
-
-       if (list_empty(&d40c->queue))
-               return NULL;
-
-       d = list_first_entry(&d40c->queue,
-                            struct d40_desc,
-                            node);
-       return d;
+       return list_first_entry_or_null(&d40c->queue, struct d40_desc, node);
 }
 
 static struct d40_desc *d40_first_done(struct d40_chan *d40c)
 {
-       if (list_empty(&d40c->done))
-               return NULL;
-
-       return list_first_entry(&d40c->done, struct d40_desc, node);
+       return list_first_entry_or_null(&d40c->done, struct d40_desc, node);
 }
 
 static int d40_psize_2_burst_size(bool is_log, int psize)
index d9731ca5e2628a0c1b406f3ee3062e970d6a515f..a030ae7b1df2eb830e1fb03debdbb78dba0c7002 100644 (file)
@@ -123,10 +123,8 @@ static inline void vchan_cyclic_callback(struct virt_dma_desc *vd)
  */
 static inline struct virt_dma_desc *vchan_next_desc(struct virt_dma_chan *vc)
 {
-       if (list_empty(&vc->desc_issued))
-               return NULL;
-
-       return list_first_entry(&vc->desc_issued, struct virt_dma_desc, node);
+       return list_first_entry_or_null(&vc->desc_issued,
+                                       struct virt_dma_desc, node);
 }
 
 /**