projects
/
platform
/
adaptation
/
renesas_rcar
/
renesas_kernel.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
| inline |
side by side
(parent:
ff49896
)
dma: cppi41: off by one in desc_to_chan()
author
Dan Carpenter
<dan.carpenter@oracle.com>
Wed, 28 Aug 2013 10:48:44 +0000
(13:48 +0300)
committer
Greg Kroah-Hartman
<gregkh@linuxfoundation.org>
Wed, 28 Aug 2013 22:46:33 +0000
(15:46 -0700)
The test here should be ">=" instead of ">". The cdd->chan_busy[] array
has "ALLOC_DECS_NUM" elements.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Vinod Koul <vinod.koul@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/dma/cppi41.c
patch
|
blob
|
history
diff --git
a/drivers/dma/cppi41.c
b/drivers/dma/cppi41.c
index
6c4e64f
..
7c82b92
100644
(file)
--- a/
drivers/dma/cppi41.c
+++ b/
drivers/dma/cppi41.c
@@
-242,7
+242,7
@@
static struct cppi41_channel *desc_to_chan(struct cppi41_dd *cdd, u32 desc)
}
desc_num = (desc - cdd->descs_phys) / sizeof(struct cppi41_desc);
- BUG_ON(desc_num > ALLOC_DECS_NUM);
+ BUG_ON(desc_num >
=
ALLOC_DECS_NUM);
c = cdd->chan_busy[desc_num];
cdd->chan_busy[desc_num] = NULL;
return c;