[pulseaudio-discuss] [PATCH] config: do not change digital controls from analog path config

[Puustinen, Ismo]

On Wed, 2014-12-10 at 18:47 +0500, Alexander E. Patrakov wrote:
> 10.12.2014 18:42, Arun Raghavan wrote:
> > On 10 December 2014 at 19:08, Ismo Puustinen <ismo.puustinen at intel.com> wrote:
> >> When the default ALSA analog path configuration is loaded, it disables
> >> digital IEC958 mixer controls. This should not be needed, since digital outputs
> >> are handled from a different sink. Therefore remove the configuration that
> >> touches the controls.
> >> ---
> >>   src/modules/alsa/mixer/paths/analog-output.conf.common | 6 ------
> >>   1 file changed, 6 deletions(-)
> >>
> >> diff --git a/src/modules/alsa/mixer/paths/analog-output.conf.common b/src/modules/alsa/mixer/paths/analog-output.conf.common
> >> index 727b776..700bf28 100644
> >> --- a/src/modules/alsa/mixer/paths/analog-output.conf.common
> >> +++ b/src/modules/alsa/mixer/paths/analog-output.conf.common
> >> @@ -152,12 +152,6 @@ priority = 0
> >>   name = output-bass-boost-off
> >>   priority = 10
> >>
> >> -[Element IEC958]
> >> -switch = off
> >> -
> >> -[Element IEC958 Optical Raw]
> >> -switch = off
> >> -
> >>   ;;; 'Analog Output'
> >>
> >>   [Element Analog Output]
> >> --
> >
> > IIRC on some devices, this would keep the TOSLINK LED on even in
> > analog mode. Is there a reason to not explicitly disable these
> > controls?
> 
> There is a commented-out 
> output:analog-stereo+output:iec958-stereo+input:analog-stereo example at 
> the end of /usr/share/pulseaudio/alsa-mixer/profile-sets/default.conf 
> that could be a potential reason to apply this patch.

Yes, I had a similar use case with two alsa sinks: the analog sink found
by udev and manually loaded sink for HDMI. The HDMI sink did not work
out of the box because the analog output muted the IEC 958 control
during udev probing.

Ismo