FS#2668 - Bugfixes in iriver pcm record & FM-record on "record" button

Attached to Project: Rockbox
Opened by Toni (ahellmann) - Monday, 05 September 2005, 10:33 GMT
Last edited by Toni (ahellmann) - Monday, 31 October 2005, 17:46 GMT
Task Type Patches
Status Closed
Assigned To No-one
Operating System
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No


This patch includes 2 major bugfixes in pcm_record.c
and allows radio recording on record button for iriver

New iriver FM recording feature:
1. direct FM recording with 'record' button ('off' button to
stop recording). Recordings are saved in '/recordings'.

Not yet done:
1. No continues multiple records
So do not press: record->record->record but press:

1. avoid audio glitch at recording startup
old pcm_record.c: DAR1 = (unsigned long)rec_buffers
new pcm_record.c: DAR1 = (unsigned long)rec_buffers

2. avoid audio lock after recording (no reboot neccessary)
old pcm_record.c: pcm_close_recording() not serialized
new pcm_record.c: pcm_close_recording() serialized
This task depends upon

Closed by  Michael Sevakis (MikeS)
Monday, 06 November 2006, 23:47 GMT
Reason for closing:  Out of Date
Additional comments about closing:  :)
Comment by Anonymous Submitter - Friday, 14 October 2005, 01:06 GMT

is it possible to add the peak meter to the next patch?
Comment by Toni (ahellmann) - Friday, 14 October 2005, 15:52 GMT

I personally don't like the peakmeter in the radio screen for
the reason that the radio input signal can get disurbed by
some electronic interference caused by the peakmeter.
Beside this it would be relatively easy to add the peakmeter
functionality. When I have time I will continue on this patch.
Comment by Dave Drury (sourcemedd) - Monday, 31 October 2005, 10:36 GMT

I have installed this patch but am not able to listen to
anything recorded (either with FM or line in, mic as the
source). THe recordXX.wav file doesn't play on iriver or
PC. Any tips??
Comment by Toni (ahellmann) - Monday, 31 October 2005, 17:46 GMT

Did you use a daily build from 20050905? If not, then I guess
there might be conflicts with current builds, since there have
been done a lot of modifications to the code. Because this
patch does not completely integrate into the rockbox source,
there is no chance for CVS commitment. So I lost interest in
updating to the current builds. :-(