diff options
| author |   Ben Dooks <ben.dooks@codethink.co.uk> | 2018-11-21 16:13:20 +0000 |
|---|---|---|
| committer |   Vinod Koul <vkoul@kernel.org> | 2019-01-07 09:52:41 +0530 |
| commit | 216a1d7da0660dd7d5f79c48e117de8d8d2da468 (patch) | |
| tree | 1eae26fd0d4431d535461e35512e077ca5d3cb5c /drivers/dma/tegra20-apb-dma.c | |
| parent | dmaengine: tegra: avoid overflow of byte tracking (diff) | |
| download | linux-dev-216a1d7da0660dd7d5f79c48e117de8d8d2da468.tar.xz linux-dev-216a1d7da0660dd7d5f79c48e117de8d8d2da468.zip | |
dmaengine: tegra: make byte counters unsigned int
The buffer byte request length and counter are declared as signed integers
but the values should never be below zero, so make these unsigned integers
instead.
Reviewed-by: Dmitry Osipenko <digetx@gmail.com>
Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk>
Signed-off-by: Vinod Koul <vkoul@kernel.org>
Diffstat (limited to 'drivers/dma/tegra20-apb-dma.c')
| -rw-r--r-- | drivers/dma/tegra20-apb-dma.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/dma/tegra20-apb-dma.c b/drivers/dma/tegra20-apb-dma.c index 8219ab88a507..adfd918baedc 100644 --- a/drivers/dma/tegra20-apb-dma.c +++ b/drivers/dma/tegra20-apb-dma.c @@ -155,7 +155,7 @@ struct tegra_dma_channel_regs { */ struct tegra_dma_sg_req { struct tegra_dma_channel_regs ch_regs; - int req_len; + unsigned int req_len; bool configured; bool last_sg; struct list_head node; @@ -169,8 +169,8 @@ struct tegra_dma_sg_req { */ struct tegra_dma_desc { struct dma_async_tx_descriptor txd; - int bytes_requested; - int bytes_transferred; + unsigned int bytes_requested; + unsigned int bytes_transferred; enum dma_status dma_status; struct list_head node; struct list_head tx_list; |