Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Bugs
  • Category FM Tuner
  • Assigned To No-one
  • Operating System Iriver H300 series
  • Severity Low
  • Priority High
  • Reported Version
  • Due in Version Version 3.0
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by Anonymous Submitter - 2006-02-23
Last edited by christi-s - 2006-03-19

FS#4714 - Preset naming

Many bugs together:

The radio seems to overload the processor, and then the selection of letters in the screen is too slow. (not properly a bug)

Sometimes, when one direction key is pressed (up, down, right, left), the program keeps repeating this key several times. And sometimes the repetition includes unpressed buttons.

When a letter is selected with NAVI, it is sometimes not inserted in the name. The only solution is then to restart the machine.

Closed by  zagor
2006-03-24 14:21
Reason for closing:  Fixed
Additional comments about closing:   Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407

Fixed in CVS.

the radio sets the cpu to idle mode and (stupidly) doesnt reset it when a button is pressed… i fixed this in the radio wps patch, and ill up a patch with just this change soon (hopefully)..
if you want to fix this your self goto apps/recorder/radio.c and comment out the line cpu_idle_mode(true) (or something similar) in the radio_screen function

Instead of posting a patch, I preferred to post here a reply with your suggested edition. Please, check if the file is properly changed. If you look for the string "#4714" you'll find the two lines commented out. One cpu_idle_mode(true) and one cpu_idle_mode(false). I see there is a switch(button) inside the while below the edited line, which controls the actions to be done with pressed buttons. I don't have the knowledge to judge if the edition is the adequate to correct the bug. Please, someone check.

   radio.c (37.3 KiB)

ye, thats fine, just copy that over the current file and it will work fine… there is actually a better way to do it, which is turning idle mode off when a button is pressed and then back on after a timeout with no button being presed but tis a bit more complicated, and not really nescacery

Two things: shouldn't a patch be sent with the change? Should I put this edited file as a patch in the bug system or not? I am new with rockbox, I just know some C programming, but I am not the one to decide as I know nothing of rockbox internals.

About the solution you comment, I already considered it. This would be preferrable. In fact I am curious because inside the while you can find two if(search_dir) in sequence (is it redundant?). Perhaps the first one sets the radio freq and the second is the one detecting the button pressed. Is it inside the second if where cpu should be set to active? And then, after the switch below, more or less where you can see if (button != BUTTON_NONE) set it to idle mode? Or is it the other way around, idle mode before the switch and idel when button!=BUTTON_NONE? And well, about this timeout, I don't know how to detect it, sorry :(

So, should a patch be added or not? Will you do it along with several other changes?

i dont like the radio code at all and have changed it.. but the higher powers havnt yet added that patch.. so we'll wait… ill sync it with the latest cvs and fix it upa bit and then nag them in irc again and we'll see what happens

Let me suggest something. I am using the H300 optimized build from Needleboy. He has the policy of including everything worth being used for H300 even though it is not in the official build (it is in fact a paral.lel build for H300 until the official one supports H300 adequately). You could send him a note to include this code of yours so we can test it. I suppose you know about it: http://www.misticriver.net/showthread.php?t=35654

Just an idea..

Patch http://www.rockbox.org/bugs/task/4745 fixes it as well.

It bumps the cpu out of idle if a button is pressed and kicks it down to sleep two seconds after buttonpress again.

As the patch 4745 is closed, this bug here is.
~lImbus

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing