| From c43f1508686e8e4746012bf87995085eeb0f5307 Mon Sep 17 00:00:00 2001 |
| From: Tomoya MORINAGA <tomoya-linux@dsn.lapis-semi.com> |
| Date: Tue, 11 Oct 2011 21:43:21 +0900 |
| Subject: pch_dma: Fix suspend issue |
| |
| From: Tomoya MORINAGA <tomoya-linux@dsn.lapis-semi.com> |
| |
| commit c43f1508686e8e4746012bf87995085eeb0f5307 upstream. |
| |
| Currently, executing suspend/hibernation, |
| memory access violation occurs. |
| |
| In pch_dma_save_regs() called by suspend(), |
| you can see the following code. |
| |
| static void pch_dma_save_regs(struct pch_dma *pd) |
| { |
| snip... |
| list_for_each_entry_safe(chan, _c, &pd->dma.channels, device_node) { |
| pd_chan = to_pd_chan(chan); |
| |
| pd->ch_regs[i].dev_addr = channel_readl(pd_chan, DEV_ADDR); |
| pd->ch_regs[i].mem_addr = channel_readl(pd_chan, MEM_ADDR); |
| pd->ch_regs[i].size = channel_readl(pd_chan, SIZE); |
| pd->ch_regs[i].next = channel_readl(pd_chan, NEXT); |
| |
| i++; |
| } |
| } |
| |
| Max loop count is 12 defined at pci_table. |
| So, this caused memory access violation. |
| |
| This patch fixes the issue |
| - Modify array size (MAX_CHAN_NR) |
| |
| Signed-off-by: Tomoya MORINAGA <tomoya-linux@dsn.lapis-semi.com> |
| Signed-off-by: Vinod Koul <vinod.koul@linux.intel.com> |
| Signed-off-by: Tomoya MORINAGA <tomoya.rohm@gmail.com> |
| Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> |
| |
| --- |
| drivers/dma/pch_dma.c | 2 +- |
| 1 file changed, 1 insertion(+), 1 deletion(-) |
| |
| --- a/drivers/dma/pch_dma.c |
| +++ b/drivers/dma/pch_dma.c |
| @@ -60,7 +60,7 @@ |
| #define DMA_DESC_FOLLOW_WITHOUT_IRQ 0x2 |
| #define DMA_DESC_FOLLOW_WITH_IRQ 0x3 |
| |
| -#define MAX_CHAN_NR 8 |
| +#define MAX_CHAN_NR 12 |
| |
| #define DMA_MASK_CTL0_MODE 0x33333333 |
| #define DMA_MASK_CTL2_MODE 0x00003333 |