aboutsummaryrefslogtreecommitdiffstats
path: root/sound/soc/intel
diff options
context:
space:
mode:
authorBen Zhang <benzh@chromium.org>2016-07-07 21:54:56 -0400
committerMark Brown <broonie@kernel.org>2016-07-14 12:30:03 -0400
commitc999675b04c146aa57f6e853a3746de979427fad (patch)
tree6656530b5226d5ef8b6bc7101f0cd49321f77ce9 /sound/soc/intel
parent9dc201cf75e04fd2d753a1d01ed9d03f4253e32f (diff)
ASoC: Intel: Fix conflicting pcm dev drvdata on haswell
soc-core sets the snd_soc_pcm_runtime->dev drvdata to snd_soc_pcm_runtime in soc_post_component_init, and access it in places like codec_reg_show. hsw_pcm_open overwrites the drvdata to point to hsw_pcm_data, confusing soc-core, and causing crashes when cat /sys/devices/pci0000:00/INT3438:00/.../System PCM/codec_reg This patch removes the set in hsw_pcm_open since it's no longer used. commit 7ff9d6714a5c ("ASoC: Intel: Split hsw_pcm_data for playback and capture") already removed all calls to snd_soc_pcm_get_drvdata(rtd). Signed-off-by: Ben Zhang <benzh@chromium.org> Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'sound/soc/intel')
-rw-r--r--sound/soc/intel/haswell/sst-haswell-pcm.c1
1 files changed, 0 insertions, 1 deletions
diff --git a/sound/soc/intel/haswell/sst-haswell-pcm.c b/sound/soc/intel/haswell/sst-haswell-pcm.c
index 994256b39b9c..3154525c2b83 100644
--- a/sound/soc/intel/haswell/sst-haswell-pcm.c
+++ b/sound/soc/intel/haswell/sst-haswell-pcm.c
@@ -819,7 +819,6 @@ static int hsw_pcm_open(struct snd_pcm_substream *substream)
819 mutex_lock(&pcm_data->mutex); 819 mutex_lock(&pcm_data->mutex);
820 pm_runtime_get_sync(pdata->dev); 820 pm_runtime_get_sync(pdata->dev);
821 821
822 snd_soc_pcm_set_drvdata(rtd, pcm_data);
823 pcm_data->substream = substream; 822 pcm_data->substream = substream;
824 823
825 snd_soc_set_runtime_hwparams(substream, &hsw_pcm_hardware); 824 snd_soc_set_runtime_hwparams(substream, &hsw_pcm_hardware);