diff options
| author | Krzysztof Kozlowski <k.kozlowski@samsung.com> | 2015-06-15 17:25:16 +0900 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2015-08-16 20:52:11 -0700 |
| commit | 9cd4b78cd56489755fcba685b5e1a243cea6e6cf (patch) | |
| tree | 63e2e25339f989f6b44ad883aeccf0fec3593dc2 /drivers/dma | |
| parent | 4e200fcacc37eebe93acfc3d751b6b7b79bf85a1 (diff) | |
dmaengine: pl330: Fix overflow when reporting residue in memcpy
commit ae128293d97404f491dc76f1843c7adacfec3441 upstream.
During memcpy operations the residue was always set to an u32 overflowed
value.
In pl330_tx_status() function number of currently transferred bytes was
subtracted from internal "bytes_requested" field. However this
"bytes_requested" was not initialized at start to length of memcpy
buffer so transferred bytes were subtracted from 0 causing overflow.
Signed-off-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
Fixes: aee4d1fac887 ("dmaengine: pl330: improve pl330_tx_status() function")
Signed-off-by: Vinod Koul <vinod.koul@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/dma')
| -rw-r--r-- | drivers/dma/pl330.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/dma/pl330.c b/drivers/dma/pl330.c index 340f9e607cd8..8652388e77d6 100644 --- a/drivers/dma/pl330.c +++ b/drivers/dma/pl330.c @@ -2621,6 +2621,7 @@ pl330_prep_dma_memcpy(struct dma_chan *chan, dma_addr_t dst, desc->rqcfg.brst_len = 1; desc->rqcfg.brst_len = get_burst_len(desc, len); + desc->bytes_requested = len; desc->txd.flags = flags; |