From: jeeja.kp@intel.com
To: alsa-devel@alsa-project.org
Cc: tiwai@suse.de, patches.audio@intel.com, broonie@kernel.org,
liam.r.girdwood@intel.com, Jeeja KP <jeeja.kp@intel.com>,
"Subhransu S. Prusty" <subhransu.s.prusty@intel.com>
Subject: [PATCH v2 03/13] ASoC: Intel: Skylake: Use set_tdm_slot to set the dma channel
Date: Wed, 18 Jan 2017 21:34:43 +0530 [thread overview]
Message-ID: <1484755493-30595-4-git-send-email-jeeja.kp@intel.com> (raw)
In-Reply-To: <1484755493-30595-1-git-send-email-jeeja.kp@intel.com>
From: Jeeja KP <jeeja.kp@intel.com>
DMA channel(stream tag) used by the HDA link need to programmed in
codec so that codec receives packet from the link associated with the
same channel.
DMA channel is allocated in link BE dai hw_params, the same needs to be
set for the BE codec dai. Instead of using get/set dma_data(), use
dai_ops snd_soc_dai_set_tdm_slot() to set the stream tag.
Signed-off-by: Subhransu S. Prusty <subhransu.s.prusty@intel.com>
Signed-off-by: Jeeja KP <jeeja.kp@intel.com>
---
sound/soc/intel/skylake/skl-pcm.c | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/sound/soc/intel/skylake/skl-pcm.c b/sound/soc/intel/skylake/skl-pcm.c
index 508144b..b0817a12 100644
--- a/sound/soc/intel/skylake/skl-pcm.c
+++ b/sound/soc/intel/skylake/skl-pcm.c
@@ -542,10 +542,10 @@ static int skl_link_hw_params(struct snd_pcm_substream *substream,
struct hdac_ext_bus *ebus = dev_get_drvdata(dai->dev);
struct hdac_ext_stream *link_dev;
struct snd_soc_pcm_runtime *rtd = snd_pcm_substream_chip(substream);
- struct hdac_ext_dma_params *dma_params;
struct snd_soc_dai *codec_dai = rtd->codec_dai;
struct skl_pipe_params p_params = {0};
struct hdac_ext_link *link;
+ int stream_tag;
link_dev = snd_hdac_ext_stream_assign(ebus, substream,
HDAC_EXT_STREAM_TYPE_LINK);
@@ -558,16 +558,16 @@ static int skl_link_hw_params(struct snd_pcm_substream *substream,
if (!link)
return -EINVAL;
+ stream_tag = hdac_stream(link_dev)->stream_tag;
+
/* set the stream tag in the codec dai dma params */
- dma_params = snd_soc_dai_get_dma_data(codec_dai, substream);
- if (dma_params)
- dma_params->stream_tag = hdac_stream(link_dev)->stream_tag;
+ snd_soc_dai_set_tdm_slot(codec_dai, stream_tag, 0, 0, 0);
p_params.s_fmt = snd_pcm_format_width(params_format(params));
p_params.ch = params_channels(params);
p_params.s_freq = params_rate(params);
p_params.stream = substream->stream;
- p_params.link_dma_id = hdac_stream(link_dev)->stream_tag - 1;
+ p_params.link_dma_id = stream_tag - 1;
p_params.link_index = link->index;
p_params.format = params_format(params);
--
2.5.0
next prev parent reply other threads:[~2017-01-18 15:53 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-01-18 16:04 [PATCH v2 00/13] ASoC: hdmi: Preparatory work to support MST audio jeeja.kp
2017-01-18 16:04 ` [PATCH v2 01/13] ASoC: hdac_hdmi: use audio component framework to read ELD jeeja.kp
2017-01-18 16:04 ` [PATCH v2 02/13] ASoC: hdac_hdmi: Register widget event handlers jeeja.kp
2017-01-20 13:35 ` Mark Brown
2017-01-23 10:05 ` Jeeja KP
2017-01-24 12:16 ` Mark Brown
2017-01-24 12:48 ` Jeeja KP
2017-01-18 16:04 ` jeeja.kp [this message]
2017-01-18 16:04 ` [PATCH v2 04/13] ASoC: hdac_hdmi: Move channel info from pin to PCM structure jeeja.kp
2017-01-18 16:04 ` [PATCH v2 05/13] ASoC: Intel: bxt: add channel map support in rt298 machine jeeja.kp
2017-01-18 16:04 ` [PATCH v2 06/13] ASoC: Intel: bxt: add channel map support in bxt_da7219_max98357a machine jeeja.kp
2017-01-18 16:04 ` [PATCH v2 07/13] ASoC: hdac_hdmi: Begin to add support for DP Multi-stream audio jeeja.kp
2017-01-18 16:04 ` [PATCH v2 08/13] ASoC: Intel: Skylake: Add route change to rt286 machine jeeja.kp
2017-01-18 16:04 ` [PATCH v2 09/13] ASoC: Intel: Skylake: Add route change to nau88l25_max98357a machine jeeja.kp
2017-01-18 16:04 ` [PATCH v2 10/13] ASoC: Intel: Skylake: Add route change to nau88l25_ssm4567 machine jeeja.kp
2017-01-18 16:04 ` [PATCH v2 11/13] ASoC: Intel: bxt: Add route change to rt298 machine jeeja.kp
2017-01-18 16:04 ` [PATCH v2 12/13] ASoC: Intel: bxt: Add route change to da7219_max98357a machine jeeja.kp
2017-01-18 16:04 ` [PATCH v2 13/13] ASoC: hdac_hdmi: Add support to handle MST capable pin jeeja.kp
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=1484755493-30595-4-git-send-email-jeeja.kp@intel.com \
--to=jeeja.kp@intel.com \
--cc=alsa-devel@alsa-project.org \
--cc=broonie@kernel.org \
--cc=liam.r.girdwood@intel.com \
--cc=patches.audio@intel.com \
--cc=subhransu.s.prusty@intel.com \
--cc=tiwai@suse.de \
/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 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).