[alsa-devel] Question on Dynamic PCM

Sebastien LEDUC sebastien.leduc at stericsson.com
Fri May 18 16:02:09 CEST 2012


In commit "ASoC: dapm: Add API call to query valid DAPM paths<http://git.kernel.org/?p=linux/kernel/git/broonie/sound.git;a=commit;h=ec2e3031b65f23f66840b5c89c4b83076831a435>" , the signature of the functions is_connected_input_ep / is_connected_output_ep has been modified so that the 2 functions now fill list of connected DAPM widgets.

What I don't understand is why the way widgets are added is not symmetric between the two:

In function is_connected_output_ep:
-----------------------------------------------

 list_for_each_entry(path, &widget->sinks, list_source) {
                ...
                err = dapm_list_add_widget(list, path->sink);
                ...
}

In function is_connected_input_ep:
---------------------------------------------

list_for_each_entry(path, &widget->sources, list_sink) {
...
                err = dapm_list_add_widget(list, path->sink);
}


I would have expected to have for the latter:
err = dapm_list_add_widget(list, path->source);

regards,
Sebastien Le Duc






More information about the Alsa-devel mailing list