Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category FM Tuner
  • Assigned To No-one
  • Operating System Sansa c200
  • 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 pixelma - 2007-10-01
Last edited by pixelma - 2007-10-02

FS#7881 - Enable radio on the Sansa c200

This patch enables the radio on the Sansa c200 - made by Mark Arigo (lowlight) who posted a link to it in IRC.

Even if it won’t be here for long (I hope), I thought it is good to post it to the tracker because it gives the opportunity to discuss, comment and improve in one place and nothing gets lost in the IRC logs.

The only thing that’s changed from the original patch is the keymap file to add volume controls in the radio screen (VOL_UP/VOL_DOWN). I’m not a hundred percent sure if my way is the right approach but it works and I saw the same “style” in other keymap files and places too. - All other targets use the settings increase/decrease actions (which is UP and DOWN on the c200) for volume controls but in the radio context there are already other actions assigned to UP and DOWN. I then added the settings increase/decrease actions to context_radio (and assigned the volume buttons to it) so that these take priority over the ones defined in context_settings (which covers the rest).

   c200radio.diff (3.9 KiB)
Closed by  pixelma
2007-10-02 21:07
Reason for closing:  Accepted
lowlight commented on 2007-10-02 01:28

I was going to commit but then realized some c200’s don’t have radios. Perhaps we need to think of real-time detection…any other targets do that? Are there e200’s without radio too?

lowlight commented on 2007-10-02 02:47

I looked and there’s some sort of runtime radio detection present already. So we need tester with a c200 without a radio (Zagor).

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing