Rockbox

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

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

FS#7634 - SDHC support

Attached to Project: Rockbox
Opened by Barry Wardell (barrywardell) - Tuesday, 21 August 2007, 02:49 GMT+2
Last edited by Barry Wardell (barrywardell) - Wednesday, 22 August 2007, 02:33 GMT+2
Task Type Patches
Category Drivers
Status Closed
Assigned To Barry Wardell (barrywardell)
Player Type Sansa e200
Severity Low
Priority Normal
Reported Version Daily build (which?)
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Private No

Details

Current microSD cards only support sizes up to 2GB. In order to support larger card sizes up to 32GB, support for the SD specification version 2.00 is required (http://66.102.9.104/search?q=cache:lVMIQpGALuUJ:www.sdcard.org/sd_memorycard/Simplified%2520Physical%2520Layer%2520Specification.PDF+http://www.sdcard.org/sd_memorycard/Simplified%2520Physical%2520Layer%2520Specification.PDF&hl=en&ct=clnk&cd=1&gl=uk&client=firefox-a). This patch should add support for these microSD High Capacity cards, while maintaining backwards compatibility with older microSD cards.

Disclaimer: I've based this patch entirely on reading the SD specification version 2.00. It hasn't been tested on an e200 yet, so since it meddles with the disk access, if I've made a mistake somewhere it could lead to data loss both from the SD card and the e200's internal memory.
   sdhc.patch (3.3 KiB) 
 firmware/target/arm/sandisk/sansa-e200/ata-e200.c |   44 +++++++++++++++++++---
 1 file changed, 38 insertions(+), 6 deletions(-)

This task depends upon

Closed by  Barry Wardell (barrywardell)
Wednesday, 22 August 2007, 02:33 GMT+2
Reason for closing:  Accepted
Comment by Barry Wardell (barrywardell) - Tuesday, 21 August 2007, 03:02 GMT+2
Updated patch to not do the bank switching stuff for SDHC cards. Still untested.
   sdhc2.patch (5.1 KiB) 
 firmware/target/arm/sandisk/sansa-e200/ata-e200.c |   89 +++++++++++++++-------
 1 file changed, 64 insertions(+), 25 deletions(-)

Comment by Frank M. (framo) - Tuesday, 21 August 2007, 03:47 GMT+2
Hi Barry, many thanks for the quick posting of this patch after my comment on my problem with seeing a 4GB microSDHC card on my Sansa e280.
After applying the patch with today's svn code, I get the following error though: "ATA error: -219 Press ON to debug" and then a lot of numbers when I press the menue button.
Comment by Ben Kohler (iamben) - Tuesday, 21 August 2007, 04:00 GMT+2
same result here. after restoring a previous build, all is well (no corruption on SD or internal flash as far as i can tell). i look forward to seeing how this turns out, thanks for your hard work.
Comment by Barry Wardell (barrywardell) - Tuesday, 21 August 2007, 12:44 GMT+2
How about this?
   sdhc3.patch (5.2 KiB) 
 firmware/target/arm/sandisk/sansa-e200/ata-e200.c |   88 +++++++++++++++-------
 1 file changed, 63 insertions(+), 25 deletions(-)

Comment by Barry Wardell (barrywardell) - Tuesday, 21 August 2007, 15:58 GMT+2
I've just tried this with my own 4GB microSDHC on my Sansa and it seems to work fine. I haven't tested thoroughly though.
Comment by Jack Suter (chrisjs169) - Tuesday, 21 August 2007, 19:46 GMT+2
I just got a stack overflow when connecting the USB cable. No other patches added, except this one. While this is one of the first times I've gotten a stack overflow, AFAIK it usually says "*PANIC* Stkov [stack name]" though mine didn't have a "stack name" and just said "*PANIC* Stkov". I wasn't able to reproduce the error, but it could possibly be related I guess.
Comment by Frank M. (framo) - Tuesday, 21 August 2007, 21:44 GMT+2
Barry, you are a star, I can't believe it works!
My 4GB SanDisk microSDHC card is readable with this patch on my Sansa e280.
It shows up as a <microSD1> folder in the root directory and I am able to browse and play songs from it.
I can also use Rockbox to copy/paste between the SDHC card and the internal memory or rename/delete files.
Cut/Paste between the card and the internal memory gives a "failed" error message but it still works.
Thank you so much!
Comment by Barry Wardell (barrywardell) - Wednesday, 22 August 2007, 02:05 GMT+2
Updated version. Reads the CSD version 2.0 properly so we now have currcard->max_read_bl_len, currcard->block_size, currcard->numblocks and currcard->capacity are filled in properly. Probably won't make any operational difference.

The cut/paste bug is a separate issue that had gone unnoticed. I've committed a fix to SVN.

I'm not sure about the stkov problem, but I'm guessing that's a separate problem too.
   sdhc4.patch (7.5 KiB) 
 firmware/target/arm/sandisk/sansa-e200/ata-e200.c |  123 ++++++++++++++++------
 1 file changed, 91 insertions(+), 32 deletions(-)

Loading...