Added kl_dumpgpio functionality as kinetis_pindump, which was

already prototyped in kinetis.h. It is enabled when CONFIG_DEBUG_GPIO
is defined.

arch/arm/src/kinetis/Make.defs

@@ -108,7 +108,7 @@ CHIP_CSRCS += kinetis_pinirq.c
 endif
 
 ifeq ($(CONFIG_DEBUG_GPIO),y)
-CHIP_CSRCS += kinetis_pindbg.c
+CHIP_CSRCS += kinetis_pindump.c
 endif
 
 ifeq ($(CONFIG_KINETIS_SDHC),y)

arch/arm/src/kinetis/kinetis.h

@@ -567,7 +567,7 @@ void kinetis_pindmadisable(uint32_t pinset);
  ************************************************************************************/
 
 #ifdef CONFIG_DEBUG_GPIO
-int kinetis_pindump(uint32_t pinset, const char *msg);
+void kinetis_pindump(uint32_t pinset, const char *msg);
 #else
 #  define kinetis_pindump(p,m)
 #endif

arch/arm/src/kinetis/kinetis_pindump.c

@@ -0,0 +1,135 @@
+/****************************************************************************
+ * arch/arm/src/kinetis/kinetis_pindump.c
+ *
+ *   Copyright (C) 2013, 2016 Gregory Nutt. All rights reserved.
+ *   Author: Gregory Nutt <gnutt@nuttx.org>
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ *
+ * 1. Redistributions of source code must retain the above copyright
+ *    notice, this list of conditions and the following disclaimer.
+ * 2. Redistributions in binary form must reproduce the above copyright
+ *    notice, this list of conditions and the following disclaimer in
+ *    the documentation and/or other materials provided with the
+ *    distribution.
+ * 3. Neither the name NuttX nor the names of its contributors may be
+ *    used to endorse or promote products derived from this software
+ *    without specific prior written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+ * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+ * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS
+ * FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE
+ * COPYRIGHT OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT,
+ * INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING,
+ * BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS
+ * OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED
+ * AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
+ * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN
+ * ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
+ * POSSIBILITY OF SUCH DAMAGE.
+ *
+ ****************************************************************************/
+
+/****************************************************************************
+ * Included Files
+ ****************************************************************************/
+
+#include <nuttx/config.h>
+
+#include <sys/types.h>
+#include <debug.h>
+
+#include <nuttx/irq.h>
+#include "up_arch.h"
+
+#include "kinetis.h"
+#include "kinetis_gpio.h"
+#include "kinetis_port.h"
+
+#ifdef CONFIG_DEBUG_GPIO
+
+/****************************************************************************
+ * Private Data
+ ****************************************************************************/
+/* Port letters for prettier debug output */
+
+static const char g_portchar[KINETIS_NPORTS] =
+{
+#if KINETIS_NPORTS > 9
+#  error "Additional support required for this number of GPIOs"
+#elif KINETIS_NPORTS > 8
+  'A', 'B', 'C', 'D', 'E', 'F', 'G', 'H', 'I'
+#elif KINETIS_NPORTS > 7
+  'A', 'B', 'C', 'D', 'E', 'F', 'G', 'H'
+#elif KINETIS_NPORTS > 6
+  'A', 'B', 'C', 'D', 'E', 'F', 'G'
+#elif KINETIS_NPORTS > 5
+  'A', 'B', 'C', 'D', 'E', 'F'
+#elif KINETIS_NPORTS > 4
+  'A', 'B', 'C', 'D', 'E'
+#elif KINETIS_NPORTS > 3
+  'A', 'B', 'C', 'D'
+#elif KINETIS_NPORTS > 2
+  'A', 'B', 'C'
+#elif KINETIS_NPORTS > 1
+  'A', 'B'
+#elif KINETIS_NPORTS > 0
+  'A'
+#else
+#  error "Bad number of GPIOs"
+#endif
+};
+
+/****************************************************************************
+ * Public Data
+ ****************************************************************************/
+
+/****************************************************************************
+ * Private Functions
+ ****************************************************************************/
+
+/****************************************************************************
+ * Public Functions
+ ****************************************************************************/
+
+/****************************************************************************
+ * Function:  kinetis_pindump
+ *
+ * Description:
+ *   Dump all GPIO registers associated with the provided pin description
+ *   along with a descriptive messasge.
+ *
+ ****************************************************************************/
+
+void kinetis_pindump(uint32_t pinset, const char *msg)
+{
+  irqstate_t flags;
+  uintptr_t base;
+  int port;
+
+  /* Decode the port and pin.  Use the port number to get the GPIO base
+   * address.
+   */
+
+  port = (pinset & _PIN_PORT_MASK) >> _PIN_PORT_SHIFT;
+  DEBUGASSERT((unsigned)port < KINETIS_NPORTS);
+  base = KINETIS_GPIO_BASE(port);
+
+  /* The following requires exclusive access to the GPIO registers */
+
+  flags = enter_critical_section();
+
+  lldbg("GPIO%c pinset: %08x base: %08x -- %s\n",
+        g_portchar[port], pinset, base, msg);
+  lldbg("  PDOR: %08x  PDIR: %08x  PDDR: %08x\n",
+        getreg32(base + KINETIS_GPIO_PDOR_OFFSET),
+        getreg32(base + KINETIS_GPIO_PDIR_OFFSET),
+        getreg32(base + KINETIS_GPIO_PDDR_OFFSET));
+
+  leave_critical_section(flags);
+}
+
+#endif /* CONFIG_DEBUG_GPIO */

arch/arm/src/kinetis/kinetis_pwm.c

@@ -88,7 +88,7 @@
 #  ifdef CONFIG_DEBUG_VERBOSE
 #    define pwmvdbg           vdbg
 #    define pwmllvdbg         llvdbg
-#    define pwm_dumpgpio(p,m) /* kinetis_dumpgpio(p,m) */
+#    define pwm_dumpgpio(p,m) kinetis_pindump(p,m)
 #  else
 #    define pwmlldbg(x...)
 #    define pwmllvdbg(x...)