FS#11140 : sd-as3525v2.c Use CD Interrupt vs. CD polling in send

Status Closed
Percent Complete
100%
Task Type Patches
Category Drivers
Assigned To No-one
Operating System Another
Severity Low
Priority Very Low
Reported Version Release 3.4
Due in Version Undecided
Due Date Undecided
Votes
Private

Attached to Project: Rockbox
Opened by FlynDice - 2010-03-23
Last edited by funman - 2010-05-03

FS#11140 - sd-as3525v2.c Use CD Interrupt vs. CD polling in send_cmd()

I believe this patch works as far as the SD part is concerned. Rockbox loads from the SD normally and I can browse the contents of both the internal and uSD. Using the CD int also allows us to check for a response timeout and a bad response crc value. Attempting to play music results in a stkov ata/sd or a Data abort at various locations. This is new territory for me so I thought I’d ask for some help here if someone may understand what’s going on!

send_cmd_int.diff (3.1 KiB)

Closed by funman
2010-05-03 22:16
Reason for closing: Accepted
Additional comments about closing: Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407

r25799

Comments (7)
Related Tasks (0/0)

funman commented on 2010-03-24 12:36

There’s no wakeup_init() , although I’m not sure if it’s a problem (bss is set to 0)

The rest looks ok (didn’t test yet)

funman commented on 2010-03-24 17:06

ATA error -10 at boot with r25316, no µSD plugged in, Clip+ 4GB

FlynDice commented on 2010-03-24 22:55

Sorry, seems using HS timings makes it a bit touchier. I’ve got the response timeout disabled and it exhibits the same behavior as before with this patch.
Browsing both internal and uSD works fine and displaying album art files works but when I attempt to play a music file the wps screen flashes up and then I get Stkov ata/sd followed by data abort.

send_cmd_int_2.diff (4.8 KiB)

funman commented on 2010-03-25 14:15

same result than you, no idea where the problems come from.

funman commented on 2010-05-03 19:45

sync + Enable DMA channel before send_cmd() ⇒ looks alright on Clip+/Clipv2

send_cmd_int_3.diff (5.9 KiB)

funman commented on 2010-05-03 20:25

write support !!!!

I ask for a (discarded) response when issuing *_multiple_blocks

It works with this patch but without it I got errors when initializing the database (which stresses read/write i think)

send_cmd_int_4.diff (6 KiB)

funman commented on 2010-05-03 21:05