IXP: Fix GPIO_INT_ACT_LOW_SET()
authorMarek Vasut <marek.vasut@gmail.com>
Tue, 6 Mar 2012 10:57:43 +0000 (11:57 +0100)
committerAlbert ARIBAUD <albert.u.boot@aribaud.net>
Mon, 26 Mar 2012 21:09:26 +0000 (23:09 +0200)
The GPIO_INT_ACT_LOW_SET was incorrectly handling interrupt lines higher than 7.
This is due to the fact that there are two registers for total of 16 lines.

Signed-off-by: Marek Vasut <marek.vasut@gmail.com>
Cc: Bryan Hundven <bryanhundven@gmail.com>
Cc: Michael Schwingen <rincewind@discworld.dascon.de>
arch/arm/include/asm/arch-ixp/ixp425.h

index 5132607..c2e9c82 100644 (file)
 #define IXP425_GPIO_GPCLKR      IXP425_GPIO_REG(IXP425_GPIO_GPCLKR_OFFSET)
 #define IXP425_GPIO_GPDBSELR    IXP425_GPIO_REG(IXP425_GPIO_GPDBSELR_OFFSET)
 
+#define IXP425_GPIO_GPITR(line)        (((line) >= 8) ? \
+                               IXP425_GPIO_GPIT2R : IXP425_GPIO_GPIT1R)
+
 /*
  * Macros to make it easy to access the GPIO registers
  */
 #define GPIO_OUTPUT_DISABLE(line)      *IXP425_GPIO_GPOER |= (1 << (line))
 #define GPIO_OUTPUT_SET(line)          *IXP425_GPIO_GPOUTR |= (1 << (line))
 #define GPIO_OUTPUT_CLEAR(line)                *IXP425_GPIO_GPOUTR &= ~(1 << (line))
-#define GPIO_INT_ACT_LOW_SET(line)     *IXP425_GPIO_GPIT1R = \
-               (*IXP425_GPIO_GPIT1R & ~(0x7 << (line * 3))) | (0x1 << (line * 3))
+#define GPIO_INT_ACT_LOW_SET(line)                             \
+       *IXP425_GPIO_GPITR(line) =                              \
+                       (*IXP425_GPIO_GPITR(line) &             \
+                       ~(0x7 << (((line) & 0x7) * 3))) |       \
+                       (0x1 << (((line) & 0x7) * 3))           \
 
 /*
  * Constants to make it easy to access Timer Control/Status registers