[pulseaudio-discuss] [PATCH 3/6] Cards now has ports directly, and device port has list of profiles

[David Henningsson]

On 11/08/2011 11:30 AM, Arun Raghavan wrote:
> On Thu, 2011-11-03 at 21:04 +0200, Tanu Kaskinen wrote:
>> Somehow keeping a list of profiles in the ports doesn't feel right -
>> it's as if that list would have been thrown there just to make things
>> convenient for some random code... But I guess there's a reason, which
>> just isn't apparent from this patch yet, for having that list there.
>
> This is my largest concern as well.

There is nothing wrong with every port knowing what profiles that port 
is a part of; it follows naturally with having port belonging to cards.

Looking at the paper that was the result of the desktop summit 
discussion, there are lines between ports and profiles, indicating 
pointers between them, or something similar. Sorry if I'm sounding harsh 
here, but I prefer not to rewrite my code once again just because you 
have changed your mind.

> It's the same concern that I had
> with Mengdong's suggestion that profiles should have an intended role --
> this feels conceptually incorrect, but becomes necessary because we
> don't know anything about the sink that will appear when the profile is
> activated.
>
> So this is my proposal -- all possible sinks for a card should be
> created upfront, in an "inactive" state. This way, from both the
> jack-detection and routing fronts, we can see what sink we want, and if
> it is inactive, we activate it by going to the profile it "belongs" to
> and activating that (clearly some conflict-resolution will be needed
> here too).
>
> This isn't a trivial change, but it's something that's been coming up
> repeatedly, and I'd be much happier if we took a little longer and did
> it right.
>
> Thoughts?

Since I don't see how that is supposed to work out, nor agree with the 
change in general, I'm not the right person to write code to do that 
change. Should you nevertheless decide to do so, I suggest you merge the 
already posted patch set first, then start working on that more radical 
change.

-- 
David Henningsson, Canonical Ltd.
http://launchpad.net/~diwic