Clean up the code. Manage the reconnection. Use the lwip error, insted
[bertos.git] / bertos / cpu / cortex-m3 / drv / sd_sam3.c
index fd4062c288dc0b01a0de34d89df1d99d917e2001..0c1c0a0628f51acd7980c6237e59df6bfd344e33 100644 (file)
  * invalidate any other reasons why the executable file might be covered by
  * the GNU General Public License.
  *
- * Copyright 2007 Develer S.r.l. (http://www.develer.com/)
+ * Copyright 2011 Develer S.r.l. (http://www.develer.com/)
  * -->
  *
  * \brief Function library for secure digital memory.
  *
- * \author Francesco Sacchi <batt@develer.com>
+ * \author Daniele Basile <asterix@develer.com>
  */
 
 
 #include <string.h> /* memset */
 
 
+#define SD_SEND_ALL_CID        BV(0)
 #define SD_STATUS_APP_CMD      BV(5)
 #define SD_STATUS_READY        BV(8)
-#define SD_STATUS_CURR_MASK    0x1E00
-#define SD_STATUS_CURR_SHIFT   9
+#define SD_CARD_IS_LOCKED      BV(25)
 
+#define SD_OCR_CCS              BV(30)     /**< SD Card Capacity Status (CCS) */
+#define SD_OCR_BUSY             BV(31)     /**< SD/MMC Card power up status bit (busy) */
+
+#define SD_OCR_VDD_27_28        BV(15)
+#define SD_OCR_VDD_28_29        BV(16)
+#define SD_OCR_VDD_29_30        BV(17)
+#define SD_OCR_VDD_30_31        BV(18)
+#define SD_OCR_VDD_31_32        BV(19)
+#define SD_OCR_VDD_32_33        BV(20)
+
+
+#define SD_HOST_VOLTAGE_RANGE     (SD_OCR_VDD_27_28 | \
+                                   SD_OCR_VDD_28_29 | \
+                                   SD_OCR_VDD_29_30 | \
+                                   SD_OCR_VDD_30_31 | \
+                                   SD_OCR_VDD_31_32 | \
+                                   SD_OCR_VDD_32_33)
+
+
+#define CMD8_V_RANGE_CHECK_PAT    0xAA
+#define CMD8_V_RANGE_27V_36V      (0x100 | CMD8_V_RANGE_CHECK_PAT)
+#define CMD8_V_RANGE_LOW          (0x1000 | CMD8_V_RANGE_CHECK_PAT)
+#define CMD8_V_ECHO_REPLY         0xFF
+#define CMD8_SUPP_V_RANGE_REPLY   0xFF00
+
+#define SD_STATUS_ERROR          BV(19)
+
+#define SD_GET_ERRORS(status)   ((status) & 0xFFF80000)
 #define SD_ADDR_TO_RCA(addr)    (uint32_t)(((addr) << 16) & 0xFFFF0000)
-#define SD_GET_STATE(status)    (uint8_t)(((status) & SD_STATUS_CURR_MASK) >> SD_STATUS_CURR_SHIFT)
+#define SD_GET_STATE(status)    (uint8_t)(((status) & 0x1E00) >> 9)
 
 static const uint32_t tran_exp[] =
 {
-    10000,      100000,     1000000,    10000000,
-    0,      0,      0,      0
+       10000,      100000,     1000000,    10000000,
+       0,      0,      0,      0
 };
 
 static const uint8_t tran_mant[] =
 {
-    0,  10, 12, 13, 15, 20, 25, 30,
-    35, 40, 45, 50, 55, 60, 70, 80,
+       0,  10, 12, 13, 15, 20, 25, 30,
+       35, 40, 45, 50, 55, 60, 70, 80,
 };
 
 
+void sd_dumpCsd(SdCSD *csd);
+void sd_dumpCid(SdCID *cid);
+void sd_dumpSsr(SdSSR *ssr);
+
+void sd_sendInit(void);
+void sd_goIdle(void);
+int sd_sendIfCond(Sd *sd);
+int sd_sendAppOpCond(Sd *sd);
+
+int sd_getCid(Sd *sd, SdCID *cid, uint32_t addr, uint8_t flag);
+int sd_getCsd(Sd *sd, SdCSD *csd);
+int sd_getSrc(Sd *sd);
+
+int sd_appStatus(Sd *sd);
+int sd_getRelativeAddr(Sd *sd);
+int sd_getStatus(Sd *sd, SdSSR *ssr, uint32_t *buf, size_t words);
+
+int sd_selectCard(Sd *sd);
+int sd_deSelectCard(Sd *sd);
+int sd_setBusWidth(Sd *sd, size_t len);
+int sd_set_BlockLen(Sd *sd, size_t len);
+void sd_setHightSpeed(Sd *sd);
+
+
+INLINE int sd_setBus4bit(Sd *sd)
+{
+       return sd_setBusWidth(sd, 4);
+}
+
+INLINE int sd_setBus1bit(Sd *sd)
+{
+       return sd_setBusWidth(sd, 1);
+}
+
+
 LOG_INFOB(
 static void dump(const char *label, uint32_t *r, size_t len)
 {
@@ -97,17 +160,59 @@ static void dump(const char *label, uint32_t *r, size_t len)
 }
 )
 
+void sd_dumpCsd(SdCSD *csd)
+{
+       ASSERT(csd);
+
+       LOG_INFO("VERSION: %d.0\n", csd->structure ? 2 : 1);
+       LOG_INFO("CARD COMMAND CLASS: %d\n", csd->ccc);
+       LOG_INFO("MAX DATA RATE: %ld\n", csd->max_data_rate);
+       LOG_INFO("WRITE BLK LEN BITS: %ld\n", csd->write_blk_bits);
+       LOG_INFO("READ BLK LEN BITS: %ld\n", csd->read_blk_bits);
+       LOG_INFO("ERASE SIZE: %ld\n", csd->erase_size);
+       LOG_INFO("BLK NUM: %ld\n", csd->block_num);
+       LOG_INFO("BLK LEN: %ld\n", csd->block_len);
+       LOG_INFO("CAPACITY %ld\n", csd->capacity);
+       LOG_INFO("FLAG Write: WP %d, W MISALIGN %d\n", csd->write_partial, csd->write_misalign);
+       LOG_INFO("FLAG Read: RP %d, R MISALIGN %d\n", csd->read_partial, csd->read_misalign);
+
+}
+
+void sd_dumpCid(SdCID *cid)
+{
+       ASSERT(cid);
+
+       LOG_INFO("MANFID: %d\n", cid->manfid);
+       LOG_INFO("OEMID: %d\n", cid->oemid);
+       LOG_INFO("SERIAL: %ld\n", cid->serial);
+       LOG_INFO("PROD_NAME: %s\n", cid->prod_name);
+       LOG_INFO("REV: %d.%d\n", cid->m_rev, cid->l_rev);
+       LOG_INFO("OFF,Y,M: %lx, %ld %ld\n", cid->year_off, (BCD_TO_INT_32BIT(cid->year_off) / 12) + 2000,
+                                                                                               (BCD_TO_INT_32BIT(cid->year_off) % 12));
+}
+
+void sd_dumpSsr(SdSSR *ssr)
+{
+       ASSERT(ssr);
 
-static int sd_decodeCsd(SDcsd *csd, uint32_t *resp, size_t len)
+       LOG_INFO("BUS_WIDTH: %d\n", ssr->bus_width);
+       LOG_INFO("TYPE: %d\n", ssr->card_type);
+       LOG_INFO("AU_TYPE: %d\n", ssr->au_size);
+       LOG_INFO("ERASE_SIZE: %d\n", ssr->erase_size);
+       LOG_INFO("SPEED_CLASS: %d\n", ssr->speed_class);
+}
+
+
+static int sd_decodeCsd(SdCSD *csd, uint32_t *resp, size_t len)
 {
        ASSERT(csd);
        ASSERT(resp);
        ASSERT(len >= 4);
 
-    csd->structure = UNSTUFF_BITS(resp, 126, 2);
+       csd->structure = UNSTUFF_BITS(resp, 126, 2);
        csd->ccc = UNSTUFF_BITS(resp, 84, 12);
 
-    csd->max_data_rate  = tran_exp[UNSTUFF_BITS(resp, 96, 3)] * tran_mant[UNSTUFF_BITS(resp, 99, 4)];
+       csd->max_data_rate  = tran_exp[UNSTUFF_BITS(resp, 96, 3)] * tran_mant[UNSTUFF_BITS(resp, 99, 4)];
 
        /*
         * CSD structure:
@@ -119,107 +224,63 @@ static int sd_decodeCsd(SDcsd *csd, uint32_t *resp, size_t len)
         * - >1: not defined.
         */
 
-    if (csd->structure == 0)
+       if (csd->structure == 0)
        {
                // (C_size + 1) x 2^(C_SIZE_MUL+2)
-               csd->blk_num = (1 + UNSTUFF_BITS(resp, 62, 12)) << (UNSTUFF_BITS(resp, 47, 3) + 2);
+               csd->block_num = (1 + UNSTUFF_BITS(resp, 62, 12)) << (UNSTUFF_BITS(resp, 47, 3) + 2);
 
                csd->read_blk_bits = UNSTUFF_BITS(resp, 80, 4);
                csd->write_blk_bits = UNSTUFF_BITS(resp, 22, 4);
 
-               csd->blk_len = 1 << csd->read_blk_bits;
-        csd->capacity  = csd->blk_num * csd->blk_len;
+               csd->block_len = 1 << csd->read_blk_bits;
+               csd->capacity  = csd->block_num * csd->block_len;
 
-        csd->read_partial = UNSTUFF_BITS(resp, 79, 1);
-        csd->read_misalign = UNSTUFF_BITS(resp, 77, 1);
+               csd->read_partial = UNSTUFF_BITS(resp, 79, 1);
+               csd->read_misalign = UNSTUFF_BITS(resp, 77, 1);
 
-        csd->write_misalign = UNSTUFF_BITS(resp, 78, 1);
-        csd->write_partial = UNSTUFF_BITS(resp, 21, 1);
+               csd->write_misalign = UNSTUFF_BITS(resp, 78, 1);
+               csd->write_partial = UNSTUFF_BITS(resp, 21, 1);
 
-        if (UNSTUFF_BITS(resp, 46, 1))
+               if (UNSTUFF_BITS(resp, 46, 1))
                {
-            csd->erase_size = 1;
-        }
+                       csd->erase_size = 1;
+               }
                else if(csd->write_blk_bits >= 9)
                {
-            csd->erase_size = UNSTUFF_BITS(resp, 39, 7) + 1;
-            csd->erase_size <<= csd->write_blk_bits - 9;
-        }
+                       csd->erase_size = UNSTUFF_BITS(resp, 39, 7) + 1;
+                       csd->erase_size <<= csd->write_blk_bits - 9;
+               }
 
                return 0;
        }
        else if (csd->structure == 1)
        {
                kprintf("csize %ld\n", UNSTUFF_BITS(resp, 48, 22));
-        csd->capacity  = (1 + UNSTUFF_BITS(resp, 48, 22)) << 10;
+               csd->capacity  = (1 + UNSTUFF_BITS(resp, 48, 22)) << 10;
 
                csd->write_blk_bits = 9;
                csd->write_partial = 0;
-        csd->write_misalign = 0;
+               csd->write_misalign = 0;
 
                csd->read_blk_bits = 9;
                csd->read_partial = 0;
-        csd->read_misalign = 0;
+               csd->read_misalign = 0;
 
-        csd->erase_size = 1;
+               csd->erase_size = 1;
                // the block size if fixed to 512kb
-               csd->blk_len = (1 << csd->write_blk_bits) << 10;
+               csd->block_len = (1 << csd->write_blk_bits) << 10;
 
-        return 0;
+               return 0;
        }
