mirror of
https://github.com/team-infusion-developers/android_kernel_samsung_msm8976.git
synced 2024-11-01 02:21:16 +00:00
b63d5e491b
commit fc7dc61d9a87011aaf8a6eb3144ebf9552adf5d2 upstream. Unbalanced calls to snd_imx_pcm_trigger() may result in endless FIQ activity and thus provoke eternal sound. While on the first glance, the switch statement looks pretty symmetric, the SUSPEND/RESUME pair is not: the suspend case comes along snd_pcm_suspend_all(), which for fsl/imx-pcm-fiq is called only at snd_soc_suspend(), but the resume case originates straight from the SNDRV_PCM_IOCTL_RESUME. This way userland may provoke an unbalanced resume, which might cause the fiq_enable counter to increase and never return to zero again, so eventually imx_pcm_fiq is never disabled. Simply removing the fiq_enable will solve the problem, as long as one never goes play and capture game simultaneously, but beware trying both at once, the early TRIGGER_STOP will cut off the other activity prematurely. So now playing and capturing is scrutinized separately, instead of by counting. Signed-off-by: Oskar Schirmer <oskar@scara.com> Signed-off-by: Mark Brown <broonie@linaro.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> |
||
|---|---|---|
| .. | ||
| efika-audio-fabric.c | ||
| eukrea-tlv320.c | ||
| fsl_dma.c | ||
| fsl_dma.h | ||
| fsl_ssi.c | ||
| fsl_ssi.h | ||
| fsl_utils.c | ||
| fsl_utils.h | ||
| imx-audmux.c | ||
| imx-audmux.h | ||
| imx-mc13783.c | ||
| imx-pcm-dma.c | ||
| imx-pcm-fiq.c | ||
| imx-pcm.c | ||
| imx-pcm.h | ||
| imx-sgtl5000.c | ||
| imx-ssi.c | ||
| imx-ssi.h | ||
| Kconfig | ||
| Makefile | ||
| mpc5200_dma.c | ||
| mpc5200_dma.h | ||
| mpc5200_psc_ac97.c | ||
| mpc5200_psc_ac97.h | ||
| mpc5200_psc_i2s.c | ||
| mpc8610_hpcd.c | ||
| mx27vis-aic32x4.c | ||
| p1022_ds.c | ||
| p1022_rdk.c | ||
| pcm030-audio-fabric.c | ||
| phycore-ac97.c | ||
| wm1133-ev1.c | ||