Rockbox

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

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

FS#9575 - Preliminary button driver for Sansa Fuze

Attached to Project: Rockbox
Opened by Thomas Martitz (kugel.) - Monday, 24 November 2008, 21:09 GMT+2
Last edited by Björn Stenberg (zagor) - Monday, 24 November 2008, 21:51 GMT+2
Task Type Patches
Category Applications
Status Closed
Assigned To No-one
Player Type All players
Severity Low
Priority Normal
Reported Version Release 3.0
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Private No

Details

Simply copied button_int from button-e200v2.c.

I also updated the V2HardwareMapping wiki page accordingly.
   button-fuze.c.diff (1.7 KiB) 
 firmware/target/arm/as3525/sansa-fuze/button-fuze.c |   50 ++++++++++++++++++++
 1 file changed, 50 insertions(+)

This task depends upon

Closed by  Björn Stenberg (zagor)
Monday, 24 November 2008, 21:51 GMT+2
Reason for closing:  Accepted
Comment by Rafaël Carré (funman) - Monday, 24 November 2008, 21:33 GMT+2
as said on irc, I prefer if the unneeded gpio*_dir are removed; since they won't be used in something else than buttons (to check with the hardware mappings wiki)

gpio*_afsel need to be preserved since they are used in lcd.

Also I'm not familiar with the rockbox threads; could an interrupt happen while these registers are set to gpio (xpb and xpc as gpio), and switch the CPU to LCD code (which would need xpb and xpc as DBOP) ?
Comment by Thomas Martitz (kugel.) - Monday, 24 November 2008, 21:35 GMT+2
That can be done after commit imho in a seperate patch, which will not only affect fuze, but also e200 of course.

Also, I did some test and wasn't quite successful.

Loading...