-    else
+       else
        {
-        kprintf("Unrecognised CSD structure version %d\n", csd->structure);
-        return -1;
-    }
-
-    return 0;
-}
-
-
-void sd_dumpCsd(Sd *sd)
-{
-       ASSERT(sd);
-
-       LOG_INFO("VERSION: %d.0\n", sd->csd.structure ? 2 : 1);
-    LOG_INFO("CARD COMMAND CLASS: %d\n", sd->csd.ccc);
-       LOG_INFO("MAX DATA RATE: %ld\n", sd->csd.max_data_rate);
-       LOG_INFO("WRITE BLK LEN BITS: %ld\n", sd->csd.write_blk_bits);
-       LOG_INFO("READ BLK LEN BITS: %ld\n", sd->csd.read_blk_bits);
-       LOG_INFO("ERASE SIZE: %ld\n", sd->csd.erase_size);
-       LOG_INFO("BLK NUM: %ld\n", sd->csd.blk_num);
-       LOG_INFO("BLK LEN: %ld\n", sd->csd.blk_len);
-       LOG_INFO("CAPACITY %ld\n", sd->csd.capacity);
-       LOG_INFO("FLAG Write: WP %d, W MISALIGN %d\n", sd->csd.write_partial, sd->csd.write_misalign);
-       LOG_INFO("FLAG Read: RP %d, R MISALIGN %d\n", sd->csd.read_partial, sd->csd.read_misalign);
-
-}
-
-void sd_dumpCid(Sd *sd)
-{
-       ASSERT(sd);
-
-       LOG_INFO("MANFID: %d\n", sd->cid.manfid);
-    LOG_INFO("OEMID: %d\n", sd->cid.oemid);
-       LOG_INFO("SERIAL: %ld\n", sd->cid.serial);
-    LOG_INFO("PROD_NAME: %s\n", sd->cid.prod_name);
-    LOG_INFO("REV: %d.%d\n", sd->cid.m_rev, sd->cid.l_rev);
-    LOG_INFO("OFF,Y,M: %lx, %ld %ld\n", sd->cid.year_off, (BCD_TO_INT_32BIT(sd->cid.year_off) / 12) + 2000,
-                                                                                               (BCD_TO_INT_32BIT(sd->cid.year_off) % 12));
-}
-
-void sd_dumpSsr(Sd *sd)
-{
-       ASSERT(sd);
+               kprintf("Unrecognised CSD structure version %d\n", csd->structure);
+               return -1;
+       }
 
