From owner-svn-src-head@FreeBSD.ORG Wed Oct 13 21:40:32 2010 Return-Path: Delivered-To: svn-src-head@freebsd.org Received: from [127.0.0.1] (freefall.freebsd.org [IPv6:2001:4f8:fff6::28]) by hub.freebsd.org (Postfix) with ESMTP id 481E8106566B; Wed, 13 Oct 2010 21:40:31 +0000 (UTC) (envelope-from jkim@FreeBSD.org) From: Jung-uk Kim To: Rui Paulo Date: Wed, 13 Oct 2010 17:40:17 -0400 User-Agent: KMail/1.6.2 References: <201010131439.o9DEdssc090571@svn.freebsd.org> <201010131703.21145.jkim@FreeBSD.org> In-Reply-To: MIME-Version: 1.0 Content-Disposition: inline Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Message-Id: <201010131740.23408.jkim@FreeBSD.org> Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r213779 - head/sys/dev/sound/pci X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 13 Oct 2010 21:40:32 -0000 On Wednesday 13 October 2010 05:17 pm, Rui Paulo wrote: > On 13 Oct 2010, at 22:03, Jung-uk Kim wrote: > > On Wednesday 13 October 2010 04:30 pm, you wrote: > >> I read the PR and the mailing list posts, but I don't see what > >> problem does "case SPICDS_TYPE_AK4381 || SPICDS_TYPE_AK4396:" > >> fix. > > > > A similar bug for right channel was fixed by netchild@ in > > r188480: > > > > http://svn.freebsd.org/viewvc/base/head/sys/dev/sound/pci/spicds. > >c?view=log > > > > but he missed the left channel, which caused volume differences > > between the two channels. > > > > "In FreeBSD 7.2 it worked just like a charm but after upgrading > > to 8.0 the left stereo channel is only half as loud as the right > > one. It can be reproduced with either speakers and headphones. I > > tracked it down to a change in revision 188480 of spicds.c, the > > change "fix: stupid bug with volume control for AK4396" breaks > > volume control for me. > > > > http://lists.freebsd.org/pipermail/freebsd-multimedia/2009-Decemb > >er/010553html > > > > Carl Johan Gustavsson submitted a correct patch: > > > > http://lists.freebsd.org/pipermail/freebsd-multimedia/2009-Decemb > >er/010558html > > > > ariff@ said he would take it but I guess he never did: > > > > http://lists.freebsd.org/pipermail/freebsd-multimedia/2009-Decemb > >er/010575html > > Oh, I read your previous email backwards. This explains it, thanks. :-) > Guess we can close the PR now. Yes, that's exactly what I wanted to tell you. Thanks! Jung-uk Kim