aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorArnd Bergmann <arnd@arndb.de>2013-03-03 15:51:28 -0500
committerVinod Koul <vinod.koul@intel.com>2013-03-29 19:04:07 -0400
commitf73bb9b35596e045feacdf4d2fd32cfb087e2411 (patch)
tree13f29ee3560d019ae3b7dcec555dc05604f1636d
parent8bb9660418e05bb1845ac1a2428444d78e322cc7 (diff)
dmaengine: dw_dma: fix endianess for DT xlate function
As reported by Wu Fengguang's build robot tracking sparse warnings, the dma_spec arguments in the dw_dma_xlate are already byte swapped on little-endian platforms and must not get swapped again. This code is currently not used anywhere, but will be used in Linux 3.10 when the ARM SPEAr platform starts using the generic DMA DT binding. Signed-off-by: Arnd Bergmann <arnd@arndb.de> Reported-by: Fengguang Wu <fengguang.wu@intel.com> Acked-by: Viresh Kumar <viresh.kumar@linaro.org> Signed-off-by: Vinod Koul <vinod.koul@intel.com>
-rw-r--r--drivers/dma/dw_dmac.c6
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/dma/dw_dmac.c b/drivers/dma/dw_dmac.c
index c599558faeda..eb81ec9d5b91 100644
--- a/drivers/dma/dw_dmac.c
+++ b/drivers/dma/dw_dmac.c
@@ -1276,9 +1276,9 @@ static struct dma_chan *dw_dma_xlate(struct of_phandle_args *dma_spec,
1276 if (dma_spec->args_count != 3) 1276 if (dma_spec->args_count != 3)
1277 return NULL; 1277 return NULL;
1278 1278
1279 fargs.req = be32_to_cpup(dma_spec->args+0); 1279 fargs.req = dma_spec->args[0];
1280 fargs.src = be32_to_cpup(dma_spec->args+1); 1280 fargs.src = dma_spec->args[1];
1281 fargs.dst = be32_to_cpup(dma_spec->args+2); 1281 fargs.dst = dma_spec->args[2];
1282 1282
1283 if (WARN_ON(fargs.req >= DW_DMA_MAX_NR_REQUESTS || 1283 if (WARN_ON(fargs.req >= DW_DMA_MAX_NR_REQUESTS ||
1284 fargs.src >= dw->nr_masters || 1284 fargs.src >= dw->nr_masters ||