-       LOG_INFO("BUS_WIDTH: %d\n", sd->ssr.bus_width);
-    LOG_INFO("TYPE: %d\n", sd->ssr.card_type);
-       LOG_INFO("AU_TYPE: %d\n", sd->ssr.au_size);
-       LOG_INFO("ERASE_SIZE: %d\n", sd->ssr.erase_size);
-       LOG_INFO("SPEED_CLASS: %d\n", sd->ssr.speed_class);
+       return 0;
 }
 
-
 void sd_sendInit(void)
 {
        hsmci_init(NULL); //TODO: REMOVE IT!
@@ -250,7 +311,6 @@ int sd_sendIfCond(Sd *sd)
                return 0;
        }
        LOG_ERR("IF_COND: %lx\n", (sd->status));
-
        return -1;
 }
 
@@ -278,8 +338,6 @@ int sd_sendAppOpCond(Sd *sd)
                                LOG_INFO("SD power up! Hight Capability [%d]\n", (bool)((sd->status) & SD_OCR_CCS));
                                return 0;
                        }
-
-                       LOG_ERR("sd not ready.\n");
                }
        }
 
@@ -287,10 +345,11 @@ int sd_sendAppOpCond(Sd *sd)
 }
 
 
-int sd_getCid(Sd *sd, uint32_t addr, uint8_t flag)
+int sd_getCid(Sd *sd, SdCID *cid, uint32_t addr, uint8_t flag)
 {
        ASSERT(sd);
-       memset(&(sd->cid), 0, sizeof(SDcid));
+       ASSERT(cid);
+       memset(cid, 0, sizeof(SdCID));
 
        uint8_t idx = 9; // CMD9 get cid from gived sd address (RCA)
        if (flag & SD_SEND_ALL_CID)
@@ -308,26 +367,27 @@ int sd_getCid(Sd *sd, uint32_t addr, uint8_t flag)
                hsmci_readResp(resp, 4);
                LOG_INFOB(dump("CID", resp, 4););
 
-               sd->cid.manfid        = UNSTUFF_BITS(resp, 120, 8);
-               sd->cid.oemid         = UNSTUFF_BITS(resp, 104, 16);
-               sd->cid.prod_name[0]      = UNSTUFF_BITS(resp, 96, 8);
-               sd->cid.prod_name[1]      = UNSTUFF_BITS(resp, 88, 8);
-               sd->cid.prod_name[2]      = UNSTUFF_BITS(resp, 80, 8);
-               sd->cid.prod_name[3]      = UNSTUFF_BITS(resp, 72, 8);
-               sd->cid.prod_name[4]      = UNSTUFF_BITS(resp, 64, 8);
-               sd->cid.m_rev         = UNSTUFF_BITS(resp, 60, 4);
-               sd->cid.l_rev         = UNSTUFF_BITS(resp, 56, 4);
-               sd->cid.serial        = (uint32_t)UNSTUFF_BITS(resp, 24, 32);
-               sd->cid.year_off      = UNSTUFF_BITS(resp, 8, 12);
+               cid->manfid        = UNSTUFF_BITS(resp, 120, 8);
+               cid->oemid         = UNSTUFF_BITS(resp, 104, 16);
+               cid->prod_name[0]      = UNSTUFF_BITS(resp, 96, 8);
+               cid->prod_name[1]      = UNSTUFF_BITS(resp, 88, 8);
+               cid->prod_name[2]      = UNSTUFF_BITS(resp, 80, 8);
+               cid->prod_name[3]      = UNSTUFF_BITS(resp, 72, 8);
+               cid->prod_name[4]      = UNSTUFF_BITS(resp, 64, 8);
+               cid->m_rev         = UNSTUFF_BITS(resp, 60, 4);
+               cid->l_rev         = UNSTUFF_BITS(resp, 56, 4);
+               cid->serial        = (uint32_t)UNSTUFF_BITS(resp, 24, 32);
+               cid->year_off      = UNSTUFF_BITS(resp, 8, 12);
        }
 
        return 0;
 }
 
