From 40174c2010758f637e9872514aab865dbb2f5d56 Mon Sep 17 00:00:00 2001
From: patacongo <patacongo@7fd9a85b-ad96-42d3-883c-3090e2eb8679>
Date: Thu, 26 Nov 2009 00:18:22 +0000
Subject: [PATCH] Numerous fixes for basic STM32 SDIO DMA access

git-svn-id: https://nuttx.svn.sourceforge.net/svnroot/nuttx/trunk@2279 7fd9a85b-ad96-42d3-883c-3090e2eb8679
---
 nuttx/ChangeLog                               |   4 +-
 nuttx/Documentation/NuttX.html                |   6 +-
 nuttx/arch/arm/src/common/up_initialize.c     |  13 ++
 nuttx/arch/arm/src/common/up_internal.h       |   3 +
 nuttx/arch/arm/src/stm32/stm32_dma.c          | 182 +++++++++++++-----
 nuttx/arch/arm/src/stm32/stm32_dma.h          |   3 +
 nuttx/arch/arm/src/stm32/stm32_internal.h     |  88 +++++----
 nuttx/arch/arm/src/stm32/stm32_sdio.c         |  31 ++-
 nuttx/configs/stm3210e-eval/RIDE/defconfig    |   2 +
 nuttx/configs/stm3210e-eval/include/board.h   |   1 +
 nuttx/configs/stm3210e-eval/nsh/defconfig     |   2 +
 nuttx/configs/stm3210e-eval/ostest/defconfig  |   2 +
 nuttx/configs/stm3210e-eval/src/up_boot.c     |  29 +--
 .../configs/stm3210e-eval/usbserial/defconfig |   2 +
 nuttx/include/debug.h                         |  32 +++
 nuttx/tools/mkconfig.c                        |  11 +-
 16 files changed, 296 insertions(+), 115 deletions(-)

diff --git a/nuttx/ChangeLog b/nuttx/ChangeLog
index e52087f32..954c7f552 100644
--- a/nuttx/ChangeLog
+++ b/nuttx/ChangeLog
@@ -946,7 +946,7 @@
 	  certain range.  Worked fine until you try to use an interrupt in that
 	  range!
 
-0.4.14 2009-xx-xx Gregory Nutt <spudmonkey@racsa.co.cr>
+4.14 2009-xx-xx Gregory Nutt <spudmonkey@racsa.co.cr>
 
 	* arch/arm/src/stm32/stm32_gpio.c - Add support for configure an input GPIO
 	  to generate an EXTI interrupt.
@@ -971,6 +971,8 @@
 	  media: The bind method could fail repeatedly until media is asserted.
 	* arch/arm/src/stm32/chip.h & stm32_dma.c -- Fixed several definitions
 	  that can cause compilation errors when DMA2 is enabled.
+	* arch/arm/src/stm32/stm32_dma.c - Integrated and debugged STM32 DMA
+	  functionality that was added in 0.4.12.
 
 
 
diff --git a/nuttx/Documentation/NuttX.html b/nuttx/Documentation/NuttX.html
index 359a158ff..4066bd9f5 100644
--- a/nuttx/Documentation/NuttX.html
+++ b/nuttx/Documentation/NuttX.html
@@ -8,7 +8,7 @@
   <tr align="center" bgcolor="#e4e4e4">
     <td>
       <h1><big><font color="#3c34ec"><i>NuttX RTOS</i></font></big></h1>
-      <p>Last Updated: November 21, 2009</p>
+      <p>Last Updated: November 25, 2009</p>
     </td>
   </tr>
 </table>
@@ -1594,7 +1594,7 @@ buildroot-0.1.7 2009-06-26 &lt;spudmonkey@racsa.co.cr&gt;
 </table>
 
 <ul><pre>
-nuttx-0.4.14 2009-xx-xx Gregory Nutt &lt;spudmonkey@racsa.co.cr&gt;
+nuttx-4.14 2009-xx-xx Gregory Nutt &lt;spudmonkey@racsa.co.cr&gt;
 
 	* arch/arm/src/stm32/stm32_gpio.c - Add support for configure an input GPIO
 	  to generate an EXTI interrupt.
@@ -1619,6 +1619,8 @@ nuttx-0.4.14 2009-xx-xx Gregory Nutt &lt;spudmonkey@racsa.co.cr&gt;
 	  media: The bind method could fail repeatedly until media is asserted.
 	* arch/arm/src/stm32/chip.h & stm32_dma.c -- Fixed several definitions
 	  that can cause compilation errors when DMA2 is enabled.
+	* arch/arm/src/stm32/stm32_dma.c - Integrated and debugged STM32 DMA
+	  functionality that was added in 0.4.12.
 
 pascal-0.1.3 2009-xx-xx Gregory Nutt &lt;spudmonkey@racsa.co.cr&gt;
 
diff --git a/nuttx/arch/arm/src/common/up_initialize.c b/nuttx/arch/arm/src/common/up_initialize.c
index 995a319e6..d12a23de1 100644
--- a/nuttx/arch/arm/src/common/up_initialize.c
+++ b/nuttx/arch/arm/src/common/up_initialize.c
@@ -124,6 +124,19 @@ void up_initialize(void)
 
   up_irqinitialize();
 
