Rockbox

I am reporting that selecting 11kHz samplerate freezes my iRiver H140. This happens from the plugin and the recording menu.

All else seems to be working correctly! I recorded 60 audio files from the internal mic. After transferring them to my PC foobar2000 tells me that it’s all working properly. Except samplerate = 11kHz of course.

Very very cool work Mike!

I tried this patch on my IAUDIO X5L 30GB, and had some difficulties:
-Voice menus not talking anymore,
-Disk did not spin down,
-When I went to the record screen and tried to start a recording, it
said my disk was full… but there’s 16G free on there…

That was built with some CVS from around Oct 3-4-5 (sorry), and no other patch.
I hope you can make something of this.

I’m new to Rockbox, got my player only this week, so please forgive me
if I’m missing something obvious. Did this depend on some other patch perhaps?

thx

Stephane:

More details would help. This sounds like a WavPack problem that has since been fixed. There was a problem where the unused data was’t being cleared at the start and overflow for high rates. I don’t know how your X5L could really act any differently than my x5 60GB so I figure you’d have to experience similar problems under similar conditions.

Recording assumes any error that is reported during recording is a disk full problem. This is no longer the case with codec recording. The error was likely due to the encoder flooding the flushing thread and corrupting the data when the buffer overflowed. This is checked for and data is dropped in that case but it never gets full anymore even with 88.2 WavPack. (WavPack encodes better than it plays back :)

Voice menus are shut down during recording BTW but do come back on. It would not be good to swap out the encoder to play voice when encoding since it would increase the likely hood of data loss.

does voice work during pre-rec?

The reason I ask is that we have several blind users, and having a voiced recording screen has long been a request that was implemented several months ago - seems the encoding framework killed that.

pre-recording in uncompressed buffer still seems the logical thing to do, you can still disable voice when actually recording

btw, do post your updated patch….

There’s been talk about using a built in codec for voice (speex?) and getting rid of swapping between voice and encoder would aid that. I will soon experiment with allowing the swapping since it hasn’t really been tried out. Having to have an encoder catch up from up to 30s behind is a bit of scary thought esp at 88.2 or 96kHz and esp. if doing WavPack. :)

There will be full duplex enabled so playing back something else during recording will be possible mainly for writing DSP plugings. Some seeds are planted for that in the next version of the patch.

I’ll post the patch real soon. I changed it to not entirely reset every chunk header after use since that shouldn’t be needed but have to be sure everything is working alright. I also changed the manner in which the iRiver pcm sample rate switching is configured and hopefully the whole list will work *praying*. I’m looking for a good deal an H1x0 ATM so I can test the SPDIF and UDA myself to speed up things when changing code around.

Details, details:
- Changed the iRiver sample rate code so hopefull it will all work now.
- Did some other updates that should add stabiliy to the stream splitting…not every case was covered before.
- Added thread boosting when very full and spinup-time sensitive watermark for flushing data.
- Added seeds for full duplex audio to pcm_playback.c which pcm_record.c uses for samplrate and source switching.
- Added lots of logf if you like to watch that on remote so you can see if it’s indeed doing what I claim :)
- Added asynchronous stops so UI doesn’t do dead after when stopping
- Added thread_get_priority and thread_set_priority returns old priority (need that for prio boost)
- More tweaking to wavpack_enc
- Simplified audio format declaration in id3.c. No array left in metadata.c and all are together for easy maintenence.
- Moved encoder decs and datatypes to enc_base.h including the function decs. for encorder only pcm_record apis.
- Samplerate arrays in pcm_sampr.h and pcm_sampr.c are automatically generated with a few flags in your config*.h
- encoders should be able to work on little endian devices as well
- no more cfg file for the encoders so no disk spinup when changing formats
- Much more important minutae…

   big-recording-update2.patch.t... (369.6 KiB)

BTW:

Should probably reset your settings to make sure because I don’t know if it’s synced with the config block version anymore. I’ll do that just before a commit.

Full update! Not just main binary!

unless I’m doing something wrong, this patch doesn apply cleanly against current CVS

That’s strange. I update just before creating it. I’ll try again.

The problem was that I have my editor set the turn tabs into four spaces and there are apparently some files in CVS that are using tabs and these diffs got in the patch. When I cleaned out the lines starting with “?”, the tabs in the patch were of course turned into spaces and it wouldn’t patch anymore of couse. I think the -l option ignores whitespace changes. This one is a diff with everything left as cvs diff produced it and works fine with up to the minute CVS.

   big-recording-update2.1.patch... (369.3 KiB)

much better, thanks

I’ll try to load it on my h340 before leaving for work so I can test there a bit

Man do a quick test with the sampr_test plugin because the sound chip configuration on the iRivers was totally changed and I can’t test it myself. It’s a major anxiety point about it atm. :0

I say that cause it might not even play if I have it wrong. :)

indeed

W A R N I N G: this patch breaks playback on iriver h1x0 and h3x0!!!

;)

sure was a quiet ride to work on the bike…

Should definitely fix the playback problem. LinusN helped sort that out. Maybe the recording will get audio now. There was an exception leaving the recording screen when he was testing but we’ll see.

   big-recording-update2.2.patch... (370.2 KiB)

Just a resync of version 2.2 with current CVS.

   big-recording-update2.21.patc... (386.2 KiB)

Another resync to CVS

   big-recording-update2.22.patc... (397.5 KiB)

My chick s#1t solution to the slave mode monitoring problem on iRivers. :)
The chip should be setup like currently when recording except the unneeded (right) ADC is turned off when using mic.

I’ll put up a patch for debugging recording in slave mode when a commit happens.

   big-recording-update2.3.patch... (417.7 KiB)