[pulseaudio-discuss] [PATCH] Alsa: Correct port availability with multiple jacks

Tanu Kaskinen tanu.kaskinen at linux.intel.com
Tue Oct 28 03:10:11 PDT 2014


On Tue, 2014-10-28 at 05:30 +0100, David Henningsson wrote:
> 
> On 2014-10-24 22:33, Sjoerd Simons wrote:
> > In case there are two independent jacks for one port (e.g. Dock
> > Headphone Jack and Headphone Jack), the availability ends up being
> > incorrect for one of the first was _NO (not plugged) and the second gets
> > _YES (plugged). Also pulse complains about the state being inconsistent
> > which isn't true.
> >
> > Fix this by preferring higher states (_YES > _NO > _UNKNOWN)
> 
> Thanks, I agree with this conclusion. I didn't really know what to do 
> with one jack being yes and another being no, but the above reasoning 
> makes sense.
> 
> I'm not sure I'm happy with just a simple ">" comparison - in case we 
> add more jack states in the future, they might not fit with the ">" 
> scheme. But maybe we should deal with that when the problem comes and 
> just take your patch as-is now. What do other people think?

I'd prefer replacing the ">" comparison with something else too.

The use case that this patch is made for seems such that there shouldn't
be any jack state conflict to worry about in the first place, though. To
me "dock headphone" and "built-in headphone" seem like obviously
separate paths, so I think we should reflect that in our configuration
by creating a new path file.

-- 
Tanu



More information about the pulseaudio-discuss mailing list