[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH] audio: split sample conversion and volumemixing
From: |
malc |
Subject: |
[Qemu-devel] Re: [PATCH] audio: split sample conversion and volumemixing |
Date: |
Wed, 12 Jan 2011 02:15:18 +0300 (MSK) |
User-agent: |
Alpine 2.00 (LNX 1167 2008-08-23) |
On Tue, 11 Jan 2011, Michael Walle wrote:
>
> Hi Wolfgang,
>
> > The fixed point path for scaling in mixeng_volume() seems to be under by
> > a factor of two, IMHO. The right shift should be by 31, not 32. (Because
> > the volume, which I assume is a signed 32 bit integer, can be 0.5 at
> > most).
> I must admit i just copied the shift from the original define. But the volume
> and the samples are signed 64bit values.
>
> @malc: any comments on this?
I don't understand the problem, the result of multiplication is converted
from 32.32 fixed point to integer by shifting right by 32.. It escapes me
how 31 fits here.
>
> > Does the code work with FLOAT_MIXENG undefined?
> At least for my input voice, it works with integer arithmetic.
>
>
--
mailto:address@hidden
- [Qemu-devel] audio: Mixing engine and mixer emulation, Michael Walle, 2011/01/03
- [Qemu-devel] Re: audio: Mixing engine and mixer emulation, malc, 2011/01/03
- [Qemu-devel] [PATCH] audio: split sample conversion and volume mixing, Michael Walle, 2011/01/04
- RE: [Qemu-devel] Re: [PATCH] audio: split sample conversion and volumemixing, Schildbach, Wolfgang, 2011/01/11
- [Qemu-devel] Re: [PATCH] audio: split sample conversion and volumemixing, Michael Walle, 2011/01/11
- [Qemu-devel] Re: [PATCH] audio: split sample conversion and volumemixing,
malc <=
- [Qemu-devel] RE: [PATCH] audio: split sample conversion and volumemixing, Schildbach, Wolfgang, 2011/01/12
- [Qemu-devel] Re: [PATCH] audio: split sample conversion and volume mixing, malc, 2011/01/12