[alsa-devel] [PATCH v2 1/2] ASoC: dapm: support user-defined stop condition in dai_get_connected_widgets

[Mark Brown]

On Fri, May 13, 2016 at 02:01:43PM +0100, Piotr Stankiewicz wrote:
> On Fri, May 13, 2016 at 01:09:05PM +0100, Mark Brown wrote:

> > This seems really weird and uncomfortable.  From the naming I would
> > expect it to be a boolean function and I'm struggling to understand when
> > it would be useful to report multiple endpoints on a widget (edge
> > widgets normally being single objects).  The whole counting thing is
> > mainly a combination of debugging and an artefact of the implementation.
> > What's the motivation here?

> I don't have a strong reason for this. My motivation here was - since I
> am adding support for stopping an arbitrary point, it could be that
> someone would find a use in stopping the graph walk part-way through, and
> still reporting the number of endpoints as normal. But, if that's
> unlikely to be of use, I'm happy to make it a boolean function.

Why would the number of endpoints at the edge of the graph ever be
something other than 0 or 1?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: not available
URL: <http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20160513/fe3efc1a/attachment.sig>