* [v3 0/2] ASoC: mediatek: mt8192: support rt1015p_rt5682s
@ 2022-03-11 2:15 Jiaxin Yu
2022-03-11 2:15 ` [v3 1/2] ASoC: dt-bindings: mt8192-mt6359: add new compatible for using rt1015p and rt5682 Jiaxin Yu
2022-03-11 2:15 ` [v3 2/2] ASoC: mediatek: mt8192: support rt1015p_rt5682s Jiaxin Yu
0 siblings, 2 replies; 4+ messages in thread
From: Jiaxin Yu @ 2022-03-11 2:15 UTC (permalink / raw)
To: broonie, robh+dt
Cc: aaronyu, matthias.bgg, trevor.wu, tzungbi, linmq006, alsa-devel,
devicetree, linux-arm-kernel, linux-mediatek, linux-kernel,
Project_Global_Chrome_Upstream_Group, Jiaxin Yu
From: Jiaxin Yu <jiaxin.yu@mediatek.corp-partner.google.com>
The series reuses mt8192-mt6359-rt10150rt5682.c for supporting machine
driver with rt1015p speaker amplifier and rt5682s headset codec.
Changes from v2:
- fix build warnings such as "data argument not used by format string"
Changes from v1:
- uses the snd_soc_of_get_dai_link_codecs to complete the
configuration of dai_link's codecs
- uses definitions to simplifies card name and compatible name
Jiaxin Yu (2):
ASoC: dt-bindings: mt8192-mt6359: add new compatible for using rt1015p
and rt5682
ASoC: mediatek: mt8192: support rt1015p_rt5682s
.../sound/mt8192-mt6359-rt1015-rt5682.yaml | 1 +
sound/soc/mediatek/Kconfig | 1 +
.../mt8192/mt8192-mt6359-rt1015-rt5682.c | 204 +++++++++++-------
3 files changed, 129 insertions(+), 77 deletions(-)
--
2.18.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 4+ messages in thread
* [v3 1/2] ASoC: dt-bindings: mt8192-mt6359: add new compatible for using rt1015p and rt5682
2022-03-11 2:15 [v3 0/2] ASoC: mediatek: mt8192: support rt1015p_rt5682s Jiaxin Yu
@ 2022-03-11 2:15 ` Jiaxin Yu
2022-03-11 2:15 ` [v3 2/2] ASoC: mediatek: mt8192: support rt1015p_rt5682s Jiaxin Yu
1 sibling, 0 replies; 4+ messages in thread
From: Jiaxin Yu @ 2022-03-11 2:15 UTC (permalink / raw)
To: broonie, robh+dt
Cc: aaronyu, matthias.bgg, trevor.wu, tzungbi, linmq006, alsa-devel,
devicetree, linux-arm-kernel, linux-mediatek, linux-kernel,
Project_Global_Chrome_Upstream_Group, Jiaxin Yu, Rob Herring
Adds new compatible string "mt8192_mt6359_rt1015p_rt5682s" for machines
with rt1015p and rt5682s.
Signed-off-by: Jiaxin Yu <jiaxin.yu@mediatek.com>
Acked-by: Rob Herring <robh@kernel.org>
---
.../devicetree/bindings/sound/mt8192-mt6359-rt1015-rt5682.yaml | 1 +
1 file changed, 1 insertion(+)
diff --git a/Documentation/devicetree/bindings/sound/mt8192-mt6359-rt1015-rt5682.yaml b/Documentation/devicetree/bindings/sound/mt8192-mt6359-rt1015-rt5682.yaml
index a781e7aaaa38..4b0b0478018a 100644
--- a/Documentation/devicetree/bindings/sound/mt8192-mt6359-rt1015-rt5682.yaml
+++ b/Documentation/devicetree/bindings/sound/mt8192-mt6359-rt1015-rt5682.yaml
@@ -18,6 +18,7 @@ properties:
enum:
- mediatek,mt8192_mt6359_rt1015_rt5682
- mediatek,mt8192_mt6359_rt1015p_rt5682
+ - mediatek,mt8192_mt6359_rt1015p_rt5682s
mediatek,platform:
$ref: "/schemas/types.yaml#/definitions/phandle"
--
2.18.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [v3 2/2] ASoC: mediatek: mt8192: support rt1015p_rt5682s
2022-03-11 2:15 [v3 0/2] ASoC: mediatek: mt8192: support rt1015p_rt5682s Jiaxin Yu
2022-03-11 2:15 ` [v3 1/2] ASoC: dt-bindings: mt8192-mt6359: add new compatible for using rt1015p and rt5682 Jiaxin Yu
@ 2022-03-11 2:15 ` Jiaxin Yu
2022-03-11 15:53 ` Jiaxin Yu
1 sibling, 1 reply; 4+ messages in thread
From: Jiaxin Yu @ 2022-03-11 2:15 UTC (permalink / raw)
To: broonie, robh+dt
Cc: aaronyu, matthias.bgg, trevor.wu, tzungbi, linmq006, alsa-devel,
devicetree, linux-arm-kernel, linux-mediatek, linux-kernel,
Project_Global_Chrome_Upstream_Group, Jiaxin Yu,
kernel test robot
Supports machines with rt1015p and rt5682s. Uses new proposed compatible
string "mt8192_mt6359_rt1015p_rt5682s". Using define to simplifies card
name and compatible name, and uses the snd_soc_of_get_dai_link_codecs()
to complete the configuration of dai_link's codecs.
Signed-off-by: Jiaxin Yu <jiaxin.yu@mediatek.com>
Reported-by: kernel test robot <lkp@intel.com>
---
sound/soc/mediatek/Kconfig | 1 +
.../mt8192/mt8192-mt6359-rt1015-rt5682.c | 204 +++++++++++-------
2 files changed, 128 insertions(+), 77 deletions(-)
diff --git a/sound/soc/mediatek/Kconfig b/sound/soc/mediatek/Kconfig
index d515613a79da..cacfbab4262d 100644
--- a/sound/soc/mediatek/Kconfig
+++ b/sound/soc/mediatek/Kconfig
@@ -176,6 +176,7 @@ config SND_SOC_MT8192_MT6359_RT1015_RT5682
select SND_SOC_RT1015
select SND_SOC_RT1015P
select SND_SOC_RT5682_I2C
+ select SND_SOC_RT5682S
select SND_SOC_DMIC
help
This adds ASoC driver for Mediatek MT8192 boards
diff --git a/sound/soc/mediatek/mt8192/mt8192-mt6359-rt1015-rt5682.c b/sound/soc/mediatek/mt8192/mt8192-mt6359-rt1015-rt5682.c
index ee91569c0911..219b38ee25ae 100644
--- a/sound/soc/mediatek/mt8192/mt8192-mt6359-rt1015-rt5682.c
+++ b/sound/soc/mediatek/mt8192/mt8192-mt6359-rt1015-rt5682.c
@@ -28,8 +28,13 @@
#define RT1015_DEV0_NAME "rt1015.1-0028"
#define RT1015_DEV1_NAME "rt1015.1-0029"
-#define RT5682_CODEC_DAI "rt5682-aif1"
-#define RT5682_DEV0_NAME "rt5682.1-001a"
+#define RT1015_RT5682_CARD_NAME "mt8192_mt6359_rt1015_rt5682"
+#define RT1015P_RT5682_CARD_NAME "mt8192_mt6359_rt1015p_rt5682"
+#define RT1015P_RT5682S_CARD_NAME "mt8192_mt6359_rt1015p_rt5682s"
+
+#define RT1015_RT5682_OF_NAME "mediatek,mt8192_mt6359_rt1015_rt5682"
+#define RT1015P_RT5682_OF_NAME "mediatek,mt8192_mt6359_rt1015p_rt5682"
+#define RT1015P_RT5682S_OF_NAME "mediatek,mt8192_mt6359_rt1015p_rt5682s"
struct mt8192_mt6359_priv {
struct snd_soc_jack headset_jack;
@@ -71,8 +76,8 @@ static int mt8192_rt1015_i2s_hw_params(struct snd_pcm_substream *substream,
return snd_soc_dai_set_sysclk(cpu_dai, 0, mclk_fs, SND_SOC_CLOCK_OUT);
}
-static int mt8192_rt5682_i2s_hw_params(struct snd_pcm_substream *substream,
- struct snd_pcm_hw_params *params)
+static int mt8192_rt5682x_i2s_hw_params(struct snd_pcm_substream *substream,
+ struct snd_pcm_hw_params *params)
{
struct snd_soc_pcm_runtime *rtd = asoc_substream_to_rtd(substream);
struct snd_soc_card *card = rtd->card;
@@ -121,8 +126,8 @@ static const struct snd_soc_ops mt8192_rt1015_i2s_ops = {
.hw_params = mt8192_rt1015_i2s_hw_params,
};
-static const struct snd_soc_ops mt8192_rt5682_i2s_ops = {
- .hw_params = mt8192_rt5682_i2s_hw_params,
+static const struct snd_soc_ops mt8192_rt5682x_i2s_ops = {
+ .hw_params = mt8192_rt5682x_i2s_hw_params,
};
static int mt8192_mt6359_mtkaif_calibration(struct snd_soc_pcm_runtime *rtd)
@@ -604,17 +609,9 @@ SND_SOC_DAILINK_DEFS(i2s2,
DAILINK_COMP_ARRAY(COMP_DUMMY()),
DAILINK_COMP_ARRAY(COMP_EMPTY()));
-SND_SOC_DAILINK_DEFS(i2s3_rt1015,
- DAILINK_COMP_ARRAY(COMP_CPU("I2S3")),
- DAILINK_COMP_ARRAY(COMP_CODEC(RT1015_DEV0_NAME,
- RT1015_CODEC_DAI),
- COMP_CODEC(RT1015_DEV1_NAME,
- RT1015_CODEC_DAI)),
- DAILINK_COMP_ARRAY(COMP_EMPTY()));
-
-SND_SOC_DAILINK_DEFS(i2s3_rt1015p,
+SND_SOC_DAILINK_DEFS(i2s3,
DAILINK_COMP_ARRAY(COMP_CPU("I2S3")),
- DAILINK_COMP_ARRAY(COMP_CODEC("rt1015p", "HiFi")),
+ DAILINK_COMP_ARRAY(COMP_EMPTY()),
DAILINK_COMP_ARRAY(COMP_EMPTY()));
SND_SOC_DAILINK_DEFS(i2s5,
@@ -634,14 +631,12 @@ SND_SOC_DAILINK_DEFS(i2s7,
SND_SOC_DAILINK_DEFS(i2s8,
DAILINK_COMP_ARRAY(COMP_CPU("I2S8")),
- DAILINK_COMP_ARRAY(COMP_CODEC(RT5682_DEV0_NAME,
- RT5682_CODEC_DAI)),
+ DAILINK_COMP_ARRAY(COMP_EMPTY()),
DAILINK_COMP_ARRAY(COMP_EMPTY()));
SND_SOC_DAILINK_DEFS(i2s9,
DAILINK_COMP_ARRAY(COMP_CPU("I2S9")),
- DAILINK_COMP_ARRAY(COMP_CODEC(RT5682_DEV0_NAME,
- RT5682_CODEC_DAI)),
+ DAILINK_COMP_ARRAY(COMP_EMPTY()),
DAILINK_COMP_ARRAY(COMP_EMPTY()));
SND_SOC_DAILINK_DEFS(connsys_i2s,
@@ -929,6 +924,7 @@ static struct snd_soc_dai_link mt8192_mt6359_dai_links[] = {
.dpcm_playback = 1,
.ignore_suspend = 1,
.be_hw_params_fixup = mt8192_i2s_hw_params_fixup,
+ SND_SOC_DAILINK_REG(i2s3),
},
{
.name = "I2S5",
@@ -962,7 +958,7 @@ static struct snd_soc_dai_link mt8192_mt6359_dai_links[] = {
.init = mt8192_rt5682_init,
.be_hw_params_fixup = mt8192_i2s_hw_params_fixup,
SND_SOC_DAILINK_REG(i2s8),
- .ops = &mt8192_rt5682_i2s_ops,
+ .ops = &mt8192_rt5682x_i2s_ops,
},
{
.name = "I2S9",
@@ -971,7 +967,7 @@ static struct snd_soc_dai_link mt8192_mt6359_dai_links[] = {
.ignore_suspend = 1,
.be_hw_params_fixup = mt8192_i2s_hw_params_fixup,
SND_SOC_DAILINK_REG(i2s9),
- .ops = &mt8192_rt5682_i2s_ops,
+ .ops = &mt8192_rt5682x_i2s_ops,
},
{
.name = "CONNSYS_I2S",
@@ -1051,7 +1047,7 @@ static struct snd_soc_codec_conf rt1015_amp_conf[] = {
};
static struct snd_soc_card mt8192_mt6359_rt1015_rt5682_card = {
- .name = "mt8192_mt6359_rt1015_rt5682",
+ .name = RT1015_RT5682_CARD_NAME,
.owner = THIS_MODULE,
.dai_link = mt8192_mt6359_dai_links,
.num_links = ARRAY_SIZE(mt8192_mt6359_dai_links),
@@ -1066,13 +1062,13 @@ static struct snd_soc_card mt8192_mt6359_rt1015_rt5682_card = {
};
static const struct snd_soc_dapm_widget
-mt8192_mt6359_rt1015p_rt5682_widgets[] = {
+mt8192_mt6359_rt1015p_rt5682x_widgets[] = {
SND_SOC_DAPM_SPK("Speakers", NULL),
SND_SOC_DAPM_HP("Headphone Jack", NULL),
SND_SOC_DAPM_MIC("Headset Mic", NULL),
};
-static const struct snd_soc_dapm_route mt8192_mt6359_rt1015p_rt5682_routes[] = {
+static const struct snd_soc_dapm_route mt8192_mt6359_rt1015p_rt5682x_routes[] = {
/* speaker */
{ "Speakers", NULL, "Speaker" },
/* headset */
@@ -1081,74 +1077,112 @@ static const struct snd_soc_dapm_route mt8192_mt6359_rt1015p_rt5682_routes[] = {
{ "IN1P", NULL, "Headset Mic" },
};
-static const struct snd_kcontrol_new mt8192_mt6359_rt1015p_rt5682_controls[] = {
+static const struct snd_kcontrol_new mt8192_mt6359_rt1015p_rt5682x_controls[] = {
SOC_DAPM_PIN_SWITCH("Speakers"),
SOC_DAPM_PIN_SWITCH("Headphone Jack"),
SOC_DAPM_PIN_SWITCH("Headset Mic"),
};
-static struct snd_soc_card mt8192_mt6359_rt1015p_rt5682_card = {
- .name = "mt8192_mt6359_rt1015p_rt5682",
+static struct snd_soc_card mt8192_mt6359_rt1015p_rt5682x_card = {
.owner = THIS_MODULE,
.dai_link = mt8192_mt6359_dai_links,
.num_links = ARRAY_SIZE(mt8192_mt6359_dai_links),
- .controls = mt8192_mt6359_rt1015p_rt5682_controls,
- .num_controls = ARRAY_SIZE(mt8192_mt6359_rt1015p_rt5682_controls),
- .dapm_widgets = mt8192_mt6359_rt1015p_rt5682_widgets,
- .num_dapm_widgets = ARRAY_SIZE(mt8192_mt6359_rt1015p_rt5682_widgets),
- .dapm_routes = mt8192_mt6359_rt1015p_rt5682_routes,
- .num_dapm_routes = ARRAY_SIZE(mt8192_mt6359_rt1015p_rt5682_routes),
+ .controls = mt8192_mt6359_rt1015p_rt5682x_controls,
+ .num_controls = ARRAY_SIZE(mt8192_mt6359_rt1015p_rt5682x_controls),
+ .dapm_widgets = mt8192_mt6359_rt1015p_rt5682x_widgets,
+ .num_dapm_widgets = ARRAY_SIZE(mt8192_mt6359_rt1015p_rt5682x_widgets),
+ .dapm_routes = mt8192_mt6359_rt1015p_rt5682x_routes,
+ .num_dapm_routes = ARRAY_SIZE(mt8192_mt6359_rt1015p_rt5682x_routes),
};
+static int mt8192_mt6359_card_set_be_link(struct snd_soc_card *card,
+ struct snd_soc_dai_link *link,
+ struct device_node *node,
+ char *link_name)
+{
+ int ret;
+
+ if (node && strcmp(link->name, link_name) == 0) {
+ ret = snd_soc_of_get_dai_link_codecs(card->dev, node, link);
+ if (ret < 0) {
+ dev_err(card->dev, "get dai link codecs fail\n");
+ return ret;
+ }
+ }
+
+ return 0;
+}
+
static int mt8192_mt6359_dev_probe(struct platform_device *pdev)
{
struct snd_soc_card *card;
- struct device_node *platform_node, *hdmi_codec;
+ struct device_node *platform_node, *hdmi_codec, *headset_codec, *speaker_codec;
int ret, i;
struct snd_soc_dai_link *dai_link;
struct mt8192_mt6359_priv *priv;
+ struct device *dev;
- platform_node = of_parse_phandle(pdev->dev.of_node,
- "mediatek,platform", 0);
- if (!platform_node) {
- dev_err(&pdev->dev, "Property 'platform' missing or invalid\n");
+ card = (struct snd_soc_card *)of_device_get_match_data(&pdev->dev);
+ if (!card)
return -EINVAL;
+ card->dev = &pdev->dev;
+ dev = &pdev->dev;
+
+ if (of_device_is_compatible(dev->of_node, RT1015P_RT5682_OF_NAME))
+ card->name = RT1015P_RT5682_CARD_NAME;
+ else if (of_device_is_compatible(dev->of_node, RT1015P_RT5682S_OF_NAME))
+ card->name = RT1015P_RT5682S_CARD_NAME;
+ else
+ dev_dbg(dev, "No need to set card name\n");
+
+ platform_node = of_parse_phandle(dev->of_node, "mediatek,platform", 0);
+ if (!platform_node) {
+ ret = -EINVAL;
+ dev_err_probe(dev, ret, "Property 'platform' missing or invalid\n");
+ goto err_platform_node;
}
- card = (struct snd_soc_card *)of_device_get_match_data(&pdev->dev);
- if (!card) {
+ hdmi_codec = of_parse_phandle(dev->of_node, "mediatek,hdmi-codec", 0);
+ if (!hdmi_codec) {
ret = -EINVAL;
- goto put_platform_node;
+ dev_err_probe(dev, ret, "Property 'hdmi-codec' missing or invalid\n");
+ goto err_hdmi_codec;
}
- card->dev = &pdev->dev;
- hdmi_codec = of_parse_phandle(pdev->dev.of_node,
- "mediatek,hdmi-codec", 0);
+ speaker_codec = of_get_child_by_name(dev->of_node, "mediatek,speaker-codec");
+ if (!speaker_codec) {
+ ret = -EINVAL;
+ dev_err_probe(dev, ret, "Property 'speaker_codec' missing or invalid\n");
+ goto err_speaker_codec;
+ }
+
+ headset_codec = of_get_child_by_name(dev->of_node, "mediatek,headset-codec");
+ if (!headset_codec) {
+ ret = -EINVAL;
+ dev_err_probe(dev, ret, "Property 'headset_codec' missing or invalid\n");
+ goto err_headset_codec;
+ }
for_each_card_prelinks(card, i, dai_link) {
- if (strcmp(dai_link->name, "I2S3") == 0) {
- if (card == &mt8192_mt6359_rt1015_rt5682_card) {
- dai_link->ops = &mt8192_rt1015_i2s_ops;
- dai_link->cpus = i2s3_rt1015_cpus;
- dai_link->num_cpus =
- ARRAY_SIZE(i2s3_rt1015_cpus);
- dai_link->codecs = i2s3_rt1015_codecs;
- dai_link->num_codecs =
- ARRAY_SIZE(i2s3_rt1015_codecs);
- dai_link->platforms = i2s3_rt1015_platforms;
- dai_link->num_platforms =
- ARRAY_SIZE(i2s3_rt1015_platforms);
- } else if (card == &mt8192_mt6359_rt1015p_rt5682_card) {
- dai_link->cpus = i2s3_rt1015p_cpus;
- dai_link->num_cpus =
- ARRAY_SIZE(i2s3_rt1015p_cpus);
- dai_link->codecs = i2s3_rt1015p_codecs;
- dai_link->num_codecs =
- ARRAY_SIZE(i2s3_rt1015p_codecs);
- dai_link->platforms = i2s3_rt1015p_platforms;
- dai_link->num_platforms =
- ARRAY_SIZE(i2s3_rt1015p_platforms);
- }
+ ret = mt8192_mt6359_card_set_be_link(card, dai_link, speaker_codec, "I2S3");
+ if (ret) {
+ dev_err_probe(&pdev->dev, ret, "%s set speaker_codec fail\n",
+ dai_link->name);
+ goto err_probe;
+ }
+
+ ret = mt8192_mt6359_card_set_be_link(card, dai_link, headset_codec, "I2S8");
+ if (ret) {
+ dev_err_probe(&pdev->dev, ret, "%s set headset_codec fail\n",
+ dai_link->name);
+ goto err_probe;
+ }
+
+ ret = mt8192_mt6359_card_set_be_link(card, dai_link, headset_codec, "I2S9");
+ if (ret) {
+ dev_err_probe(&pdev->dev, ret, "%s set %s headset_codec fail\n",
+ dai_link->name);
+ goto err_probe;
}
if (hdmi_codec && strcmp(dai_link->name, "TDM") == 0) {
@@ -1156,6 +1190,9 @@ static int mt8192_mt6359_dev_probe(struct platform_device *pdev)
dai_link->ignore = 0;
}
+ if (strcmp(dai_link->codecs[0].dai_name, RT1015_CODEC_DAI) == 0)
+ dai_link->ops = &mt8192_rt1015_i2s_ops;
+
if (!dai_link->platforms->name)
dai_link->platforms->of_node = platform_node;
}
@@ -1163,34 +1200,47 @@ static int mt8192_mt6359_dev_probe(struct platform_device *pdev)
priv = devm_kzalloc(&pdev->dev, sizeof(*priv), GFP_KERNEL);
if (!priv) {
ret = -ENOMEM;
- goto put_hdmi_codec;
+ goto err_probe;
}
snd_soc_card_set_drvdata(card, priv);
ret = mt8192_afe_gpio_init(&pdev->dev);
if (ret) {
- dev_err(&pdev->dev, "init gpio error %d\n", ret);
- goto put_hdmi_codec;
+ dev_err_probe(&pdev->dev, ret, "%s init gpio error\n", __func__);
+ goto err_probe;
}
ret = devm_snd_soc_register_card(&pdev->dev, card);
-
-put_hdmi_codec:
+ if (ret)
+ dev_err_probe(&pdev->dev, ret,
+ "%s snd_soc_register_card fail\n", __func__);
+
+err_probe:
+ of_node_put(headset_codec);
+err_headset_codec:
+ of_node_put(speaker_codec);
+err_speaker_codec:
of_node_put(hdmi_codec);
-put_platform_node:
+err_hdmi_codec:
of_node_put(platform_node);
+err_platform_node:
+
return ret;
}
#ifdef CONFIG_OF
static const struct of_device_id mt8192_mt6359_dt_match[] = {
{
- .compatible = "mediatek,mt8192_mt6359_rt1015_rt5682",
+ .compatible = RT1015_RT5682_OF_NAME,
.data = &mt8192_mt6359_rt1015_rt5682_card,
},
{
- .compatible = "mediatek,mt8192_mt6359_rt1015p_rt5682",
- .data = &mt8192_mt6359_rt1015p_rt5682_card,
+ .compatible = RT1015P_RT5682_OF_NAME,
+ .data = &mt8192_mt6359_rt1015p_rt5682x_card,
+ },
+ {
+ .compatible = RT1015P_RT5682S_OF_NAME,
+ .data = &mt8192_mt6359_rt1015p_rt5682x_card,
},
{}
};
--
2.18.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [v3 2/2] ASoC: mediatek: mt8192: support rt1015p_rt5682s
2022-03-11 2:15 ` [v3 2/2] ASoC: mediatek: mt8192: support rt1015p_rt5682s Jiaxin Yu
@ 2022-03-11 15:53 ` Jiaxin Yu
0 siblings, 0 replies; 4+ messages in thread
From: Jiaxin Yu @ 2022-03-11 15:53 UTC (permalink / raw)
To: broonie, robh+dt
Cc: aaronyu, matthias.bgg, Trevor Wu (吳文良),
tzungbi, linmq006, alsa-devel, devicetree, linux-arm-kernel,
linux-mediatek, linux-kernel,
Project_Global_Chrome_Upstream_Group, kernel test robot
On Fri, 2022-03-11 at 10:15 +0800, Jiaxin Yu wrote:
> Supports machines with rt1015p and rt5682s. Uses new proposed
> compatible
> string "mt8192_mt6359_rt1015p_rt5682s". Using define to simplifies
> card
> name and compatible name, and uses the
> snd_soc_of_get_dai_link_codecs()
> to complete the configuration of dai_link's codecs.
>
> Signed-off-by: Jiaxin Yu <jiaxin.yu@mediatek.com>
> Reported-by: kernel test robot <lkp@intel.com>
> ---
> sound/soc/mediatek/Kconfig | 1 +
> .../mt8192/mt8192-mt6359-rt1015-rt5682.c | 204 +++++++++++-----
> --
> 2 files changed, 128 insertions(+), 77 deletions(-)
>
> diff --git a/sound/soc/mediatek/Kconfig b/sound/soc/mediatek/Kconfig
> index d515613a79da..cacfbab4262d 100644
> --- a/sound/soc/mediatek/Kconfig
> +++ b/sound/soc/mediatek/Kconfig
> @@ -176,6 +176,7 @@ config SND_SOC_MT8192_MT6359_RT1015_RT5682
> select SND_SOC_RT1015
> select SND_SOC_RT1015P
> select SND_SOC_RT5682_I2C
> + select SND_SOC_RT5682S
> select SND_SOC_DMIC
> help
> This adds ASoC driver for Mediatek MT8192 boards
> diff --git a/sound/soc/mediatek/mt8192/mt8192-mt6359-rt1015-rt5682.c
> b/sound/soc/mediatek/mt8192/mt8192-mt6359-rt1015-rt5682.c
> index ee91569c0911..219b38ee25ae 100644
> --- a/sound/soc/mediatek/mt8192/mt8192-mt6359-rt1015-rt5682.c
> +++ b/sound/soc/mediatek/mt8192/mt8192-mt6359-rt1015-rt5682.c
> @@ -28,8 +28,13 @@
> #define RT1015_DEV0_NAME "rt1015.1-0028"
> #define RT1015_DEV1_NAME "rt1015.1-0029"
>
> -#define RT5682_CODEC_DAI "rt5682-aif1"
> -#define RT5682_DEV0_NAME "rt5682.1-001a"
> +#define RT1015_RT5682_CARD_NAME "mt8192_mt6359_rt1015_rt5682"
> +#define RT1015P_RT5682_CARD_NAME "mt8192_mt6359_rt1015p_rt5682"
> +#define RT1015P_RT5682S_CARD_NAME "mt8192_mt6359_rt1015p_rt5682s"
> +
> +#define RT1015_RT5682_OF_NAME "mediatek,mt8192_mt6359_rt1015_rt5682"
> +#define RT1015P_RT5682_OF_NAME
> "mediatek,mt8192_mt6359_rt1015p_rt5682"
> +#define RT1015P_RT5682S_OF_NAME
> "mediatek,mt8192_mt6359_rt1015p_rt5682s"
>
> struct mt8192_mt6359_priv {
> struct snd_soc_jack headset_jack;
> @@ -71,8 +76,8 @@ static int mt8192_rt1015_i2s_hw_params(struct
> snd_pcm_substream *substream,
> return snd_soc_dai_set_sysclk(cpu_dai, 0, mclk_fs,
> SND_SOC_CLOCK_OUT);
> }
>
> -static int mt8192_rt5682_i2s_hw_params(struct snd_pcm_substream
> *substream,
> - struct snd_pcm_hw_params
> *params)
> +static int mt8192_rt5682x_i2s_hw_params(struct snd_pcm_substream
> *substream,
> + struct snd_pcm_hw_params
> *params)
> {
> struct snd_soc_pcm_runtime *rtd =
> asoc_substream_to_rtd(substream);
> struct snd_soc_card *card = rtd->card;
> @@ -121,8 +126,8 @@ static const struct snd_soc_ops
> mt8192_rt1015_i2s_ops = {
> .hw_params = mt8192_rt1015_i2s_hw_params,
> };
>
> -static const struct snd_soc_ops mt8192_rt5682_i2s_ops = {
> - .hw_params = mt8192_rt5682_i2s_hw_params,
> +static const struct snd_soc_ops mt8192_rt5682x_i2s_ops = {
> + .hw_params = mt8192_rt5682x_i2s_hw_params,
> };
>
> static int mt8192_mt6359_mtkaif_calibration(struct
> snd_soc_pcm_runtime *rtd)
> @@ -604,17 +609,9 @@ SND_SOC_DAILINK_DEFS(i2s2,
> DAILINK_COMP_ARRAY(COMP_DUMMY()),
> DAILINK_COMP_ARRAY(COMP_EMPTY()));
>
> -SND_SOC_DAILINK_DEFS(i2s3_rt1015,
> - DAILINK_COMP_ARRAY(COMP_CPU("I2S3")),
> - DAILINK_COMP_ARRAY(COMP_CODEC(RT1015_DEV0_NAME,
> - RT1015_CODEC_DAI),
> - COMP_CODEC(RT1015_DEV1_NAME,
> - RT1015_CODEC_DAI)),
> - DAILINK_COMP_ARRAY(COMP_EMPTY()));
> -
> -SND_SOC_DAILINK_DEFS(i2s3_rt1015p,
> +SND_SOC_DAILINK_DEFS(i2s3,
> DAILINK_COMP_ARRAY(COMP_CPU("I2S3")),
> - DAILINK_COMP_ARRAY(COMP_CODEC("rt1015p", "HiFi")),
> + DAILINK_COMP_ARRAY(COMP_EMPTY()),
> DAILINK_COMP_ARRAY(COMP_EMPTY()));
>
> SND_SOC_DAILINK_DEFS(i2s5,
> @@ -634,14 +631,12 @@ SND_SOC_DAILINK_DEFS(i2s7,
>
> SND_SOC_DAILINK_DEFS(i2s8,
> DAILINK_COMP_ARRAY(COMP_CPU("I2S8")),
> - DAILINK_COMP_ARRAY(COMP_CODEC(RT5682_DEV0_NAME,
> - RT5682_CODEC_DAI)),
> + DAILINK_COMP_ARRAY(COMP_EMPTY()),
> DAILINK_COMP_ARRAY(COMP_EMPTY()));
>
> SND_SOC_DAILINK_DEFS(i2s9,
> DAILINK_COMP_ARRAY(COMP_CPU("I2S9")),
> - DAILINK_COMP_ARRAY(COMP_CODEC(RT5682_DEV0_NAME,
> - RT5682_CODEC_DAI)),
> + DAILINK_COMP_ARRAY(COMP_EMPTY()),
> DAILINK_COMP_ARRAY(COMP_EMPTY()));
>
> SND_SOC_DAILINK_DEFS(connsys_i2s,
> @@ -929,6 +924,7 @@ static struct snd_soc_dai_link
> mt8192_mt6359_dai_links[] = {
> .dpcm_playback = 1,
> .ignore_suspend = 1,
> .be_hw_params_fixup = mt8192_i2s_hw_params_fixup,
> + SND_SOC_DAILINK_REG(i2s3),
> },
> {
> .name = "I2S5",
> @@ -962,7 +958,7 @@ static struct snd_soc_dai_link
> mt8192_mt6359_dai_links[] = {
> .init = mt8192_rt5682_init,
> .be_hw_params_fixup = mt8192_i2s_hw_params_fixup,
> SND_SOC_DAILINK_REG(i2s8),
> - .ops = &mt8192_rt5682_i2s_ops,
> + .ops = &mt8192_rt5682x_i2s_ops,
> },
> {
> .name = "I2S9",
> @@ -971,7 +967,7 @@ static struct snd_soc_dai_link
> mt8192_mt6359_dai_links[] = {
> .ignore_suspend = 1,
> .be_hw_params_fixup = mt8192_i2s_hw_params_fixup,
> SND_SOC_DAILINK_REG(i2s9),
> - .ops = &mt8192_rt5682_i2s_ops,
> + .ops = &mt8192_rt5682x_i2s_ops,
> },
> {
> .name = "CONNSYS_I2S",
> @@ -1051,7 +1047,7 @@ static struct snd_soc_codec_conf
> rt1015_amp_conf[] = {
> };
>
> static struct snd_soc_card mt8192_mt6359_rt1015_rt5682_card = {
> - .name = "mt8192_mt6359_rt1015_rt5682",
> + .name = RT1015_RT5682_CARD_NAME,
> .owner = THIS_MODULE,
> .dai_link = mt8192_mt6359_dai_links,
> .num_links = ARRAY_SIZE(mt8192_mt6359_dai_links),
> @@ -1066,13 +1062,13 @@ static struct snd_soc_card
> mt8192_mt6359_rt1015_rt5682_card = {
> };
>
> static const struct snd_soc_dapm_widget
> -mt8192_mt6359_rt1015p_rt5682_widgets[] = {
> +mt8192_mt6359_rt1015p_rt5682x_widgets[] = {
> SND_SOC_DAPM_SPK("Speakers", NULL),
> SND_SOC_DAPM_HP("Headphone Jack", NULL),
> SND_SOC_DAPM_MIC("Headset Mic", NULL),
> };
>
> -static const struct snd_soc_dapm_route
> mt8192_mt6359_rt1015p_rt5682_routes[] = {
> +static const struct snd_soc_dapm_route
> mt8192_mt6359_rt1015p_rt5682x_routes[] = {
> /* speaker */
> { "Speakers", NULL, "Speaker" },
> /* headset */
> @@ -1081,74 +1077,112 @@ static const struct snd_soc_dapm_route
> mt8192_mt6359_rt1015p_rt5682_routes[] = {
> { "IN1P", NULL, "Headset Mic" },
> };
>
> -static const struct snd_kcontrol_new
> mt8192_mt6359_rt1015p_rt5682_controls[] = {
> +static const struct snd_kcontrol_new
> mt8192_mt6359_rt1015p_rt5682x_controls[] = {
> SOC_DAPM_PIN_SWITCH("Speakers"),
> SOC_DAPM_PIN_SWITCH("Headphone Jack"),
> SOC_DAPM_PIN_SWITCH("Headset Mic"),
> };
>
> -static struct snd_soc_card mt8192_mt6359_rt1015p_rt5682_card = {
> - .name = "mt8192_mt6359_rt1015p_rt5682",
> +static struct snd_soc_card mt8192_mt6359_rt1015p_rt5682x_card = {
> .owner = THIS_MODULE,
> .dai_link = mt8192_mt6359_dai_links,
> .num_links = ARRAY_SIZE(mt8192_mt6359_dai_links),
> - .controls = mt8192_mt6359_rt1015p_rt5682_controls,
> - .num_controls =
> ARRAY_SIZE(mt8192_mt6359_rt1015p_rt5682_controls),
> - .dapm_widgets = mt8192_mt6359_rt1015p_rt5682_widgets,
> - .num_dapm_widgets =
> ARRAY_SIZE(mt8192_mt6359_rt1015p_rt5682_widgets),
> - .dapm_routes = mt8192_mt6359_rt1015p_rt5682_routes,
> - .num_dapm_routes =
> ARRAY_SIZE(mt8192_mt6359_rt1015p_rt5682_routes),
> + .controls = mt8192_mt6359_rt1015p_rt5682x_controls,
> + .num_controls =
> ARRAY_SIZE(mt8192_mt6359_rt1015p_rt5682x_controls),
> + .dapm_widgets = mt8192_mt6359_rt1015p_rt5682x_widgets,
> + .num_dapm_widgets =
> ARRAY_SIZE(mt8192_mt6359_rt1015p_rt5682x_widgets),
> + .dapm_routes = mt8192_mt6359_rt1015p_rt5682x_routes,
> + .num_dapm_routes =
> ARRAY_SIZE(mt8192_mt6359_rt1015p_rt5682x_routes),
> };
>
> +static int mt8192_mt6359_card_set_be_link(struct snd_soc_card *card,
> + struct snd_soc_dai_link
> *link,
> + struct device_node *node,
> + char *link_name)
> +{
> + int ret;
> +
> + if (node && strcmp(link->name, link_name) == 0) {
> + ret = snd_soc_of_get_dai_link_codecs(card->dev, node,
> link);
> + if (ret < 0) {
> + dev_err(card->dev, "get dai link codecs
> fail\n");
> + return ret;
> + }
> + }
> +
> + return 0;
> +}
> +
> static int mt8192_mt6359_dev_probe(struct platform_device *pdev)
> {
> struct snd_soc_card *card;
> - struct device_node *platform_node, *hdmi_codec;
> + struct device_node *platform_node, *hdmi_codec, *headset_codec,
> *speaker_codec;
> int ret, i;
> struct snd_soc_dai_link *dai_link;
> struct mt8192_mt6359_priv *priv;
> + struct device *dev;
>
> - platform_node = of_parse_phandle(pdev->dev.of_node,
> - "mediatek,platform", 0);
> - if (!platform_node) {
> - dev_err(&pdev->dev, "Property 'platform' missing or
> invalid\n");
> + card = (struct snd_soc_card *)of_device_get_match_data(&pdev-
> >dev);
> + if (!card)
> return -EINVAL;
> + card->dev = &pdev->dev;
> + dev = &pdev->dev;
> +
> + if (of_device_is_compatible(dev->of_node,
> RT1015P_RT5682_OF_NAME))
> + card->name = RT1015P_RT5682_CARD_NAME;
> + else if (of_device_is_compatible(dev->of_node,
> RT1015P_RT5682S_OF_NAME))
> + card->name = RT1015P_RT5682S_CARD_NAME;
> + else
> + dev_dbg(dev, "No need to set card name\n");
> +
> + platform_node = of_parse_phandle(dev->of_node,
> "mediatek,platform", 0);
> + if (!platform_node) {
> + ret = -EINVAL;
> + dev_err_probe(dev, ret, "Property 'platform' missing or
> invalid\n");
> + goto err_platform_node;
> }
>
> - card = (struct snd_soc_card *)of_device_get_match_data(&pdev-
> >dev);
> - if (!card) {
> + hdmi_codec = of_parse_phandle(dev->of_node, "mediatek,hdmi-
> codec", 0);
> + if (!hdmi_codec) {
> ret = -EINVAL;
> - goto put_platform_node;
> + dev_err_probe(dev, ret, "Property 'hdmi-codec' missing
> or invalid\n");
> + goto err_hdmi_codec;
> }
> - card->dev = &pdev->dev;
>
> - hdmi_codec = of_parse_phandle(pdev->dev.of_node,
> - "mediatek,hdmi-codec", 0);
> + speaker_codec = of_get_child_by_name(dev->of_node,
> "mediatek,speaker-codec");
> + if (!speaker_codec) {
> + ret = -EINVAL;
> + dev_err_probe(dev, ret, "Property 'speaker_codec'
> missing or invalid\n");
> + goto err_speaker_codec;
> + }
> +
> + headset_codec = of_get_child_by_name(dev->of_node,
> "mediatek,headset-codec");
> + if (!headset_codec) {
> + ret = -EINVAL;
> + dev_err_probe(dev, ret, "Property 'headset_codec'
> missing or invalid\n");
> + goto err_headset_codec;
> + }
>
> for_each_card_prelinks(card, i, dai_link) {
> - if (strcmp(dai_link->name, "I2S3") == 0) {
> - if (card == &mt8192_mt6359_rt1015_rt5682_card)
> {
> - dai_link->ops = &mt8192_rt1015_i2s_ops;
> - dai_link->cpus = i2s3_rt1015_cpus;
> - dai_link->num_cpus =
> - ARRAY_SIZE(i2s3_rt1015_cpus);
> - dai_link->codecs = i2s3_rt1015_codecs;
> - dai_link->num_codecs =
> - ARRAY_SIZE(i2s3_rt1015_codecs);
> - dai_link->platforms =
> i2s3_rt1015_platforms;
> - dai_link->num_platforms =
> - ARRAY_SIZE(i2s3_rt1015_platform
> s);
> - } else if (card ==
> &mt8192_mt6359_rt1015p_rt5682_card) {
> - dai_link->cpus = i2s3_rt1015p_cpus;
> - dai_link->num_cpus =
> - ARRAY_SIZE(i2s3_rt1015p_cpus);
> - dai_link->codecs = i2s3_rt1015p_codecs;
> - dai_link->num_codecs =
> - ARRAY_SIZE(i2s3_rt1015p_codecs)
> ;
> - dai_link->platforms =
> i2s3_rt1015p_platforms;
> - dai_link->num_platforms =
> - ARRAY_SIZE(i2s3_rt1015p_platfor
> ms);
> - }
> + ret = mt8192_mt6359_card_set_be_link(card, dai_link,
> speaker_codec, "I2S3");
> + if (ret) {
> + dev_err_probe(&pdev->dev, ret, "%s set
> speaker_codec fail\n",
> + dai_link->name);
> + goto err_probe;
> + }
> +
> + ret = mt8192_mt6359_card_set_be_link(card, dai_link,
> headset_codec, "I2S8");
> + if (ret) {
> + dev_err_probe(&pdev->dev, ret, "%s set
> headset_codec fail\n",
> + dai_link->name);
> + goto err_probe;
> + }
> +
> + ret = mt8192_mt6359_card_set_be_link(card, dai_link,
> headset_codec, "I2S9");
> + if (ret) {
> + dev_err_probe(&pdev->dev, ret, "%s set %s
> headset_codec fail\n",
> + dai_link->name);
I'm terribly sorry, this will cause a build error, but my compiler
will not show it. I will fix it and send again.
> + goto err_probe;
> }
>
> if (hdmi_codec && strcmp(dai_link->name, "TDM") == 0) {
> @@ -1156,6 +1190,9 @@ static int mt8192_mt6359_dev_probe(struct
> platform_device *pdev)
> dai_link->ignore = 0;
> }
>
> + if (strcmp(dai_link->codecs[0].dai_name,
> RT1015_CODEC_DAI) == 0)
> + dai_link->ops = &mt8192_rt1015_i2s_ops;
> +
> if (!dai_link->platforms->name)
> dai_link->platforms->of_node = platform_node;
> }
> @@ -1163,34 +1200,47 @@ static int mt8192_mt6359_dev_probe(struct
> platform_device *pdev)
> priv = devm_kzalloc(&pdev->dev, sizeof(*priv), GFP_KERNEL);
> if (!priv) {
> ret = -ENOMEM;
> - goto put_hdmi_codec;
> + goto err_probe;
> }
> snd_soc_card_set_drvdata(card, priv);
>
> ret = mt8192_afe_gpio_init(&pdev->dev);
> if (ret) {
> - dev_err(&pdev->dev, "init gpio error %d\n", ret);
> - goto put_hdmi_codec;
> + dev_err_probe(&pdev->dev, ret, "%s init gpio error\n",
> __func__);
> + goto err_probe;
> }
>
> ret = devm_snd_soc_register_card(&pdev->dev, card);
> -
> -put_hdmi_codec:
> + if (ret)
> + dev_err_probe(&pdev->dev, ret,
> + "%s snd_soc_register_card fail\n",
> __func__);
> +
> +err_probe:
> + of_node_put(headset_codec);
> +err_headset_codec:
> + of_node_put(speaker_codec);
> +err_speaker_codec:
> of_node_put(hdmi_codec);
> -put_platform_node:
> +err_hdmi_codec:
> of_node_put(platform_node);
> +err_platform_node:
> +
> return ret;
> }
>
> #ifdef CONFIG_OF
> static const struct of_device_id mt8192_mt6359_dt_match[] = {
> {
> - .compatible = "mediatek,mt8192_mt6359_rt1015_rt5682",
> + .compatible = RT1015_RT5682_OF_NAME,
> .data = &mt8192_mt6359_rt1015_rt5682_card,
> },
> {
> - .compatible = "mediatek,mt8192_mt6359_rt1015p_rt5682",
> - .data = &mt8192_mt6359_rt1015p_rt5682_card,
> + .compatible = RT1015P_RT5682_OF_NAME,
> + .data = &mt8192_mt6359_rt1015p_rt5682x_card,
> + },
> + {
> + .compatible = RT1015P_RT5682S_OF_NAME,
> + .data = &mt8192_mt6359_rt1015p_rt5682x_card,
> },
> {}
> };
> --
> 2.18.0
>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2022-03-11 15:55 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-03-11 2:15 [v3 0/2] ASoC: mediatek: mt8192: support rt1015p_rt5682s Jiaxin Yu
2022-03-11 2:15 ` [v3 1/2] ASoC: dt-bindings: mt8192-mt6359: add new compatible for using rt1015p and rt5682 Jiaxin Yu
2022-03-11 2:15 ` [v3 2/2] ASoC: mediatek: mt8192: support rt1015p_rt5682s Jiaxin Yu
2022-03-11 15:53 ` Jiaxin Yu
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).