nios2 : convert altera_pio to driver model
authorThomas Chou <thomas@wytron.com.tw>
Wed, 21 Oct 2015 13:33:45 +0000 (21:33 +0800)
committerThomas Chou <thomas@wytron.com.tw>
Thu, 22 Oct 2015 23:37:03 +0000 (07:37 +0800)
Convert altera_pio to driver model.

Signed-off-by: Thomas Chou <thomas@wytron.com.tw>
Acked-by: Chin Liang See <clsee@altera.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
arch/nios2/dts/3c120_devboard.dts
arch/nios2/include/asm/gpio.h
board/altera/nios2-generic/nios2-generic.c
configs/nios2-generic_defconfig
doc/device-tree-bindings/gpio/altera_pio.txt [new file with mode: 0644]
drivers/gpio/Kconfig
drivers/gpio/altera_pio.c
include/configs/nios2-generic.h

index 3d76ec4b9e2887048393e6666675a881add5ecaf..4a34ce56db6604d5284c8effb377d65fe26f6f34 100644 (file)
@@ -68,6 +68,9 @@
                                <0x00004400 0x08004400 0x00000040>,
                                <0x00004800 0x08004800 0x00000040>,
                                <0x00004c80 0x08004c80 0x00000020>,
+                               <0x00004cc0 0x08004cc0 0x00000010>,
+                               <0x00004ce0 0x08004ce0 0x00000010>,
+                               <0x00004d00 0x08004d00 0x00000010>,
                                <0x00004d50 0x08004d50 0x00000008>,
                                <0x00008000 0x08008000 0x00000020>,
                                <0x00400000 0x08400000 0x00000020>;
                                current-speed = <115200>;
                                clock-frequency = <62500000>;
                        };
+
+                       user_led_pio_8out: gpio@0x4cc0 {
+                               compatible = "altr,pio-1.0";
+                               reg = <0x00004cc0 0x00000010>;
+                               resetvalue = <255>;
+                               altr,gpio-bank-width = <8>;
+                               #gpio-cells = <2>;
+                               gpio-controller;
+                               gpio-bank-name = "led";
+                       };
+
+                       user_dipsw_pio_8in: gpio@0x4ce0 {
+                               compatible = "altr,pio-1.0";
+                               reg = <0x00004ce0 0x00000010>;
+                               interrupt-parent = <&cpu>;
+                               interrupts = <8>;
+                               edge_type = <2>;
+                               level_trigger = <0>;
+                               resetvalue = <0>;
+                               altr,gpio-bank-width = <8>;
+                               #gpio-cells = <2>;
+                               gpio-controller;
+                               gpio-bank-name = "dipsw";
+                       };
+
+                       user_pb_pio_4in: gpio@0x4d00 {
+                               compatible = "altr,pio-1.0";
+                               reg = <0x00004d00 0x00000010>;
+                               interrupt-parent = <&cpu>;
+                               interrupts = <9>;
+                               edge_type = <2>;
+                               level_trigger = <0>;
+                               resetvalue = <0>;
+                               altr,gpio-bank-width = <4>;
+                               #gpio-cells = <2>;
+                               gpio-controller;
+                               gpio-bank-name = "pb";
+                       };
                };
 
                cfi_flash_64m: flash@0x0 {
index 908381f5f7b07dbfa1b1480de65e27f58221ecad..306ab4c9f2aa3c73f2e3c710807e08787e845c04 100644 (file)
@@ -1,79 +1 @@
-/*
- * nios2 gpio driver
- *
- * This gpio core is described in http://nioswiki.com/GPIO
- * bit[0] data
- * bit[1] output enable
- *
- * When CONFIG_SYS_GPIO_BASE is not defined, the board may either
- * provide its own driver or the altera_pio driver may be used.
- *
- * Copyright (C) 2010 Thomas Chou <thomas@wytron.com.tw>
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License version 2 as
- * published by the Free Software Foundation.
- */
-
-#ifndef _ASM_NIOS2_GPIO_H_
-#define _ASM_NIOS2_GPIO_H_
-
-#ifdef CONFIG_SYS_GPIO_BASE
-#include <asm/io.h>
-
-static inline int gpio_request(unsigned gpio, const char *label)
-{
-       return 0;
-}
-
-static inline int gpio_free(unsigned gpio)
-{
-       return 0;
-}
-
-static inline int gpio_direction_input(unsigned gpio)
-{
-       writel(1, CONFIG_SYS_GPIO_BASE + (gpio << 2));
-       return 0;
-}
-
-static inline int gpio_direction_output(unsigned gpio, int value)
-{
-       writel(value ? 3 : 2, CONFIG_SYS_GPIO_BASE + (gpio << 2));
-       return 0;
-}
-
-static inline int gpio_get_value(unsigned gpio)
-{
-       return readl(CONFIG_SYS_GPIO_BASE + (gpio << 2));
-}
-
-static inline void gpio_set_value(unsigned gpio, int value)
-{
-       writel(value ? 3 : 2, CONFIG_SYS_GPIO_BASE + (gpio << 2));
-}
-
-static inline int gpio_is_valid(int number)
-{
-       return ((unsigned)number) < CONFIG_SYS_GPIO_WIDTH;
-}
-#else
-#ifdef CONFIG_ALTERA_PIO
-extern int altera_pio_init(u32 base, u8 width, char iot,
-                          u32 rstval, u32 negmask,
-                          const char *label);
-
-extern void altera_pio_info(void);
-#define gpio_status() altera_pio_info()
-#endif
-
-extern int gpio_request(unsigned gpio, const char *label);
-extern int gpio_free(unsigned gpio);
-extern int gpio_direction_input(unsigned gpio);
-extern int gpio_direction_output(unsigned gpio, int value);
-extern int gpio_get_value(unsigned gpio);
-extern void gpio_set_value(unsigned gpio, int value);
-extern int gpio_is_valid(int number);
-#endif /* CONFIG_SYS_GPIO_BASE */
-
-#endif /* _ASM_NIOS2_GPIO_H_ */
+#include <asm-generic/gpio.h>
index 384fee9b75d4a3e15f49747a9766ac02854c3263..cb98a08895548eeb80b56d833f68ebbaa5ee8dc0 100644 (file)
@@ -35,18 +35,6 @@ int board_early_init_f(void)
        return 0;
 }
 
