Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 64eae986 authored by Kuninori Morimoto's avatar Kuninori Morimoto Committed by Mark Brown
Browse files

ASoC: rsnd: fixup rsnd_gen_dma_addr() for Gen1



ad32d0c7
(ASoC: rsnd: add rsnd_gen_dma_addr() for DMAC addr)
added rsnd_gen_dma_addr() to calculate DMA addr,
but, it is necessary only for Gen2.
This patch ignores Gen1 case.
Kernel will be panic without this patch.
Special thanks to Simon

Reported-by: default avatarSimon Horman <horms@verge.net.au>
Signed-off-by: default avatarKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Tested-by: default avatarSimon Horman <horms+renesas@verge.net.au>
Signed-off-by: default avatarMark Brown <broonie@linaro.org>
parent 4cf61278
Loading
Loading
Loading
Loading
+21 −12
Original line number Diff line number Diff line
@@ -184,7 +184,7 @@ static int rsnd_gen_regmap_init(struct rsnd_priv *priv,
#define RDMA_CMD_O_N(addr, i)	(addr ##_reg - 0x004f8000 + (0x400 * i))
#define RDMA_CMD_O_P(addr, i)	(addr ##_reg - 0x001f8000 + (0x400 * i))

void rsnd_gen_dma_addr(struct rsnd_priv *priv,
static void rsnd_gen2_dma_addr(struct rsnd_priv *priv,
		       struct rsnd_dma *dma,
		       struct dma_slave_config *cfg,
		       int is_play, int slave_id)
@@ -226,6 +226,24 @@ void rsnd_gen_dma_addr(struct rsnd_priv *priv,
		}
	};

	/* it shouldn't happen */
	if (use_dvc & !use_src) {
		dev_err(dev, "DVC is selected without SRC\n");
		return;
	}

	cfg->src_addr = dma_addrs[is_ssi][is_play][use_src + use_dvc].src_addr;
	cfg->dst_addr = dma_addrs[is_ssi][is_play][use_src + use_dvc].dst_addr;

	dev_dbg(dev, "dma%d addr - src : %x / dst : %x\n",
		id, cfg->src_addr, cfg->dst_addr);
}

void rsnd_gen_dma_addr(struct rsnd_priv *priv,
		       struct rsnd_dma *dma,
		       struct dma_slave_config *cfg,
		       int is_play, int slave_id)
{
	cfg->slave_id   = slave_id;
	cfg->src_addr   = 0;
	cfg->dst_addr   = 0;
@@ -237,18 +255,9 @@ void rsnd_gen_dma_addr(struct rsnd_priv *priv,
	if (rsnd_is_gen1(priv))
		return;

	/* it shouldn't happen */
	if (use_dvc & !use_src) {
		dev_err(dev, "DVC is selected without SRC\n");
		return;
	rsnd_gen2_dma_addr(priv, dma, cfg, is_play, slave_id);
}

	cfg->src_addr = dma_addrs[is_ssi][is_play][use_src + use_dvc].src_addr;
	cfg->dst_addr = dma_addrs[is_ssi][is_play][use_src + use_dvc].dst_addr;

	dev_dbg(dev, "dma%d addr - src : %x / dst : %x\n",
		id, cfg->src_addr, cfg->dst_addr);
}

/*
 *		Gen2