dmaengine: k3dma: Off by one in k3_of_dma_simple_xlate()
authorDan Carpenter <dan.carpenter@oracle.com>
Fri, 22 Jun 2018 11:15:47 +0000 (14:15 +0300)
committerVinod Koul <vkoul@kernel.org>
Thu, 28 Jun 2018 06:27:21 +0000 (11:57 +0530)
The d->chans[] array has d->dma_requests elements so the > should be
>= here.

Fixes: 8e6152bc660e ("dmaengine: Add hisilicon k3 DMA engine driver")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Vinod Koul <vkoul@kernel.org>
drivers/dma/k3dma.c

index fa31cccbe04faf5fa6a8adb07abf6b48a4a6cdd2..6bfa217ed6d0de81d1ef51b37accdba9ae4cd04b 100644 (file)
@@ -794,7 +794,7 @@ static struct dma_chan *k3_of_dma_simple_xlate(struct of_phandle_args *dma_spec,
        struct k3_dma_dev *d = ofdma->of_dma_data;
        unsigned int request = dma_spec->args[0];
 
-       if (request > d->dma_requests)
+       if (request >= d->dma_requests)
                return NULL;
 
        return dma_get_slave_channel(&(d->chans[request].vc.chan));