| Re: [hatari-devel] STe DMA sound -> new downsampling method |
[ Thread Index |
Date Index
| More lists.tuxfamily.org/hatari-devel Archives
]
- To: hatari-devel@xxxxxxxxxxxxxxxxxxx
- Subject: Re: [hatari-devel] STe DMA sound -> new downsampling method
- From: Vincent Rivière <vincent.riviere@xxxxxxxxxxx>
- Date: Sat, 5 Oct 2019 12:43:30 +0200
- Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=sender:from:subject:to:references:message-id:date:user-agent :mime-version:in-reply-to:content-language:content-transfer-encoding; bh=yrYqU/g7NmfqRasngbD1iC1jerkMTaXRQnjeHsZXCYA=; b=mVfjdxSYh9IdDTgnXvo3H7LS3qU+auPPS3KiV6TsYtdyv+DELAoZUiZ19x+i0yGFJs 1NrbIlYkbcfnpCeMuo86FyxGf7HIBRZVvwRtiapWVHX/Vqtq8XEfK2vqOLI9xibqRBzf jmQh89kHejg7rIhSLKmrqC8PS2b0IivH3qBhAdobYhq9n0zM281R1ak7k0WDLKUtRjLv 3xhFO/gSXyf4Tw6cxq6EXbi4Nqz4Ito0xgywsmfYqWHEjbl1CrsAvOatgO0gcUCcnvhY BFeATKvyrjTU/jwmaRiiU6gyR5HGGxIvAjKSoFOTRndVCJ56NKdC9qU7+2FV9bA6dCKK cesw==
On 05/10/2019 at 11:26, Nicolas Pomarède wrote:
Fix is available in latest git.
Thanks for the quick fix.
Now I can hear all the voices with correct frequency.
And the initial bug is fixed!
No more high-pitched noise in Lethal Xcess :-D
Many thanks for that.
On my 64 bit linux, the correct value of interval_fract is 0x5af18 because
0x10000L is defaulting to 64 bit. But with mingw/cygwin for example, 0x1000L
is 32 bit and the operation gets truncated after the multiplication, yelding
a wrong value of interval_fract=0x139c9 :(
So this was a simple 32/64-bit issue. Nothing specifically related to
Windows/Cygwin.
PS : you can now remove the youtube video, no need to keep trace of hatari's
bug :)
Don't worry, that video is Unlisted. So it is only visible from the link I
posted here.
I will do some cleanup on YouTube later.
--
Vincent Rivière