[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 03/11] audio: use a nominal volume of UINT_MAX
From: |
Marc-André Lureau |
Subject: |
Re: [Qemu-devel] [PATCH 03/11] audio: use a nominal volume of UINT_MAX |
Date: |
Mon, 12 Mar 2012 22:07:07 +0100 |
On Mon, Mar 12, 2012 at 10:00 PM, malc <address@hidden> wrote:
> I really do not understand your point at all, mixeng_volume fields
> are int64_t's it's irrelevant what you initialize them with, they
> are still 64bit integers, and UINT_MAX as i already said is not even
> correct, it's 1/(2**32-1) less than one in fixed point.
But the current code does this:
AUD_set_volume_out (SWVoiceOut *sw, int mute, uint8_t lvol..)
sw->vol.l = nominal_volume.l * lvol / 255;
So the range was [0..2^8-1] and then it becomes [0..2^32], which looks
wrong to me.
And Spice uses [0..2^16-1]. So it is simpler to stay within the range
of an integer..
--
Marc-André Lureau
- [Qemu-devel] [PATCH 00/11] apply volume on client side (v3), Marc-André Lureau, 2012/03/12
- [Qemu-devel] [PATCH 01/11] audio: add VOICE_VOLUME ctl, Marc-André Lureau, 2012/03/12
- [Qemu-devel] [PATCH 02/11] audio: don't apply volume effect if backend has VOICE_VOLUME_CAP, Marc-André Lureau, 2012/03/12
- [Qemu-devel] [PATCH 03/11] audio: use a nominal volume of UINT_MAX, Marc-André Lureau, 2012/03/12
- Re: [Qemu-devel] [PATCH 03/11] audio: use a nominal volume of UINT_MAX, malc, 2012/03/12
- Re: [Qemu-devel] [PATCH 03/11] audio: use a nominal volume of UINT_MAX, Marc-André Lureau, 2012/03/12
- Re: [Qemu-devel] [PATCH 03/11] audio: use a nominal volume of UINT_MAX, malc, 2012/03/12
- Re: [Qemu-devel] [PATCH 03/11] audio: use a nominal volume of UINT_MAX, Marc-André Lureau, 2012/03/12
- Re: [Qemu-devel] [PATCH 03/11] audio: use a nominal volume of UINT_MAX, malc, 2012/03/12
- Re: [Qemu-devel] [PATCH 03/11] audio: use a nominal volume of UINT_MAX,
Marc-André Lureau <=
- Re: [Qemu-devel] [PATCH 03/11] audio: use a nominal volume of UINT_MAX, malc, 2012/03/12
- Re: [Qemu-devel] [PATCH 03/11] audio: use a nominal volume of UINT_MAX, Marc-André Lureau, 2012/03/12
- Re: [Qemu-devel] [PATCH 03/11] audio: use a nominal volume of UINT_MAX, malc, 2012/03/12
- Re: [Qemu-devel] [PATCH 03/11] audio: use a nominal volume of UINT_MAX, Marc-André Lureau, 2012/03/12
- Re: [Qemu-devel] [PATCH 03/11] audio: use a nominal volume of UINT_MAX, malc, 2012/03/12
- Re: [Qemu-devel] [PATCH 03/11] audio: use a nominal volume of UINT_MAX, Marc-André Lureau, 2012/03/12
[Qemu-devel] [PATCH 05/11] hw/ac97: the volume mask is not only 0x1f, Marc-André Lureau, 2012/03/12
[Qemu-devel] [PATCH 04/11] hw/ac97: remove USE_MIXER code, Marc-André Lureau, 2012/03/12
[Qemu-devel] [PATCH 06/11] hw/ac97: add support for volume control, Marc-André Lureau, 2012/03/12