Re: [alsa-devel] [PATCH v2] ASoC: dapm - Add API call to query the widgets of valid DAPM paths.

On Wed, Jul 20, 2011 at 12:23:15PM +0100, Liam Girdwood wrote:

•                   dev_dbg(widget->dapm->dev," %c : %s -> %s -> %s\n",
  

•                           path->sink && path->connect ? '*' : ' ',
  

•                           widget->name, path->name, path->sink->name);
  

We should fix the debugfs stuff to include unconnected paths too, only showing currently active paths was a mistake. Might be worth dialing down to vdbg too, we look at DAPM a *lot* during scenario changes and on big systems it can obscure what's going on.

+/**

• • snd_soc_dapm_get_connected_widgets - query audio path and it's widgets.
• • @dapm: the dapm context.
• • @stream_name: stream name.
• • @list: list of active widgets for this stream.
• • @stream: stream direction.
• • Queries DAPM graph as to whether an valid audio stream path exists for
• • the initial widget specified by name. This takes into account

s/initial widget/stream/

• /* get stream root widget AIF from stream string and direction */
• list_for_each_entry(w, &dapm->card->widgets, list) {

• if (w->id != type)
  

• 	continue;
  

• if (strstr(w->sname, stream_name))
  

• 	goto found;
  

• }
• dev_err(dapm->dev, "root widget for %s not found\n", stream_name);

Hrm, this will only check the widgets connected to the first widget it sees for an AIF. For streams with multiple widgets connected (eg, left and right channel) it's only going to see things on the first widget it comes across which probably isn't what we were expecting. Unless we change to match the comment and query by widget name in which case we're already requiring uniqueness.