Re: [Qemu-devel] [4321] First cut at WM8750 volume control (Jan Kiszka).

[andrzej zaborowski]

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).

Regards