summaryrefslogtreecommitdiff
path: root/nuttx/arch/arm/src/kinetis/kinetis_sdhc.c
diff options
context:
space:
mode:
Diffstat (limited to 'nuttx/arch/arm/src/kinetis/kinetis_sdhc.c')
-rw-r--r--nuttx/arch/arm/src/kinetis/kinetis_sdhc.c79
1 files changed, 30 insertions, 49 deletions
diff --git a/nuttx/arch/arm/src/kinetis/kinetis_sdhc.c b/nuttx/arch/arm/src/kinetis/kinetis_sdhc.c
index aab0282f6..d9da13cf2 100644
--- a/nuttx/arch/arm/src/kinetis/kinetis_sdhc.c
+++ b/nuttx/arch/arm/src/kinetis/kinetis_sdhc.c
@@ -172,8 +172,8 @@
# define SAMPLENDX_BEFORE_SETUP 0
# define SAMPLENDX_BEFORE_ENABLE 1
# define SAMPLENDX_AFTER_SETUP 2
-# define SAMPLENDX_END_TRANSFER 3
-# define SAMPLENDX_DMA_CALLBACK 4
+# define SAMPLENDX_DMA_CALLBACK 3
+# define SAMPLENDX_END_TRANSFER 4
# define DEBUG_NSAMPLES 5
# else
# define SAMPLENDX_BEFORE_SETUP 0
@@ -680,13 +680,13 @@ static void kinetis_dumpsamples(struct kinetis_dev_s *priv)
}
#endif
kinetis_dumpsample(priv, &g_sampleregs[SAMPLENDX_AFTER_SETUP], "After setup");
- kinetis_dumpsample(priv, &g_sampleregs[SAMPLENDX_END_TRANSFER], "End of transfer");
#if defined(CONFIG_DEBUG_DMA) && defined(CONFIG_SDIO_DMA)
if (priv->dmamode)
{
kinetis_dumpsample(priv, &g_sampleregs[SAMPLENDX_DMA_CALLBACK], "DMA Callback");
}
#endif
+ kinetis_dumpsample(priv, &g_sampleregs[SAMPLENDX_END_TRANSFER], "End of transfer");
}
#endif
@@ -1037,10 +1037,6 @@ static void kinetis_endtransfer(struct kinetis_dev_s *priv, sdio_eventset_t wkup
#ifdef CONFIG_SDIO_DMA
if (priv->dmamode)
{
- /* DMA debug instrumentation */
-
- kinetis_sample(priv, SAMPLENDX_END_TRANSFER);
-
/* Make sure that the DMA is stopped (it will be stopped automatically
* on normal transfers, but not necessarily when the transfer terminates
* on an error condition).
@@ -1054,6 +1050,10 @@ static void kinetis_endtransfer(struct kinetis_dev_s *priv, sdio_eventset_t wkup
priv->remaining = 0;
+ /* Debug instrumentation */
+
+ kinetis_sample(priv, SAMPLENDX_END_TRANSFER);
+
/* Is a thread wait for these data transfer complete events? */
if ((priv->waitevents & wkupevent) != 0)
@@ -1780,30 +1780,31 @@ static void kinetis_sendcmd(FAR struct sdio_dev_s *dev, uint32_t cmd, uint32_t a
switch (cmd & MMCSD_RESPONSE_MASK)
{
- case MMCSD_NO_RESPONSE:
- regval |= SDHC_XFERTYP_RSPTYP_NONE; /* No response */
+ case MMCSD_NO_RESPONSE: /* No response */
+ regval |= SDHC_XFERTYP_RSPTYP_NONE;
break;
- case MMCSD_R1B_RESPONSE:
- regval |= SDHC_XFERTYP_RSPTYP_LEN48BSY; /* Response length 48, check busy */
+ case MMCSD_R1B_RESPONSE: /* Response length 48, check busy & cmdindex*/
+ regval |= (SDHC_XFERTYP_RSPTYP_LEN48BSY | SDHC_XFERTYP_CICEN);
+ break;
- case MMCSD_R1_RESPONSE:
- case MMCSD_R3_RESPONSE:
+ case MMCSD_R1_RESPONSE: /* Response length 48, check cmdindex */
+ case MMCSD_R6_RESPONSE:
+ regval |= (SDHC_XFERTYP_RSPTYP_LEN48 | SDHC_XFERTYP_CICEN);
+ break;
+
+ case MMCSD_R3_RESPONSE: /* Response length 48 */
case MMCSD_R4_RESPONSE:
case MMCSD_R5_RESPONSE:
- case MMCSD_R6_RESPONSE:
case MMCSD_R7_RESPONSE:
- regval |= SDHC_XFERTYP_RSPTYP_LEN48; /* Response length 48 */
+ regval |= SDHC_XFERTYP_RSPTYP_LEN48;
break;
- case MMCSD_R2_RESPONSE:
- regval |= SDHC_XFERTYP_RSPTYP_LEN136; /* Response length 136 */
+ case MMCSD_R2_RESPONSE: /* Response length 136 */
+ regval |= SDHC_XFERTYP_RSPTYP_LEN136;
break;
}
-# warning "Missing other settings?"
- fvdbg("cmd: %08x arg: %08x regval: %08x\n", cmd, arg, regval);
-
/* Enable DMA */
#ifdef CONFIG_SDIO_DMA
@@ -1815,16 +1816,11 @@ static void kinetis_sendcmd(FAR struct sdio_dev_s *dev, uint32_t cmd, uint32_t a
}
#endif
- /* Other bits (revisit -- these probably should not alway be set).
- *
- * CICEN=1: The SDHC will check the index field in the response to see if
- * it has the same value as the command index. If it is not, it
- * is reported as a command index error.
- * Also... what about CMDTYP?
- */
+ /* Other bits? What about CMDTYP? */
#warning "Revisit"
- regval |= SDHC_XFERTYP_CICEN;
+
+ fvdbg("cmd: %08x arg: %08x regval: %08x\n", cmd, arg, regval);
/* Set the SDHC Argument value */
@@ -2093,9 +2089,6 @@ static int kinetis_waitresponse(FAR struct sdio_dev_s *dev, uint32_t cmd)
static int kinetis_recvshortcrc(FAR struct sdio_dev_s *dev, uint32_t cmd, uint32_t *rshort)
{
-#ifdef CONFIG_DEBUG
- uint32_t respcmd;
-#endif
uint32_t regval;
int ret = OK;
@@ -2154,19 +2147,6 @@ static int kinetis_recvshortcrc(FAR struct sdio_dev_s *dev, uint32_t cmd, uint32
fdbg("ERROR: CRC failure: %08x\n", regval);
ret = -EIO;
}
-#ifdef CONFIG_DEBUG
- else
- {
- /* Check response received is of desired command */
-
- respcmd = getreg32(KINETIS_SDHC_RESPCMD);
- if ((uint8_t)(respcmd & SDHC_RESPCMD_MASK) != (cmd & MMCSD_CMDIDX_MASK))
- {
- fdbg("ERROR: RESCMD=%02x CMD=%08x\n", respcmd, cmd);
- ret = -EINVAL;
- }
- }
-#endif
}
/* Clear all pending message completion events and return the R1/R6 response */
@@ -2461,6 +2441,7 @@ static sdio_eventset_t kinetis_eventwait(FAR struct sdio_dev_s *dev,
priv->xfrflags = 0;
#endif
+ kinetis_dumpsamples(priv);
return wkupevent;
}
@@ -2789,7 +2770,7 @@ static void kinetis_callback(void *arg)
****************************************************************************/
/****************************************************************************
- * Name: sdio_initialize
+ * Name: sdhc_initialize
*
* Description:
* Initialize SDIO for operation.
@@ -2802,7 +2783,7 @@ static void kinetis_callback(void *arg)
*
****************************************************************************/
-FAR struct sdio_dev_s *sdio_initialize(int slotno)
+FAR struct sdio_dev_s *sdhc_initialize(int slotno)
{
uint32_t regval;
@@ -2888,7 +2869,7 @@ FAR struct sdio_dev_s *sdio_initialize(int slotno)
}
/****************************************************************************
- * Name: sdio_mediachange
+ * Name: sdhc_mediachange
*
* Description:
* Called by board-specific logic -- posssible from an interrupt handler --
@@ -2906,7 +2887,7 @@ FAR struct sdio_dev_s *sdio_initialize(int slotno)
*
****************************************************************************/
-void sdio_mediachange(FAR struct sdio_dev_s *dev, bool cardinslot)
+void sdhc_mediachange(FAR struct sdio_dev_s *dev, bool cardinslot)
{
struct kinetis_dev_s *priv = (struct kinetis_dev_s *)dev;
uint8_t cdstatus;
@@ -2951,7 +2932,7 @@ void sdio_mediachange(FAR struct sdio_dev_s *dev, bool cardinslot)
*
****************************************************************************/
-void sdio_wrprotect(FAR struct sdio_dev_s *dev, bool wrprotect)
+void sdhc_wrprotect(FAR struct sdio_dev_s *dev, bool wrprotect)
{
struct kinetis_dev_s *priv = (struct kinetis_dev_s *)dev;
irqstate_t flags;