From: Marek Vasut <marek.va...@gmail.com>

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.va...@gmail.com>
Cc: Bryan Hundven <bryanhund...@gmail.com>
Cc: Michael Schwingen <rincew...@discworld.dascon.de>
---
 arch/arm/include/asm/arch-ixp/ixp425.h |   10 ++++++++--
 1 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/arch/arm/include/asm/arch-ixp/ixp425.h 
b/arch/arm/include/asm/arch-ixp/ixp425.h
index 5132607..c2e9c82 100644
--- a/arch/arm/include/asm/arch-ixp/ixp425.h
+++ b/arch/arm/include/asm/arch-ixp/ixp425.h
@@ -358,6 +358,9 @@
 #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
  */
@@ -365,8 +368,11 @@
 #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
-- 
1.7.9

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to