+  /* Initialize the DMA subsystem if the weak function stm32_dmainitialize has been
+   * brought into the build
+   */
+
+#ifdef CONFIG_ARCH_DMA
+#ifdef CONFIG_HAVE_WEAKFUNCTIONS
+  if (up_dmainitialize)
+#endif
+    {
+      up_dmainitialize();
+    }
+#endif
+
   /* Initialize the system timer interrupt */
 
 #if !defined(CONFIG_SUPPRESS_INTERRUPTS) && !defined(CONFIG_SUPPRESS_TIMER_INTS)
diff --git a/nuttx/arch/arm/src/common/up_internal.h b/nuttx/arch/arm/src/common/up_internal.h
index ee5864794..9abf4a0b6 100644
--- a/nuttx/arch/arm/src/common/up_internal.h
+++ b/nuttx/arch/arm/src/common/up_internal.h
@@ -156,6 +156,9 @@ extern void up_boot(void);
 extern void up_copystate(uint32 *dest, uint32 *src);
 extern void up_decodeirq(uint32 *regs);
 extern void up_irqinitialize(void);
+#ifdef CONFIG_ARCH_DMA
+extern void weak_function up_dmainitialize(void);
+#endif
 extern int  up_saveusercontext(uint32 *saveregs);
 extern void up_fullcontextrestore(uint32 *restoreregs) __attribute__ ((noreturn));
 extern void up_switchcontext(uint32 *saveregs, uint32 *restoreregs);
diff --git a/nuttx/arch/arm/src/stm32/stm32_dma.c b/nuttx/arch/arm/src/stm32/stm32_dma.c
index 7fc5cb6f5..feda25a4e 100755
--- a/nuttx/arch/arm/src/stm32/stm32_dma.c
+++ b/nuttx/arch/arm/src/stm32/stm32_dma.c
@@ -44,9 +44,12 @@
 #include <debug.h>
 #include <errno.h>
 
+#include <nuttx/irq.h>
+#include <nuttx/arch.h>
 #include <arch/irq.h>
 
 #include "up_arch.h"
+#include "up_internal.h"
 #include "os_internal.h"
 #include "chip.h"
 #include "stm32_dma.h"
@@ -64,6 +67,10 @@
 #  define DMA_NCHANNELS  DMA1_NCHANNELS
 #endif
 
+#ifndef CONFIG_DMA_PRI
+#  define CONFIG_DMA_PRI NVIC_SYSH_PRIORITY_DEFAULT
+#endif
+
 /* Convert the DMA channel base address to the DMA register block address */
  
 #define DMA_BASE(ch)     (ch & 0xfffffc00)
