Nyaaori/rockbox
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

rockdoom: Mixing 512 samples in an interrupt handler is too much. Gigabeast FIFOs are very tight and it will channel swap and crackle. It's better to minimize time in the audio callback anyway so change it to 128. It should probably even be 64. Use faster saturation code too.

Browse source 
git-svn-id: svn://svn.rockbox.org/rockbox/trunk@19679 a1c6a512-1295-4272-9138-f99709370657
This commit is contained in:
Michael Sevakis 2009-01-05 02:29:58 +00:00
parent 96474e3b4d
commit 6920a5be44
1 changed files with 11 additions and 15 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

26
apps/plugins/doom/i_sound.c
View file

@ -48,7 +48,7 @@
// mixing buffer, and the samplerate of the raw data.
// Needed for calling the actual sound output.
#define SAMPLECOUNT 512
#define SAMPLECOUNT 128
#define NUM_CHANNELS 24
// It is 2 for 16bit, and 2 for two channels.
@ -99,6 +99,13 @@ int *vol_lookup; // Volume lookups.
int *steptable; // Pitch to stepping lookup. (Not setup properly right now)
static inline int32_t clip_sample16(int32_t sample)
{
if ((int16_t)sample != sample)
sample = 0x7fff ^ (sample >> 31);
return sample;
}
//
// This function loads the sound data from the WAD lump for single sound.
// It is used to cache all the sounddata at startup.
@ -371,7 +378,7 @@ void I_UpdateSound( void )
signed short* leftend;
// Step in mixbuffer, left and right, thus two.
int step;
const int step = 2;
// Mixing channel index.
int chan;
@ -380,7 +387,6 @@ void I_UpdateSound( void )
// are in global mixbuffer, alternating.
leftout = mixbuffer;
rightout = mixbuffer +1;
step = 2;
// Determine end, for left channel only
// (right channel is implicit).
@ -431,20 +437,10 @@ void I_UpdateSound( void )
// else if (dl < -128) *leftout = -128;
// else *leftout = dl;
if (dl > 0x7fff)
*leftout = 0x7fff;
else if (dl < -0x8000)
*leftout = -0x8000;
else
*leftout = (signed short)dl;
*leftout = clip_sample16(dl);
// Same for right hardware channel.
if (dr > 0x7fff)
*rightout = 0x7fff;
else if (dr < -0x8000)
*rightout = -0x8000;
else
*rightout = (signed short)dr;
*rightout = clip_sample16(dr);
// Increment current pointers in mixbuffer.
leftout += step;