diff options
| author | Amaury Pouly <amaury.pouly@gmail.com> | 2013-07-02 00:42:10 +0200 |
|---|---|---|
| committer | Amaury Pouly <amaury.pouly@gmail.com> | 2013-07-02 00:45:00 +0200 |
| commit | 20ee453edce207a4285638f0399926b0b9f506df (patch) | |
| tree | e5a035e8a841fee8fee3a14a607e31290e48ed28 | |
| parent | d8024df1052966aa554424d88712244ae5978f2a (diff) | |
| download | rockbox-20ee453edce207a4285638f0399926b0b9f506df.zip rockbox-20ee453edce207a4285638f0399926b0b9f506df.tar.gz rockbox-20ee453edce207a4285638f0399926b0b9f506df.tar.bz2 rockbox-20ee453edce207a4285638f0399926b0b9f506df.tar.xz | |
imx233: fix sd init sequence
Implement the switch function as specified by the specification,
that is wait for the response AND transfer 64 bytes of data. This
fixes some issue when the SD card take a long time to switch. In
particular waiting 100ms (max per spec) will not work if no data
is transfered in some cases.
Change-Id: Ia22350468018b842e57ce6f6c1a8d676eba97fb8
| -rw-r--r-- | firmware/target/arm/imx233/sdmmc-imx233.c | 16 |
1 files changed, 12 insertions, 4 deletions
diff --git a/firmware/target/arm/imx233/sdmmc-imx233.c b/firmware/target/arm/imx233/sdmmc-imx233.c index b74c21c..c389a1d 100644 --- a/firmware/target/arm/imx233/sdmmc-imx233.c +++ b/firmware/target/arm/imx233/sdmmc-imx233.c @@ -353,8 +353,17 @@ static int init_sd_card(int drive) return -6; /* CMD6 */ - if(!send_cmd(drive, SD_SWITCH_FUNC, 0x80fffff1, MCI_RESP, NULL)) - return -7; + { + /* only transfer 64 bytes */ + imx233_ssp_set_block_size(ssp, /*log2(64)*/6); + if(imx233_ssp_sd_mmc_transfer(ssp, SD_SWITCH_FUNC, 0x80fffff1, + SSP_SHORT_RESP, aligned_buffer[drive], 1, true, true, NULL)) + { + imx233_ssp_set_block_size(ssp, /*log2(512)*/9); + return -12; + } + imx233_ssp_set_block_size(ssp, /*log2(512)*/9); + } /* go back to STBY state so we can read csd */ /* CMD7 w/rca=0: Deselect card to put it in STBY state */ @@ -465,10 +474,9 @@ static int init_mmc_drive(int drive) /* read extended CSD */ { - uint8_t ext_csd[512]; + uint8_t *ext_csd = aligned_buffer[drive]; if(imx233_ssp_sd_mmc_transfer(ssp, 8, 0, SSP_SHORT_RESP, aligned_buffer[drive], 1, true, true, &status)) return -12; - memcpy(ext_csd, aligned_buffer[drive], 512); uint32_t *sec_count = (void *)&ext_csd[212]; window_start[drive] = 0; window_end[drive] = *sec_count; |