@@ -76,7 +83,7 @@
 
 struct stm32_dma_s
 {
-  ubyte          chan;     /* DMA channel number */
+  ubyte          chan;     /* DMA channel number (0-6) */
   ubyte          irq;      /* DMA channel IRQ number */
   sem_t          sem;      /* Used to wait for DMA channel to become available */
   uint32         base;     /* DMA register channel base address */
@@ -93,69 +100,69 @@ struct stm32_dma_s
 static struct stm32_dma_s g_dma[DMA_NCHANNELS] =
 {
   {
-	.chan     = STM32_DMA1_CHAN1,
-	.irq      = STM32_IRQ_DMA1CH1,
+    .chan     = 0,
+    .irq      = STM32_IRQ_DMA1CH1,
     .base     = STM32_DMA1_BASE + STM32_DMACHAN_OFFSET(0),
   },
   {
-	.chan     = STM32_DMA1_CHAN2,
-	.irq      = STM32_IRQ_DMA1CH2,
+    .chan     = 1,
+    .irq      = STM32_IRQ_DMA1CH2,
     .base     = STM32_DMA1_BASE + STM32_DMACHAN_OFFSET(1),
   },
   {
-	.chan     = STM32_DMA1_CHAN3,
-	.irq      = STM32_IRQ_DMA1CH3,
+    .chan     = 2,
+    .irq      = STM32_IRQ_DMA1CH3,
     .base     = STM32_DMA1_BASE + STM32_DMACHAN_OFFSET(2),
   },
   {
-	.chan     = STM32_DMA1_CHAN4,
-	.irq      = STM32_IRQ_DMA1CH4,
+    .chan     = 3,
+    .irq      = STM32_IRQ_DMA1CH4,
     .base     = STM32_DMA1_BASE + STM32_DMACHAN_OFFSET(3),
   },
   {
-	.chan     = STM32_DMA1_CHAN5,
-	.irq      = STM32_IRQ_DMA1CH5,
+    .chan     = 4,
+    .irq      = STM32_IRQ_DMA1CH5,
     .base     = STM32_DMA1_BASE + STM32_DMACHAN_OFFSET(4),
   },
   {
-	.chan     = STM32_DMA1_CHAN6,
-	.irq      = STM32_IRQ_DMA1CH6,
+    .chan     = 5,
+    .irq      = STM32_IRQ_DMA1CH6,
     .base     = STM32_DMA1_BASE + STM32_DMACHAN_OFFSET(5),
   },
   {
-	.chan     = STM32_DMA1_CHAN7,
-	.irq      = STM32_IRQ_DMA1CH7,
+    .chan     = 6,
+    .irq      = STM32_IRQ_DMA1CH7,
     .base     = STM32_DMA1_BASE + STM32_DMACHAN_OFFSET(6),
   },
 #if STM32_NDMA > 1
   {
-	.chan     = STM32_DMA2_CHAN1,
-	.irq      = STM32_IRQ_DMA2CH1,
+    .chan     = 0,
+    .irq      = STM32_IRQ_DMA2CH1,
     .base     = STM32_DMA2_BASE + STM32_DMACHAN_OFFSET(0),
   },
   {
-	.chan     = STM32_DMA2_CHAN2,
-	.irq      = STM32_IRQ_DMA2CH2,
+    .chan     = 1,
+    .irq      = STM32_IRQ_DMA2CH2,
     .base     = STM32_DMA2_BASE + STM32_DMACHAN_OFFSET(1),
   },
   {
-	.chan     = STM32_DMA2_CHAN3,
-	.irq      = STM32_IRQ_DMA2CH3,
+    .chan     = 2,
+    .irq      = STM32_IRQ_DMA2CH3,
     .base     = STM32_DMA2_BASE + STM32_DMACHAN_OFFSET(2),
   },
   {
-	.chan     = STM32_DMA2_CHAN4,
+    .chan     = 3,
 #ifdef CONFIG_STM32_CONNECTIVITY_LINE
-	.irq      = STM32_IRQ_DMA2CH4,
+    .irq      = STM32_IRQ_DMA2CH4,
 #else
     .irq      = STM32_IRQ_DMA2CH45,
 #endif
     .base     = STM32_DMA2_BASE + STM32_DMACHAN_OFFSET(3),
   },
   {
-	.chan     = STM32_DMA2_CHAN5,
+    .chan     = 4,
 #ifdef CONFIG_STM32_CONNECTIVITY_LINE
-	.irq      = STM32_IRQ_DMA2CH5,
+    .irq      = STM32_IRQ_DMA2CH5,
 #else
     .irq      = STM32_IRQ_DMA2CH45,
 #endif
@@ -227,6 +234,33 @@ static inline void stm32_dmagive(FAR struct stm32_dma_s *dmach)
   (void)sem_post(&dmach->sem);
 }
 
+/************************************************************************************
+ * Name: stm32_dmachandisable
+ *
+ * Description:
+ *  Disable the DMA channel
+ *
+ ************************************************************************************/
+
+static void stm32_dmachandisable(struct stm32_dma_s *dmach)
+{
+  uint32 regval;
+
+  /* Disable all interrupts at the DMA controller */
+
+  regval = dmachan_getreg(dmach, STM32_DMACHAN_CCR_OFFSET);
+  regval &= ~DMA_CCR_ALLINTS;
+
+  /* Disable the DMA channel */
+
+  regval &= ~DMA_CCR_EN;
+  dmachan_putreg(dmach, STM32_DMACHAN_CCR_OFFSET, regval);
+
+  /* Clear pending channel interrupts */
+
+  dmabase_putreg(dmach, STM32_DMA_IFCR_OFFSET, DMA_ISR_CHAN_MASK(dmach->chan));
+}
+
 /************************************************************************************
  * Name: stm32_dmainterrupt
  *
@@ -239,13 +273,13 @@ static int stm32_dmainterrupt(int irq, void *context)
 {
   struct stm32_dma_s *dmach;
   uint32 isr;
-  int chan;
+  int chndx;
 
   /* Get the channel structure from the interrupt number */
 
   if (irq >= STM32_IRQ_DMA1CH1 && irq <= STM32_IRQ_DMA1CH7)
     {
-      chan = irq - STM32_IRQ_DMA1CH1;
+      chndx = irq - STM32_IRQ_DMA1CH1;
     }
   else
 #if STM32_NDMA > 1
@@ -255,28 +289,28 @@ static int stm32_dmainterrupt(int irq, void *context)
   if (irq >= STM32_IRQ_DMA2CH1 && irq <= STM32_IRQ_DMA2CH45)
 #endif
     {
-      chan = irq - STM32_IRQ_DMA2CH1 + DMA1_NCHANNELS;
+      chndx = irq - STM32_IRQ_DMA2CH1 + DMA1_NCHANNELS;
     }
   else
 #endif
     {
       PANIC(OSERR_INTERNAL);
     }
-  dmach = &g_dma[chan];
+  dmach = &g_dma[chndx];
 
   /* Get the interrupt status (for this channel only) */
 
-  isr = dmabase_getreg(dmach, STM32_DMA_ISR_OFFSET) & ~DMA_ISR_CHAN_MASK(chan);
+  isr = dmabase_getreg(dmach, STM32_DMA_ISR_OFFSET) & DMA_ISR_CHAN_MASK(dmach->chan);
 
-  /* Clear pending interrupts (for this channel only) */
+  /* Disable the DMA channel */
 
-  dmabase_putreg(dmach, STM32_DMA_IFCR_OFFSET, isr);
+  stm32_dmachandisable(dmach);
 
   /* Invoke the callback */
 
   if (dmach->callback)
     {
-      dmach->callback(dmach, isr >> DMA_ISR_CHAN_SHIFT(chan), dmach->arg);
+      dmach->callback(dmach, isr >> DMA_ISR_CHAN_SHIFT(dmach->chan), dmach->arg);
     }
   return OK;
 }
@@ -296,16 +330,33 @@ static int stm32_dmainterrupt(int irq, void *context)
  *
  ****************************************************************************/
 
-void weak_function stm32_dmainitialize(void)
+void weak_function up_dmainitialize(void)
 {
-  int chan;
+  struct stm32_dma_s *dmach;
+  int chndx;
 
-  /* Attach DMA interrupt vectors */
+  /* Initialize each DMA channel */
 
-  for (chan = 0; chan < DMA_NCHANNELS; chan++)
+  for (chndx = 0; chndx < DMA_NCHANNELS; chndx++)
     {
-      sem_init(&g_dma[chan].sem, 0, 1);	
-      irq_attach(g_dma[chan].irq, stm32_dmainterrupt);
+      dmach = &g_dma[chndx];
+      sem_init(&dmach->sem, 0, 1);
+
+      /* Attach DMA interrupt vectors */
+
+      (void)irq_attach(dmach->irq, stm32_dmainterrupt);
+
+      /* Disable the DMA channel */
+
+      stm32_dmachandisable(dmach);
+
+      /* Enable the IRQ at the NVIC (still disabled at the DMA controller) */
+
+      up_enable_irq(dmach->irq);
+ 
+      /* Set the interrrupt priority */
+
+      up_prioritize_irq(dmach->irq, CONFIG_DMA_PRI);
     }
 }
 
@@ -314,7 +365,7 @@ void weak_function stm32_dmainitialize(void)
  *
  * Description:
  *   Allocate a DMA channel.  This function gives the caller mutually
- *   exclusive access to the DMA channel specified by the 'chan' argument.
+ *   exclusive access to the DMA channel specified by the 'chndx' argument.
  *   DMA channels are shared on the STM32:  Devices sharing the same DMA
  *   channel cannot do DMA concurrently!  See the DMACHAN_* definitions in
  *   stm32_dma.h.
@@ -330,8 +381,8 @@ void weak_function stm32_dmainitialize(void)
  *   version.  Feel free to do that if that is what you need.
  *
  * Returned Value:
- *   Provided that 'chan' is valid, this function ALWAYS returns a non-NULL,
- *   void* DMA channel handle.  (If 'chan' is invalid, the function will
+ *   Provided that 'chndx' is valid, this function ALWAYS returns a non-NULL,
+ *   void* DMA channel handle.  (If 'chndx' is invalid, the function will
  *   assert if debug is enabled or do something ignorant otherwise).
  *
  * Assumptions:
@@ -341,11 +392,11 @@ void weak_function stm32_dmainitialize(void)
  *
  ****************************************************************************/
 
-DMA_HANDLE stm32_dmachannel(int chan)
+DMA_HANDLE stm32_dmachannel(int chndx)
 {
-  struct stm32_dma_s *dmach = &g_dma[chan];
+  struct stm32_dma_s *dmach = &g_dma[chndx];
 
-  DEBUGASSERT(chan < DMA_NCHANNELS);
+  DEBUGASSERT(chndx < DMA_NCHANNELS);
 
   /* Get exclusive access to the DMA channel -- OR wait until the channel
    * is available if it is currently being used by another driver
@@ -478,3 +529,44 @@ void stm32_dmastart(DMA_HANDLE handle, dma_callback_t callback, void *arg, boole
   ccr |= (half ? (DMA_CCR_HTIE|DMA_CCR_TEIE) : (DMA_CCR_TCIE|DMA_CCR_TEIE));
   dmachan_putreg(dmach, STM32_DMACHAN_CCR_OFFSET, ccr);
 }
+
+/****************************************************************************
+ * Name: stm32_dmadump
+ *
+ * Description:
+ *   Dump DMA register contents
+ *
+ * Assumptions:
+ *   - DMA handle allocated by stm32_dmachannel()
+ *
+ ****************************************************************************/
+
+#ifdef CONFIG_DEBUG_DMA
+void stm32_dmadump(DMA_HANDLE handle, const char *msg)
+{
+  struct stm32_dma_s *dmach = (struct stm32_dma_s *)handle;
+  uint32 dmabase = DMA_BASE(dmach->base);
+  irqstate_t flags;
+  uint32 addr;
+
+  dmalldbg("%s: base: %08x Channel base: %08x \n", msg, dmabase, dmach->base);
+
+  flags = irqsave();
+  addr = dmabase + STM32_DMA_ISR_OFFSET;
+  dmalldbg("  ISRC[%08x]: %08x\n", addr, getreg32(addr));
+
+  addr = dmach->base + STM32_DMACHAN_CCR_OFFSET;
+  dmalldbg("   CCR[%08x]: %08x\n", addr, getreg32(addr));
+
+  addr = dmach->base + STM32_DMACHAN_CNDTR_OFFSET;
+  dmalldbg(" CNDTR[%08x]: %08x\n", addr, getreg32(addr));
+
+  addr = dmach->base + STM32_DMACHAN_CPAR_OFFSET;
+  dmalldbg("  CPAR[%08x]: %08x\n", addr, getreg32(addr));
+
+  addr = dmach->base + STM32_DMACHAN_CMAR_OFFSET;
+  dmalldbg("  CMAR[%08x]: %08x\n", addr, getreg32(addr));
+  irqrestore(flags);
+}
+#endif
+
diff --git a/nuttx/arch/arm/src/stm32/stm32_dma.h b/nuttx/arch/arm/src/stm32/stm32_dma.h
index f5409d7f6..e2e16edbd 100644
--- a/nuttx/arch/arm/src/stm32/stm32_dma.h
+++ b/nuttx/arch/arm/src/stm32/stm32_dma.h
@@ -229,6 +229,7 @@
 #define DMA_IFCR_CHAN6_MASK       (DMA_CHAN_MASK <<  DMA_IFCR_CHAN6_SHIFT)
 #define DMA_IFCR_CHAN7_SHIFT      (24)      /* Bits 27-24:  DMA Channel 7 interrupt flag clear */
 #define DMA_IFCR_CHAN7_MASK       (DMA_CHAN_MASK <<  DMA_IFCR_CHAN7_SHIFT)
+#define DMA_IFCR_ALLCHANNELS      (0x0fffffff)
 
 #define DMA_IFCR_CGIF(n)          (DMA_CHAN_GIF_BIT << DMA_IFCR_CHAN_SHIFT(n))
 #define DMA_IFCR_CTCIF(n)         (DMA_CHAN_TCIF_BIT << DMA_IFCR_CHAN_SHIFT(n))
@@ -263,6 +264,8 @@
 #define DMA_CCR_TCIE              (1 << 1)  /* Bit 1: Transfer complete interrupt enable */
 #define DMA_CCR_EN                (1 << 0)  /* Bit 0: Channel enable */
 
+#define DMA_CCR_ALLINTS           (DMA_CCR_TEIE|DMA_CCR_HTIE|DMA_CCR_TCIE)
+
 /* DMA channel number of data register */
 
 #define DMA_CNDTR_NDT_SHIFT       (0)       /* Bits 15-0: Number of data to Transfer */
diff --git a/nuttx/arch/arm/src/stm32/stm32_internal.h b/nuttx/arch/arm/src/stm32/stm32_internal.h
index 9964c1130..b567c5d0f 100755
--- a/nuttx/arch/arm/src/stm32/stm32_internal.h
+++ b/nuttx/arch/arm/src/stm32/stm32_internal.h
@@ -50,6 +50,12 @@
  * Definitions
  ************************************************************************************/
 
+/* Configuration ********************************************************************/
+
+#if !defined(CONFIG_DEBUG) || !defined(CONFIG_DEBUG_VERBOSE)
+#  undef CONFIG_DEBUG_DMA
+#endif
+
 /* NVIC priority levels *************************************************************/
 
 #define NVIC_SYSH_PRIORITY_MIN     0xff /* All bits set in minimum priority */
@@ -265,19 +271,6 @@ EXTERN int stm32_dumpgpio(uint32 pinset, const char *msg);
 #  define stm32_dumpgpio(p,m)
 #endif
 
-/****************************************************************************
- * Name: stm32_dmainitialize
- *
- * Description:
- *   Initialize the DMA subsystem
- *
- * Returned Value:
- *   None
- *
- ****************************************************************************/
-
-EXTERN void weak_function stm32_dmainitialize(void);
-
 /****************************************************************************
  * Name: stm32_dmachannel
  *
@@ -359,6 +352,23 @@ EXTERN void stm32_dmasetup(DMA_HANDLE handle, uint32 paddr, uint32 maddr,
 EXTERN void stm32_dmastart(DMA_HANDLE handle, dma_callback_t callback,
                            void *arg, boolean half);
 
+/****************************************************************************
+ * Name: stm32_dmadump
+ *
+ * Description:
+ *   Dump DMA register contents
+ *
+ * Assumptions:
+ *   - DMA handle allocated by stm32_dmachannel()
+ *
+ ****************************************************************************/
+
+#ifdef CONFIG_DEBUG_DMA
+EXTERN void stm32_dmadump(DMA_HANDLE handle, const char *msg);
+#else
+#  define stm32_dmadump(handle)
+#endif
+
 /************************************************************************************
  * Function: stm32_ethinitialize
  *
@@ -445,31 +455,31 @@ EXTERN int stm32_usbpullup(FAR struct usbdev_s *dev,  boolean enable);
 struct usbdev_s;
 EXTERN void stm32_usbsuspend(FAR struct usbdev_s *dev, boolean resume);
 
-/****************************************************************************
- * Name: sdio_initialize
- *
- * Description:
- *   Initialize SDIO for operation.
- *
+/****************************************************************************
+ * Name: sdio_initialize
+ *
+ * Description:
+ *   Initialize SDIO for operation.
+ *
  * Input Parameters:
  *   slotno - Not used.
  *
  * Returned Values:
  *   A reference to an SDIO interface structure.  NULL is returned on failures.
- *
- ****************************************************************************/
+ *
+ ****************************************************************************/
 
-struct sdio_dev_s; /* See include/nuttx/sdio.h */
-EXTERN FAR struct sdio_dev_s *sdio_initialize(int slotno);
-
-/****************************************************************************
- * Name: sdio_mediachange
- *
+struct sdio_dev_s; /* See include/nuttx/sdio.h */
+EXTERN FAR struct sdio_dev_s *sdio_initialize(int slotno);
+
+/****************************************************************************
+ * Name: sdio_mediachange
+ *
  * Description:
  *   Called by board-specific logic -- posssible from an interrupt handler --
  *   in order to signal to the driver that a card has been inserted or
- *   removed from the slot
- *
+ *   removed from the slot
+ *
  * Input Parameters:
  *   dev        - An instance of the SDIO driver device state structure.
  *   cardinslot - TRUE is a card has been detected in the slot; FALSE if a 
@@ -478,27 +488,27 @@ EXTERN FAR struct sdio_dev_s *sdio_initialize(int slotno);
  *
  * Returned Values:
  *   None
- *
- ****************************************************************************/
-
+ *
+ ****************************************************************************/
+
 EXTERN void sdio_mediachange(FAR struct sdio_dev_s *dev, boolean cardinslot);
 
-/****************************************************************************
- * Name: sdio_wrprotect
- *
+/****************************************************************************
+ * Name: sdio_wrprotect
+ *
  * Description:
  *   Called by board-specific logic to report if the card in the slot is
  *   mechanically write protected.
- *
+ *
  * Input Parameters:
  *   dev       - An instance of the SDIO driver device state structure.
  *   wrprotect - TRUE is a card is writeprotected.
  *
  * Returned Values:
  *   None
- *
- ****************************************************************************/
-
+ *
+ ****************************************************************************/
+
 EXTERN void sdio_wrprotect(FAR struct sdio_dev_s *dev, boolean wrprotect);
 
 #undef EXTERN
diff --git a/nuttx/arch/arm/src/stm32/stm32_sdio.c b/nuttx/arch/arm/src/stm32/stm32_sdio.c
index b46147f04..b3b6d47fd 100644
--- a/nuttx/arch/arm/src/stm32/stm32_sdio.c
+++ b/nuttx/arch/arm/src/stm32/stm32_sdio.c
@@ -118,10 +118,10 @@
 
 /* DMA CCR register settings */
 
-#define SDIO_RXDMA16_CONFIG      (CONFIG_SDIO_DMAPRIO|DMA_CCR_MSIZE_16BITS|\
-                                  DMA_CCR_PSIZE_16BITS|DMA_CCR_MINC)
-#define SDIO_TXDMA16_CONFIG      (CONFIG_SDIO_DMAPRIO|DMA_CCR_MSIZE_16BITS|\
-                                  DMA_CCR_PSIZE_16BITS|DMA_CCR_MINC|DMA_CCR_DIR)
+#define SDIO_RXDMA32_CONFIG      (CONFIG_SDIO_DMAPRIO|DMA_CCR_MSIZE_32BITS|\
+                                  DMA_CCR_PSIZE_32BITS|DMA_CCR_MINC)
+#define SDIO_TXDMA32_CONFIG      (CONFIG_SDIO_DMAPRIO|DMA_CCR_MSIZE_32BITS|\
+                                  DMA_CCR_PSIZE_32BITS|DMA_CCR_MINC|DMA_CCR_DIR)
 
 /* FIFO sizes */
 
