AuxXxilium/linux_dsm_epyc7002
1
0
Fork 0
mirror of https://github.com/AuxXxilium/linux_dsm_epyc7002.git synced 2025-02-26 02:54:00 +07:00
Code Issues Actions Packages Projects Releases Wiki Activity

greybus: audio: Update pm runtime support in dai_ops callback

Browse Source 
Ensure pm runtime get_sync/put protection in codec_dai ops callback
functions before accessing apbridge.

Signed-off-by: Vaibhav Agarwal <vaibhav.agarwal@linaro.org>
Reviewed-by: Mark Greer <mgreer@animalcreek.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@google.com>
This commit is contained in:
Vaibhav Agarwal 2016-08-04 15:14:37 +05:30 committed by Greg Kroah-Hartman
parent 956adf749d
commit c388ae7696
1 changed files with 30 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

30
drivers/staging/greybus/audio_codec.c
View File

@ -405,6 +405,7 @@ static int gbcodec_hw_params(struct snd_pcm_substream *substream,
uint32_t format, rate; uint32_t format, rate;
struct gbaudio_module_info *module; struct gbaudio_module_info *module;
struct gbaudio_data_connection *data; struct gbaudio_data_connection *data;
struct gb_bundle *bundle;
struct gbaudio_codec_info *codec = dev_get_drvdata(dai->dev); struct gbaudio_codec_info *codec = dev_get_drvdata(dai->dev);
mutex_lock(&codec->lock); mutex_lock(&codec->lock);
@ -456,6 +457,13 @@ static int gbcodec_hw_params(struct snd_pcm_substream *substream,
return -EINVAL; return -EINVAL;
} }
bundle = to_gb_bundle(module->dev);
ret = gb_pm_runtime_get_sync(bundle);
if (ret) {
mutex_unlock(&codec->lock);
return ret;
}
ret = gb_audio_apbridgea_set_config(data->connection, 0, ret = gb_audio_apbridgea_set_config(data->connection, 0,
AUDIO_APBRIDGEA_PCM_FMT_16, AUDIO_APBRIDGEA_PCM_FMT_16,
AUDIO_APBRIDGEA_PCM_RATE_48000, AUDIO_APBRIDGEA_PCM_RATE_48000,
@ -466,6 +474,9 @@ static int gbcodec_hw_params(struct snd_pcm_substream *substream,
mutex_unlock(&codec->lock); mutex_unlock(&codec->lock);
return ret; return ret;
} }
gb_pm_runtime_put_noidle(bundle);
codec->stream[substream->stream].state = GBAUDIO_CODEC_HWPARAMS; codec->stream[substream->stream].state = GBAUDIO_CODEC_HWPARAMS;
codec->stream[substream->stream].format = format; codec->stream[substream->stream].format = format;
codec->stream[substream->stream].rate = rate; codec->stream[substream->stream].rate = rate;
@ -482,6 +493,7 @@ static int gbcodec_prepare(struct snd_pcm_substream *substream,
int ret; int ret;
struct gbaudio_module_info *module; struct gbaudio_module_info *module;
struct gbaudio_data_connection *data; struct gbaudio_data_connection *data;
struct gb_bundle *bundle;
struct gbaudio_codec_info *codec = dev_get_drvdata(dai->dev); struct gbaudio_codec_info *codec = dev_get_drvdata(dai->dev);
mutex_lock(&codec->lock); mutex_lock(&codec->lock);
@ -504,6 +516,13 @@ static int gbcodec_prepare(struct snd_pcm_substream *substream,
return -ENODEV; return -ENODEV;
} }
bundle = to_gb_bundle(module->dev);
ret = gb_pm_runtime_get_sync(bundle);
if (ret) {
mutex_unlock(&codec->lock);
return ret;
}
switch (substream->stream) { switch (substream->stream) {
case SNDRV_PCM_STREAM_PLAYBACK: case SNDRV_PCM_STREAM_PLAYBACK:
ret = gb_audio_apbridgea_set_tx_data_size(data->connection, 0, ret = gb_audio_apbridgea_set_tx_data_size(data->connection, 0,
@ -521,6 +540,8 @@ static int gbcodec_prepare(struct snd_pcm_substream *substream,
return ret; return ret;
} }
gb_pm_runtime_put_noidle(bundle);
codec->stream[substream->stream].state = GBAUDIO_CODEC_PREPARE; codec->stream[substream->stream].state = GBAUDIO_CODEC_PREPARE;
mutex_unlock(&codec->lock); mutex_unlock(&codec->lock);
return 0; return 0;
@ -531,6 +552,7 @@ static int gbcodec_mute_stream(struct snd_soc_dai *dai, int mute, int stream)
int ret; int ret;
struct gbaudio_data_connection *data; struct gbaudio_data_connection *data;
struct gbaudio_module_info *module; struct gbaudio_module_info *module;
struct gb_bundle *bundle;
struct gbaudio_codec_info *codec = dev_get_drvdata(dai->dev); struct gbaudio_codec_info *codec = dev_get_drvdata(dai->dev);
@ -563,6 +585,13 @@ static int gbcodec_mute_stream(struct snd_soc_dai *dai, int mute, int stream)
return -ENODEV; return -ENODEV;
} }
bundle = to_gb_bundle(module->dev);
ret = gb_pm_runtime_get_sync(bundle);
if (ret) {
mutex_unlock(&codec->lock);
return ret;
}
if (!mute && !stream) {/* start playback */ if (!mute && !stream) {/* start playback */
ret = gb_audio_apbridgea_prepare_tx(data->connection, ret = gb_audio_apbridgea_prepare_tx(data->connection,
0); 0);
@ -597,6 +626,7 @@ static int gbcodec_mute_stream(struct snd_soc_dai *dai, int mute, int stream)
module->name, mute ? "Mute" : "Unmute", module->name, mute ? "Mute" : "Unmute",
stream ? "Capture" : "Playback", ret); stream ? "Capture" : "Playback", ret);
gb_pm_runtime_put_noidle(bundle);
mutex_unlock(&codec->lock); mutex_unlock(&codec->lock);
return ret; return ret;
} }