ASoC: mediatek: mt8195: Migrate to mtk_soundcard_common_probe
authorAngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
Tue, 16 Apr 2024 07:13:56 +0000 (09:13 +0200)
committerMark Brown <broonie@kernel.org>
Tue, 16 Apr 2024 10:59:56 +0000 (19:59 +0900)
Add mtk_soundcard_pdata platform data for the MediaTek common sound card
probe mechanism, including a driver/soc-specific probe extension (used
for bits that cannot be commonized  hence specific to this driver), and
change the probe function to mtk_soundcard_common_probe.

This is also adding the possibility of specifying the links and routing
with the audio-routing property and (x)-dai-link nodes in device trees
to stop hardcoding machine specific links in the card driver assupported
by the common probe function, but support for legacy device trees is
retained with a legacy_probe function, which is used only in case the
new properties are not found.

Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
Link: https://lore.kernel.org/r/20240416071410.75620-5-angelogioacchino.delregno@collabora.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/mediatek/mt8195/mt8195-mt6359.c

index 53fd8a897b9d27a5894006eaa0e96743b2e728c6..f694618e76354b8a66d99fd5712de3b287084a5a 100644 (file)
@@ -22,6 +22,7 @@
 #include "../common/mtk-afe-platform-driver.h"
 #include "../common/mtk-dsp-sof-common.h"
 #include "../common/mtk-soc-card.h"
+#include "../common/mtk-soundcard-driver.h"
 #include "mt8195-afe-clk.h"
 #include "mt8195-afe-common.h"
 
 #define RT1019_SPEAKER_AMP_PRESENT             BIT(1)
 #define MAX98390_SPEAKER_AMP_PRESENT           BIT(2)
 
+#define DUMB_CODEC_INIT                                BIT(0)
+#define MT6359_CODEC_INIT                      BIT(1)
+#define RT1011_CODEC_INIT                      BIT(2)
+#define RT1019_CODEC_INIT                      BIT(3)
+#define MAX98390_CODEC_INIT                    BIT(4)
+#define RT5682_CODEC_INIT                      BIT(5)
+
 #define RT1011_CODEC_DAI       "rt1011-aif"
 #define RT1011_DEV0_NAME       "rt1011.2-0038"
 #define RT1011_DEV1_NAME       "rt1011.2-0039"
 #define SOF_DMA_UL4 "SOF_DMA_UL4"
 #define SOF_DMA_UL5 "SOF_DMA_UL5"
 
-struct mt8195_card_data {
-       const char *name;
-       unsigned long quirk;
-};
-
 struct mt8195_mt6359_priv {
-       struct snd_soc_jack headset_jack;
-       struct snd_soc_jack dp_jack;
-       struct snd_soc_jack hdmi_jack;
        struct clk *i2so1_mclk;
 };
 
