Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Bugs
  • Category FM Tuner
  • Assigned To No-one
  • Operating System All players
  • Severity Low
  • Priority Very Low
  • Reported Version Version 3.1
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by bertrik - 2009-01-25
Last edited by bertrik - 2009-09-24

FS#9824 - Fix FM channel spacing for Europe

This patch fixes the channel spacing for the “Europe” regional setting. In Europe, a channel spacing of 100 kHz (0.1 MHz) is used for FM radio stations, as per recommendation BS.412 of the ITU-R (International Telecommunications Union - Radiocommunications sector). The current channel spacing of 50 kHz (0.05 MHz) for Europe causes scanning for stations to take twice as long as needed and allows for accidental tuning to incorrect frequencies at x.x5 MHz. This patch fixes that.

The “Korea” regional setting is removed as it is now identical to the “Europe” regional setting. A few stations in Italy still use the 50 kHz (0.05 MHz) spacing so in order to receive these, a “World” regional setting with 50 kHz (0.05 MHz) channel spacing has been added.

Closed by  bertrik
2009-09-24 22:06
Reason for closing:  Accepted
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

Committed as svn r22822

Wouldn’t a Korean use the World setting, rather than the Europe one, which is meant to be used for Korea?

And wouldn’t an Italian use the Europe setting, rather than the world one?

I really think having some redundancy as I suggested on the ML is a more friendly solution.

Ok, maybe the current patch is no so friendly towards the Koreans or some Italians (although I think it’s already an improvement). One solution would be add specific countries (not regions) to the list, but in that case I can see the list easily grow out of control. Perhaps we could cover it by adding an “Asia” setting. Does anyone know what kind of FM settings other countries in Asia use? I don’t really like the solution of explicitly specifying the channel spacing, but maybe that’s the only way to go.

I don’t see the issue in having these settings (other than a bit of waste):

North America: 87.9 - 107.9 MHz, 200 kHz steps, 75 us de-emphasis
Europe: 87.5 - 108.0 MHz, 100 kHz steps, 50 us de-emphasis
Italy: 87.5 - 108.0 MHz, 50 kHz steps, 50 us de-emphasis
Japan: 76.0 - 90.0 MHz, 100 kHz steps, 50 us de-emphasis
Other: 87.5 - 108.0 MHz, 50 kHz steps, 50 us de-emphasis

Italians would pick “Italy” (since that’s a better fit than Europe), Koreans would pick “Other” (since there’s nothing else that applies to them).

Koreans are supposed to pick europe IIUC.

Ah right. So call it Europe/Korea instead :-)

I just downloaded the list of Korean FM stations from fmlist.org and guess what … all stations are spaced on a 200 kHz (not 100 kHz) grid, just like North America!

Okay, so add a Korea option instead.

There’s already a Korea option, this patch removes it ;)

I meant to my suggestion:

North America: 87.9 - 107.9 MHz, 200 kHz steps, 75 us de-emphasis
Europe: 87.5 - 108.0 MHz, 100 kHz steps, 50 us de-emphasis
Korea: 87.5 - 108.0 MHz, 200 kHz steps, 50 us de-emphasis
Italy: 87.5 - 108.0 MHz, 50 kHz steps, 50 us de-emphasis
Japan: 76.0 - 90.0 MHz, 100 kHz steps, 50 us de-emphasis
Other: 87.5 - 108.0 MHz, 50 kHz steps, 50 us de-emphasis

These sound good imo.

I think I’m going to close this task since the patch is not going in its current shape anyway. I’m not really motivated to sort out the mess called fm region settings (and as a European I’ll just keep using the Korean setting…).

What’s the issue with my last suggestion? IIRC it covers everyone, and everyone will get the right setting if they pick the region that’s most specific to them.

This patch implements my suggestion. I’ve tested that it compiles on a range of players, but haven’t tested it on target yet.

Resynced against 22731, with intention to commit when rockbox 3.4 freeze ends.

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing