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 Sikken - 2009-01-25
Last edited by Bertrik Sikken - 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 Sikken
2009-09-24 22:06
Reason for closing:  Accepted
Additional comments about closing:  

Committed as svn r22822

Thomas Martitz commented on 2009-01-25 15:24

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

Jonas Häggqvist commented on 2009-01-25 19:04

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.

Bertrik Sikken commented on 2009-01-25 19:33

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.

Jonas Häggqvist commented on 2009-01-25 20:31

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).

Thomas Martitz commented on 2009-01-25 20:32

Koreans are supposed to pick europe IIUC.

Jonas Häggqvist commented on 2009-01-25 20:39

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

Bertrik Sikken commented on 2009-01-25 20:45

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!

Jonas Häggqvist commented on 2009-01-25 20:46

Okay, so add a Korea option instead.

Thomas Martitz commented on 2009-01-25 21:19

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

Jonas Häggqvist commented on 2009-01-25 21:25

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

Thomas Martitz commented on 2009-01-25 21:26

These sound good imo.

Bertrik Sikken commented on 2009-08-08 14:16

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…).

Jonas Häggqvist commented on 2009-08-08 14:19

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.

Jonas Häggqvist commented on 2009-08-08 15:29

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

Bertrik Sikken commented on 2009-09-20 20:02

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

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing