[8/9,v4] ASoC: fsi: remove slave_id settings for DMAEngine

Commit Message

Kuninori Morimoto Jan. 29, 2015, 1:03 a.m. UTC

From: Kuninori Morimoto <[e-mail]>

Current fsi sets dma_slave_config :: slave_id field for DMAEngine,
but it is no longer needed. Let's remove it.

Signed-off-by: Kuninori Morimoto <[e-mail]>
---
v3 -> v4

 - no change

 sound/soc/sh/fsi.c |    3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

Comments

Mark Brown Jan. 29, 2015, 11:35 a.m. UTC | #1

On Thu, Jan 29, 2015 at 01:03:42AM +0000, Kuninori Morimoto wrote:
> From: Kuninori Morimoto <[e-mail]>
> 
> Current fsi sets dma_slave_config :: slave_id field for DMAEngine,
> but it is no longer needed. Let's remove it.

Acked-by: Mark Brown <[e-mail]>

Patch

diff --git a/sound/soc/sh/fsi.c b/sound/soc/sh/fsi.c
index 8869971..c4eb234 100644
--- a/sound/soc/sh/fsi.c
+++ b/sound/soc/sh/fsi.c
@@ -1374,10 +1374,9 @@  static int fsi_dma_probe(struct fsi_priv *fsi, struct fsi_stream *io, struct dev
 				shdma_chan_filter, (void *)io->dma_id,
 				dev, is_play ? "tx" : "rx");
 	if (io->chan) {
-		struct dma_slave_config cfg;
+		struct dma_slave_config cfg = {};
 		int ret;
 
-		cfg.slave_id	= io->dma_id;
 		cfg.dst_addr	= 0; /* use default addr */
 		cfg.src_addr	= 0; /* use default addr */
 		cfg.direction	= is_play ? DMA_MEM_TO_DEV : DMA_DEV_TO_MEM;