Rockbox

Another update to make the patch up to date with cvs.
Also I tidied up the code a bit and added a nice icon for
the radio fmr presets.

question: what does “if(radio_get_status() !=
FMRADIO_POWERED)…” code do (I mean what kind of
problem is it trying to solve).

It checks if the radio if it is in stop mode (off) or idle
(playing while in filetree).Then if idle, it will not
initialize the radio in order not to make a ‘pop’ when the
user enters the radio again.

P.S: On a clean build enter and exit (with the click button)
multiple times, you should hear a ‘pop’ when you enter the
FM radio again.

I’ve taken small bits and pieces into my patch. I’ll have a look
at presets loading/saving as soon as I commit first batch of
changes.

A small update of the patch to build cleanly with new CVS changes.

Major Update:

This update will compile with latest major changes in radio.
A whole new different approach is used in this patch.

From now on the saving of an fmr file will occur only on a
manual save or on exit (if there are changes), instead of
saving every time the list is updated.

Removed usage of fmr files outside the radio screen (though
fmr files are still viewable in filetree)

Update: Code policing and minor fixes.

Hi!
Would it be possible to update this so it works with the
current CVS?

Thank you!

Roolku

Ok I will try to update it today or tomorrow.

Upadated to apply on current CVS.

Updated with latest CVS changes.

Revert to “Load temporary an fmr file outside of the
designated folder and fire up the radio” approach.

Update: Remove the CVS bug that sometimes the screen is not
updated.

Make the code more bug proof.

Updated to CVS.

   radio.diff (29.3 KiB)

Again Updated!

   radio.diff (28.7 KiB)