Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 13 Oct 2010 17:40:17 -0400
From:      Jung-uk Kim <jkim@FreeBSD.org>
To:        Rui Paulo <rpaulo@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
Message-ID:  <201010131740.23408.jkim@FreeBSD.org>
In-Reply-To: <C8F8A6C0-CBFB-4874-9D71-F311D64EA0B3@freebsd.org>
References:  <201010131439.o9DEdssc090571@svn.freebsd.org> <201010131703.21145.jkim@FreeBSD.org> <C8F8A6C0-CBFB-4874-9D71-F311D64EA0B3@freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
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



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201010131740.23408.jkim>