[pulseaudio-discuss] [PATCH 5/5] alsa-sink: Don't assume we were able to enable hw-volume or sync-volume (v1.1)

Colin Guthrie gmane at colin.guthr.ie
Sat Jan 15 05:55:06 PST 2011


'Twas brillig, and oku at iki.fi at 13/01/11 14:44 did gyre and gimble:
> From: Jyri Sarha <jyri.sarha at nokia.com>
> 
> This patch also disables mixer callback code if we do not have neither
> HW-volume or HW-mute.


Forgive me if I'm just having a slow day, but the check you are now using:

if (u->sink->flags & (PA_SINK_HW_VOLUME_CTRL|PA_SINK_HW_MUTE_CTRL))

would seem to be checking that it has *both* VOLUME and MUTE_CTRL,
rather than *either* of them.

I've merged the other 4 patches but can you take a look at this issue
(or just correct my general dumbness)?

Cheers

Col

-- 

Colin Guthrie
gmane(at)colin.guthr.ie
http://colin.guthr.ie/

Day Job:
  Tribalogic Limited [http://www.tribalogic.net/]
Open Source:
  Mageia Contributor [http://www.mageia.org/]
  PulseAudio Hacker [http://www.pulseaudio.org/]
  Trac Hacker [http://trac.edgewall.org/]



More information about the pulseaudio-discuss mailing list