ARM i.MX23/28: do not use complicated macros if not necessary
authorSascha Hauer <s.hauer@pengutronix.de>
Mon, 24 Jan 2011 12:09:41 +0000 (13:09 +0100)
committerSascha Hauer <s.hauer@pengutronix.de>
Fri, 28 Jan 2011 07:28:46 +0000 (08:28 +0100)
Get rid of ## preprocessor construct where it only makes the
code harder to read.

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Acked-by: Shawn Guo <shawn.guo@freescale.com>
Acked-by: Wolfram Sang <w.sang@pengutronix.de>
arch/arm/mach-mxs/gpio.c

index 36df2d7..8bcb340 100644 (file)
@@ -287,21 +287,23 @@ int __init mxs_gpio_init(struct mxs_gpio_port *port, int cnt)
        return 0;
 }
 
-#define DEFINE_MXS_GPIO_PORT(soc, _id)                                 \
+#define MX23_GPIO_BASE MX23_IO_ADDRESS(MX23_PINCTRL_BASE_ADDR)
+#define MX28_GPIO_BASE MX28_IO_ADDRESS(MX28_PINCTRL_BASE_ADDR)
+
+#define DEFINE_MXS_GPIO_PORT(_base, _irq, _id)                         \
        {                                                               \
                .chip.label = "gpio-" #_id,                             \
                .id = _id,                                              \
-               .irq = soc ## _INT_GPIO ## _id,                         \
-               .base = soc ## _IO_ADDRESS(                             \
-                               soc ## _PINCTRL ## _BASE_ADDR),         \
+               .irq = _irq,                                            \
+               .base = _base,                                          \
                .virtual_irq_start = MXS_GPIO_IRQ_START + (_id) * 32,   \
        }
 
 #ifdef CONFIG_SOC_IMX23
 static struct mxs_gpio_port mx23_gpio_ports[] = {
-       DEFINE_MXS_GPIO_PORT(MX23, 0),
-       DEFINE_MXS_GPIO_PORT(MX23, 1),
-       DEFINE_MXS_GPIO_PORT(MX23, 2),
+       DEFINE_MXS_GPIO_PORT(MX23_GPIO_BASE, MX23_INT_GPIO0, 0),
+       DEFINE_MXS_GPIO_PORT(MX23_GPIO_BASE, MX23_INT_GPIO1, 1),
+       DEFINE_MXS_GPIO_PORT(MX23_GPIO_BASE, MX23_INT_GPIO2, 2),
 };
 
 int __init mx23_register_gpios(void)
@@ -312,11 +314,11 @@ int __init mx23_register_gpios(void)
 
 #ifdef CONFIG_SOC_IMX28
 static struct mxs_gpio_port mx28_gpio_ports[] = {
-       DEFINE_MXS_GPIO_PORT(MX28, 0),
-       DEFINE_MXS_GPIO_PORT(MX28, 1),
-       DEFINE_MXS_GPIO_PORT(MX28, 2),
-       DEFINE_MXS_GPIO_PORT(MX28, 3),
-       DEFINE_MXS_GPIO_PORT(MX28, 4),
+       DEFINE_MXS_GPIO_PORT(MX28_GPIO_BASE, MX28_INT_GPIO0, 0),
+       DEFINE_MXS_GPIO_PORT(MX28_GPIO_BASE, MX28_INT_GPIO1, 1),
+       DEFINE_MXS_GPIO_PORT(MX28_GPIO_BASE, MX28_INT_GPIO2, 2),
+       DEFINE_MXS_GPIO_PORT(MX28_GPIO_BASE, MX28_INT_GPIO3, 3),
+       DEFINE_MXS_GPIO_PORT(MX28_GPIO_BASE, MX28_INT_GPIO4, 4),
 };
 
 int __init mx28_register_gpios(void)