summaryrefslogtreecommitdiffstats
path: root/drivers/dma
diff options
context:
space:
mode:
authorLars-Peter Clausen <lars@metafoo.de>2014-07-06 20:32:27 +0200
committerVinod Koul <vinod.koul@intel.com>2014-07-15 22:01:03 +0530
commit1b446d2a61c436f7ad542fcdb212d4e51845ac28 (patch)
tree40c63dc47043054c158426deebb37294ac507457 /drivers/dma
parent65ad60600f5a3bf4c4edfbb95992af6ff2dfc18a (diff)
downloadop-kernel-dev-1b446d2a61c436f7ad542fcdb212d4e51845ac28.zip
op-kernel-dev-1b446d2a61c436f7ad542fcdb212d4e51845ac28.tar.gz
dmaengine: pl330: Remove uneccessary ccr validation
We know that we do not create invalid ccr settings in this driver. There is no need to validate them. Signed-off-by: Lars-Peter Clausen <lars@metafoo.de> Signed-off-by: Vinod Koul <vinod.koul@intel.com>
Diffstat (limited to 'drivers/dma')
-rw-r--r--drivers/dma/pl330.c23
1 files changed, 0 insertions, 23 deletions
diff --git a/drivers/dma/pl330.c b/drivers/dma/pl330.c
index f0e9f47..0209823 100644
--- a/drivers/dma/pl330.c
+++ b/drivers/dma/pl330.c
@@ -1420,21 +1420,6 @@ static inline u32 _prepare_ccr(const struct pl330_reqcfg *rqc)
return ccr;
}
-static inline bool _is_valid(u32 ccr)
-{
- enum pl330_cachectrl dcctl;
- enum pl330_cachectrl scctl;
-
- dcctl = (ccr >> CC_DSTCCTRL_SHFT) & CC_DRCCCTRL_MASK;
- scctl = (ccr >> CC_SRCCCTRL_SHFT) & CC_SRCCCTRL_MASK;
-
- if (dcctl == INVALID1 || dcctl == INVALID2
- || scctl == INVALID1 || scctl == INVALID2)
- return false;
- else
- return true;
-}
-
/*
* Submit a list of xfers after which the client wants notification.
* Client is not notified after each xfer unit, just once after all
@@ -1495,14 +1480,6 @@ static int pl330_submit_req(struct pl330_thread *thrd, struct pl330_req *r)
ccr = readl(regs + CC(thrd->id));
}
- /* If this req doesn't have valid xfer settings */
- if (!_is_valid(ccr)) {
- ret = -EINVAL;
- dev_info(thrd->dmac->pinfo->dev, "%s:%d Invalid CCR(%x)!\n",
- __func__, __LINE__, ccr);
- goto xfer_exit;
- }
-
idx = IS_FREE(&thrd->req[0]) ? 0 : 1;
xs.ccr = ccr;
OpenPOWER on IntegriCloud