Commit f33e7bb3 authored by Sameer Pujar's avatar Sameer Pujar Committed by Vinod Koul

dmaengine: tegra210-adma: restore channel status

Status of ADMA channel registers is not saved and restored during system
suspend. During active playback if system enters suspend, this results in
wrong state of channel registers during system resume and playback fails
to resume properly. Fix this by saving following channel registers in
runtime suspend and restore during runtime resume.
 * ADMA_CH_LOWER_SRC_ADDR
 * ADMA_CH_LOWER_TRG_ADDR
 * ADMA_CH_FIFO_CTRL
 * ADMA_CH_CONFIG
 * ADMA_CH_CTRL
 * ADMA_CH_CMD
 * ADMA_CH_TC
Runtime PM calls will be inovked during system resume path if a playback
or capture needs to be resumed. Hence above changes work fine for system
suspend case.

Fixes: f46b1957 ("dmaengine: tegra-adma: Add support for Tegra210 ADMA")
Signed-off-by: default avatarSameer Pujar <spujar@nvidia.com>
Reviewed-by: default avatarJon Hunter <jonathanh@nvidia.com>
Signed-off-by: default avatarVinod Koul <vkoul@kernel.org>
parent f030e419
...@@ -112,6 +112,7 @@ struct tegra_adma_chan_regs { ...@@ -112,6 +112,7 @@ struct tegra_adma_chan_regs {
unsigned int src_addr; unsigned int src_addr;
unsigned int trg_addr; unsigned int trg_addr;
unsigned int fifo_ctrl; unsigned int fifo_ctrl;
unsigned int cmd;
unsigned int tc; unsigned int tc;
}; };
...@@ -141,6 +142,7 @@ struct tegra_adma_chan { ...@@ -141,6 +142,7 @@ struct tegra_adma_chan {
enum dma_transfer_direction sreq_dir; enum dma_transfer_direction sreq_dir;
unsigned int sreq_index; unsigned int sreq_index;
bool sreq_reserved; bool sreq_reserved;
struct tegra_adma_chan_regs ch_regs;
/* Transfer count and position info */ /* Transfer count and position info */
unsigned int tx_buf_count; unsigned int tx_buf_count;
...@@ -711,8 +713,30 @@ static struct dma_chan *tegra_dma_of_xlate(struct of_phandle_args *dma_spec, ...@@ -711,8 +713,30 @@ static struct dma_chan *tegra_dma_of_xlate(struct of_phandle_args *dma_spec,
static int tegra_adma_runtime_suspend(struct device *dev) static int tegra_adma_runtime_suspend(struct device *dev)
{ {
struct tegra_adma *tdma = dev_get_drvdata(dev); struct tegra_adma *tdma = dev_get_drvdata(dev);
struct tegra_adma_chan_regs *ch_reg;
struct tegra_adma_chan *tdc;
int i;
tdma->global_cmd = tdma_read(tdma, ADMA_GLOBAL_CMD); tdma->global_cmd = tdma_read(tdma, ADMA_GLOBAL_CMD);
if (!tdma->global_cmd)
goto clk_disable;
for (i = 0; i < tdma->nr_channels; i++) {
tdc = &tdma->channels[i];
ch_reg = &tdc->ch_regs;
ch_reg->cmd = tdma_ch_read(tdc, ADMA_CH_CMD);
/* skip if channel is not active */
if (!ch_reg->cmd)
continue;
ch_reg->tc = tdma_ch_read(tdc, ADMA_CH_TC);
ch_reg->src_addr = tdma_ch_read(tdc, ADMA_CH_LOWER_SRC_ADDR);
ch_reg->trg_addr = tdma_ch_read(tdc, ADMA_CH_LOWER_TRG_ADDR);
ch_reg->ctrl = tdma_ch_read(tdc, ADMA_CH_CTRL);
ch_reg->fifo_ctrl = tdma_ch_read(tdc, ADMA_CH_FIFO_CTRL);
ch_reg->config = tdma_ch_read(tdc, ADMA_CH_CONFIG);
}
clk_disable:
clk_disable_unprepare(tdma->ahub_clk); clk_disable_unprepare(tdma->ahub_clk);
return 0; return 0;
...@@ -721,7 +745,9 @@ static int tegra_adma_runtime_suspend(struct device *dev) ...@@ -721,7 +745,9 @@ static int tegra_adma_runtime_suspend(struct device *dev)
static int tegra_adma_runtime_resume(struct device *dev) static int tegra_adma_runtime_resume(struct device *dev)
{ {
struct tegra_adma *tdma = dev_get_drvdata(dev); struct tegra_adma *tdma = dev_get_drvdata(dev);
int ret; struct tegra_adma_chan_regs *ch_reg;
struct tegra_adma_chan *tdc;
int ret, i;
ret = clk_prepare_enable(tdma->ahub_clk); ret = clk_prepare_enable(tdma->ahub_clk);
if (ret) { if (ret) {
...@@ -730,6 +756,24 @@ static int tegra_adma_runtime_resume(struct device *dev) ...@@ -730,6 +756,24 @@ static int tegra_adma_runtime_resume(struct device *dev)
} }
tdma_write(tdma, ADMA_GLOBAL_CMD, tdma->global_cmd); tdma_write(tdma, ADMA_GLOBAL_CMD, tdma->global_cmd);
if (!tdma->global_cmd)
return 0;
for (i = 0; i < tdma->nr_channels; i++) {
tdc = &tdma->channels[i];
ch_reg = &tdc->ch_regs;
/* skip if channel was not active earlier */
if (!ch_reg->cmd)
continue;
tdma_ch_write(tdc, ADMA_CH_TC, ch_reg->tc);
tdma_ch_write(tdc, ADMA_CH_LOWER_SRC_ADDR, ch_reg->src_addr);
tdma_ch_write(tdc, ADMA_CH_LOWER_TRG_ADDR, ch_reg->trg_addr);
tdma_ch_write(tdc, ADMA_CH_CTRL, ch_reg->ctrl);
tdma_ch_write(tdc, ADMA_CH_FIFO_CTRL, ch_reg->fifo_ctrl);
tdma_ch_write(tdc, ADMA_CH_CONFIG, ch_reg->config);
tdma_ch_write(tdc, ADMA_CH_CMD, ch_reg->cmd);
}
return 0; return 0;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment