Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category
  • Assigned To No-one
  • Operating System
  • Severity Low
  • Priority Very Low
  • Reported Version
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by Toni - 2005-09-05
Last edited by Toni - 2005-10-31

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

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

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:
record→off→record→off

Bugfixes:
1. avoid audio glitch at recording startup
old pcm_record.c: DAR1 = (unsigned long)rec_buffers
[write_index++];
new pcm_record.c: DAR1 = (unsigned long)rec_buffers
[write_index];

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

Closed by  Michael Sevakis
2006-11-06 23:47
Reason for closing:  Out of Date
Additional comments about closing:  

:)

Anonymous Submitter commented on 2005-10-14 01:06

is it possible to add the peak meter to the next patch?

Toni commented on 2005-10-14 15:52

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.

Dave Drury commented on 2005-10-31 10:36

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??

Toni commented on 2005-10-31 17:46

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. :-(

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing