ARM: S5PC110: Use the correct external GPIO config

From: Kyungmin Park <[email protected]>

Actually external GPIO don't have interrupt.

It's handled from differnt place, irq-eint.c

So use the correct external interrupt config and calculate the base adddress.

Signed-off-by Kyungmin Park <[email protected]>

---

arch/arm/mach-s5pv210/gpiolib.c +7 -0

index 9ea8972..f6a32c4 100644
@@ -31,11 +31,20 @@ static struct s3c_gpio_cfg gpio_cfg_noint = {
 ? ? ? ?.get_pull ? ? ? = s3c_gpio_getpull_updown,
 ?};
 
+static struct s3c_gpio_cfg gpio_cfg_extint = {
+ ? ? ? .set_config ? ? = s3c_gpio_setcfg_s3c64xx_4bit,
+ ? ? ? .set_pull ? ? ? = s3c_gpio_setpull_updown,
+ ? ? ? .get_pull ? ? ? = s3c_gpio_getpull_updown,
+};
+
+

?/* GPIO bank's base address given the index of the bank in the ?* list of all gpio banks.
?*/
?#define S5PV210_BANK_BASE(bank_nr) ? ? (S5P_VA_GPIO + ((bank_nr) * 0x20))

arch/arm/mach-s5pv210/gpiolib.c +2 -0

+#define EXTINT_OFFSET ? ? ? ? ? ? ? ? ?(0xC00)
+

?/*
?* Following are the gpio banks in v210.
?*

arch/arm/mach-s5pv210/gpiolib.c +9 -9

@@ -207,32 +216,28 @@ static struct s3c_gpio_chip s5pv210_gpio_4bit[] = {
 ? ? ? ? ? ? ? ? ? ? ? ?.label ?= "MP03",
 ? ? ? ? ? ? ? ?},
 ? ? ? ?}, {
- ? ? ? ? ? ? ? .base ? = (S5P_VA_GPIO + 0xC00),
- ? ? ? ? ? ? ? .config = &gpio_cfg_noint,
+ ? ? ? ? ? ? ? .config = &gpio_cfg_extint,
 ? ? ? ? ? ? ? ?.chip ? = {
 ? ? ? ? ? ? ? ? ? ? ? ?.base ? = S5PV210_GPH0(0),
 ? ? ? ? ? ? ? ? ? ? ? ?.ngpio ?= S5PV210_GPIO_H0_NR,
 ? ? ? ? ? ? ? ? ? ? ? ?.label ?= "GPH0",
 ? ? ? ? ? ? ? ?},
 ? ? ? ?}, {
- ? ? ? ? ? ? ? .base ? = (S5P_VA_GPIO + 0xC20),
- ? ? ? ? ? ? ? .config = &gpio_cfg_noint,
+ ? ? ? ? ? ? ? .config = &gpio_cfg_extint,
 ? ? ? ? ? ? ? ?.chip ? = {
 ? ? ? ? ? ? ? ? ? ? ? ?.base ? = S5PV210_GPH1(0),
 ? ? ? ? ? ? ? ? ? ? ? ?.ngpio ?= S5PV210_GPIO_H1_NR,
 ? ? ? ? ? ? ? ? ? ? ? ?.label ?= "GPH1",
 ? ? ? ? ? ? ? ?},
 ? ? ? ?}, {
- ? ? ? ? ? ? ? .base ? = (S5P_VA_GPIO + 0xC40),
- ? ? ? ? ? ? ? .config = &gpio_cfg_noint,
+ ? ? ? ? ? ? ? .config = &gpio_cfg_extint,
 ? ? ? ? ? ? ? ?.chip ? = {
 ? ? ? ? ? ? ? ? ? ? ? ?.base ? = S5PV210_GPH2(0),
 ? ? ? ? ? ? ? ? ? ? ? ?.ngpio ?= S5PV210_GPIO_H2_NR,
 ? ? ? ? ? ? ? ? ? ? ? ?.label ?= "GPH2",
 ? ? ? ? ? ? ? ?},
 ? ? ? ?}, {
- ? ? ? ? ? ? ? .base ? = (S5P_VA_GPIO + 0xC60),
- ? ? ? ? ? ? ? .config = &gpio_cfg_noint,
+ ? ? ? ? ? ? ? .config = &gpio_cfg_extint,
 ? ? ? ? ? ? ? ?.chip ? = {
 ? ? ? ? ? ? ? ? ? ? ? ?.base ? = S5PV210_GPH3(0),
 ? ? ? ? ? ? ? ? ? ? ? ?.ngpio ?= S5PV210_GPIO_H3_NR,

@@ -245,11 +250,15 @@ static __init int s5pv210_gpiolib_init(void)
 ?{
 ? ? ? ?struct s3c_gpio_chip *chip = s5pv210_gpio_4bit;
 ? ? ? ?int nr_chips = ARRAY_SIZE(s5pv210_gpio_4bit);
- ? ? ? int i = 0;
+ ? ? ? int i, extint = 0;
 
 ? ? ? ?for (i = 0; i < nr_chips; i++, chip++) {
 ? ? ? ? ? ? ? ?if (chip->config == NULL)
 ? ? ? ? ? ? ? ? ? ? ? ?chip->config = &gpio_cfg;
+ ? ? ? ? ? ? ? if (chip->config == &gpio_cfg_extint) {
+ ? ? ? ? ? ? ? ? ? ? ? chip->base = S5PV210_BANK_BASE(extint) + EXTINT_OFFSET;
+ ? ? ? ? ? ? ? ? ? ? ? extint++;
+ ? ? ? ? ? ? ? }
 ? ? ? ? ? ? ? ?if (chip->base == NULL)
 ? ? ? ? ? ? ? ? ? ? ? ?chip->base = S5PV210_BANK_BASE(i);
 ? ? ? ?}

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel at lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel