ASoC: fsl: drop unneeded snd_soc_dai_set_drvdata
authorJulia Lawall <Julia.Lawall@inria.fr>
Sat, 13 Feb 2021 10:19:07 +0000 (11:19 +0100)
committerMark Brown <broonie@kernel.org>
Wed, 10 Mar 2021 13:07:08 +0000 (13:07 +0000)
snd_soc_dai_set_drvdata is not needed when the set data comes from
snd_soc_dai_get_drvdata or dev_get_drvdata.  The problem was fixed
usingthe following semantic patch: (http://coccinelle.lip6.fr/)

// <smpl>
@@
expression x,y,e;
@@
x = dev_get_drvdata(y->dev)
... when != x = e
- snd_soc_dai_set_drvdata(y,x);

@@
expression x,y,e;
@@
x = snd_soc_dai_get_drvdata(y)
... when != x = e
- snd_soc_dai_set_drvdata(y,x);
// </smpl>

Signed-off-by: Julia Lawall <Julia.Lawall@inria.fr>
Acked-by: Nicolin Chen <nicoleotsuka@gmail.com>
Link: https://lore.kernel.org/r/20210213101907.1318496-5-Julia.Lawall@inria.fr
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/fsl/fsl_micfil.c
sound/soc/fsl/fsl_sai.c
sound/soc/fsl/fsl_xcvr.c

index 5935af2e5ff478adeb9f056e3cbeb8899d3ec8c2..2b9edd4bb94d766431251393d3e951cda175eb11 100644 (file)
@@ -423,8 +423,6 @@ static int fsl_micfil_dai_probe(struct snd_soc_dai *cpu_dai)
                return ret;
        }
 
-       snd_soc_dai_set_drvdata(cpu_dai, micfil);
-
        return 0;
 }
 
index 5e65b456d3e21b3143256d4bf3473a6ae47b5080..8876d0ed37d990f1080c2254aee64cf05834a305 100644 (file)
@@ -727,8 +727,6 @@ static int fsl_sai_dai_probe(struct snd_soc_dai *cpu_dai)
        snd_soc_dai_init_dma_data(cpu_dai, &sai->dma_params_tx,
                                &sai->dma_params_rx);
 
-       snd_soc_dai_set_drvdata(cpu_dai, sai);
-
        return 0;
 }
 
index 6dd0a5fcd4556af99c1a631447db31d1ef930d1d..4f33dbe29c786e3b2b01f723befedc9021792ae4 100644 (file)
@@ -869,7 +869,6 @@ static int fsl_xcvr_dai_probe(struct snd_soc_dai *dai)
        struct fsl_xcvr *xcvr = snd_soc_dai_get_drvdata(dai);
 
        snd_soc_dai_init_dma_data(dai, &xcvr->dma_prms_tx, &xcvr->dma_prms_rx);
-       snd_soc_dai_set_drvdata(dai, xcvr);
 
        snd_soc_add_dai_controls(dai, &fsl_xcvr_mode_kctl, 1);
        snd_soc_add_dai_controls(dai, &fsl_xcvr_arc_mode_kctl, 1);