On Fri, Aug 23, 2013 at 12:30:56PM +0300, Dan Carpenter wrote: > The test here should be ">=" instead of ">". The cdd->chan_busy[] array > has "ALLOC_DECS_NUM" elements. > > Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> This needs to go thru USB tree Acked-by Vinod Koul <vinod.koul@xxxxxxxxx> ~Vinod > > diff --git a/drivers/dma/cppi41.c b/drivers/dma/cppi41.c > index 5dcebca..7f6524c 100644 > --- a/drivers/dma/cppi41.c > +++ b/drivers/dma/cppi41.c > @@ -241,7 +241,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; -- -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