+enum mt8195_jacks {
+       MT8195_JACK_HEADSET,
+       MT8195_JACK_DP,
+       MT8195_JACK_HDMI,
+       MT8195_JACK_MAX,
+};
+
 /* Headset jack detection DAPM pins */
 static struct snd_soc_jack_pin mt8195_jack_pins[] = {
        {
@@ -382,33 +389,31 @@ static const struct snd_soc_ops mt8195_dptx_ops = {
 static int mt8195_dptx_codec_init(struct snd_soc_pcm_runtime *rtd)
 {
        struct mtk_soc_card_data *soc_card_data = snd_soc_card_get_drvdata(rtd->card);
-       struct mt8195_mt6359_priv *priv = soc_card_data->mach_priv;
+       struct snd_soc_jack *jack = &soc_card_data->card_data->jacks[MT8195_JACK_DP];
        struct snd_soc_component *cmpnt_codec =
                snd_soc_rtd_to_codec(rtd, 0)->component;
        int ret;
 
-       ret = snd_soc_card_jack_new(rtd->card, "DP Jack", SND_JACK_LINEOUT,
-                                   &priv->dp_jack);
+       ret = snd_soc_card_jack_new(rtd->card, "DP Jack", SND_JACK_LINEOUT, jack);
        if (ret)
                return ret;
 
-       return snd_soc_component_set_jack(cmpnt_codec, &priv->dp_jack, NULL);
+       return snd_soc_component_set_jack(cmpnt_codec, jack, NULL);
 }
 
 static int mt8195_hdmi_codec_init(struct snd_soc_pcm_runtime *rtd)
 {
        struct mtk_soc_card_data *soc_card_data = snd_soc_card_get_drvdata(rtd->card);
-       struct mt8195_mt6359_priv *priv = soc_card_data->mach_priv;
+       struct snd_soc_jack *jack = &soc_card_data->card_data->jacks[MT8195_JACK_HDMI];
        struct snd_soc_component *cmpnt_codec =
                snd_soc_rtd_to_codec(rtd, 0)->component;
        int ret;
 
-       ret = snd_soc_card_jack_new(rtd->card, "HDMI Jack", SND_JACK_LINEOUT,
-                                   &priv->hdmi_jack);
+       ret = snd_soc_card_jack_new(rtd->card, "HDMI Jack", SND_JACK_LINEOUT, jack);
        if (ret)
                return ret;
 
-       return snd_soc_component_set_jack(cmpnt_codec, &priv->hdmi_jack, NULL);
+       return snd_soc_component_set_jack(cmpnt_codec, jack, NULL);
 }
 
 static int mt8195_dptx_hw_params_fixup(struct snd_soc_pcm_runtime *rtd,
@@ -566,7 +571,7 @@ static int mt8195_rt5682_init(struct snd_soc_pcm_runtime *rtd)
                snd_soc_rtd_to_codec(rtd, 0)->component;
        struct mtk_soc_card_data *soc_card_data = snd_soc_card_get_drvdata(rtd->card);
        struct mt8195_mt6359_priv *priv = soc_card_data->mach_priv;
-       struct snd_soc_jack *jack = &priv->headset_jack;
+       struct snd_soc_jack *jack = &soc_card_data->card_data->jacks[MT8195_JACK_HEADSET];
        struct snd_soc_component *cmpnt_afe =
                snd_soc_rtdcom_lookup(rtd, AFE_PCM_NAME);
        struct mtk_base_afe *afe = snd_soc_component_get_drvdata(cmpnt_afe);
@@ -687,7 +692,7 @@ static int mt8195_rt1011_init(struct snd_soc_pcm_runtime *rtd)
        return ret;
 }
 
-static int mt8195_rt1019_init(struct snd_soc_pcm_runtime *rtd)
+static int mt8195_dumb_amp_init(struct snd_soc_pcm_runtime *rtd)
 {
        struct snd_soc_card *card = rtd->card;
        int ret;
@@ -707,6 +712,18 @@ static int mt8195_rt1019_init(struct snd_soc_pcm_runtime *rtd)
                return ret;
        }
 
+       return 0;
+}
+
+static int mt8195_rt1019_init(struct snd_soc_pcm_runtime *rtd)
+{
+       struct snd_soc_card *card = rtd->card;
+       int ret;
+
+       ret = mt8195_dumb_amp_init(rtd);
+       if (ret)
+               return ret;
+
        ret = snd_soc_dapm_add_routes(&card->dapm, mt8195_rt1019_routes,
                                      ARRAY_SIZE(mt8195_rt1019_routes));
        if (ret)
@@ -1371,108 +1388,31 @@ static int mt8195_dai_link_fixup(struct snd_soc_pcm_runtime *rtd,
        return ret;
 }
 
-static int mt8195_mt6359_dev_probe(struct platform_device *pdev)
+static int mt8195_mt6359_legacy_probe(struct mtk_soc_card_data *soc_card_data)
 {
-       struct snd_soc_card *card = &mt8195_mt6359_soc_card;
+       struct mtk_platform_card_data *card_data = soc_card_data->card_data;
+       struct snd_soc_card *card = card_data->card;
+       struct device_node *codec_node, *dp_node, *hdmi_node;
        struct snd_soc_dai_link *dai_link;
-       struct mtk_soc_card_data *soc_card_data;
-       struct mt8195_mt6359_priv *mach_priv;
-       struct device_node *platform_node, *adsp_node, *codec_node, *dp_node, *hdmi_node;
-       struct mt8195_card_data *card_data;
-       int is5682s = 0;
-       int init6359 = 0;
-       int sof_on = 0;
-       int ret, i;
-
-       card_data = (struct mt8195_card_data *)of_device_get_match_data(&pdev->dev);
-       card->dev = &pdev->dev;
-
-       ret = snd_soc_of_parse_card_name(card, "model");
-       if (ret) {
-               dev_err(&pdev->dev, "%s new card name parsing error %d\n",
-                       __func__, ret);
-               return ret;
-       }
-
-       if (!card->name)
-               card->name = card_data->name;
+       struct device *dev = card->dev;
+       bool is5682s, init6359 = false;
+       int i;
 
        if (strstr(card->name, "_5682s")) {
                codec_node = of_find_compatible_node(NULL, NULL, "realtek,rt5682s");
-               is5682s = 1;
-       } else
-               codec_node = of_find_compatible_node(NULL, NULL, "realtek,rt5682i");
-
-       soc_card_data = devm_kzalloc(&pdev->dev, sizeof(*card_data), GFP_KERNEL);
-       if (!soc_card_data)
-               return -ENOMEM;
-
-       mach_priv = devm_kzalloc(&pdev->dev, sizeof(*mach_priv), GFP_KERNEL);
-       if (!mach_priv)
-               return -ENOMEM;
-
-       soc_card_data->mach_priv = mach_priv;
-
-       adsp_node = of_parse_phandle(pdev->dev.of_node, "mediatek,adsp", 0);
-       if (adsp_node) {
-               struct mtk_sof_priv *sof_priv;
-
-               sof_priv = devm_kzalloc(&pdev->dev, sizeof(*sof_priv), GFP_KERNEL);
-               if (!sof_priv) {
-                       ret = -ENOMEM;
-                       goto err_kzalloc;
-               }
-               sof_priv->conn_streams = g_sof_conn_streams;
-               sof_priv->num_streams = ARRAY_SIZE(g_sof_conn_streams);
-               sof_priv->sof_dai_link_fixup = mt8195_dai_link_fixup;
-               soc_card_data->sof_priv = sof_priv;
-               card->probe = mtk_sof_card_probe;
-               card->late_probe = mtk_sof_card_late_probe;
-               if (!card->topology_shortname_created) {
-                       snprintf(card->topology_shortname, 32, "sof-%s", card->name);
-                       card->topology_shortname_created = true;
-               }
-               card->name = card->topology_shortname;
-               sof_on = 1;
-       }
-
-       if (of_property_read_bool(pdev->dev.of_node, "mediatek,dai-link")) {
-               ret = mtk_sof_dailink_parse_of(card, pdev->dev.of_node,
-                                              "mediatek,dai-link",
-                                              mt8195_mt6359_dai_links,
-                                              ARRAY_SIZE(mt8195_mt6359_dai_links));
-               if (ret) {
-                       dev_dbg(&pdev->dev, "Parse dai-link fail\n");
-                       goto err_parse_of;
-               }
+               is5682s = true;
        } else {
-               if (!sof_on)
-                       card->num_links = DAI_LINK_REGULAR_NUM;
-       }
-
-       platform_node = of_parse_phandle(pdev->dev.of_node,
-                                        "mediatek,platform", 0);
-       if (!platform_node) {
-               dev_dbg(&pdev->dev, "Property 'platform' missing or invalid\n");
-               ret = -EINVAL;
-               goto err_platform_node;
+               codec_node = of_find_compatible_node(NULL, NULL, "realtek,rt5682i");
+               is5682s = false;
        }
 
-       dp_node = of_parse_phandle(pdev->dev.of_node, "mediatek,dptx-codec", 0);
-       hdmi_node = of_parse_phandle(pdev->dev.of_node,
-                                    "mediatek,hdmi-codec", 0);
+       dp_node = of_parse_phandle(dev->of_node, "mediatek,dptx-codec", 0);
+       hdmi_node = of_parse_phandle(dev->of_node, "mediatek,hdmi-codec", 0);
 
        for_each_card_prelinks(card, i, dai_link) {
-               if (!dai_link->platforms->name) {
-                       if (!strncmp(dai_link->name, "AFE_SOF", strlen("AFE_SOF")) && sof_on)
-                               dai_link->platforms->of_node = adsp_node;
-                       else
-                               dai_link->platforms->of_node = platform_node;
-               }
-
                if (strcmp(dai_link->name, "DPTX_BE") == 0) {
                        if (!dp_node) {
-                               dev_dbg(&pdev->dev, "No property 'dptx-codec'\n");
+                               dev_dbg(dev, "No property 'dptx-codec'\n");
                        } else {
                                dai_link->codecs->of_node = dp_node;
                                dai_link->codecs->name = NULL;
@@ -1481,7 +1421,7 @@ static int mt8195_mt6359_dev_probe(struct platform_device *pdev)
                        }
                } else if (strcmp(dai_link->name, "ETDM3_OUT_BE") == 0) {
                        if (!hdmi_node) {
-                               dev_dbg(&pdev->dev, "No property 'hdmi-codec'\n");
+                               dev_dbg(dev, "No property 'hdmi-codec'\n");
                        } else {
                                dai_link->codecs->of_node = hdmi_node;
                                dai_link->codecs->name = NULL;
@@ -1490,7 +1430,7 @@ static int mt8195_mt6359_dev_probe(struct platform_device *pdev)
                        }
                } else if (strcmp(dai_link->name, "ETDM1_OUT_BE") == 0) {
                        if (!codec_node) {
-                               dev_err(&pdev->dev, "Codec not found!\n");
+                               dev_err(dev, "Codec not found!\n");
                        } else {
                                dai_link->codecs->of_node = codec_node;
                                dai_link->codecs->name = NULL;
@@ -1501,7 +1441,7 @@ static int mt8195_mt6359_dev_probe(struct platform_device *pdev)
                        }
                } else if (strcmp(dai_link->name, "ETDM2_IN_BE") == 0) {
                        if (!codec_node) {
-                               dev_err(&pdev->dev, "Codec not found!\n");
+                               dev_err(dev, "Codec not found!\n");
                        } else {
                                dai_link->codecs->of_node = codec_node;
                                dai_link->codecs->name = NULL;
@@ -1514,10 +1454,10 @@ static int mt8195_mt6359_dev_probe(struct platform_device *pdev)
                           strcmp(dai_link->name, "UL_SRC2_BE") == 0) {
                        if (!init6359) {
                                dai_link->init = mt8195_mt6359_init;
-                               init6359 = 1;
+                               init6359 = true;
                        }
                } else if (strcmp(dai_link->name, "ETDM2_OUT_BE") == 0) {
-                       switch (card_data->quirk) {
+                       switch (card_data->flags) {
                        case RT1011_SPEAKER_AMP_PRESENT:
                                dai_link->codecs = rt1011_comps;
                                dai_link->num_codecs = ARRAY_SIZE(rt1011_comps);
@@ -1545,33 +1485,119 @@ static int mt8195_mt6359_dev_probe(struct platform_device *pdev)
                }
        }
 
-       snd_soc_card_set_drvdata(card, soc_card_data);
+       return 0;
+}
 
-       ret = devm_snd_soc_register_card(&pdev->dev, card);
+static int mt8195_mt6359_soc_card_probe(struct mtk_soc_card_data *soc_card_data, bool legacy)
+{
+       struct mtk_platform_card_data *card_data = soc_card_data->card_data;
+       struct snd_soc_card *card = card_data->card;
+       struct mt8195_mt6359_priv *mach_priv;
+       struct snd_soc_dai_link *dai_link;
+       u8 codec_init = 0;
+       int i;
 
-       of_node_put(platform_node);
-       of_node_put(dp_node);
-       of_node_put(hdmi_node);
-err_kzalloc:
-err_parse_of:
-err_platform_node:
-       of_node_put(adsp_node);
-       return ret;
+       mach_priv = devm_kzalloc(card->dev, sizeof(*mach_priv), GFP_KERNEL);
+       if (!mach_priv)
+               return -ENOMEM;
+
+       soc_card_data->mach_priv = mach_priv;
+
+       if (legacy)
+               return mt8195_mt6359_legacy_probe(soc_card_data);
+
+       for_each_card_prelinks(card, i, dai_link) {
+               if (strcmp(dai_link->name, "DPTX_BE") == 0) {
+                       if (strcmp(dai_link->codecs->dai_name, "snd-soc-dummy-dai"))
+                               dai_link->init = mt8195_dptx_codec_init;
+               } else if (strcmp(dai_link->name, "ETDM3_OUT_BE") == 0) {
+                       if (strcmp(dai_link->codecs->dai_name, "snd-soc-dummy-dai"))
+                               dai_link->init = mt8195_hdmi_codec_init;
+               } else if (strcmp(dai_link->name, "DL_SRC_BE") == 0 ||
+                          strcmp(dai_link->name, "UL_SRC1_BE") == 0 ||
+                          strcmp(dai_link->name, "UL_SRC2_BE") == 0) {
+                       if (!(codec_init & MT6359_CODEC_INIT)) {
+                               dai_link->init = mt8195_mt6359_init;
+                               codec_init |= MT6359_CODEC_INIT;
+                       }
+               } else if (strcmp(dai_link->name, "ETDM1_OUT_BE") == 0 ||
+                          strcmp(dai_link->name, "ETDM2_OUT_BE") == 0 ||
+                          strcmp(dai_link->name, "ETDM1_IN_BE") == 0 ||
+                          strcmp(dai_link->name, "ETDM2_IN_BE") == 0) {
+                       if (!strcmp(dai_link->codecs->dai_name, MAX98390_CODEC_DAI)) {
+                               if (!(codec_init & MAX98390_CODEC_INIT)) {
+                                       dai_link->init = mt8195_max98390_init;
+                                       codec_init |= MAX98390_CODEC_INIT;
+                               }
+                       } else if (!strcmp(dai_link->codecs->dai_name, RT1011_CODEC_DAI)) {
+                               dai_link->ops = &mt8195_rt1011_etdm_ops;
+                               if (!(codec_init & RT1011_CODEC_INIT)) {
+                                       dai_link->init = mt8195_rt1011_init;
+                                       codec_init |= RT1011_CODEC_INIT;
+                               }
+                       } else if (!strcmp(dai_link->codecs->dai_name, RT1019_CODEC_DAI)) {
+                               if (!(codec_init & RT1019_CODEC_INIT)) {
+                                       dai_link->init = mt8195_rt1019_init;
+                                       codec_init |= RT1019_CODEC_INIT;
+                               }
+                       } else if (!strcmp(dai_link->codecs->dai_name, RT5682_CODEC_DAI) ||
+                                  !strcmp(dai_link->codecs->dai_name, RT5682S_CODEC_DAI)) {
+                               dai_link->ops = &mt8195_rt5682_etdm_ops;
+                               if (!(codec_init & RT5682_CODEC_INIT)) {
+                                       dai_link->init = mt8195_rt5682_init;
+                                       codec_init |= RT5682_CODEC_INIT;
+                               }
+                       } else {
+                               if (strcmp(dai_link->codecs->dai_name, "snd-soc-dummy-dai")) {
+                                       if (!(codec_init & DUMB_CODEC_INIT)) {
+                                               dai_link->init = mt8195_dumb_amp_init;
+                                               codec_init |= DUMB_CODEC_INIT;
+                                       }
+                               }
+                       }
+               }
+       }
+
+       return 0;
 }
 
-static struct mt8195_card_data mt8195_mt6359_rt1019_rt5682_card = {
-       .name = "mt8195_r1019_5682",
-       .quirk = RT1019_SPEAKER_AMP_PRESENT,
+static const struct mtk_sof_priv mt8195_sof_priv = {
+       .conn_streams = g_sof_conn_streams,
+       .num_streams = ARRAY_SIZE(g_sof_conn_streams),
+       .sof_dai_link_fixup = mt8195_dai_link_fixup
 };
 
-static struct mt8195_card_data mt8195_mt6359_rt1011_rt5682_card = {
-       .name = "mt8195_r1011_5682",
-       .quirk = RT1011_SPEAKER_AMP_PRESENT,
+static const struct mtk_soundcard_pdata mt8195_mt6359_rt1019_rt5682_card = {
+       .card_name = "mt8195_r1019_5682",
+       .card_data = &(struct mtk_platform_card_data) {
+               .card = &mt8195_mt6359_soc_card,
+               .num_jacks = MT8195_JACK_MAX,
+               .flags = RT1019_SPEAKER_AMP_PRESENT
+       },
+       .sof_priv = &mt8195_sof_priv,
+       .soc_probe = mt8195_mt6359_soc_card_probe
 };
 
-static struct mt8195_card_data mt8195_mt6359_max98390_rt5682_card = {
-       .name = "mt8195_m98390_r5682",
-       .quirk = MAX98390_SPEAKER_AMP_PRESENT,
+static const struct mtk_soundcard_pdata mt8195_mt6359_rt1011_rt5682_card = {
+       .card_name = "mt8195_r1011_5682",
+       .card_data = &(struct mtk_platform_card_data) {
+               .card = &mt8195_mt6359_soc_card,
+               .num_jacks = MT8195_JACK_MAX,
+               .flags = RT1011_SPEAKER_AMP_PRESENT
+       },
+       .sof_priv = &mt8195_sof_priv,
+       .soc_probe = mt8195_mt6359_soc_card_probe
+};
+
+static const struct mtk_soundcard_pdata mt8195_mt6359_max98390_rt5682_card = {
+       .card_name = "mt8195_m98390_r5682",
+       .card_data = &(struct mtk_platform_card_data) {
+               .card = &mt8195_mt6359_soc_card,
+               .num_jacks = MT8195_JACK_MAX,
+               .flags = MAX98390_SPEAKER_AMP_PRESENT
+       },
+       .sof_priv = &mt8195_sof_priv,
+       .soc_probe = mt8195_mt6359_soc_card_probe
 };
 
 static const struct of_device_id mt8195_mt6359_dt_match[] = {
@@ -1597,7 +1623,7 @@ static struct platform_driver mt8195_mt6359_driver = {
                .of_match_table = mt8195_mt6359_dt_match,
                .pm = &snd_soc_pm_ops,
        },
-       .probe = mt8195_mt6359_dev_probe,
+       .probe = mtk_soundcard_common_probe,
 };
 
 module_platform_driver(mt8195_mt6359_driver);