diff options
author | Vaibhav Agarwal <vaibhav.agarwal@linaro.org> | 2016-06-10 12:59:11 +0530 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@google.com> | 2016-06-10 11:46:06 -0700 |
commit | c6722ab5d3c3d23021dd32bd1ae569665ee2263b (patch) | |
tree | 128e7dea9e6302599894a8e4f6e570401fb28037 /drivers/staging/greybus/audio_topology.c | |
parent | f2b6303d4505b5b046691a936c346a0e376cd9f3 (diff) |
greybus: audio: Ratelimit err messages in bundle, topology driver
Earlier I have shared a patch to rate limit err messages in audio_codec
driver. However, missed to include suggestion from Mark to do similar
changes in audio bundle & topology parser as well. Doing it now.
Testing Done: Compile tested
Fixes: 4cb3d109e5fc ("audio: Ratelimit err messages")
Signed-off-by: Vaibhav Agarwal <vaibhav.agarwal@linaro.org>
Reviewed-by: Vaibhav Hiremath <vaibhav.hiremath@linaro.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@google.com>
Diffstat (limited to 'drivers/staging/greybus/audio_topology.c')
-rw-r--r-- | drivers/staging/greybus/audio_topology.c | 23 |
1 files changed, 12 insertions, 11 deletions
diff --git a/drivers/staging/greybus/audio_topology.c b/drivers/staging/greybus/audio_topology.c index dfe120baecad..1e0768670b69 100644 --- a/drivers/staging/greybus/audio_topology.c +++ b/drivers/staging/greybus/audio_topology.c @@ -173,8 +173,8 @@ static int gbcodec_mixer_ctl_get(struct snd_kcontrol *kcontrol, ret = gb_audio_gb_get_control(module->mgmt_connection, data->ctl_id, GB_AUDIO_INVALID_INDEX, &gbvalue); if (ret) { - dev_err(codec->dev, "%d:Error in %s for %s\n", ret, __func__, - kcontrol->id.name); + dev_err_ratelimited(codec->dev, "%d:Error in %s for %s\n", ret, + __func__, kcontrol->id.name); return ret; } @@ -253,8 +253,8 @@ static int gbcodec_mixer_ctl_put(struct snd_kcontrol *kcontrol, ret = gb_audio_gb_set_control(module->mgmt_connection, data->ctl_id, GB_AUDIO_INVALID_INDEX, &gbvalue); if (ret) { - dev_err(codec->dev, "%d:Error in %s for %s\n", ret, __func__, - kcontrol->id.name); + dev_err_ratelimited(codec->dev, "%d:Error in %s for %s\n", ret, + __func__, kcontrol->id.name); } return ret; @@ -335,8 +335,8 @@ static int gbcodec_mixer_dapm_ctl_get(struct snd_kcontrol *kcontrol, ret = gb_audio_gb_get_control(module->mgmt_connection, data->ctl_id, GB_AUDIO_INVALID_INDEX, &gbvalue); if (ret) { - dev_err(codec->dev, "%d:Error in %s for %s\n", ret, __func__, - kcontrol->id.name); + dev_err_ratelimited(codec->dev, "%d:Error in %s for %s\n", ret, + __func__, kcontrol->id.name); return ret; } /* update ucontrol */ @@ -393,9 +393,9 @@ static int gbcodec_mixer_dapm_ctl_put(struct snd_kcontrol *kcontrol, data->ctl_id, GB_AUDIO_INVALID_INDEX, &gbvalue); if (ret) { - dev_err(codec->dev, - "%d:Error in %s for %s\n", ret, __func__, - kcontrol->id.name); + dev_err_ratelimited(codec->dev, + "%d:Error in %s for %s\n", ret, + __func__, kcontrol->id.name); } } @@ -600,8 +600,9 @@ static int gbaudio_widget_event(struct snd_soc_dapm_widget *w, break; } if (ret) - dev_err(codec->dev, "%d: widget, event:%d failed:%d\n", wid, - event, ret); + dev_err_ratelimited(codec->dev, + "%d: widget, event:%d failed:%d\n", wid, + event, ret); return ret; } |