-int sd_getCsd(Sd *sd)
+int sd_getCsd(Sd *sd, SdCSD *csd)
 {
        ASSERT(sd);
-       memset(&(sd->csd), 0, sizeof(SDcsd));
+       ASSERT(csd);
+       memset(csd, 0, sizeof(SdCSD));
 
        LOG_INFO("Send to RCA: %lx\n", SD_ADDR_TO_RCA(sd->addr));
        if (hsmci_sendCmd(9, SD_ADDR_TO_RCA(sd->addr), HSMCI_CMDR_RSPTYP_136_BIT))
@@ -340,7 +400,7 @@ int sd_getCsd(Sd *sd)
                uint32_t resp[4];
                hsmci_readResp(resp, 4);
                LOG_INFOB(dump("CSD", resp, 4););
-               sd_decodeCsd(&(sd->csd), resp, 4);
+               sd_decodeCsd(csd, resp, 4);
        }
 
        return 0;
@@ -370,6 +430,7 @@ int sd_appStatus(Sd *sd)
        if (hsmci_sendCmd(13, SD_ADDR_TO_RCA(sd->addr), HSMCI_CMDR_RSPTYP_48_BIT))
        {
                LOG_ERR("STATUS: %lx\n", HSMCI_SR);
+               sd->status |= SD_STATUS_ERROR;
                return -1;
        }
 
