ASoC: codecs: wcd9335: Drop unused dmic rate handling

Driver was prepared to adjust DMIC microphone parameters according to
DMIC clock rate which is read from platform data or Devicetree.  The
latter part never happened, so the code is always called with
dmic_clk_rate=0, prints error and uses default/fallback values.  All
this part can be simplified by dropping dead parts of code.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Link: https://msgid.link/r/20240612-asoc-wcd9xxx-wide-cleanups-v1-6-0d15885b2a06@linaro.org
Signed-off-by: Mark Brown <broonie@kernel.org>

diff --git a/sound/soc/codecs/wcd9335.c b/sound/soc/codecs/wcd9335.c
index 6a1376b3b98c6efd4d186d4935ee53fe9642a419..f5fe2583b807473d9f5fb88d7365c51c7ef325c6 100644 (file)
--- a/sound/soc/codecs/wcd9335.c
+++ b/sound/soc/codecs/wcd9335.c
@@ -344,8 +344,6 @@ struct wcd9335_codec {
        int dmic_0_1_clk_cnt;
        int dmic_2_3_clk_cnt;
        int dmic_4_5_clk_cnt;
-       int dmic_sample_rate;
-       int mad_dmic_sample_rate;
 };
 
 struct wcd9335_irq {
@@ -2844,52 +2842,15 @@ out:
 }
 
 static u8 wcd9335_get_dmic_clk_val(struct snd_soc_component *component,
-                                u32 mclk_rate, u32 dmic_clk_rate)
+                                u32 mclk_rate)
 {
-       u32 div_factor;
        u8 dmic_ctl_val;
 
-       /* Default value to return in case of error */
        if (mclk_rate == WCD9335_MCLK_CLK_9P6MHZ)
                dmic_ctl_val = WCD9335_DMIC_CLK_DIV_2;
        else
                dmic_ctl_val = WCD9335_DMIC_CLK_DIV_3;
 
-       if (dmic_clk_rate == 0) {
-               dev_err(component->dev,
-                       "%s: dmic_sample_rate cannot be 0\n",
-                       __func__);
-               goto done;
-       }
-
-       div_factor = mclk_rate / dmic_clk_rate;
-       switch (div_factor) {
-       case 2:
-               dmic_ctl_val = WCD9335_DMIC_CLK_DIV_2;
-               break;
-       case 3:
-               dmic_ctl_val = WCD9335_DMIC_CLK_DIV_3;
-               break;
-       case 4:
-               dmic_ctl_val = WCD9335_DMIC_CLK_DIV_4;
-               break;
-       case 6:
-               dmic_ctl_val = WCD9335_DMIC_CLK_DIV_6;
-               break;
-       case 8:
-               dmic_ctl_val = WCD9335_DMIC_CLK_DIV_8;
-               break;
-       case 16:
-               dmic_ctl_val = WCD9335_DMIC_CLK_DIV_16;
-               break;
-       default:
-               dev_err(component->dev,
-                       "%s: Invalid div_factor %u, clk_rate(%u), dmic_rate(%u)\n",
-                       __func__, div_factor, mclk_rate, dmic_clk_rate);
-               break;
-       }
-
-done:
        return dmic_ctl_val;
 }
 
@@ -2943,11 +2904,7 @@ static int wcd9335_codec_enable_dmic(struct snd_soc_dapm_widget *w,
 
        switch (event) {
        case SND_SOC_DAPM_PRE_PMU:
-               dmic_rate_val =
-                       wcd9335_get_dmic_clk_val(comp,
-                                       wcd->mclk_rate,
-                                       wcd->dmic_sample_rate);
-
+               dmic_rate_val = wcd9335_get_dmic_clk_val(comp, wcd->mclk_rate);
                (*dmic_clk_cnt)++;
                if (*dmic_clk_cnt == 1) {
                        snd_soc_component_update_bits(comp, dmic_clk_reg,
@@ -2959,10 +2916,7 @@ static int wcd9335_codec_enable_dmic(struct snd_soc_dapm_widget *w,
 
                break;
        case SND_SOC_DAPM_POST_PMD:
-               dmic_rate_val =
-                       wcd9335_get_dmic_clk_val(comp,
-                                       wcd->mclk_rate,
-                                       wcd->mad_dmic_sample_rate);
+               dmic_rate_val = wcd9335_get_dmic_clk_val(comp, wcd->mclk_rate);
                (*dmic_clk_cnt)--;
                if (*dmic_clk_cnt  == 0) {
                        snd_soc_component_update_bits(comp, dmic_clk_reg,