diff options
author | Petr Kulhavy <petr@barix.com> | 2015-03-23 16:35:01 -0400 |
---|---|---|
committer | Vinod Koul <vinod.koul@intel.com> | 2015-03-31 23:45:57 -0400 |
commit | ab7add30ce63a7147f045f00c65a44c0f14ad6b2 (patch) | |
tree | 45f8d52a4256b0e1333b301645834bbc37be0626 /drivers/dma/edma.c | |
parent | de6b641e2d23d116dcf6b6a67e2ee4856e3c3026 (diff) |
dmaengine: edma: fixed wrongly initialized data parameter to the edma callback
The "data" parameter passed indirectly to the edma_callback() should be
edma_chan and not the dma_chan.
This bug was so far harmless since the offset of struct dma_chan within struct
edma_chan is 0. However as soon as someone changes struct edma_chan this would
cause troubles.
Signed-off-by: Petr Kulhavy <petr@barix.com>
Acked-by: Peter Ujfalusi <peter.ujfalusi@ti.com>
Signed-off-by: Vinod Koul <vinod.koul@intel.com>
Diffstat (limited to 'drivers/dma/edma.c')
-rw-r--r-- | drivers/dma/edma.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/dma/edma.c b/drivers/dma/edma.c index 276157f22612..0ea813d98998 100644 --- a/drivers/dma/edma.c +++ b/drivers/dma/edma.c | |||
@@ -805,7 +805,7 @@ static int edma_alloc_chan_resources(struct dma_chan *chan) | |||
805 | LIST_HEAD(descs); | 805 | LIST_HEAD(descs); |
806 | 806 | ||
807 | a_ch_num = edma_alloc_channel(echan->ch_num, edma_callback, | 807 | a_ch_num = edma_alloc_channel(echan->ch_num, edma_callback, |
808 | chan, EVENTQ_DEFAULT); | 808 | echan, EVENTQ_DEFAULT); |
809 | 809 | ||
810 | if (a_ch_num < 0) { | 810 | if (a_ch_num < 0) { |
811 | ret = -ENODEV; | 811 | ret = -ENODEV; |