alsa-devel.alsa-project.org archive mirror
 help / color / mirror / Atom feed
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

  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).