• Status Closed
  • Percent Complete
  • 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!

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


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)

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

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.

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

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

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)

Remove panicf() if send_cmd() fails (it seems to work fine)

Add a comment for dummy response


Available keyboard shortcuts


Task Details

Task Editing