Rockbox

This is the bug/patch tracker for Rockbox. Click here for more information.

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

FS#8767 - Tuner bandwidth extension for LV24020LP based tuners (72.4Mhz - 121.3Mhz)

Attached to Project: Rockbox
Opened by Joel Garske (gibbon_) - Thursday, 20 March 2008, 16:38 GMT+2
Last edited by Dominik Riebeling (bluebrother) - Sunday, 23 September 2012, 21:35 GMT+2
Task Type Patches
Category FM Tuner
Status Closed
Assigned To No-one
Player Type Another
Severity Low
Priority Normal
Reported Version Daily build (which?)
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Private No

Details

Supported are players using the LV24020LP tuner.

This patch adds 5 Regions to the FM Tuner Region menu where supported

72.400Mhz - 121.300Mhz step: 12.5kHz
72.400Mhz - 121.300Mhz step: 20.0kHz
72.415Mhz - 121.295Mhz step: 20.0kHz (german BOS raster)
72.400Mhz - 121.300Mhz step: 50.0kHz
72.400Mhz - 121.300Mhz step:100.0kHz

and enables some basic creative tuner usage. The limits have been determined by bruteforce. I tried as long as i found the point where the frequency took a lot longer to tune.

Should be save for all targets.
   additional_regions.patch (5.7 KiB) 
 apps/recorder/radio.c   |    2 -
 apps/recorder/radio.h   |    2 -
 apps/lang/english.lang  |   80 ++++++++++++++++++++++++++++++++++++++++++++++++
 apps/settings_list.c    |   22 ++++++++++++-
 firmware/export/tuner.h |    5 +++
 firmware/tuner.c        |   23 +++++++++++++
 6 files changed, 129 insertions(+), 5 deletions(-)

This task depends upon

Closed by  Dominik Riebeling (bluebrother)
Sunday, 23 September 2012, 21:35 GMT+2
Reason for closing:  Out of Date
Additional comments about closing:  Nobody was interested in this in several years. Additionally, we intentionally only allow "useful" frequencies, so something like this is unlikely to make it into Rockbox, so someone interested in such a feature has to create his own build anyway.
Comment by Dominik Riebeling (bluebrother) - Thursday, 20 March 2008, 18:48 GMT+2
You included changed apart from the mentioned additional regions (changing the number of maximum presets etc.) Also, the TEA5767 can also tune further than the region values are limiting, and the Archos tuner as well. Besides, we do not want supporting radio settings that are out of range for the chosen location, so this most likely will get rejected.
Comment by Joel Garske (gibbon_) - Thursday, 20 March 2008, 22:54 GMT+2
Ah well... the number of presets slipped in from another approach, sorry :/

Thanks anyway
Comment by Joel Garske (gibbon_) - Thursday, 20 March 2008, 22:58 GMT+2
and you mention the TEA5767... i don't have that device. I supposed that it was capable of tuning to different frequencies, but because i didn't exactly know the margins... so i left it out.

I totally accept not supporting that region settings, but may i ask a reason? I don't mind if there just isn't one and you just want to keep clear about legal issues.
Comment by Marcin Łukasik (Moarc) - Sunday, 12 October 2008, 16:35 GMT+2
Patch needs resync, now fails at apps/settings_list.c.
   settings_list.c.rej (1.1 KiB)
Comment by Marcin Łukasik (Moarc) - Saturday, 13 December 2008, 07:46 GMT+2
I fixed this patch, now applies to r19415 without any problems.
   additional_regions.patch (5.8 KiB) 
 apps/recorder/radio.c   |    2 -
 apps/recorder/radio.h   |    2 -
 apps/lang/english.lang  |   80 ++++++++++++++++++++++++++++++++++++++++++++++++
 apps/settings_list.c    |   23 ++++++++++++-
 firmware/export/tuner.h |    5 +++
 firmware/tuner.c        |   23 +++++++++++++
 6 files changed, 129 insertions(+), 6 deletions(-)

Loading...