All of lore.kernel.org
 help / color / mirror / Atom feed
From: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
To: linux-sound@vger.kernel.org
Cc: alsa-devel@alsa-project.org, tiwai@suse.de, broonie@kernel.org,
	"Charles Keepax" <ckeepax@opensource.cirrus.com>,
	"Richard Fitzgerald" <rf@opensource.cirrus.com>,
	"Shuming Fan" <shumingf@realtek.com>,
	"Jack Yu" <jack.yu@realtek.com>,
	"Oder Chiou" <oder_chiou@realtek.com>,
	"Bard liao" <yung-chuan.liao@linux.intel.com>,
	"Chao Song" <chao.song@linux.intel.com>,
	"Péter Ujfalusi" <peter.ujfalusi@linux.intel.com>,
	"Pierre-Louis Bossart" <pierre-louis.bossart@linux.intel.com>
Subject: [PATCH 26/34] ASoC: Intel: sof_sdw_rt722_sdca: set rtd_init in codec_info_list[]
Date: Tue, 26 Mar 2024 11:04:21 -0500	[thread overview]
Message-ID: <20240326160429.13560-27-pierre-louis.bossart@linux.intel.com> (raw)
In-Reply-To: <20240326160429.13560-1-pierre-louis.bossart@linux.intel.com>

From: Bard Liao <yung-chuan.liao@linux.intel.com>

Add rt722 into rt_sdca_jack_rtd_init() supported list, and set
rtd_init callback directly in codec_info_list[]. No need to use a
init callback to set the rtd_init callback.
Besides, sof_sdw_rt_sdca_jack_init() doesn't set .rtd_init anymore.

Reviewed-by: Chao Song <chao.song@linux.intel.com>
Reviewed-by: Péter Ujfalusi <peter.ujfalusi@linux.intel.com>
Signed-off-by: Bard Liao <yung-chuan.liao@linux.intel.com>
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
---
 sound/soc/intel/boards/sof_sdw.c              |  5 ++--
 sound/soc/intel/boards/sof_sdw_common.h       | 14 ++--------
 sound/soc/intel/boards/sof_sdw_rt722_sdca.c   | 26 ++-----------------
 .../boards/sof_sdw_rt_sdca_jack_common.c      |  2 +-
 4 files changed, 8 insertions(+), 39 deletions(-)

diff --git a/sound/soc/intel/boards/sof_sdw.c b/sound/soc/intel/boards/sof_sdw.c
index 0e376cb0ce0b..bb8429a83467 100644
--- a/sound/soc/intel/boards/sof_sdw.c
+++ b/sound/soc/intel/boards/sof_sdw.c
@@ -902,6 +902,7 @@ static struct sof_sdw_codec_info codec_info_list[] = {
 				.dailink = {SDW_JACK_OUT_DAI_ID, SDW_JACK_IN_DAI_ID},
 				.init = sof_sdw_rt_sdca_jack_init,
 				.exit = sof_sdw_rt_sdca_jack_exit,
+				.rtd_init = rt_sdca_jack_rtd_init,
 			},
 			{
 				.direction = {true, false},
@@ -909,14 +910,14 @@ static struct sof_sdw_codec_info codec_info_list[] = {
 				.dai_type = SOF_SDW_DAI_TYPE_AMP,
 				/* No feedback capability is provided by rt722-sdca codec driver*/
 				.dailink = {SDW_AMP_OUT_DAI_ID, SDW_UNUSED_DAI_ID},
-				.init = sof_sdw_rt722_spk_init,
+				.rtd_init = rt722_spk_rtd_init,
 			},
 			{
 				.direction = {false, true},
 				.dai_name = "rt722-sdca-aif3",
 				.dai_type = SOF_SDW_DAI_TYPE_MIC,
 				.dailink = {SDW_UNUSED_DAI_ID, SDW_DMIC_DAI_ID},
-				.init = sof_sdw_rt722_sdca_dmic_init,
+				.rtd_init = rt722_sdca_dmic_rtd_init,
 			},
 		},
 		.dai_num = 3,