@@ -392,6 +453,7 @@ INLINE int sd_cardSelection(Sd *sd, uint32_t rca)
        if (hsmci_sendCmd(7, rca, HSMCI_CMDR_RSPTYP_R1B))
        {
                LOG_ERR("SELECT_SD: %lx\n", HSMCI_SR);
+               sd->status |= SD_STATUS_ERROR;
                return -1;
        }
 
@@ -462,7 +524,6 @@ int sd_setBusWidth(Sd *sd, size_t len)
        hsmci_readResp(&(sd->status), 1);
        if ((sd->status) & (SD_STATUS_APP_CMD | SD_STATUS_READY))
        {
-               hsmci_setBusWidth(len);
 
                uint8_t arg = 0;
                if (len == 4)
@@ -480,7 +541,10 @@ int sd_setBusWidth(Sd *sd, size_t len)
                LOG_INFO("State[%d]\n", SD_GET_STATE(sd->status));
 
                if (sd->status & SD_STATUS_READY)
+               {
+                       hsmci_setBusWidth(len);
                        return 0;
+               }
        }
 
        LOG_ERR("SET_BUS_WIDTH REP %lx\n", (sd->status));
@@ -503,7 +567,7 @@ int sd_set_BlockLen(Sd *sd, size_t len)
        LOG_INFOB(dump("SET_BLK_LEN", &(sd->status), 1););
        LOG_INFO("State[%d]\n", SD_GET_STATE(sd->status));
 
