[Speex-dev] Re: Patch, related to TI DSP C54x C55x C6x builds

[Jean-Marc Valin]

Hi Jim,

Thank for the patch. I'll apply it when I have a few minutes. If I
haven't done so after a few weeks, please send it again. I'm in the
process of relocating to Australia, so everything's a bit of a mess
around here. Also, please post the c5X-specific files to the list
(.cmd, .pjt, ...) so they'll be archived. Last thing, I see you defined
spx_word64_t as long long for the C5X, isn't there a way to tell the
compiler that the 40-bit accumulator is fine?

	Jean-Marc

Le jeudi 18 août 2005 à 16:18 -0400, Jim Crichton a écrit :
> Jean-Marc,
> 
> I have attached a small patch with modifications to arch.h, bits.c, and 
> misc.c.  This contains the few mods remaining to support the various fixed 
> point TI DSPs after the work that you did at the end of May (thank you for 
> this).
> 
> arch.h: Add switch for compilers not supporting "long long" (C55x does, C54x 
> and older C64x does not)
> bits.c: Allow external definition for max buffer size, change 
> MAX_BYTES_PER_FRAME
>         to MAX_CHARS_PER_FRAME for consistency
> misc.c: Added override switches to alloc routines, conditional include of 
> user file
>         changes to allow manual memory allocation rather than using heap
> 
> Note that none of these changes are necessary to get Speex to run on C55x.
> 
> The arch.h change allows operation with older versions of Code Composer 
> Studio.
> The bits.c change reduces the data memory usage, for applications running a 
> subset of the Speex coding modes.
> The misc.c change allows private memory allocation, for cases where it is 
> not desirable to use the normal heap.
> This makes it possible to sort of comply with the TI algorithm interface 
> standard (xDAIS).
> 
> I changed my code today to follow the pattern used for the Sharc port 
> (OVERRIDE switches), but you might prefer a different style for including 
> the user-provided allocation routines.
> 
> Regards,
> 
> Jim Crichton
-- 
Jean-Marc Valin <Jean-Marc.Valin at USherbrooke.ca>
Université de Sherbrooke