diff --git a/sound/soc/intel/boards/sof_sdw_common.h b/sound/soc/intel/boards/sof_sdw_common.h
index 84715d9ca942..315b2a298195 100644
--- a/sound/soc/intel/boards/sof_sdw_common.h
+++ b/sound/soc/intel/boards/sof_sdw_common.h
@@ -157,18 +157,6 @@ int sof_sdw_rt_amp_init(struct snd_soc_card *card,
 			bool playback);
 int sof_sdw_rt_amp_exit(struct snd_soc_card *card, struct snd_soc_dai_link *dai_link);
 
-/* RT722-SDCA support */
-int sof_sdw_rt722_spk_init(struct snd_soc_card *card,
-			   const struct snd_soc_acpi_link_adr *link,
-			   struct snd_soc_dai_link *dai_links,
-			   struct sof_sdw_codec_info *info,
-			   bool playback);
-int sof_sdw_rt722_sdca_dmic_init(struct snd_soc_card *card,
-				 const struct snd_soc_acpi_link_adr *link,
-				 struct snd_soc_dai_link *dai_links,
-				 struct sof_sdw_codec_info *info,
-				 bool playback);
-
 /* MAXIM codec support */
 int sof_sdw_maxim_init(struct snd_soc_card *card,
 		       const struct snd_soc_acpi_link_adr *link,
@@ -194,6 +182,8 @@ int rt5682_rtd_init(struct snd_soc_pcm_runtime *rtd);
 int rt700_rtd_init(struct snd_soc_pcm_runtime *rtd);
 int rt711_rtd_init(struct snd_soc_pcm_runtime *rtd);
 int rt712_spk_rtd_init(struct snd_soc_pcm_runtime *rtd);
+int rt722_spk_rtd_init(struct snd_soc_pcm_runtime *rtd);
+int rt722_sdca_dmic_rtd_init(struct snd_soc_pcm_runtime *rtd);
 int rt_dmic_rtd_init(struct snd_soc_pcm_runtime *rtd);
 int rt_amp_spk_rtd_init(struct snd_soc_pcm_runtime *rtd);
 int rt_sdca_jack_rtd_init(struct snd_soc_pcm_runtime *rtd);
diff --git a/sound/soc/intel/boards/sof_sdw_rt722_sdca.c b/sound/soc/intel/boards/sof_sdw_rt722_sdca.c
index fe3a2bff95bc..4947d16c70db 100644
--- a/sound/soc/intel/boards/sof_sdw_rt722_sdca.c
+++ b/sound/soc/intel/boards/sof_sdw_rt722_sdca.c
@@ -27,7 +27,7 @@ static const struct snd_kcontrol_new rt722_spk_controls[] = {
 	SOC_DAPM_PIN_SWITCH("Speaker"),
 };
 
-static int rt722_spk_init(struct snd_soc_pcm_runtime *rtd)
+int rt722_spk_rtd_init(struct snd_soc_pcm_runtime *rtd)
 {
 	struct snd_soc_card *card = rtd->card;
 	int ret;
@@ -59,18 +59,7 @@ static int rt722_spk_init(struct snd_soc_pcm_runtime *rtd)
 	return ret;
 }
 
-int sof_sdw_rt722_spk_init(struct snd_soc_card *card,
-			   const struct snd_soc_acpi_link_adr *link,
-			   struct snd_soc_dai_link *dai_links,
-			   struct sof_sdw_codec_info *info,
-			   bool playback)
-{
-	dai_links->init = rt722_spk_init;
-
-	return 0;
-}
-
-static int rt722_sdca_dmic_rtd_init(struct snd_soc_pcm_runtime *rtd)
+int rt722_sdca_dmic_rtd_init(struct snd_soc_pcm_runtime *rtd)
 {
 	struct snd_soc_card *card = rtd->card;
 	struct snd_soc_dai *codec_dai = snd_soc_rtd_to_codec(rtd, 0);
@@ -84,14 +73,3 @@ static int rt722_sdca_dmic_rtd_init(struct snd_soc_pcm_runtime *rtd)
 
 	return 0;
 }
-
-int sof_sdw_rt722_sdca_dmic_init(struct snd_soc_card *card,
-				 const struct snd_soc_acpi_link_adr *link,
-				 struct snd_soc_dai_link *dai_links,
-				 struct sof_sdw_codec_info *info,
-				 bool playback)
-{
-	dai_links->init = rt722_sdca_dmic_rtd_init;
-
-	return 0;
-}
diff --git a/sound/soc/intel/boards/sof_sdw_rt_sdca_jack_common.c b/sound/soc/intel/boards/sof_sdw_rt_sdca_jack_common.c
index fe59a144efef..006771485486 100644
--- a/sound/soc/intel/boards/sof_sdw_rt_sdca_jack_common.c
+++ b/sound/soc/intel/boards/sof_sdw_rt_sdca_jack_common.c
@@ -85,7 +85,7 @@ static struct snd_soc_jack_pin rt_sdca_jack_pins[] = {
 };
 
 static const char * const jack_codecs[] = {
-	"rt711", "rt712", "rt713"
+	"rt711", "rt712", "rt713", "rt722"
 };
 
 int rt_sdca_jack_rtd_init(struct snd_soc_pcm_runtime *rtd)
-- 
2.40.1


  parent reply	other threads:[~2024-03-26 16:05 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-03-26 16:03 [PATCH 00/34] ASoC: Intel: boards: updates for 6.10 - part2 Pierre-Louis Bossart
2024-03-26 16:03 ` [PATCH 01/34] ASoC: intel: sof_sdw: Make find_codec_info_part() return a pointer Pierre-Louis Bossart
2024-03-26 16:03 ` [PATCH 02/34] ASoC: intel: sof_sdw: Make find_codec_info_acpi() " Pierre-Louis Bossart
2024-03-26 16:03 ` [PATCH 03/34] ASoC: intel: sof_sdw: Make find_codec_info_dai() " Pierre-Louis Bossart
2024-03-26 16:03 ` [PATCH 04/34] ASoC: intel: sof_sdw: Only pass codec_conf pointer around Pierre-Louis Bossart
2024-03-26 16:04 ` [PATCH 05/34] ASoC: intel: sof_sdw: Set channel map directly from endpoints Pierre-Louis Bossart
2024-03-26 16:04 ` [PATCH 06/34] ASoC: Intel: sof_sdw: Move get_codec_dai_by_name() into sof_sdw itself Pierre-Louis Bossart
2024-03-26 16:04 ` [PATCH 07/34] ASoC: Intel: sof_sdw: Move flags to private struct Pierre-Louis Bossart
2024-03-26 16:04 ` [PATCH 08/34] ASoC: Intel: sof_sdw: Only pass dai_link pointer around Pierre-Louis Bossart
2024-03-26 16:04 ` [PATCH 09/34] ASoC: Intel: sof_sdw: Use for_each_set_bit Pierre-Louis Bossart
2024-03-26 16:04 ` [PATCH 10/34] ASoC: Intel: sof_sdw: Factor out SSP DAI creation Pierre-Louis Bossart
2024-03-26 16:04 ` [PATCH 11/34] ASoC: Intel: sof_sdw: Factor out DMIC " Pierre-Louis Bossart
2024-03-26 16:04 ` [PATCH 12/34] ASoC: Intel: sof_sdw: Factor out HDMI " Pierre-Louis Bossart
2024-03-26 16:04 ` [PATCH 13/34] ASoC: Intel: sof_sdw: Factor out BlueTooth " Pierre-Louis Bossart
2024-03-26 16:04 ` [PATCH 14/34] ASoC: Intel: sof_sdw: Factor out codec name generation Pierre-Louis Bossart
2024-03-26 16:04 ` [PATCH 15/34] ASoC: Intel: soc-acpi-intel-arl-match: Add rt711 sdca codec support Pierre-Louis Bossart
2024-03-26 16:04 ` [PATCH 16/34] ASoC: Intel: sof_sdw: Remove no longer supported quirk Pierre-Louis Bossart
2024-03-26 16:04 ` [PATCH 17/34] ASoC: intel: soc-acpi: Add missing cs42l43 endpoints Pierre-Louis Bossart
2024-03-26 16:04 ` [PATCH 18/34] ASoC: Intel: sof-sdw: Add new code for parsing the snd_soc_acpi structs Pierre-Louis Bossart
2024-03-26 16:04 ` [PATCH 19/34] ASoC: Intel: sof_sdw: Move counting and codec_conf to new parsing Pierre-Louis Bossart
2024-03-26 16:04 ` [PATCH 20/34] ASoC: Intel: sof_sdw: Move ignore_pch_dmic " Pierre-Louis Bossart
2024-03-26 16:04 ` [PATCH 21/34] ASoC: Intel: sof_sdw: Move append_dai_type " Pierre-Louis Bossart
2024-03-26 16:04 ` [PATCH 22/34] ASoC: Intel: sof_sdw: Move generation of DAI links " Pierre-Louis Bossart
2024-03-26 16:04 ` [PATCH 23/34] ASoC: intel: sof_sdw: Factor out SoundWire DAI creation Pierre-Louis Bossart
2024-03-26 16:04 ` [PATCH 24/34] ASoC: Intel: sof_sdw: use generic rtd_init function for Realtek SDW DMICs Pierre-Louis Bossart
2024-03-26 16:04 ` [PATCH 25/34] ASoC: Intel: sof_sdw: remove unused rt dmic rtd_init Pierre-Louis Bossart
2024-03-26 16:04 ` Pierre-Louis Bossart [this message]
2024-03-26 16:04 ` [PATCH 27/34] ASoC: Intel: sof_sdw_rt722_sdca: use rt_dmic_rtd_init Pierre-Louis Bossart
2024-03-26 16:04 ` [PATCH 28/34] ASoC: rt715-sdca: rename dai name with rt715-sdca prefix Pierre-Louis Bossart
2024-03-26 16:04 ` [PATCH 29/34] ASoC: Intel: sof_sdw: change rt715-sdca dai name Pierre-Louis Bossart
2024-03-26 16:04 ` [PATCH 30/34] ASoC: Intel: change cs35l56 name_prefix Pierre-Louis Bossart
2024-03-26 16:04 ` [PATCH 31/34] ASoC: Intel: sof_sdw: Don't pass acpi_link_adr to init functions Pierre-Louis Bossart
2024-03-26 16:04 ` [PATCH 32/34] ASoC: Intel: sof_sdw: Remove redundant initialisations Pierre-Louis Bossart
2024-03-26 16:04 ` [PATCH 33/34] ASoC: Intel: sof_sdw: Add quirk for optional codec speakers Pierre-Louis Bossart
2024-03-26 16:04 ` [PATCH 34/34] ASoC: Intel: sof_sdw: Add support for cs42l43 optional speaker output Pierre-Louis Bossart
2024-03-27 15:10 ` [PATCH 00/34] ASoC: Intel: boards: updates for 6.10 - part2 Mark Brown

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20240326160429.13560-27-pierre-louis.bossart@linux.intel.com \
    --to=pierre-louis.bossart@linux.intel.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=broonie@kernel.org \
    --cc=chao.song@linux.intel.com \
    --cc=ckeepax@opensource.cirrus.com \
    --cc=jack.yu@realtek.com \
    --cc=linux-sound@vger.kernel.org \
    --cc=oder_chiou@realtek.com \
    --cc=peter.ujfalusi@linux.intel.com \
    --cc=rf@opensource.cirrus.com \
    --cc=shumingf@realtek.com \
    --cc=tiwai@suse.de \
    --cc=yung-chuan.liao@linux.intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.