Rockbox

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

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

FS#11403 - Support for mystery FM chip in some Sansa Clip+

Attached to Project: Rockbox
Opened by Bertrik Sikken (bertrik) - Monday, 14 June 2010, 23:40 GMT+2
Last edited by Bertrik Sikken (bertrik) - Tuesday, 15 June 2010, 23:04 GMT+2
Task Type Patches
Category FM Tuner
Status Closed
Assigned To No-one
Player Type All players
Severity Low
Priority Normal
Reported Version Release 3.4
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Private No

Details

Some Sansa Clip+ players don't have a si4700 chip like many other AMS sansas, but some unidentified other SiLabs FM tuner chip.

This patch adds support for this chip. It uses the multiple tuner mechanism so I had to add detection code to decide between a si4700 or the mystery chip. This may have impact on other players using an si4700.

Things that do not work right yet:
- I've seen it get auto-scanned presets off by 1 channel (e.g. 0.1 MHz)
- Stereo indication is not supported yet
- It's tricky to select the proper debug menu info, so it currently shows debug info for the si4700 only (even though the player may have detected the mystery FM chip).

Things to test:
- Verify that radio on sansa clip+ players with an si4700 still works
- Verify that radio on other players with an si4700 still works (AMS sansas, gigabeat S). It still works on my Clip v1.
   clipplus_mystery_fm_1.patch (20.5 KiB) 
 apps/debug_menu.c                                          |   23 
 firmware/export/tuner.h                                    |    8 
 firmware/export/si4700.h                                   |    1 
 firmware/export/config.h                                   |    1 
 firmware/export/config/sansaclipplus.h                     |    2 
 firmware/export/fmclipplus.h                               |   54 ++
 firmware/SOURCES                                           |    4 
 firmware/target/arm/as3525/sansa-clipplus/tuner-clipplus.c |   40 +
 firmware/drivers/tuner/fmclipplus.c                        |  332 +++++++++++++
 firmware/drivers/tuner/si4700.c                            |   29 -
 firmware/tuner.c                                           |   18 
 11 files changed, 485 insertions(+), 27 deletions(-)

This task depends upon

Closed by  Bertrik Sikken (bertrik)
Tuesday, 15 June 2010, 23:04 GMT+2
Reason for closing:  Accepted
Additional comments about closing:  Committed as svn r 26864
Comment by Michael Chicoine (mc2739) - Tuesday, 15 June 2010, 14:48 GMT+2
tested with r26857 and radio works on my e260v2. My radio id is 1242 0850
Comment by Alex Parker (BigBambi) - Tuesday, 15 June 2010, 19:11 GMT+2
Works fine on my Gigabeat S
Comment by Bertrik Sikken (bertrik) - Tuesday, 15 June 2010, 22:34 GMT+2
FlynDice tested on a clip+ with a si4700.

Final patch, soon to go in.
I fixed the debug screen to show either the si4700 or fmclipplus radio register.
   clipplus_mystery_fm_2.patch (21.6 KiB) 
 apps/debug_menu.c                                          |   55 +-
 firmware/export/tuner.h                                    |    6 
 firmware/export/si4700.h                                   |    1 
 firmware/export/config.h                                   |    1 
 firmware/export/config/sansaclipplus.h                     |    2 
 firmware/export/fmclipplus.h                               |   54 ++
 firmware/SOURCES                                           |    4 
 firmware/target/arm/as3525/sansa-clipplus/tuner-clipplus.c |   39 +
 firmware/drivers/tuner/fmclipplus.c                        |  332 +++++++++++++
 firmware/drivers/tuner/si4700.c                            |   29 -
 firmware/tuner.c                                           |   18 
 11 files changed, 510 insertions(+), 31 deletions(-)

Loading...