Messages in this thread | | | Date | Fri, 10 Jun 2022 12:01:54 +0200 | Subject | Re: [PATCH v7 3/8] ASoC: mediatek: mt8186: add mt8186-mt6366 common driver | From | AngeloGioacchino Del Regno <> |
| |
Il 10/06/22 10:27, Jiaxin Yu ha scritto: > Add mt8186-mt6366 common driver for mt8186 series machine. > > Signed-off-by: Jiaxin Yu <jiaxin.yu@mediatek.com> > --- > sound/soc/mediatek/mt8186/Makefile | 1 + > .../mediatek/mt8186/mt8186-mt6366-common.c | 59 +++++++++++++++++++ > .../mediatek/mt8186/mt8186-mt6366-common.h | 17 ++++++ > 3 files changed, 77 insertions(+) > create mode 100644 sound/soc/mediatek/mt8186/mt8186-mt6366-common.c > create mode 100644 sound/soc/mediatek/mt8186/mt8186-mt6366-common.h >
..snip..
> diff --git a/sound/soc/mediatek/mt8186/mt8186-mt6366-common.c b/sound/soc/mediatek/mt8186/mt8186-mt6366-common.c > new file mode 100644 > index 000000000000..94e1128e8128 > --- /dev/null > +++ b/sound/soc/mediatek/mt8186/mt8186-mt6366-common.c > @@ -0,0 +1,59 @@ > +// SPDX-License-Identifier: GPL-2.0 > +// > +// mt8186-mt6366-common.c > +// -- MT8186 MT6366 ALSA common driver > +// > +// Copyright (c) 2022 MediaTek Inc. > +// Author: Jiaxin Yu <jiaxin.yu@mediatek.com> > +// > +#include <sound/soc.h> > + > +#include "../../codecs/mt6358.h" > +#include "../common/mtk-afe-platform-driver.h" > +#include "mt8186-afe-common.h" > +#include "mt8186-mt6366-common.h" > + > +int mt8186_mt6366_init(struct snd_soc_pcm_runtime *rtd) > +{ > + struct snd_soc_component *cmpnt_afe = > + snd_soc_rtdcom_lookup(rtd, AFE_PCM_NAME); > + struct snd_soc_component *cmpnt_codec = > + asoc_rtd_to_codec(rtd, 0)->component; > + struct mtk_base_afe *afe = snd_soc_component_get_drvdata(cmpnt_afe); > + struct mt8186_afe_private *afe_priv = afe->platform_priv; > + struct snd_soc_dapm_context *dapm = &rtd->card->dapm; > + int ret; > + > + /* set mtkaif protocol */ > + mt6358_set_mtkaif_protocol(cmpnt_codec, > + MT6358_MTKAIF_PROTOCOL_1); > + afe_priv->mtkaif_protocol = MT6358_MTKAIF_PROTOCOL_1; > + > + ret = snd_soc_dapm_sync(dapm); > + if (ret) { > + dev_info(rtd->dev, "failed to snd_soc_dapm_sync\n");
This should be a dev_err().
> + return ret; > + } > + > + return 0; > +} > +EXPORT_SYMBOL_GPL(mt8186_mt6366_init); > + > +int mt8186_mt6366_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_probe(card->dev, ret, "get dai link codecs fail\n"); > + return ret;
You may at this point just 'return dev_err_probe(...)'.
Regards, Angelo
| |