From: Peter Ujfalusi <peter.ujfalusi@ti.com>
To: Daniel Mack <zonque@gmail.com>
Cc: ALSA development <alsa-devel@alsa-project.org>,
Mark Brown <broonie@kernel.org>,
Liam Girdwood <lgirdwood@gmail.com>
Subject: Re: Ambiguous DAPM widget names and DPCM
Date: Fri, 26 Jul 2013 11:13:17 +0200 [thread overview]
Message-ID: <51F23DAD.70305@ti.com> (raw)
In-Reply-To: <51EEA2A3.8030106@gmail.com>
On 07/23/2013 05:34 PM, Daniel Mack wrote:
> With printk() in snd_soc_dapm_new_control(), I see the following widgets
> being created during boot (one pair for each Codec, and two pairs for
> the platform dai):
>
> [ 1.963501] snd_soc_dapm_new_control() ceb6b680 name Playback
> [ 1.971739] snd_soc_dapm_new_control() ceb6b5c0 name Capture
> [ 1.996001] snd_soc_dapm_new_control() ceb6b500 name Playback
> [ 2.004354] snd_soc_dapm_new_control() ceb6b440 name Capture
> [ 2.019684] snd_soc_dapm_new_control() ceb6b380 name Playback
> [ 2.028053] snd_soc_dapm_new_control() ceb6b2c0 name Capture
> [ 2.047526] snd_soc_dapm_new_control() ceb6b200 name Playback
> [ 2.055846] snd_soc_dapm_new_control() ceb6b140 name Capture
> [ 2.071168] snd_soc_dapm_new_control() ceb6b080 name Playback
> [ 2.079461] snd_soc_dapm_new_control() ceb8ff00 name Capture
Can you print these with dev_err(dapm->dev, ...); so we can see the context
where it is going to be added?
It might worth asking the core to prefix the codecs DAPM, so you avoid name
duplication on the same context.
Both cs4271 and ak4101 have 'Playback'/'Capture' names for the streams.
BTW: do you see error messages about name duplication ("widget %s already
exists for context\n")? Or warning about already existing debugfs entries from
DAPM?
--
Péter
next prev parent reply other threads:[~2013-07-26 9:13 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-07-23 8:30 Ambiguous DAPM widget names and DPCM Daniel Mack
2013-07-23 12:30 ` Mark Brown
2013-07-23 15:34 ` Daniel Mack
2013-07-23 18:30 ` Mark Brown
2013-07-25 12:24 ` Daniel Mack
2013-07-25 12:48 ` Mark Brown
2013-07-25 19:53 ` Liam Girdwood
2013-07-26 9:13 ` Peter Ujfalusi [this message]
2013-07-23 12:48 ` Liam Girdwood
2014-03-21 16:50 ` Patrick Lai
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=51F23DAD.70305@ti.com \
--to=peter.ujfalusi@ti.com \
--cc=alsa-devel@alsa-project.org \
--cc=broonie@kernel.org \
--cc=lgirdwood@gmail.com \
--cc=zonque@gmail.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 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).