[Qemu-devel] [PATCH] Enable volume control globally (was: First cut at WM8750 volume control)

[Jan Kiszka]

andrzej zaborowski wrote:
> On 04/05/2008, Jan Kiszka <address@hidden> wrote:
>> andrzej zaborowski wrote:
>>  > On 04/05/2008, Jan Kiszka <address@hidden> 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).

Well, let's start like below, and if someone complains, think about a
step-wise move towards out-of-the-box mixer support in QEMU.

----------

As we now have the possibility to set per-channel volume levels, thus to
emulated sound hardware more realistically, we just need to enable the
application of this values by disabling 'NOVOL'. Users can still disable
this support if desired or required, but it should be the aim now to
fix potential remaining issues of the soft-mixer.

This patch also unbreaks the MusicPal emulation, which depends on the
mute control so that users have to tweak the source code right now.

Signed-off-by: Jan Kiszka <address@hidden>
---
 audio/mixeng.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Index: b/audio/mixeng.c
===================================================================
--- a/audio/mixeng.c
+++ b/audio/mixeng.c
@@ -28,7 +28,7 @@
 #define AUDIO_CAP "mixeng"
 #include "audio_int.h"
 
-#define NOVOL
+//#define NOVOL
 
 /* 8 bit */
 #define ENDIAN_CONVERSION natural