[Dsp-tremor-devel] 0.0.1 alpha "release"
Simon Pickering
S.G.Pickering at bath.ac.uk
Mon Jan 28 18:19:30 EET 2008
Just a quick email to introduce the 0.0.1 alpha "release".
This is work in progress, the ARM-side code is very simple and simply passes
data to the DSP-side code which does all of the decoding. Both of these
pieces of code compile and run but do not actually decode a vorbis stream!
The problem is that the DSP-side code does not recognise the vorbis stream
as being valid. You can follow the way the code runs by looking at the
error.txt file to see the debugging output. There is also x86 code in this
release which I am using to generate the "correct" outputs for comparison.
Some parts of the code were originally written in C54 asm, these need to be
ported to C (and then possibly back to C55 asm if they are not fast enough).
I replaced these asm functions with C code from the lowmem-no-byte branch. I
did this as the C54 code was not returning values correctly (i.e. crc code)
and because it would be useful to have some baseline C code to work from
when optimising.
I did not check/rewrite the pieces of C code that I added in for 16bit-byte
correctness but instead have been sorting out each piece of code as it's
called in the header setup/decode chain. This was mainly done so I could get
some part-working code to encourage myself and also to allow me to have a
better idea of what the code does by attacking each part one by one.
I've rewritten the bit reading code, which now appears to work. The
re-written crc code also looks like it works okay. I assume that the current
error in the code (i.e. the fact that it doesn't recognise a stream which it
should do) is down to some of my unchecked replaced code. I'm in the process
of finding the next piece of replaced code and debugging it.
So, even if you don't have a working DSP toolchain, looking at and
submitting patches for these replaced functions would be helpful. You can
see which functions have been replaced by looking at which were declared in
the .asm files.
Cheers,
Simon
More information about the Dsp-tremor-devel
mailing list