Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Bugs
  • Category FM Tuner
  • Assigned To No-one
  • Operating System Gigabeat S
  • Severity Low
  • Priority Very Low
  • Reported Version Daily build (which?)
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by EricJorgensen - 2009-07-30
Last edited by bertrik - 2010-06-21

FS#10474 - FM tuner on Gigabeat S fails to tune last frequency on start

in r22084 and some previous versions (I know I’ve noticed it before) the FM tuner is not tuned when the tuner function is launched.

The frequency is remembered from the last time the fm tuner was used, but only static is heard until i seek in either direction and back to the frequency.

This behavior on the Gigabeat S differs from the way the fm tuner behaves on other targets.

Closed by  bertrik
2010-06-21 16:45
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

Should be fixed in SVN r27018

I forgot to say, this occurs only on cold boot - if the beast has been turned off since the last time the tuner was used

Actually, this happens on the ams sansas too (which use the si4700 fm tuner too). I’ve seen it on my clip. When it happened, the FM debug screen showed that register 0×03 had been written with the correct value (channel index), but somehow the channel readback register 0x0B still showed 0. This means that the correct frequency had indeed been written to the fm tuner chip, but somehow it has not tuned to it.

A tuner bug from the AMS sansa that may also exist on the gigabeat S is that the player hangs when accessing the radio when it is powered off (e.g. paused). Could you try the following and see if it has the same behaviour as on the Clip? : go to the radio screen, pause, then skip channels → hang.

Yes, i get the same hang on my S60 when i pause and then skip.

Also interesting - the last two times i turned on my S60 and went into the fm radio function, it started up tuned - so it would appear to be intermittent behavior.

MikeS commented on 2009-08-03 19:25

Hmmm…I can’t get my S30 to do this at all after about a dozen times of trying. :-\

But, for the hang. Yes. Perhaps the tuner shouldn’t be placed in standby during pause but instead just muted. I’m not really surprised there. It’s not a hard lock as there is still backlight action. So, is the loop to wait for tuning not exiting?

I caught my gigabeat doing this three times in a row and then posted the bug report.

I can’t get it to do it now.

I can quite easily reproduce this on my clip but can’t see a pattern yet. My clip has a device id of 0×1242 and a chip id of 0×1053. When it happened the READCHAN register was 0 while the CHANNEL register was set. If I look at the code, this means that somehow the driver thought tuning was complete because the STC bit was set. Maybe we should add a check here on READCHAN being equal to CHANNEL (I vaguely remember discussing this earlier or seeing it in the OF), I can’t find a requirement for this in the datasheet though. The STC bit is also used during seeking, maybe this is interfering.

MikeS commented on 2009-08-04 21:05

I haven’t updated my RB on the S for awhile (r20571 - 090330) so maybe something crept in since then? The S goes through precisely the same interface selection procedure as the OF (which is also the same as the DS) when initializing the FM chip though I’m not sure it matters here. At least it being on both types of devices basically rules out a SoC-specific issue.

The need for a tuning check might be an errata of sorts not covered in the docs we have?

Seeking as currently implemented is just tuning and then checking a level and so not fundamentally different. I can’t see where that would matter. Perhaps it doesn’t reset the bit fast enough before the tuning sequence is initiated. Maybe check if a delay matters…say, one tick? (some guesses, no code in front of me atm :)

MikeS commented on 2010-05-24 21:15

I just fixed the hang problem while paused. I also tried using FM as the startup screen and got a blast of noise in my left ear for a fraction of a second. It could be trying to start the tuner before the aync audio hardware init is finished (guess). It did tune and go to normal volume however.

I looked at si4700 tuning code recently (for the Sansa Clip+) and it seems to indeed check both the STC bit (seek/tune complete) _and_ compare READCHAN (actually tuned channel) with CHAN (desired channel). If not satisfied, the TUNE bit is taken down and the tune is tried again (a few times) until it does tune. This should fix it. I hope to post a patch soon (sometime in the next 7 days)

Attached is the fix I discussed earlier. I haven’t seen any failures anymore (static just after boot).

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing