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

Tanu Kaskinen tanu.kaskinen at linux.intel.com
Sun Nov 2 10:18:33 PST 2014


On Fri, 2014-10-31 at 22:17 +0100, Sjoerd Simons wrote:
> On Tue, 2014-10-28 at 12:10 +0200, Tanu Kaskinen wrote:
> > 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.
> 
> Ah now I remembered why i didn't do that in the end (i started with two
> different path files). It's because with the default setup on the X220
> you can't mute/disable the two paths seperately. 

What do you mean by "the default setup"? Is there some non-default setup
where they can be separately muted?

-- 
Tanu



More information about the pulseaudio-discuss mailing list