From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1Jso7B-0005aZ-UV for qemu-devel@nongnu.org; Sun, 04 May 2008 19:59:33 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1Jso7A-0005aG-Cu for qemu-devel@nongnu.org; Sun, 04 May 2008 19:59:33 -0400 Received: from [199.232.76.173] (port=36234 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1Jso7A-0005aC-B2 for qemu-devel@nongnu.org; Sun, 04 May 2008 19:59:32 -0400 Received: from ug-out-1314.google.com ([66.249.92.175]) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1Jso7A-0004Wd-2a for qemu-devel@nongnu.org; Sun, 04 May 2008 19:59:32 -0400 Received: by ug-out-1314.google.com with SMTP id m2so638651uge.4 for ; Sun, 04 May 2008 16:59:29 -0700 (PDT) Message-ID: Date: Mon, 5 May 2008 01:59:28 +0200 From: "andrzej zaborowski" Subject: Re: [Qemu-devel] [4321] First cut at WM8750 volume control (Jan Kiszka). In-Reply-To: <481DF2EB.9070604@web.de> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Content-Disposition: inline References: <481D944E.7090703@web.de> <481DF2EB.9070604@web.de> Reply-To: qemu-devel@nongnu.org List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org On 04/05/2008, Jan Kiszka wrote: > andrzej zaborowski wrote: > > On 04/05/2008, Jan Kiszka wrote: > >> -#define NOVOL > >> +//#define NOVOL > >> > >> /* 8 bit */ > >> #define ENDIAN_CONVERSION natural > >> > >> to make the MusicPal work out-of-the-box (muting is now broken again, > >> causing loud noise during channel switches and while in suspended mode). > > > > Right, as I said you need to disable NOVOL manually if you want volume > > control. Changing this would affect users of all machines of all > > architectures and until now everyone was fine with using the host > > mixer instead of software mixing in qemu. > > > Well, this is a weak argument. So far there was no alternative available. I'm sure there would be an alternative if anybody wanted it. > > I don't let you go with this ;): How should we find out what needs to be > fixed - given that hardly anyone is able to test on its own all the > supported boards with appropriate sound scenarios? What regressions are > you aware of when we switch NOVOL off? The regression is the overhead of two 32bit multiplications per sample per channel even if the virtual sound card you use has no volume control. For the wm8750 that's not easily noticeable but I'm personally not making it a default and not globally. (But I won't oppose if there's a wider agreement). Regards