-       sd->csd.blk_len = len;
+       sd->b.blk_size = len;
 
        if (sd->status & SD_STATUS_READY)
                return 0;
@@ -511,12 +575,13 @@ int sd_set_BlockLen(Sd *sd, size_t len)
        return -1;
 }
 
-int sd_getStatus(Sd *sd, uint32_t *buf, size_t words)
+int sd_getStatus(Sd *sd, SdSSR *ssr, uint32_t *buf, size_t words)
 {
        ASSERT(sd);
+       ASSERT(ssr);
 
        // Status reply with 512bit data, so the block size in byte is 64
-       hsmci_prgRxDMA(buf, words, 64);
+       hsmci_read(buf, words, 64);
 
        if (hsmci_sendCmd(55, SD_ADDR_TO_RCA(sd->addr), HSMCI_CMDR_RSPTYP_48_BIT))
        {
@@ -524,8 +589,8 @@ int sd_getStatus(Sd *sd, uint32_t *buf, size_t words)
                return -1;
        }
 
-       uint32_t status = HSMCI_RSPR;
-       if (status & (SD_STATUS_APP_CMD | SD_STATUS_READY))
+       hsmci_readResp(&(sd->status), 1);
+       if (sd->status & (SD_STATUS_APP_CMD | SD_STATUS_READY))
        {
                if (hsmci_sendCmd(13, 0, HSMCI_CMDR_RSPTYP_48_BIT |
                                BV(HSMCI_CMDR_TRDIR) | HSMCI_CMDR_TRCMD_START_DATA | HSMCI_CMDR_TRTYP_SINGLE))
@@ -543,13 +608,12 @@ int sd_getStatus(Sd *sd, uint32_t *buf, size_t words)
                        hsmci_waitTransfer();
 
                        LOG_INFOB(dump("STATUS", buf, words););
-                       memset(&(sd->ssr), 0, sizeof(SDssr));
-
-                       sd->ssr.bus_width  = UNSTUFF_BITS(buf, 510, 2);
-                       sd->ssr.card_type  = UNSTUFF_BITS(buf, 480, 16);
-                       sd->ssr.au_size  = UNSTUFF_BITS(buf, 432, 8);
-                       sd->ssr.speed_class  = UNSTUFF_BITS(buf, 440, 8);
-                       sd->ssr.erase_size = UNSTUFF_BITS(buf, 408, 24);
+                       memset(ssr, 0, sizeof(SdSSR));
+                       ssr->bus_width  = UNSTUFF_BITS(buf, 510, 2);
+                       ssr->card_type  = UNSTUFF_BITS(buf, 480, 16);
+                       ssr->au_size  = UNSTUFF_BITS(buf, 432, 8);
+                       ssr->speed_class  = UNSTUFF_BITS(buf, 440, 8);
+                       ssr->erase_size = UNSTUFF_BITS(buf, 408, 24);
 
                        return 0;
                }
@@ -562,26 +626,25 @@ int sd_getStatus(Sd *sd, uint32_t *buf, size_t words)
 void sd_setHightSpeed(Sd *sd)
 {
        (void)sd;
-       hsmci_setSpeed(2100000, true);
+       hsmci_setSpeed(HSMCI_HIGH_SPEED, HSMCI_HS_MODE);
 }
 
 
 static size_t sd_SdReadDirect(struct KBlock *b, block_idx_t idx, void *buf, size_t offset, size_t size)
 {
        ASSERT(buf);
+       ASSERT(!((uint32_t)buf & 0x3));
+
        Sd *sd = SD_CAST(b);
        LOG_INFO("reading from block %ld, offset %d, size %d\n", idx, offset, size);
 
-       if (sd_selectCard(sd) < 0)
-               return -1;
-
-       hsmci_prgRxDMA(buf, size / 4, sd->csd.blk_len);
+       hsmci_waitTransfer();
+       hsmci_read(buf, size / 4, sd->b.blk_size);
 
-       if (hsmci_sendCmd(17, idx * sd->csd.blk_len + offset, HSMCI_CMDR_RSPTYP_48_BIT |
+       if (hsmci_sendCmd(17, idx * sd->b.blk_size + offset, HSMCI_CMDR_RSPTYP_48_BIT |
                        BV(HSMCI_CMDR_TRDIR) | HSMCI_CMDR_TRCMD_START_DATA | HSMCI_CMDR_TRTYP_SINGLE))
        {
                LOG_ERR("SIGLE_BLK_READ: %lx\n", HSMCI_SR);
-               sd_deSelectCard(sd);
                return -1;
        }
 
@@ -593,31 +656,27 @@ static size_t sd_SdReadDirect(struct KBlock *b, block_idx_t idx, void *buf, size
        if (sd->status & SD_STATUS_READY)
        {
                hsmci_waitTransfer();
-               sd_deSelectCard(sd);
                return size;
        }
-
-       sd_deSelectCard(sd);
        return -1;
 }
 
 static size_t sd_SdWriteDirect(KBlock *b, block_idx_t idx, const void *buf, size_t offset, size_t size)
 {
        ASSERT(buf);
+       ASSERT(!((uint32_t)buf & 0x3));
+
        Sd *sd = SD_CAST(b);
        const uint32_t *_buf = (const uint32_t *)buf;
-       LOG_INFO("reading from block %ld, offset %d, size %d\n", idx, offset, size);
-
-       if (sd_selectCard(sd) < 0)
-               return 0;
+       LOG_INFO("writing block %ld, offset %d, size %d\n", idx, offset, size);
 
-       hsmci_prgTxDMA(_buf, size / 4, sd->csd.blk_len);
+       hsmci_waitTransfer();
+       hsmci_write(_buf, size / 4, sd->b.blk_size);
 
-       if (hsmci_sendCmd(24, idx * sd->csd.blk_len + offset, HSMCI_CMDR_RSPTYP_48_BIT |
+       if (hsmci_sendCmd(24, idx * sd->b.blk_size + offset, HSMCI_CMDR_RSPTYP_48_BIT |
                                                HSMCI_CMDR_TRCMD_START_DATA | HSMCI_CMDR_TRTYP_SINGLE))
        {
                LOG_ERR("SIGLE_BLK_WRITE: %lx\n", HSMCI_SR);
-               sd_deSelectCard(sd);
                return -1;
        }
 
@@ -628,12 +687,9 @@ static size_t sd_SdWriteDirect(KBlock *b, block_idx_t idx, const void *buf, size
 
        if (sd->status & SD_STATUS_READY)
        {
-               hsmci_waitTransfer();
-               sd_deSelectCard(sd);
                return size;
        }
 
-       sd_deSelectCard(sd);
        return -1;
 }
 
@@ -641,7 +697,7 @@ static size_t sd_SdWriteDirect(KBlock *b, block_idx_t idx, const void *buf, size
 static int sd_SdError(KBlock *b)
 {
        Sd *sd = SD_CAST(b);
-       return 0;//sd->status;
+       return SD_GET_ERRORS(sd->status);
 }
 
 static void sd_SdClearerr(KBlock *b)
@@ -652,6 +708,7 @@ static void sd_SdClearerr(KBlock *b)
 
 static bool sd_blockInit(Sd *sd, KFile *ch)
 {
+       (void)ch;
        ASSERT(sd);
        memset(sd, 0, sizeof(*sd));
        DB(sd->b.priv.type = KBT_SD);
@@ -680,11 +737,12 @@ static bool sd_blockInit(Sd *sd, KFile *ch)
 
        if (sd_power_on)
        {
-               if(sd_getCid(sd, 0, SD_SEND_ALL_CID) < 0)
+               SdCID cid;
+               if(sd_getCid(sd, &cid, 0, SD_SEND_ALL_CID) < 0)
                        return false;
                else
                {
-                       sd_dumpCid(sd);
+                       sd_dumpCid(&cid);
                }
 
                if (sd_getRelativeAddr(sd) < 0)
@@ -694,12 +752,14 @@ static bool sd_blockInit(Sd *sd, KFile *ch)
                        LOG_INFO("RCA: %0lx\n", sd->addr);
                }
 
-
-               if (sd_getCsd(sd) < 0)
+               SdCSD csd;
+               if (sd_getCsd(sd, &csd) < 0)
                        return false;
                else
                {
-                       sd_dumpCsd(sd);
+                       sd->b.blk_cnt = csd.block_num * (csd.block_len / SD_DEFAULT_BLOCKLEN);
+                       LOG_INFO("blk_size %d, blk_cnt %ld\n", sd->b.blk_size, sd->b.blk_cnt);
+                       sd_dumpCsd(&csd);
                }
 
                if (sd_appStatus(sd) < 0)
@@ -714,17 +774,12 @@ static bool sd_blockInit(Sd *sd, KFile *ch)
                        return false;
                }
 
-               if (sd->status & SD_READY_FOR_DATA)
+               if (sd->status & SD_STATUS_READY)
                {
                        sd_selectCard(sd);
                        sd_set_BlockLen(sd, SD_DEFAULT_BLOCKLEN);
                        sd_setBus4bit(sd);
                        sd_setHightSpeed(sd);
-                       sd_deSelectCard(sd);
-
-                       sd->b.blk_size = SD_DEFAULT_BLOCKLEN;
-                       sd->b.blk_cnt = sd->csd.blk_num * (sd->csd.blk_len / SD_DEFAULT_BLOCKLEN);
-                       LOG_INFO("blk_size %d, blk_cnt %ld\n", sd->b.blk_size, sd->b.blk_cnt);
 
                        #if CONFIG_SD_AUTOASSIGN_FAT
                                disk_assignDrive(&sd->b, 0);
@@ -733,7 +788,7 @@ static bool sd_blockInit(Sd *sd, KFile *ch)
                        return true;
                }
        }
-
+       LOG_ERR("SD not ready.\n");
        return false;
 }