All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dan Aloni <da-x@colinux.org>
To: Takashi Iwai <tiwai@suse.de>
Cc: Clemens Ladisch <clemens@ladisch.de>, alsa-devel@lists.sourceforge.net
Subject: Re: Re: ALC658 rev 0 - a problem with Rear Right
Date: Mon, 7 Mar 2005 22:21:18 +0200	[thread overview]
Message-ID: <20050307202118.GA15401@localdomain> (raw)
In-Reply-To: <s5hpsybuv5f.wl@alsa2.suse.de>

On Mon, Mar 07, 2005 at 08:13:16PM +0100, Takashi Iwai wrote:
> At Mon, 7 Mar 2005 20:55:14 +0200,
> Dan Aloni wrote:
> > 
> > On Mon, Mar 07, 2005 at 07:28:28PM +0100, Takashi Iwai wrote:
> > > At Mon, 7 Mar 2005 20:24:14 +0200,
> > > Dan Aloni wrote:
> > > > 
> > > > On Mon, Mar 07, 2005 at 06:13:44PM +0100, Takashi Iwai wrote:
> > > > > At Mon, 7 Mar 2005 18:57:02 +0200,
> > > > > Dan Aloni wrote:
> > > > > > 
> > > > > > On Mon, Mar 07, 2005 at 05:33:56PM +0100, Takashi Iwai wrote:
> > > > > > > At Mon, 7 Mar 2005 17:39:21 +0200,
> > > > > > > Dan Aloni wrote:
> > > > > > > > 
> > > > > > > > On Mon, Mar 07, 2005 at 02:10:14PM +0100, Clemens Ladisch wrote:
> > > > > > > > > Dan Aloni wrote:
> > > > > > > > > > The problem was introduced in the following changeset, which
> > > > > > > > > > was supposed to cleanup things but accidently broke something.
> > > > > > > > > >
> > > > > > > > > > "[ALSA] Clean up and fix stereo mutes" :
> > > > > > > > > 
> > > > > > > > > Most AC'97 codecs have one mute bit for both channels, but some can
> > > > > > > > > have one for each channel (that's called "stereo mute").
> > > > > > > > > 
> > > > > > > > > It seems the new driver enables the stereo mute bits, but then forgets
> > > > > > > > > to unmute the second channel.
> > > > > > > > > 
> > > > > > > > > Does anything happen if you toggle both mute bits in alsamixer?
> > > > > > > > 
> > > > > > > > Using alsamixer I am unable to mute any channel by itself - it 
> > > > > > > > always mutes or unmutes boths channels at the same time (I tried
> > > > > > > > any possible combination of m M, < or >). However if I revert the
> > > > > > > > changeset I am able to control each channel alone using < or > and
> > > > > > > > there's no problem.
> > > > > > > 
> > > > > > > Hmm, are you sure that it's ALC658?  AFAIK, the codec doesn't  support
> > > > > > > the stereo mutes (at least ALSA code doesn't enable it).
> > > > > > 
> > > > > > Unless the driver misidentifies the chipset, this is ALC658.
> > > > > 
> > > > > Oh yes, indeed.  It's strange that the older version did have the
> > > > > stereo mute controls.
> > > > > 
> > > > > Could you compare the contents of
> > > > > /proc/asound/card0/codec97#0/ac97#0-0+regs file between the older
> > > > > ("working") and the latest versions?
> > > > 
> > > > I compared and saw no changes in the registers between the latest version 
> > > > and my version with the reverted changeset.
> > > 
> > > Hmm, interesting.
> > > 
> > > Just to be sure (I'm back from the travel shortly ago, so might be
> > > missing something):  the problem persists only on the surround right
> > > channel?
> > 
> > Yes, only that channel.
> >  
> > > And you could mute on/off of "Surround" left/right channels via
> > > alsamixer independently in the older version?
> > 
> > I can only mute both at the same time with the latest version, but I 
> > can mute each channel alone with the working version.
>[..]
> 
> OK.  Then please try the attached patch.  This will enable the stereo
> mutes on the surround volume control again.
> 

Works. Thanks,

-- 
Dan Aloni
da-x@colinux.org


-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now.
http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click

      reply	other threads:[~2005-03-07 20:21 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-03-05 15:35 ALC658 rev 0 - a problem with Rear Right Dan Aloni
2005-03-07 10:42 ` Dan Aloni
2005-03-07 12:28   ` Dan Aloni
2005-03-07 13:10     ` Clemens Ladisch
2005-03-07 15:39       ` Dan Aloni
2005-03-07 16:33         ` Takashi Iwai
2005-03-07 16:57           ` Dan Aloni
2005-03-07 17:13             ` Takashi Iwai
2005-03-07 18:24               ` Dan Aloni
2005-03-07 18:28                 ` Takashi Iwai
2005-03-07 18:55                   ` Dan Aloni
2005-03-07 19:13                     ` Takashi Iwai
2005-03-07 20:21                       ` Dan Aloni [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20050307202118.GA15401@localdomain \
    --to=da-x@colinux.org \
    --cc=alsa-devel@lists.sourceforge.net \
    --cc=clemens@ladisch.de \
    --cc=tiwai@suse.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.