At Fri, 25 Jan 2008 08:37:35 -0600,
Jerone Young wrote:
>
> I tested both patches. They work and master volume works for both
> Speakers & IEC958. It can controll up and down volume.
>
> Speakers now mute.
>
> One issue though when you mute the Master it does not mute IEC958. It
> will mute the speakers but not the IEC958 PCM stream. I am not sure if
> this desired behavior?
No, it should be also controlled by the master switch.
Could you check the below
% amixer cget name="IEC958 Playback Switch"
whether this element exists. Then,
% amixer cset name="IEC958 Playback Switch" off
to turn the SPDIF off.
> Oh also AC3 passthrough still works fine (you really can't manipulate
> this audio). I streamed some Doubly 5.1 streams over and they worked
> fine.
That's great. Then the things become really eaiser :)
> Besides that one issue you should put those 2 patches upstream.
The first patch is already on HG tree. The second patch will be
pushed soon.
thanks,
Takashi
> On Jan 25, 2008 8:08 AM, Takashi Iwai <tiwai@xxxxxxx> wrote:
> > At Fri, 25 Jan 2008 07:41:48 -0600,
> > Jerone Young wrote:
> > >
> > > Ok less explication this time ;-) .. it was a late night. I have found
> > > something interesting though.
> >
> > Thanks ;)
> >
> > > I took some time and actually evaluated the behavior of the Windows
> > > Vista Driver by Analog Devices was done for the AD1984. They do not
> > > have a PCM mixer, instead each indivdual application has there own
> > > mixer (this is a windows feature), and when you manipulate the
> > > individual mixer it manipulates the audio of that application. But
> > > there is not a PCM mixer for all apps. Just a Master. Also they have
> > > IEC958 (SDPIF) seen as a second sound device. It appears that they
> > > made what we call PCM the Master.
> >
> > Maybe Windows driver doesn't use the analog loopback feature inside
> > the codec, but does everything on software.
> >
> > > Now looking at our driver, it appears we should be doing the same. I
> > > actually have a good example why. If I am running playback through
> > > IEC958 (SDPIF) and I do anything to the PCM mixer it has no effect on
> > > PCM audio coming from applications that are running through IEC958.
> > > Only when I go to the application (assuming the application has it's
> > > own mixer code .. we will say VLC is a good example), does the actual
> > > PCM for that application get effected. So if I mute the PCM mixer, PCM
> > > audio runs over IEC958 without being muted, it only mutes it for
> > > speaker and the headphones.
> >
> > OK now I see the problem. IEC958 isn't assigned to the master
> > volume/switch list indeed.
> >
> > > What I see here is what we label as PCM is actually just the Master
> > > volume for Speaker & Headphones. But does not actually manipulate the
> > > PCM audio.
> >
> > Correct. It's because the SPDIF has an individual (a sort of) DAC.
> > So, we need to adjust this as well.
> >
> > > So really PCM really should be made Master Playback as it really does
> > > not manipulate PCM audio but just the how it is played back through
> > > Speaker & Headphones.
> >
> > A fix is easy, but this might cause another problem when a non-audio
> > signal is sent.
> >
> > Could you try the patch below with the latest HG tree? (Or, you can
> > apply this after my previous patch to 1.0.16-rc1.)
> > If the SPDIF volume/switch are changed together with the master
> > successfully, then please test AC3 or DTS output if possible. I'd
> > like to know whether it breaks now.
> > If it still works, I'll happily apply the patch to the upstream tree.
> >
> >
> > Thanks,
> >
> > Takashi
> >
> > ---
> >
> > diff -r 19029da61288 pci/hda/patch_analog.c
> > --- a/pci/hda/patch_analog.c Fri Jan 25 11:56:57 2008 +0100
> > +++ b/pci/hda/patch_analog.c Fri Jan 25 15:04:01 2008 +0100
> > @@ -139,6 +139,7 @@ static const char *ad_slave_vols[] = {
> > "Headphone Playback Volume",
> > "Mono Playback Volume",
> > "Speaker Playback Volume",
> > + "IEC958 Playback Volume",
> > NULL
> > };
> >
> > @@ -151,6 +152,7 @@ static const char *ad_slave_sws[] = {
> > "Headphone Playback Switch",
> > "Mono Playback Switch",
> > "Speaker Playback Switch",
> > + "IEC958 Playback Volume",
> > NULL
> > };
> >
> > @@ -3078,6 +3080,7 @@ static const char *ad1884_slave_vols[] =
> > "Internal Mic Playback Volume",
> > "Docking Mic Playback Volume"
> > "Beep Playback Volume",
> > + "IEC958 Playback Volume",
> > NULL
> > };
> >
> >
>
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
