Rockbox

Tasklist

FS#10416 - Pure Data for Rockbox plugin

Attached to Project: Rockbox
Opened by Wincent Balin (wincent) - Wednesday, 08 July 2009, 20:26 GMT
Last edited by Peter D'Hoye (petur) - Thursday, 23 July 2009, 21:38 GMT
Task Type Patches
Category Plugins
Status Closed
Assigned To No-one
Operating System SW-codec
Severity Low
Priority Normal
Reported Version Version 3.3
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

PDBox plug-in is working with pdpod_test.pd file from the PureData.zip archive (download it from here: http://www.ipodlinux.org/wiki/PdPod ).

The first patch is the diff to the current source tree.

The second patch is the rockbox integration of the PDBox plug-in.
This task depends upon

Closed by  Peter D'Hoye (petur)
Thursday, 23 July 2009, 21:38 GMT
Reason for closing:  Accepted
Additional comments about closing:  last pdbox patch, you can commit changes yourself from now on ;)
Comment by Wincent Balin (wincent) - Thursday, 09 July 2009, 23:20 GMT
If http://www.ipodlinux.org/ does not work, you might want to get the .pd files (unpacked) from http://trac.katastrophos.net/PDQt/browser/trunk/PureData .
Comment by Peter D'Hoye (petur) - Saturday, 11 July 2009, 20:53 GMT
Wincent, what are the exact requirements for pdbox to run? Display, memory, cpu,...
I ask this because you added it now for targets with bitmap displays except the mrobe 500

I would think it should be more near mpegplayer, because of the memory requirements.
Comment by Wincent Balin (wincent) - Saturday, 11 July 2009, 23:50 GMT
You are right. Also, domonoky tried to compile pdbox for Sansa e200, but pdbox turned out to be too large. Therefore I think we should wait with the inclusion into the build process a little, till I make some changes to get pdbox somewhat smaller.

I attach a patch which makes the file pdbox-3-working-system-integration.diff obsolete.
Comment by Peter D'Hoye (petur) - Sunday, 12 July 2009, 19:01 GMT
patches committed
Comment by Wincent Balin (wincent) - Sunday, 12 July 2009, 19:04 GMT
This is a bugfix for warnings about strict aliasing. Basically, pointer casts were replaced with conversions through float-int-unions.
Comment by Peter D'Hoye (petur) - Sunday, 12 July 2009, 19:21 GMT
committed as well :)
Comment by Wincent Balin (wincent) - Tuesday, 21 July 2009, 01:46 GMT
Cleaned up unneeded prototypes in s_path.c
Comment by Wincent Balin (wincent) - Tuesday, 21 July 2009, 12:55 GMT
TLSF memory allocator imported. Examples were left out. There are some printf-s in the source which will be commented out in the later patch.
Comment by Wincent Balin (wincent) - Tuesday, 21 July 2009, 13:39 GMT
The patch added is the corrected version of the previous one -- meaning that it makes it obsolete!
Comment by Peter D'Hoye (petur) - Tuesday, 21 July 2009, 13:53 GMT
same patch with svn properties corrected
   p.diff (46 KiB)
Comment by Peter D'Hoye (petur) - Tuesday, 21 July 2009, 13:57 GMT
tip: to change the properties for a lot of files, do this:

svn propset -R svn:keywords 'Author Date Id Revision' *
svn propset -R svn:eol-style native *
Comment by Wincent Balin (wincent) - Tuesday, 21 July 2009, 15:41 GMT
Made TLSF compile without warnings.
Comment by Wincent Balin (wincent) - Tuesday, 21 July 2009, 15:55 GMT
Replaced dbestfit with TLSF. Left references to wfirstfit in the source. Made some functions use local variables instead of allocated memory. Increased size of both the audio output buffer to remove crackling in resulting sound and the core and GUI thread stacks to diminish possibility of an error because of insufficient memory.

Loading...