ASoC: da7213: Extend support for the MCK in range [2, 50] MHz

According to DA7212 HW manual, the codec's PLL input divider can operate
with MCLK range from 2MHz to 50MHz but current driver only set the
minimum supported MCLK frequency to 5MHz. That cause 11.025kHz audio
which is corresponding to MCLK of 2.8224MHz (11.025kHz * 256) unable to
play in case audio-simple-card is used.

Signed-off-by: Hao Bui <hao.bui.yg@renesas.com>
Co-developed-by: Claudiu Beznea <claudiu.beznea.uj@bp.renesas.com>
Signed-off-by: Claudiu Beznea <claudiu.beznea.uj@bp.renesas.com>
Link: https://patch.msgid.link/20241106081826.1211088-27-claudiu.beznea.uj@bp.renesas.com
Signed-off-by: Mark Brown <broonie@kernel.org>

diff --git a/sound/soc/codecs/da7213.c b/sound/soc/codecs/da7213.c
index af38b2b5e1744abe4a7a1872543000b7b1ce4aa5..ca4cc954efa8e657c32cb920ba8a264ca5d23880 100644 (file)
--- a/sound/soc/codecs/da7213.c
+++ b/sound/soc/codecs/da7213.c
@@ -20,6 +20,7 @@
 #include <sound/pcm.h>
 #include <sound/pcm_params.h>
 #include <linux/pm_runtime.h>
+#include <linux/units.h>
 #include <sound/soc.h>
 #include <sound/initval.h>
 #include <sound/tlv.h>
@@ -1559,7 +1560,7 @@ static int da7213_set_component_sysclk(struct snd_soc_component *component,
        if (freq == 0)
                return 0;
 
-       if (((freq < 5000000) && (freq != 32768)) || (freq > 54000000)) {
+       if (((freq < da7213->fin_min_rate) && (freq != 32768)) || (freq > 54000000)) {
                dev_err(component->dev, "Unsupported MCLK value %d\n",
                        freq);
                return -EINVAL;
@@ -1858,11 +1859,14 @@ static int da7213_set_bias_level(struct snd_soc_component *component,
        return 0;
 }
 
+#define DA7213_FIN_MIN_RATE    (5 * MEGA)
+#define DA7212_FIN_MIN_RATE    (2 * MEGA)
+
 #if defined(CONFIG_OF)
 /* DT */
 static const struct of_device_id da7213_of_match[] = {
-       { .compatible = "dlg,da7212", },
-       { .compatible = "dlg,da7213", },
+       { .compatible = "dlg,da7212", .data = (void *)DA7212_FIN_MIN_RATE },
+       { .compatible = "dlg,da7213", .data = (void *)DA7213_FIN_MIN_RATE },
        { }
 };
 MODULE_DEVICE_TABLE(of, da7213_of_match);
@@ -1870,8 +1874,8 @@ MODULE_DEVICE_TABLE(of, da7213_of_match);
 
 #ifdef CONFIG_ACPI
 static const struct acpi_device_id da7213_acpi_match[] = {
-       { "DLGS7212", 0},
-       { "DLGS7213", 0},
+       { "DLGS7212", DA7212_FIN_MIN_RATE },
+       { "DLGS7213", DA7213_FIN_MIN_RATE },
        { },
 };
 MODULE_DEVICE_TABLE(acpi, da7213_acpi_match);
@@ -2167,6 +2171,10 @@ static int da7213_i2c_probe(struct i2c_client *i2c)
        if (!da7213)
                return -ENOMEM;
 
+       da7213->fin_min_rate = (uintptr_t)i2c_get_match_data(i2c);
+       if (!da7213->fin_min_rate)
+               return -EINVAL;
+
        i2c_set_clientdata(i2c, da7213);
 
        /* Get required supplies */

diff --git a/sound/soc/codecs/da7213.h b/sound/soc/codecs/da7213.h
index 505b731c0adb93808207ce460d64cf60bda3be13..b9ab791d6b883b1baaa65e35e99284821435b353 100644 (file)
--- a/sound/soc/codecs/da7213.h
+++ b/sound/soc/codecs/da7213.h
@@ -600,6 +600,7 @@ struct da7213_priv {
        struct clk *mclk;
        unsigned int mclk_rate;
        unsigned int out_rate;
+       unsigned int fin_min_rate;
        int clk_src;
        bool master;
        bool alc_calib_auto;