OSDN Git Service

ASoC: mt8192: Remove spammy log messages
authorMark Brown <broonie@kernel.org>
Fri, 24 Feb 2023 14:03:55 +0000 (14:03 +0000)
committerMark Brown <broonie@kernel.org>
Tue, 28 Feb 2023 13:58:52 +0000 (13:58 +0000)
There are a lot of info level log messages in the mt8192 ADDA driver which
are trivially triggerable from userspace, many in normal operation. Remove
these to avoid spamming the console.

Reviewed-by: NĂ­colas F. R. A. Prado <nfraprado@collabora.com>
Tested-by: NĂ­colas F. R. A. Prado <nfraprado@collabora.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
Link: https://lore.kernel.org/r/20230223-asoc-mt8192-quick-fixes-v1-1-9a85f90368e1@kernel.org
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/mediatek/mt8192/mt8192-dai-adda.c

index f8c73e8..bc8753f 100644 (file)
@@ -303,9 +303,6 @@ static int mtk_adda_ul_event(struct snd_soc_dapm_widget *w,
        struct mt8192_afe_private *afe_priv = afe->platform_priv;
        int mtkaif_dmic = afe_priv->mtkaif_dmic;
 
-       dev_info(afe->dev, "%s(), name %s, event 0x%x, mtkaif_dmic %d\n",
-                __func__, w->name, event, mtkaif_dmic);
-
        switch (event) {
        case SND_SOC_DAPM_PRE_PMU:
                mt8192_afe_gpio_request(afe->dev, true, MT8192_DAI_ADDA, 1);
@@ -345,10 +342,6 @@ static int mtk_adda_ch34_ul_event(struct snd_soc_dapm_widget *w,
        int mtkaif_dmic = afe_priv->mtkaif_dmic_ch34;
        int mtkaif_adda6_only = afe_priv->mtkaif_adda6_only;
 
-       dev_info(afe->dev,
-                "%s(), name %s, event 0x%x, mtkaif_dmic %d, mtkaif_adda6_only %d\n",
-                __func__, w->name, event, mtkaif_dmic, mtkaif_adda6_only);
-
        switch (event) {
        case SND_SOC_DAPM_PRE_PMU:
                mt8192_afe_gpio_request(afe->dev, true, MT8192_DAI_ADDA_CH34,
@@ -538,9 +531,6 @@ static int mtk_adda_dl_event(struct snd_soc_dapm_widget *w,
        struct snd_soc_component *cmpnt = snd_soc_dapm_to_component(w->dapm);
        struct mtk_base_afe *afe = snd_soc_component_get_drvdata(cmpnt);
 
-       dev_info(afe->dev, "%s(), name %s, event 0x%x\n",
-                __func__, w->name, event);
-
        switch (event) {
        case SND_SOC_DAPM_PRE_PMU:
                mt8192_afe_gpio_request(afe->dev, true, MT8192_DAI_ADDA, 0);
@@ -564,9 +554,6 @@ static int mtk_adda_ch34_dl_event(struct snd_soc_dapm_widget *w,
        struct snd_soc_component *cmpnt = snd_soc_dapm_to_component(w->dapm);
        struct mtk_base_afe *afe = snd_soc_component_get_drvdata(cmpnt);
 
-       dev_info(afe->dev, "%s(), name %s, event 0x%x\n",
-                __func__, w->name, event);
-
        switch (event) {
        case SND_SOC_DAPM_PRE_PMU:
                mt8192_afe_gpio_request(afe->dev, true, MT8192_DAI_ADDA_CH34,
@@ -612,9 +599,6 @@ static int stf_positive_gain_set(struct snd_kcontrol *kcontrol,
                                   AFE_SIDETONE_GAIN,
                                   POSITIVE_GAIN_MASK_SFT,
                                   (gain_db / 6) << POSITIVE_GAIN_SFT);
-       } else {
-               dev_warn(afe->dev, "%s(), gain_db %d invalid\n",
-                        __func__, gain_db);
        }
        return 0;
 }
@@ -640,9 +624,6 @@ static int mt8192_adda_dmic_set(struct snd_kcontrol *kcontrol,
 
        dmic_on = ucontrol->value.integer.value[0];
 
-       dev_info(afe->dev, "%s(), kcontrol name %s, dmic_on %d\n",
-                __func__, kcontrol->id.name, dmic_on);
-
        afe_priv->mtkaif_dmic = dmic_on;
        afe_priv->mtkaif_dmic_ch34 = dmic_on;
        return 0;
@@ -669,9 +650,6 @@ static int mt8192_adda6_only_set(struct snd_kcontrol *kcontrol,
 
        mtkaif_adda6_only = ucontrol->value.integer.value[0];
 
-       dev_info(afe->dev, "%s(), kcontrol name %s, mtkaif_adda6_only %d\n",
-                __func__, kcontrol->id.name, mtkaif_adda6_only);
-
        afe_priv->mtkaif_adda6_only = mtkaif_adda6_only;
        return 0;
 }
@@ -750,9 +728,6 @@ static int mtk_stf_event(struct snd_soc_dapm_widget *w,
 
        regmap_read(afe->regmap, AFE_SIDETONE_CON1, &reg_value);
 
-       dev_info(afe->dev, "%s(), name %s, event 0x%x, ul_rate 0x%x, AFE_SIDETONE_CON1 0x%x\n",
-                __func__, w->name, event, ul_rate, reg_value);
-
        switch (event) {
        case SND_SOC_DAPM_PRE_PMU:
                /* set side tone gain = 0 */
@@ -1163,12 +1138,6 @@ static int mtk_dai_adda_hw_params(struct snd_pcm_substream *substream,
        unsigned int rate = params_rate(params);
        int id = dai->id;
 
-       dev_info(afe->dev, "%s(), id %d, stream %d, rate %d\n",
-                __func__,
-                id,
-                substream->stream,
-                rate);
-
        if (substream->stream == SNDRV_PCM_STREAM_PLAYBACK) {
                unsigned int dl_src2_con0 = 0;
                unsigned int dl_src2_con1 = 0;
@@ -1441,8 +1410,6 @@ int mt8192_dai_adda_register(struct mtk_base_afe *afe)
        struct mtk_base_afe_dai *dai;
        struct mt8192_afe_private *afe_priv = afe->platform_priv;
 
-       dev_info(afe->dev, "%s()\n", __func__);
-
        dai = devm_kzalloc(afe->dev, sizeof(*dai), GFP_KERNEL);
        if (!dai)
                return -ENOMEM;