@@ -528,6 +528,8 @@ static void stm32_dmacallback(DMA_HANDLE handle, ubyte isr, void *arg)
   /* We don't really do anything at the completion of DMA.  The termination
    * of the transfer is driven by the SDIO interrupts.
    */
+
+  stm32_dmadump(handle, "DMA Callback");
 }
 #endif
 
@@ -837,6 +839,15 @@ static void stm32_endtransfer(struct stm32_dev_s *priv, sdio_eventset_t wkupeven
 
   priv->remaining = 0;
 
+  /* DMA debug instrumentation */
+
+#if defined(CONFIG_SDIO_DMA) && defined(CONFIG_DEBUG_DMA)
+  if (priv->dmamode)
+    {
+      stm32_dmadump(priv->dma, "End of Transfer");
+    }
+#endif
+
   /* Is a data transfer complete event expected? */
 
   if ((priv->waitevents & wkupevent) != 0)
@@ -1978,6 +1989,8 @@ static int stm32_dmarecvsetup(FAR struct sdio_dev_s *dev, FAR ubyte *buffer,
 
   if (priv->widebus)
     {
+      stm32_dmadump(priv->dma, "Before RECV Setup");
+
       /* Save the destination buffer information for use by the interrupt handler */
 
       priv->buffer    = (uint32*)buffer;
@@ -1995,11 +2008,12 @@ static int stm32_dmarecvsetup(FAR struct sdio_dev_s *dev, FAR ubyte *buffer,
 
       putreg32(1, SDIO_DCTRL_DMAEN_BB);
       stm32_dmasetup(priv->dma, STM32_SDIO_FIFO, (uint32)buffer,
-                     (buflen + 3) >> 2, SDIO_RXDMA16_CONFIG);
+                     (buflen + 3) >> 2, SDIO_RXDMA32_CONFIG);
  
      /* Start the DMA */
 
       stm32_dmastart(priv->dma, stm32_dmacallback, priv, FALSE);
+      stm32_dmadump(priv->dma, "After RECV Setup");
       ret = OK;
     }
   return ret;
@@ -2027,7 +2041,7 @@ static int stm32_dmarecvsetup(FAR struct sdio_dev_s *dev, FAR ubyte *buffer,
 
 #ifdef CONFIG_SDIO_DMA
 static int stm32_dmasendsetup(FAR struct sdio_dev_s *dev,
-                               FAR const ubyte *buffer, size_t buflen)
+                              FAR const ubyte *buffer, size_t buflen)
 {
   struct stm32_dev_s *priv = (struct stm32_dev_s *)dev;
   uint32 dblocksize;
@@ -2044,6 +2058,8 @@ static int stm32_dmasendsetup(FAR struct sdio_dev_s *dev,
 
   if (priv->widebus)
     {
+      stm32_dmadump(priv->dma, "Before SEND Setup");
+
       /* Save the source buffer information for use by the interrupt handler */
 
       priv->buffer    = (uint32*)buffer;
@@ -2062,12 +2078,13 @@ static int stm32_dmasendsetup(FAR struct sdio_dev_s *dev,
       /* Configure the TX DMA */
 
       stm32_dmasetup(priv->dma, STM32_SDIO_FIFO, (uint32)buffer,
-                     (buflen + 3) >> 2, SDIO_TXDMA16_CONFIG);
+                     (buflen + 3) >> 2, SDIO_TXDMA32_CONFIG);
       putreg32(1, SDIO_DCTRL_DMAEN_BB);
 
       /* Start the DMA */
 
       stm32_dmastart(priv->dma, stm32_dmacallback, priv, FALSE);
+      stm32_dmadump(priv->dma, "After SEND Setup");
       ret = OK;
     }
   return ret;
diff --git a/nuttx/configs/stm3210e-eval/RIDE/defconfig b/nuttx/configs/stm3210e-eval/RIDE/defconfig
index cc08dc9c0..5ad48fd6f 100755
--- a/nuttx/configs/stm3210e-eval/RIDE/defconfig
+++ b/nuttx/configs/stm3210e-eval/RIDE/defconfig
@@ -67,6 +67,7 @@
 #   CONFIG_BOARD_LOOPSPERMSEC.  You simply use a stop watch to measure
 #   the 100 second delay then adjust CONFIG_BOARD_LOOPSPERMSEC until
 #   the delay actually is 100 seconds.
+# CONFIG_ARCH_DMA - Support DMA initialization
 #
 CONFIG_ARCH=arm
 CONFIG_ARCH_ARM=y
@@ -86,6 +87,7 @@ CONFIG_ARCH_BOOTLOADER=n
 CONFIG_ARCH_LEDS=y
 CONFIG_ARCH_BUTTONS=n
 CONFIG_ARCH_CALIBRATION=n
+CONFIG_ARCH_DMA=n
 
 #
 # Identify toolchain
diff --git a/nuttx/configs/stm3210e-eval/include/board.h b/nuttx/configs/stm3210e-eval/include/board.h
index 4792ab6af..795054136 100755
--- a/nuttx/configs/stm3210e-eval/include/board.h
+++ b/nuttx/configs/stm3210e-eval/include/board.h
@@ -41,6 +41,7 @@
  * Included Files
  ************************************************************************************/
 
+#include <nuttx/config.h>
 #ifndef __ASSEMBLY__
 # include <sys/types.h>
 #endif
diff --git a/nuttx/configs/stm3210e-eval/nsh/defconfig b/nuttx/configs/stm3210e-eval/nsh/defconfig
index 5a41b5b1e..bcf572388 100755
--- a/nuttx/configs/stm3210e-eval/nsh/defconfig
+++ b/nuttx/configs/stm3210e-eval/nsh/defconfig
@@ -67,6 +67,7 @@
 #   CONFIG_BOARD_LOOPSPERMSEC.  You simply use a stop watch to measure
 #   the 100 second delay then adjust CONFIG_BOARD_LOOPSPERMSEC until
 #   the delay actually is 100 seconds.
+# CONFIG_ARCH_DMA - Support DMA initialization
 #
 CONFIG_ARCH=arm
 CONFIG_ARCH_ARM=y
@@ -86,6 +87,7 @@ CONFIG_ARCH_BOOTLOADER=n
 CONFIG_ARCH_LEDS=y
 CONFIG_ARCH_BUTTONS=n
 CONFIG_ARCH_CALIBRATION=n
+CONFIG_ARCH_DMA=y
 
 #
 # Identify toolchain and linker options
diff --git a/nuttx/configs/stm3210e-eval/ostest/defconfig b/nuttx/configs/stm3210e-eval/ostest/defconfig
index 0ea76f6cd..b6fd693a3 100755
--- a/nuttx/configs/stm3210e-eval/ostest/defconfig
+++ b/nuttx/configs/stm3210e-eval/ostest/defconfig
@@ -67,6 +67,7 @@
 #   CONFIG_BOARD_LOOPSPERMSEC.  You simply use a stop watch to measure
 #   the 100 second delay then adjust CONFIG_BOARD_LOOPSPERMSEC until
 #   the delay actually is 100 seconds.
+# CONFIG_ARCH_DMA - Support DMA initialization
 #
 CONFIG_ARCH=arm
 CONFIG_ARCH_ARM=y
@@ -86,6 +87,7 @@ CONFIG_ARCH_BOOTLOADER=n
 CONFIG_ARCH_LEDS=y
 CONFIG_ARCH_BUTTONS=n
 CONFIG_ARCH_CALIBRATION=n
+CONFIG_ARCH_DMA=n
 
 #
 # Identify toolchain and linker options
diff --git a/nuttx/configs/stm3210e-eval/src/up_boot.c b/nuttx/configs/stm3210e-eval/src/up_boot.c
index afa03d074..527f9d919 100755
--- a/nuttx/configs/stm3210e-eval/src/up_boot.c
+++ b/nuttx/configs/stm3210e-eval/src/up_boot.c
@@ -71,18 +71,7 @@
  ************************************************************************************/
 
 void stm32_boardinitialize(void)
-{
-  /* Initialize the DMA subsystem if the weak function stm32_dmainitialize has been
-   * brought into the build
-   */
-
-#if defined(CONFIG_STM32_DMA1) || defined(CONFIG_STM32_DMA2)
-  if (stm32_dmainitialize)
-    {
-      stm32_dmainitialize();
-    }
-#endif
-
+{
   /* Configure SPI chip selects if 1) SPI is not disabled, and 2) the weak function
    * stm32_spiinitialize() has been brought into the link.
    */
@@ -93,18 +82,18 @@ void stm32_boardinitialize(void)
       stm32_spiinitialize();
     }
 #endif
-
-   /* Initialize USB is 1) USBDEV is selected, 2) the USB controller is not
-    * disabled, and 3) the weak function stm32_usbinitialize() has been brought
-    * into the build.
-    */
-
-#if defined(CONFIG_USBDEV) && defined(CONFIG_STM32_USB)
+
+   /* Initialize USB is 1) USBDEV is selected, 2) the USB controller is not
+    * disabled, and 3) the weak function stm32_usbinitialize() has been brought
+    * into the build.
+    */
+
+#if defined(CONFIG_USBDEV) && defined(CONFIG_STM32_USB)
   if (stm32_usbinitialize)
     {
       stm32_usbinitialize();
     }
-#endif
+#endif
 
   /* Configure on-board LEDs if LED support has been selected. */
 
diff --git a/nuttx/configs/stm3210e-eval/usbserial/defconfig b/nuttx/configs/stm3210e-eval/usbserial/defconfig
index 23910eb2d..77fd6212b 100755
--- a/nuttx/configs/stm3210e-eval/usbserial/defconfig
+++ b/nuttx/configs/stm3210e-eval/usbserial/defconfig
@@ -67,6 +67,7 @@
 #   CONFIG_BOARD_LOOPSPERMSEC.  You simply use a stop watch to measure
 #   the 100 second delay then adjust CONFIG_BOARD_LOOPSPERMSEC until
 #   the delay actually is 100 seconds.
+# CONFIG_ARCH_DMA - Support DMA initialization
 #
 CONFIG_ARCH=arm
 CONFIG_ARCH_ARM=y
@@ -86,6 +87,7 @@ CONFIG_ARCH_BOOTLOADER=n
 CONFIG_ARCH_LEDS=y
 CONFIG_ARCH_BUTTONS=n
 CONFIG_ARCH_CALIBRATION=n
+CONFIG_ARCH_DMA=n
 
 #
 # Identify toolchain and linker options
diff --git a/nuttx/include/debug.h b/nuttx/include/debug.h
index c4f8ea346..10b30df42 100644
--- a/nuttx/include/debug.h
+++ b/nuttx/include/debug.h
@@ -163,6 +163,18 @@
 # define sllvdbg(x...)
 #endif
 
+#ifdef CONFIG_DEBUG_DMA
+# define dmadbg(format, arg...)    dbg(format, ##arg)
+# define dmalldbg(format, arg...)  lldbg(format, ##arg)
+# define dmavdbg(format, arg...)   vdbg(format, ##arg)
+# define dmallvdbg(format, arg...) llvdbg(format, ##arg)
+#else
+# define dmadbg(x...)
+# define dmalldbg(x...)
+# define dmavdbg(x...)
+# define dmallvdbg(x...)
+#endif
+
 #ifdef CONFIG_DEBUG_NET
 # define ndbg(format, arg...)    dbg(format, ##arg)
 # define nlldbg(format, arg...)  lldbg(format, ##arg)
@@ -284,6 +296,18 @@
 # define sllvdbg (void)
 #endif
 
+#ifdef CONFIG_DEBUG_DMA
+# define dmadbg    dbg
+# define dmalldbg  lldbg
+# define dmavdbg   vdbg
+# define dmallvdbg llvdbg
+#else
+# define dmadbg    (void)
+# define dmalldbg  (void)
+# define dmavdbg   (void)
+# define dmallvdbg (void)
+#endif
+
 #ifdef CONFIG_DEBUG_NET
 # define ndbg    dbg
 # define nlldbg  lldbg
@@ -390,6 +414,14 @@
 #  define svdbgdumpbuffer(m,b,n)
 #endif
 
+#ifdef CONFIG_DEBUG_DMA
+#  define dmadbgdumpbuffer(m,b,n)  dbgdumpbuffer(m,b,n)
+#  define dmavdbgdumpbuffer(m,b,n) vdbgdumpbuffer(m,b,n)
+#else
+#  define dmadbgdumpbuffer(m,b,n)
+#  define dmavdbgdumpbuffer(m,b,n)
+#endif
+
 #ifdef CONFIG_DEBUG_NET
 #  define ndbgdumpbuffer(m,b,n)  dbgdumpbuffer(m,b,n)
 #  define nvdbgdumpbuffer(m,b,n) vdbgdumpbuffer(m,b,n)
diff --git a/nuttx/tools/mkconfig.c b/nuttx/tools/mkconfig.c
index b4afe7bf7..3a9eda8a4 100644
--- a/nuttx/tools/mkconfig.c
+++ b/nuttx/tools/mkconfig.c
@@ -319,9 +319,18 @@ int main(int argc, char **argv, char **envp)
   printf("# undef CONFIG_NET_UDP\n");
   printf("# undef CONFIG_NET_ICMP\n");
   printf("#endif\n\n");
-  printf("/* Verbose debug only makes sense if debug is enabled */\n\n");
+  printf("/* Verbose debug and sub-system debug only make sense if debug is enabled */\n\n");
   printf("#ifndef CONFIG_DEBUG\n");
   printf("# undef CONFIG_DEBUG_VERBOSE\n");
+  printf("# undef CONFIG_DEBUG_SCHED\n");
+  printf("# undef CONFIG_DEBUG_MM\n");
+  printf("# undef CONFIG_DEBUG_DMA\n");
+  printf("# undef CONFIG_DEBUG_FS\n");
+  printf("# undef CONFIG_DEBUG_LIB\n");
+  printf("# undef CONFIG_DEBUG_BINFMT\n");
+  printf("# undef CONFIG_DEBUG_NET\n");
+  printf("# undef CONFIG_DEBUG_USB\n");
+  printf("# undef CONFIG_DEBUG_GRAPHICS\n");
   printf("#endif\n\n");
   printf("#endif /* __ARCH_CONFIG_H */\n");
   fclose(stream);