Rockbox

Tasklist

FS#7881 - Enable radio on the Sansa c200

Attached to Project: Rockbox
Opened by Marianne Arnold (pixelma) - Monday, 01 October 2007, 22:20 GMT
Last edited by Marianne Arnold (pixelma) - Tuesday, 02 October 2007, 21:07 GMT
Task Type Patches
Category FM Tuner
Status Closed
Assigned To No-one
Operating System Sansa c200
Severity Low
Priority Normal
Reported Version Daily build (which?)
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

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

This task depends upon

Closed by  Marianne Arnold (pixelma)
Tuesday, 02 October 2007, 21:07 GMT
Reason for closing:  Accepted
Comment by Mark Arigo (lowlight) - Tuesday, 02 October 2007, 01:28 GMT
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?
Comment by Mark Arigo (lowlight) - Tuesday, 02 October 2007, 02:47 GMT
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...