-int board_early_init_r(void)
-{
-#ifdef CONFIG_ALTERA_PIO
-#ifdef LED_PIO_BASE
-       altera_pio_init(LED_PIO_BASE, LED_PIO_WIDTH, 'o',
-                       LED_PIO_RSTVAL, (1 << LED_PIO_WIDTH) - 1,
-                       "led");
-#endif
-#endif
-       return 0;
-}
-
 int checkboard(void)
 {
        printf("BOARD : %s\n", CONFIG_BOARD_NAME);
index 9dc6a7292b2ecfb7e91822ecb22c67c15411f68c..ad7227265ae6afeddef669e13004d897fb3d4b3d 100644 (file)
@@ -1,5 +1,6 @@
 CONFIG_NIOS2=y
 CONFIG_DM_SERIAL=y
+CONFIG_DM_GPIO=y
 CONFIG_TARGET_NIOS2_GENERIC=y
 CONFIG_DEFAULT_DEVICE_TREE="3c120_devboard"
 CONFIG_HUSH_PARSER=y
@@ -15,5 +16,6 @@ CONFIG_CMD_PING=y
 CONFIG_OF_CONTROL=y
 CONFIG_NET_RANDOM_ETHADDR=y
 CONFIG_DM=y
+CONFIG_ALTERA_PIO=y
 CONFIG_ALTERA_JTAG_UART=y
 CONFIG_ALTERA_JTAG_UART_BYPASS=y
diff --git a/doc/device-tree-bindings/gpio/altera_pio.txt b/doc/device-tree-bindings/gpio/altera_pio.txt
new file mode 100644 (file)
index 0000000..cf71eb2
--- /dev/null
@@ -0,0 +1,28 @@
+Altera GPIO controller bindings
+
+Required properties:
+- compatible:
+  - "altr,pio-1.0"
+- reg: Physical base address and length of the controller's registers.
+
+Optional properties:
+- altr,gpio-bank-width: Width of the GPIO bank. This defines how many pins the
+  GPIO device has. Ranges between 1-32. Optional and defaults to 32 if not
+  specified.
+- gpio-bank-name: bank name attached to this device.
+
+Example:
+
+user_led_pio_8out: gpio@0x4cc0 {
+       compatible = "altr,pio-1.0";
+       reg = <0x00004cc0 0x00000010>;
+       resetvalue = <255>;
+       altr,gpio-bank-width = <8>;
+       #gpio-cells = <2>;
+       gpio-controller;
+       gpio-bank-name = "led";
+};
+
+In this example, the gpio can be accessed as led[0..7] using gpio command of
+u-boot.
+==> gpio clear led0
index ef57a89ea2f4c897605f40754e6a9d6068a10633..9e494713fbcc9648dece7bf0f3e9ab29af9cb3ca 100644 (file)
@@ -14,6 +14,13 @@ config DM_GPIO
          particular GPIOs that they provide. The uclass interface
          is defined in include/asm-generic/gpio.h.
 
+config ALTERA_PIO
+       bool "Altera PIO driver"
+       depends on DM_GPIO
+       help
+         Select this to enable PIO for Altera devices. Please find
+         details on the "Embedded Peripherals IP User Guide" of Altera.
+
 config DWAPB_GPIO
        bool "DWAPB GPIO driver"
        depends on DM && DM_GPIO
index 3ca590700c4ce86e07a15958fe2961b6086ca219..7ceb80e9510f3320956a1253603b0e6e9717a0c6 100644 (file)
 /*
- * Driver for Altera's PIO ip core
- *
+ * Copyright (C) 2015  Thomas Chou <thomas@wytron.com.tw>
  * Copyright (C) 2011  Missing Link Electronics
  *                     Joachim Foerster <joachim@missinglinkelectronics.com>
  *
  * SPDX-License-Identifier:    GPL-2.0+
- *
- * To use this driver, in your board's config. header:
- * #define CONFIG_ALTERA_PIO
- * #define CONFIG_SYS_ALTERA_PIO_NUM <number-of-pio-cores>
- * #define CONFIG_SYS_ALTERA_PIO_GPIO_NUM <total-number-of-gpios>
- * And in your board's early setup routine:
- * altera_pio_init(<baseaddr>, <width>, 'i'|'o'|'t',
- *                 <reset-value>, <neg-mask>, "label");
- *  - 'i'|'o'|'t': PIO is input-only/output-only/tri-state
- *  - <reset-value>: for correct initial status display, output-only
- *  - <neg-mask> is meant to be used to in cases of active-low
- *    GPIOs, such as LEDs and buttons (on/pressed == 0). Each bit
- *    which is 1 in <neg-mask> inverts the corresponding GPIO's value
- *    before set/after get. So: gpio_set_value(gpio, 1) => LED on .
- *
- * Do NOT define CONFIG_SYS_GPIO_BASE !
- *
- * Optionally, in your board's config. header:
- * - To force a GPIO numbering scheme like in Linux ...
- * #define CONFIG_GPIO_DOWNTO_NUMBERING
- * ... starting with 255 (default)
- * #define CONFIG_GPIO_DOWNTO_MAX 255
  */
 #include <common.h>
+#include <dm.h>
+#include <errno.h>
+#include <malloc.h>
+#include <fdtdec.h>
 #include <asm/io.h>
 #include <asm/gpio.h>
 
-#ifdef CONFIG_GPIO_DOWNTO_NUMBERING
-#ifndef CONFIG_GPIO_DOWNTO_MAX
-#define CONFIG_GPIO_DOWNTO_MAX 255
-#endif
-#endif
-
-#define ALTERA_PIO_DATA                0x0
-#define ALTERA_PIO_DIR         0x4
-
-#define GPIO_LABEL_SIZE                9
+DECLARE_GLOBAL_DATA_PTR;
 
+struct altera_pio_regs {
+       u32     data;                   /* Data register */
+       u32     direction;              /* Direction register */
+};
 
-static struct altera_pio {
-       u32 base;
-       u8 width;
-       char iot;
-       u32 negmask;
-       u32 sh_data;
-       u32 sh_dir;
-       int gidx;
-       char label[GPIO_LABEL_SIZE];
-} pios[CONFIG_SYS_ALTERA_PIO_NUM];
+struct altera_pio_platdata {
+       struct altera_pio_regs *regs;
+       int gpio_count;
+       const char *bank_name;
+};
 
-static int pio_num;
-
-static struct altera_pio_gpio {
-       unsigned num;
-       struct altera_pio *pio;
-       char reqlabel[GPIO_LABEL_SIZE];
-} gpios[CONFIG_SYS_ALTERA_PIO_GPIO_NUM];
-
-static int pio_gpio_num;
-
-
-static int altera_pio_gidx(unsigned gpio)
+static int altera_pio_direction_input(struct udevice *dev, unsigned pin)
 {
-       int i;
+       struct altera_pio_platdata *plat = dev_get_platdata(dev);
+       struct altera_pio_regs *const regs = plat->regs;
 
-       for (i = 0; i < pio_gpio_num; ++i) {
-               if (gpio == gpios[i].num)
-                       break;
-       }
-       if (i >= pio_gpio_num)
-               return -1;
-       return i;
-}
+       clrbits_le32(&regs->direction, 1 << pin);
 
-static struct altera_pio *altera_pio_get_and_mask(unsigned gpio, u32 *mask)
-{
-       int gidx = altera_pio_gidx(gpio);
-       if (gidx < 0)
-               return NULL;
-       if (mask)
-               *mask = 1 << (gidx - gpios[gidx].pio->gidx);
-       return gpios[gidx].pio;
+       return 0;
 }
 
-#define altera_pio_use_gidx(_gidx, _reqlabel) \
-       { strncpy(gpios[_gidx].reqlabel, _reqlabel, GPIO_LABEL_SIZE); }
-#define altera_pio_unuse_gidx(_gidx) { gpios[_gidx].reqlabel[0] = '\0'; }
-#define altera_pio_is_gidx_used(_gidx) (gpios[_gidx].reqlabel[0] != '\0')
-
-static int altera_pio_gpio_init(struct altera_pio *pio, u8 width)
+static int altera_pio_direction_output(struct udevice *dev, unsigned pin,
+                                    int val)
 {
-       u8 gidx = pio_gpio_num;
-       int i;
-
-       if (!width)
-               return -1;
-       if ((pio_gpio_num + width) > CONFIG_SYS_ALTERA_PIO_GPIO_NUM)
-               return -1;
-
-       for (i = 0; i < width; ++i) {
-#ifdef CONFIG_GPIO_DOWNTO_NUMBERING
-               gpios[pio_gpio_num + i].num = \
-                       CONFIG_GPIO_DOWNTO_MAX + 1 - gidx - width + i;
-#else
-               gpios[pio_gpio_num + i].num = pio_gpio_num + i;
-#endif
-               gpios[pio_gpio_num + i].pio = pio;
-               altera_pio_unuse_gidx(pio_gpio_num + i);
-       }
-       pio_gpio_num += width;
-       return gidx;
-}
+       struct altera_pio_platdata *plat = dev_get_platdata(dev);
+       struct altera_pio_regs *const regs = plat->regs;
 
-int altera_pio_init(u32 base, u8 width, char iot, u32 rstval, u32 negmask,
-                const char *label)
-{
-       if (pio_num >= CONFIG_SYS_ALTERA_PIO_NUM)
-               return -1;
+       if (val)
+               setbits_le32(&regs->data, 1 << pin);
+       else
+               clrbits_le32(&regs->data, 1 << pin);
+       /* change the data first, then the direction. to avoid glitch */
+       setbits_le32(&regs->direction, 1 << pin);
 
-       pios[pio_num].base = base;
-       pios[pio_num].width = width;
-       pios[pio_num].iot = iot;
-       switch (iot) {
-       case 'i':
-               /* input only */
-               pios[pio_num].sh_dir = 0;
-               pios[pio_num].sh_data = readl(base + ALTERA_PIO_DATA);
-               break;
-       case 'o':
-               /* output only */
-               pios[pio_num].sh_dir = 0xffffffff & ((1 << width) - 1);
-               pios[pio_num].sh_data = rstval;
-               break;
-       case 't':
-               /* bidir, tri-state */
-               pios[pio_num].sh_dir = readl(base + ALTERA_PIO_DIR);
-               pios[pio_num].sh_data = readl(base + ALTERA_PIO_DATA);
-               break;
-       default:
-               return -1;
-       }
-       pios[pio_num].negmask = negmask & ((1 << width) - 1);
-       pios[pio_num].gidx = altera_pio_gpio_init(&pios[pio_num], width);
-       if (pios[pio_num].gidx < 0)
-               return -1;
-       strncpy(pios[pio_num].label, label, GPIO_LABEL_SIZE);
-       return pio_num++;
+       return 0;
 }
 
-void altera_pio_info(void)
+static int altera_pio_get_value(struct udevice *dev, unsigned pin)
 {
-       int i;
-       int j;
-       int gidx;
-       u32 mask;
+       struct altera_pio_platdata *plat = dev_get_platdata(dev);
+       struct altera_pio_regs *const regs = plat->regs;
 
-       for (i = 0; i < pio_num; ++i) {
-               printf("Altera PIO % 2d, @0x%08x, "
-                       "width: %u, label: %s\n",
-                      i, pios[i].base, pios[i].width, pios[i].label);
-               gidx = pios[i].gidx;
-               for (j = gidx; j < (gidx + pios[i].width); ++j) {
-                       mask = 1 << (j - gidx);
-                       printf("\tGPIO % 4d: %s %s [%c] %s\n",
-                               gpios[j].num,
-                               gpios[j].pio->sh_dir & mask ? "out" : " in",
-                               gpio_get_value(gpios[j].num) ? "set" : "clr",
-                               altera_pio_is_gidx_used(j) ? 'x' : ' ',
-                               gpios[j].reqlabel);
-               }
-       }
+       return readl(&regs->data) & (1 << pin);
 }
 
 
-int gpio_request(unsigned gpio, const char *label)
+static int altera_pio_set_value(struct udevice *dev, unsigned pin, int val)
 {
-       int gidx = altera_pio_gidx(gpio);
-       if (gidx < 0)
-               return gidx;
-       if (altera_pio_is_gidx_used(gidx))
-               return -1;
-
-       altera_pio_use_gidx(gidx, label);
-       return 0;
-}
+       struct altera_pio_platdata *plat = dev_get_platdata(dev);
+       struct altera_pio_regs *const regs = plat->regs;
 
-int gpio_free(unsigned gpio)
-{
-       int gidx = altera_pio_gidx(gpio);
-       if (gidx < 0)
-               return gidx;
-       if (!altera_pio_is_gidx_used(gidx))
-               return -1;
+       if (val)
+               setbits_le32(&regs->data, 1 << pin);
+       else
+               clrbits_le32(&regs->data, 1 << pin);
 
-       altera_pio_unuse_gidx(gidx);
        return 0;
 }
 
-int gpio_direction_input(unsigned gpio)
+static int altera_pio_probe(struct udevice *dev)
 {
-       u32 mask;
-       struct altera_pio *pio;
+       struct gpio_dev_priv *uc_priv = dev_get_uclass_priv(dev);
+       struct altera_pio_platdata *plat = dev_get_platdata(dev);
 
-       pio = altera_pio_get_and_mask(gpio, &mask);
-       if (!pio)
-               return -1;
-       if (pio->iot == 'o')
-               return -1;
+       uc_priv->gpio_count = plat->gpio_count;
+       uc_priv->bank_name = plat->bank_name;
 
-       writel(pio->sh_dir &= ~mask, pio->base + ALTERA_PIO_DIR);
        return 0;
 }
 
-int gpio_direction_output(unsigned gpio, int value)
+static int altera_pio_ofdata_to_platdata(struct udevice *dev)
 {
-       u32 mask;
-       struct altera_pio *pio;
+       struct altera_pio_platdata *plat = dev_get_platdata(dev);
 
-       pio = altera_pio_get_and_mask(gpio, &mask);
-       if (!pio)
-               return -1;
-       if (pio->iot == 'i')
-               return -1;
+       plat->regs = ioremap(dev_get_addr(dev),
+               sizeof(struct altera_pio_regs));
+       plat->gpio_count = fdtdec_get_int(gd->fdt_blob, dev->of_offset,
+               "altr,gpio-bank-width", 32);
+       plat->bank_name = fdt_getprop(gd->fdt_blob, dev->of_offset,
+               "gpio-bank-name", NULL);
 
-       value = (pio->negmask & mask) ? !value : value;
-       if (value)
-               pio->sh_data |= mask;
-       else
-               pio->sh_data &= ~mask;
-       writel(pio->sh_data, pio->base + ALTERA_PIO_DATA);
-       writel(pio->sh_dir |= mask, pio->base + ALTERA_PIO_DIR);
        return 0;
 }
 
-int gpio_get_value(unsigned gpio)
-{
-       u32 mask;
-       struct altera_pio *pio;
-       u32 val;
-
-       pio = altera_pio_get_and_mask(gpio, &mask);
-       if (!pio)
-               return -1;
-
-       if ((pio->sh_dir & mask) || (pio->iot == 'o'))
-               val = pio->sh_data & mask;
-       else
-               val = readl(pio->base + ALTERA_PIO_DATA) & mask;
-       return (pio->negmask & mask) ? !val : val;
-}
-
-void gpio_set_value(unsigned gpio, int value)
-{
-       u32 mask;
-       struct altera_pio *pio;
-
-       pio = altera_pio_get_and_mask(gpio, &mask);
-       if (!pio)
-               return;
-       if (pio->iot == 'i')
-               return;
-
-       value = (pio->negmask & mask) ? !value : value;
-       if (value)
-               pio->sh_data |= mask;
-       else
-               pio->sh_data &= ~mask;
-       writel(pio->sh_data, pio->base + ALTERA_PIO_DATA);
-       return;
-}
-
-int gpio_is_valid(int number)
-{
-       int gidx = altera_pio_gidx(number);
-
-       if (gidx < 0)
-               return 1;
-       return 0;
-}
+static const struct dm_gpio_ops altera_pio_ops = {
+       .direction_input        = altera_pio_direction_input,
+       .direction_output       = altera_pio_direction_output,
+       .get_value              = altera_pio_get_value,
+       .set_value              = altera_pio_set_value,
+};
+
+static const struct udevice_id altera_pio_ids[] = {
+       { .compatible = "altr,pio-1.0" },
+       { }
+};
+
+U_BOOT_DRIVER(altera_pio) = {
+       .name           = "altera_pio",
+       .id             = UCLASS_GPIO,
+       .of_match       = altera_pio_ids,
+       .ops            = &altera_pio_ops,
+       .ofdata_to_platdata = altera_pio_ofdata_to_platdata,
+       .platdata_auto_alloc_size = sizeof(struct altera_pio_platdata),
+       .probe          = altera_pio_probe,
+};
index c330c0aa4cb2f1597865c3c5d1cc177e80b92a86..62f7db963dd9f09913ff7f8af7f9460a79ec898b 100644 (file)
@@ -15,7 +15,6 @@
 #include "../board/altera/nios2-generic/custom_fpga.h" /* fpga parameters */
 #define CONFIG_BOARD_NAME "nios2-generic" /* custom board name */
 #define CONFIG_BOARD_EARLY_INIT_F      /* enable early board-spec. init */
-#define CONFIG_BOARD_EARLY_INIT_R
 #define CONFIG_DISPLAY_CPUINFO
 #define CONFIG_DISPLAY_BOARDINFO
 #define CONFIG_SYS_NIOS_SYSID_BASE     CONFIG_SYS_SYSID_BASE
  */
 #define CONFIG_SYS_TIMER_RATE          CONFIG_SYS_TIMER_FREQ
 
-/*
- * STATUS LED
- */
-#define CONFIG_ALTERA_PIO
-#define CONFIG_SYS_ALTERA_PIO_NUM      1
-#define CONFIG_SYS_ALTERA_PIO_GPIO_NUM LED_PIO_WIDTH
-
-#define CONFIG_STATUS_LED              /* Enable status driver */
-#define CONFIG_BOARD_SPECIFIC_LED
-#define CONFIG_GPIO_LED                /* Enable GPIO LED driver */
-#define CONFIG_GPIO                    /* Enable GPIO driver */
-#define LED_PIO_BASE                   USER_LED_PIO_8OUT_BASE
-#define LED_PIO_WIDTH                  8
-#define LED_PIO_RSTVAL                 0xff
-
-#define STATUS_LED_BIT                 0       /* Bit-0 on GPIO */
-#define STATUS_LED_STATE               1       /* Blinking */
-#define STATUS_LED_PERIOD      (CONFIG_SYS_HZ / 2)     /* 500 msec */
-
 /*
  * BOOTP options
  */
 #define CONFIG_SYS_MEMTEST_START       CONFIG_SYS_SDRAM_BASE
 #define CONFIG_SYS_MEMTEST_END         (CONFIG_SYS_INIT_SP - 0x20000)
 #define CONFIG_CMDLINE_EDITING
+#define CONFIG_CMD_GPIO
 
 #endif /* __CONFIG_H */