From: Lars-Peter Clausen <lars@metafoo.de>
Date: Sun, 6 Jul 2014 18:32:28 +0000 (+0200)
Subject: dmaengine: pl330: Simplify is_manager()
X-Git-Tag: v5.15~17417^2~50
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=fbbcd9be96a0295e9d127e124f72fa0039f53d8e;p=platform%2Fkernel%2Flinux-starfive.git

dmaengine: pl330: Simplify is_manager()

Since we keep a pointer to the manager thread it is fairly easy to check if a
thread is the manager thread.

Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
Signed-off-by: Vinod Koul <vinod.koul@intel.com>
---

diff --git a/drivers/dma/pl330.c b/drivers/dma/pl330.c
index 0209823..c5eeb64 100644
--- a/drivers/dma/pl330.c
+++ b/drivers/dma/pl330.c
@@ -562,13 +562,7 @@ static inline bool _queue_full(struct pl330_thread *thrd)
 
 static inline bool is_manager(struct pl330_thread *thrd)
 {
-	struct pl330_dmac *pl330 = thrd->dmac;
-
-	/* MANAGER is indexed at the end */
-	if (thrd->id == pl330->pinfo->pcfg.num_chan)
-		return true;
-	else
-		return false;
+	return thrd->dmac->manager == thrd;
 }
 
 /* If manager of the thread is in Non-Secure mode */