* [PATCH] gpio: Fix bit masking in Kontron PLD GPIO driver
@ 2013-08-09 15:33 Michael Brunner
2013-08-09 19:25 ` Guenter Roeck
` (2 more replies)
0 siblings, 3 replies; 7+ messages in thread
From: Michael Brunner @ 2013-08-09 15:33 UTC (permalink / raw)
To: linux-kernel@vger.kernel.org
Cc: Guenter Roeck, Kevin Strasser, Darren Hart, Samuel Ortiz,
Chris Healy, Linus Walleij, linux-gpio
This patch fixes the bit masking within the GPIO driver. The masking is
basically done twice which causes the wrong GPIOs to be addressed.
Signed-off-by: Michael Brunner <michael.brunner@kontron.com>
---
drivers/gpio/gpio-kempld.c | 24 +++++++++---------------
1 file changed, 9 insertions(+), 15 deletions(-)
diff --git a/drivers/gpio/gpio-kempld.c b/drivers/gpio/gpio-kempld.c
index d3ed563..6118b66 100644
--- a/drivers/gpio/gpio-kempld.c
+++ b/drivers/gpio/gpio-kempld.c
@@ -46,9 +46,9 @@ static void kempld_gpio_bitop(struct kempld_device_data *pld,
status = kempld_read8(pld, reg);
if (val)
- status |= (1 << bit);
+ status |= KEMPLD_GPIO_MASK(bit);
else
- status &= ~(1 << bit);
+ status &= ~KEMPLD_GPIO_MASK(bit);
kempld_write8(pld, reg, status);
}
@@ -60,7 +60,7 @@ static int kempld_gpio_get_bit(struct kempld_device_data *pld, u8 reg, u8 bit)
status = kempld_read8(pld, reg);
kempld_release_mutex(pld);
- return !!(status & (1 << bit));
+ return !!(status & KEMPLD_GPIO_MASK(bit));
}
static int kempld_gpio_get(struct gpio_chip *chip, unsigned offset)
@@ -69,8 +69,7 @@ static int kempld_gpio_get(struct gpio_chip *chip, unsigned offset)
= container_of(chip, struct kempld_gpio_data, chip);
struct kempld_device_data *pld = gpio->pld;
- return kempld_gpio_get_bit(pld, KEMPLD_GPIO_LVL_NUM(offset),
- KEMPLD_GPIO_MASK(offset));
+ return kempld_gpio_get_bit(pld, KEMPLD_GPIO_LVL_NUM(offset), offset);
}
static void kempld_gpio_set(struct gpio_chip *chip, unsigned offset, int value)
@@ -80,8 +79,7 @@ static void kempld_gpio_set(struct gpio_chip *chip, unsigned offset, int value)
struct kempld_device_data *pld = gpio->pld;
kempld_get_mutex(pld);
- kempld_gpio_bitop(pld, KEMPLD_GPIO_LVL_NUM(offset),
- KEMPLD_GPIO_MASK(offset), value);
+ kempld_gpio_bitop(pld, KEMPLD_GPIO_LVL_NUM(offset), offset, value);
kempld_release_mutex(pld);
}
@@ -92,8 +90,7 @@ static int kempld_gpio_direction_input(struct gpio_chip *chip, unsigned offset)
struct kempld_device_data *pld = gpio->pld;
kempld_get_mutex(pld);
- kempld_gpio_bitop(pld, KEMPLD_GPIO_DIR_NUM(offset),
- KEMPLD_GPIO_MASK(offset), 0);
+ kempld_gpio_bitop(pld, KEMPLD_GPIO_DIR_NUM(offset), offset, 0);
kempld_release_mutex(pld);
return 0;
@@ -107,10 +104,8 @@ static int kempld_gpio_direction_output(struct gpio_chip *chip, unsigned offset,
struct kempld_device_data *pld = gpio->pld;
kempld_get_mutex(pld);
- kempld_gpio_bitop(pld, KEMPLD_GPIO_LVL_NUM(offset),
- KEMPLD_GPIO_MASK(offset), value);
- kempld_gpio_bitop(pld, KEMPLD_GPIO_DIR_NUM(offset),
- KEMPLD_GPIO_MASK(offset), 1);
+ kempld_gpio_bitop(pld, KEMPLD_GPIO_LVL_NUM(offset), offset, value);
+ kempld_gpio_bitop(pld, KEMPLD_GPIO_DIR_NUM(offset), offset, 1);
kempld_release_mutex(pld);
return 0;
@@ -122,8 +117,7 @@ static int kempld_gpio_get_direction(struct gpio_chip *chip, unsigned offset)
= container_of(chip, struct kempld_gpio_data, chip);
struct kempld_device_data *pld = gpio->pld;
- return kempld_gpio_get_bit(pld, KEMPLD_GPIO_DIR_NUM(offset),
- KEMPLD_GPIO_MASK(offset));
+ return kempld_gpio_get_bit(pld, KEMPLD_GPIO_DIR_NUM(offset), offset);
}
static int kempld_gpio_pincount(struct kempld_device_data *pld)
^ permalink raw reply related [flat|nested] 7+ messages in thread* Re: [PATCH] gpio: Fix bit masking in Kontron PLD GPIO driver
2013-08-09 15:33 [PATCH] gpio: Fix bit masking in Kontron PLD GPIO driver Michael Brunner
@ 2013-08-09 19:25 ` Guenter Roeck
2013-08-10 22:12 ` Kevin Strasser
2013-08-16 15:09 ` Linus Walleij
2 siblings, 0 replies; 7+ messages in thread
From: Guenter Roeck @ 2013-08-09 19:25 UTC (permalink / raw)
To: Michael Brunner
Cc: linux-kernel@vger.kernel.org, Kevin Strasser, Darren Hart,
Samuel Ortiz, Chris Healy, Linus Walleij, linux-gpio
On Fri, Aug 09, 2013 at 05:33:26PM +0200, Michael Brunner wrote:
> This patch fixes the bit masking within the GPIO driver. The masking is
> basically done twice which causes the wrong GPIOs to be addressed.
>
> Signed-off-by: Michael Brunner <michael.brunner@kontron.com>
Reviewed-by: Guenter Roeck <linux@roeck-us.net>
> ---
> drivers/gpio/gpio-kempld.c | 24 +++++++++---------------
> 1 file changed, 9 insertions(+), 15 deletions(-)
>
> diff --git a/drivers/gpio/gpio-kempld.c b/drivers/gpio/gpio-kempld.c
> index d3ed563..6118b66 100644
> --- a/drivers/gpio/gpio-kempld.c
> +++ b/drivers/gpio/gpio-kempld.c
> @@ -46,9 +46,9 @@ static void kempld_gpio_bitop(struct kempld_device_data *pld,
>
> status = kempld_read8(pld, reg);
> if (val)
> - status |= (1 << bit);
> + status |= KEMPLD_GPIO_MASK(bit);
> else
> - status &= ~(1 << bit);
> + status &= ~KEMPLD_GPIO_MASK(bit);
> kempld_write8(pld, reg, status);
> }
>
> @@ -60,7 +60,7 @@ static int kempld_gpio_get_bit(struct kempld_device_data *pld, u8 reg, u8 bit)
> status = kempld_read8(pld, reg);
> kempld_release_mutex(pld);
>
> - return !!(status & (1 << bit));
> + return !!(status & KEMPLD_GPIO_MASK(bit));
> }
>
> static int kempld_gpio_get(struct gpio_chip *chip, unsigned offset)
> @@ -69,8 +69,7 @@ static int kempld_gpio_get(struct gpio_chip *chip, unsigned offset)
> = container_of(chip, struct kempld_gpio_data, chip);
> struct kempld_device_data *pld = gpio->pld;
>
> - return kempld_gpio_get_bit(pld, KEMPLD_GPIO_LVL_NUM(offset),
> - KEMPLD_GPIO_MASK(offset));
> + return kempld_gpio_get_bit(pld, KEMPLD_GPIO_LVL_NUM(offset), offset);
> }
>
> static void kempld_gpio_set(struct gpio_chip *chip, unsigned offset, int value)
> @@ -80,8 +79,7 @@ static void kempld_gpio_set(struct gpio_chip *chip, unsigned offset, int value)
> struct kempld_device_data *pld = gpio->pld;
>
> kempld_get_mutex(pld);
> - kempld_gpio_bitop(pld, KEMPLD_GPIO_LVL_NUM(offset),
> - KEMPLD_GPIO_MASK(offset), value);
> + kempld_gpio_bitop(pld, KEMPLD_GPIO_LVL_NUM(offset), offset, value);
> kempld_release_mutex(pld);
> }
>
> @@ -92,8 +90,7 @@ static int kempld_gpio_direction_input(struct gpio_chip *chip, unsigned offset)
> struct kempld_device_data *pld = gpio->pld;
>
> kempld_get_mutex(pld);
> - kempld_gpio_bitop(pld, KEMPLD_GPIO_DIR_NUM(offset),
> - KEMPLD_GPIO_MASK(offset), 0);
> + kempld_gpio_bitop(pld, KEMPLD_GPIO_DIR_NUM(offset), offset, 0);
> kempld_release_mutex(pld);
>
> return 0;
> @@ -107,10 +104,8 @@ static int kempld_gpio_direction_output(struct gpio_chip *chip, unsigned offset,
> struct kempld_device_data *pld = gpio->pld;
>
> kempld_get_mutex(pld);
> - kempld_gpio_bitop(pld, KEMPLD_GPIO_LVL_NUM(offset),
> - KEMPLD_GPIO_MASK(offset), value);
> - kempld_gpio_bitop(pld, KEMPLD_GPIO_DIR_NUM(offset),
> - KEMPLD_GPIO_MASK(offset), 1);
> + kempld_gpio_bitop(pld, KEMPLD_GPIO_LVL_NUM(offset), offset, value);
> + kempld_gpio_bitop(pld, KEMPLD_GPIO_DIR_NUM(offset), offset, 1);
> kempld_release_mutex(pld);
>
> return 0;
> @@ -122,8 +117,7 @@ static int kempld_gpio_get_direction(struct gpio_chip *chip, unsigned offset)
> = container_of(chip, struct kempld_gpio_data, chip);
> struct kempld_device_data *pld = gpio->pld;
>
> - return kempld_gpio_get_bit(pld, KEMPLD_GPIO_DIR_NUM(offset),
> - KEMPLD_GPIO_MASK(offset));
> + return kempld_gpio_get_bit(pld, KEMPLD_GPIO_DIR_NUM(offset), offset);
> }
>
> static int kempld_gpio_pincount(struct kempld_device_data *pld)
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] gpio: Fix bit masking in Kontron PLD GPIO driver
2013-08-09 15:33 [PATCH] gpio: Fix bit masking in Kontron PLD GPIO driver Michael Brunner
2013-08-09 19:25 ` Guenter Roeck
@ 2013-08-10 22:12 ` Kevin Strasser
2013-08-16 15:09 ` Linus Walleij
2 siblings, 0 replies; 7+ messages in thread
From: Kevin Strasser @ 2013-08-10 22:12 UTC (permalink / raw)
To: Michael Brunner
Cc: linux-kernel@vger.kernel.org, Guenter Roeck, Darren Hart,
Samuel Ortiz, Chris Healy, Linus Walleij, linux-gpio
On Fri, Aug 09, 2013 at 05:33:26PM +0200, Michael Brunner wrote:
> This patch fixes the bit masking within the GPIO driver. The masking is
> basically done twice which causes the wrong GPIOs to be addressed.
>
> Signed-off-by: Michael Brunner <michael.brunner@kontron.com>
Acked-by: Kevin Strasser <strassek@engr.orst.edu>
> ---
> drivers/gpio/gpio-kempld.c | 24 +++++++++---------------
> 1 file changed, 9 insertions(+), 15 deletions(-)
>
> diff --git a/drivers/gpio/gpio-kempld.c b/drivers/gpio/gpio-kempld.c
> index d3ed563..6118b66 100644
> --- a/drivers/gpio/gpio-kempld.c
> +++ b/drivers/gpio/gpio-kempld.c
> @@ -46,9 +46,9 @@ static void kempld_gpio_bitop(struct kempld_device_data *pld,
>
> status = kempld_read8(pld, reg);
> if (val)
> - status |= (1 << bit);
> + status |= KEMPLD_GPIO_MASK(bit);
> else
> - status &= ~(1 << bit);
> + status &= ~KEMPLD_GPIO_MASK(bit);
> kempld_write8(pld, reg, status);
> }
>
> @@ -60,7 +60,7 @@ static int kempld_gpio_get_bit(struct kempld_device_data *pld, u8 reg, u8 bit)
> status = kempld_read8(pld, reg);
> kempld_release_mutex(pld);
>
> - return !!(status & (1 << bit));
> + return !!(status & KEMPLD_GPIO_MASK(bit));
> }
>
> static int kempld_gpio_get(struct gpio_chip *chip, unsigned offset)
> @@ -69,8 +69,7 @@ static int kempld_gpio_get(struct gpio_chip *chip, unsigned offset)
> = container_of(chip, struct kempld_gpio_data, chip);
> struct kempld_device_data *pld = gpio->pld;
>
> - return kempld_gpio_get_bit(pld, KEMPLD_GPIO_LVL_NUM(offset),
> - KEMPLD_GPIO_MASK(offset));
> + return kempld_gpio_get_bit(pld, KEMPLD_GPIO_LVL_NUM(offset), offset);
> }
>
> static void kempld_gpio_set(struct gpio_chip *chip, unsigned offset, int value)
> @@ -80,8 +79,7 @@ static void kempld_gpio_set(struct gpio_chip *chip, unsigned offset, int value)
> struct kempld_device_data *pld = gpio->pld;
>
> kempld_get_mutex(pld);
> - kempld_gpio_bitop(pld, KEMPLD_GPIO_LVL_NUM(offset),
> - KEMPLD_GPIO_MASK(offset), value);
> + kempld_gpio_bitop(pld, KEMPLD_GPIO_LVL_NUM(offset), offset, value);
> kempld_release_mutex(pld);
> }
>
> @@ -92,8 +90,7 @@ static int kempld_gpio_direction_input(struct gpio_chip *chip, unsigned offset)
> struct kempld_device_data *pld = gpio->pld;
>
> kempld_get_mutex(pld);
> - kempld_gpio_bitop(pld, KEMPLD_GPIO_DIR_NUM(offset),
> - KEMPLD_GPIO_MASK(offset), 0);
> + kempld_gpio_bitop(pld, KEMPLD_GPIO_DIR_NUM(offset), offset, 0);
> kempld_release_mutex(pld);
>
> return 0;
> @@ -107,10 +104,8 @@ static int kempld_gpio_direction_output(struct gpio_chip *chip, unsigned offset,
> struct kempld_device_data *pld = gpio->pld;
>
> kempld_get_mutex(pld);
> - kempld_gpio_bitop(pld, KEMPLD_GPIO_LVL_NUM(offset),
> - KEMPLD_GPIO_MASK(offset), value);
> - kempld_gpio_bitop(pld, KEMPLD_GPIO_DIR_NUM(offset),
> - KEMPLD_GPIO_MASK(offset), 1);
> + kempld_gpio_bitop(pld, KEMPLD_GPIO_LVL_NUM(offset), offset, value);
> + kempld_gpio_bitop(pld, KEMPLD_GPIO_DIR_NUM(offset), offset, 1);
> kempld_release_mutex(pld);
>
> return 0;
> @@ -122,8 +117,7 @@ static int kempld_gpio_get_direction(struct gpio_chip *chip, unsigned offset)
> = container_of(chip, struct kempld_gpio_data, chip);
> struct kempld_device_data *pld = gpio->pld;
>
> - return kempld_gpio_get_bit(pld, KEMPLD_GPIO_DIR_NUM(offset),
> - KEMPLD_GPIO_MASK(offset));
> + return kempld_gpio_get_bit(pld, KEMPLD_GPIO_DIR_NUM(offset), offset);
> }
>
> static int kempld_gpio_pincount(struct kempld_device_data *pld)
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] gpio: Fix bit masking in Kontron PLD GPIO driver
2013-08-09 15:33 [PATCH] gpio: Fix bit masking in Kontron PLD GPIO driver Michael Brunner
2013-08-09 19:25 ` Guenter Roeck
2013-08-10 22:12 ` Kevin Strasser
@ 2013-08-16 15:09 ` Linus Walleij
2013-08-16 15:51 ` Guenter Roeck
2 siblings, 1 reply; 7+ messages in thread
From: Linus Walleij @ 2013-08-16 15:09 UTC (permalink / raw)
To: Michael Brunner
Cc: linux-kernel@vger.kernel.org, Guenter Roeck, Kevin Strasser,
Darren Hart, Samuel Ortiz, Chris Healy,
linux-gpio@vger.kernel.org
On Fri, Aug 9, 2013 at 5:33 PM, Michael Brunner <mibru@gmx.de> wrote:
> This patch fixes the bit masking within the GPIO driver. The masking is
> basically done twice which causes the wrong GPIOs to be addressed.
>
> Signed-off-by: Michael Brunner <michael.brunner@kontron.com>
This looks good but just doesn't apply. Look:
git am --signoff brunner2.patch
Tillämpar: gpio: Fix bit masking in Kontron PLD GPIO driver
error: patch misslyckades: drivers/gpio/gpio-kempld.c:46
error: drivers/gpio/gpio-kempld.c: patchen kan inte tillämpas
Then:
patch -p1 < brunner2.patch
patching file drivers/gpio/gpio-kempld.c
Hunk #1 FAILED at 46.
Hunk #2 FAILED at 60.
Hunk #3 FAILED at 69.
Hunk #4 FAILED at 80.
Hunk #5 FAILED at 92.
Hunk #6 FAILED at 107.
Hunk #7 FAILED at 122.
7 out of 7 hunks FAILED -- saving rejects to file drivers/gpio/gpio-kempld.c.rej
I think there may be some whitespace mangling in your client
or mail server or whatever.
I had the same problem with the previous patch but hand-coded it
right since it was just a oneline.
Please try to figure out what is causing this.
Consult:
Documentation/email-clients.txt
Yours,
Linus Walleij
--
To unsubscribe from this list: send the line "unsubscribe linux-gpio" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] gpio: Fix bit masking in Kontron PLD GPIO driver
2013-08-16 15:09 ` Linus Walleij
@ 2013-08-16 15:51 ` Guenter Roeck
2013-08-16 15:58 ` Linus Walleij
0 siblings, 1 reply; 7+ messages in thread
From: Guenter Roeck @ 2013-08-16 15:51 UTC (permalink / raw)
To: Linus Walleij
Cc: Michael Brunner, linux-kernel@vger.kernel.org, Kevin Strasser,
Darren Hart, Samuel Ortiz, Chris Healy,
linux-gpio@vger.kernel.org
On Fri, Aug 16, 2013 at 05:09:04PM +0200, Linus Walleij wrote:
> On Fri, Aug 9, 2013 at 5:33 PM, Michael Brunner <mibru@gmx.de> wrote:
>
> > This patch fixes the bit masking within the GPIO driver. The masking is
> > basically done twice which causes the wrong GPIOs to be addressed.
> >
> > Signed-off-by: Michael Brunner <michael.brunner@kontron.com>
>
> This looks good but just doesn't apply. Look:
>
> git am --signoff brunner2.patch
> Tillämpar: gpio: Fix bit masking in Kontron PLD GPIO driver
> error: patch misslyckades: drivers/gpio/gpio-kempld.c:46
> error: drivers/gpio/gpio-kempld.c: patchen kan inte tillämpas
>
> Then:
> patch -p1 < brunner2.patch
> patching file drivers/gpio/gpio-kempld.c
> Hunk #1 FAILED at 46.
> Hunk #2 FAILED at 60.
> Hunk #3 FAILED at 69.
> Hunk #4 FAILED at 80.
> Hunk #5 FAILED at 92.
> Hunk #6 FAILED at 107.
> Hunk #7 FAILED at 122.
> 7 out of 7 hunks FAILED -- saving rejects to file drivers/gpio/gpio-kempld.c.rej
>
> I think there may be some whitespace mangling in your client
> or mail server or whatever.
>
> I had the same problem with the previous patch but hand-coded it
> right since it was just a oneline.
>
> Please try to figure out what is causing this.
> Consult:
> Documentation/email-clients.txt
>
I think Michael just went on vacation.
Clean version is below. He must have sent it to me separately
because it worked for me.
Guenter
---
From: Brunner Michael <Michael.Brunner@kontron.com>
Date: Wed, 31 Jul 2013 20:55:39 +0200
Subject: [PATCH] gpio: Fix bit masking in Kontron PLD GPIO driver
This patch fixes the bit masking within the GPIO driver. The masking is
basically done twice which causes the wrong GPIOs to be addressed.
Signed-off-by: Michael Brunner <michael.brunner@kontron.com>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
---
drivers/gpio/gpio-kempld.c | 24 +++++++++---------------
1 file changed, 9 insertions(+), 15 deletions(-)
diff --git a/drivers/gpio/gpio-kempld.c b/drivers/gpio/gpio-kempld.c
index d3ed563..6118b66 100644
--- a/drivers/gpio/gpio-kempld.c
+++ b/drivers/gpio/gpio-kempld.c
@@ -46,9 +46,9 @@ static void kempld_gpio_bitop(struct kempld_device_data *pld,
status = kempld_read8(pld, reg);
if (val)
- status |= (1 << bit);
+ status |= KEMPLD_GPIO_MASK(bit);
else
- status &= ~(1 << bit);
+ status &= ~KEMPLD_GPIO_MASK(bit);
kempld_write8(pld, reg, status);
}
@@ -60,7 +60,7 @@ static int kempld_gpio_get_bit(struct kempld_device_data *pld, u8 reg, u8 bit)
status = kempld_read8(pld, reg);
kempld_release_mutex(pld);
- return !!(status & (1 << bit));
+ return !!(status & KEMPLD_GPIO_MASK(bit));
}
static int kempld_gpio_get(struct gpio_chip *chip, unsigned offset)
@@ -69,8 +69,7 @@ static int kempld_gpio_get(struct gpio_chip *chip, unsigned offset)
= container_of(chip, struct kempld_gpio_data, chip);
struct kempld_device_data *pld = gpio->pld;
- return kempld_gpio_get_bit(pld, KEMPLD_GPIO_LVL_NUM(offset),
- KEMPLD_GPIO_MASK(offset));
+ return kempld_gpio_get_bit(pld, KEMPLD_GPIO_LVL_NUM(offset), offset);
}
static void kempld_gpio_set(struct gpio_chip *chip, unsigned offset, int value)
@@ -80,8 +79,7 @@ static void kempld_gpio_set(struct gpio_chip *chip, unsigned offset, int value)
struct kempld_device_data *pld = gpio->pld;
kempld_get_mutex(pld);
- kempld_gpio_bitop(pld, KEMPLD_GPIO_LVL_NUM(offset),
- KEMPLD_GPIO_MASK(offset), value);
+ kempld_gpio_bitop(pld, KEMPLD_GPIO_LVL_NUM(offset), offset, value);
kempld_release_mutex(pld);
}
@@ -92,8 +90,7 @@ static int kempld_gpio_direction_input(struct gpio_chip *chip, unsigned offset)
struct kempld_device_data *pld = gpio->pld;
kempld_get_mutex(pld);
- kempld_gpio_bitop(pld, KEMPLD_GPIO_DIR_NUM(offset),
- KEMPLD_GPIO_MASK(offset), 0);
+ kempld_gpio_bitop(pld, KEMPLD_GPIO_DIR_NUM(offset), offset, 0);
kempld_release_mutex(pld);
return 0;
@@ -107,10 +104,8 @@ static int kempld_gpio_direction_output(struct gpio_chip *chip, unsigned offset,
struct kempld_device_data *pld = gpio->pld;
kempld_get_mutex(pld);
- kempld_gpio_bitop(pld, KEMPLD_GPIO_LVL_NUM(offset),
- KEMPLD_GPIO_MASK(offset), value);
- kempld_gpio_bitop(pld, KEMPLD_GPIO_DIR_NUM(offset),
- KEMPLD_GPIO_MASK(offset), 1);
+ kempld_gpio_bitop(pld, KEMPLD_GPIO_LVL_NUM(offset), offset, value);
+ kempld_gpio_bitop(pld, KEMPLD_GPIO_DIR_NUM(offset), offset, 1);
kempld_release_mutex(pld);
return 0;
@@ -122,8 +117,7 @@ static int kempld_gpio_get_direction(struct gpio_chip *chip, unsigned offset)
= container_of(chip, struct kempld_gpio_data, chip);
struct kempld_device_data *pld = gpio->pld;
- return kempld_gpio_get_bit(pld, KEMPLD_GPIO_DIR_NUM(offset),
- KEMPLD_GPIO_MASK(offset));
+ return kempld_gpio_get_bit(pld, KEMPLD_GPIO_DIR_NUM(offset), offset);
}
static int kempld_gpio_pincount(struct kempld_device_data *pld)
--
1.7.9.7
--
To unsubscribe from this list: send the line "unsubscribe linux-gpio" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH] gpio: Fix bit masking in Kontron PLD GPIO driver
2013-08-16 15:51 ` Guenter Roeck
@ 2013-08-16 15:58 ` Linus Walleij
2013-08-16 15:59 ` Guenter Roeck
0 siblings, 1 reply; 7+ messages in thread
From: Linus Walleij @ 2013-08-16 15:58 UTC (permalink / raw)
To: Guenter Roeck
Cc: Michael Brunner, linux-kernel@vger.kernel.org, Kevin Strasser,
Darren Hart, Samuel Ortiz, Chris Healy,
linux-gpio@vger.kernel.org
On Fri, Aug 16, 2013 at 5:51 PM, Guenter Roeck <linux@roeck-us.net> wrote:
> I think Michael just went on vacation.
>
> Clean version is below. He must have sent it to me separately
> because it worked for me.
Yeah this worked fine ... strange.
Patch applied.
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] gpio: Fix bit masking in Kontron PLD GPIO driver
2013-08-16 15:58 ` Linus Walleij
@ 2013-08-16 15:59 ` Guenter Roeck
0 siblings, 0 replies; 7+ messages in thread
From: Guenter Roeck @ 2013-08-16 15:59 UTC (permalink / raw)
To: Linus Walleij
Cc: Michael Brunner, linux-kernel@vger.kernel.org, Kevin Strasser,
Darren Hart, Samuel Ortiz, Chris Healy,
linux-gpio@vger.kernel.org
On Fri, Aug 16, 2013 at 05:58:02PM +0200, Linus Walleij wrote:
> On Fri, Aug 16, 2013 at 5:51 PM, Guenter Roeck <linux@roeck-us.net> wrote:
>
> > I think Michael just went on vacation.
> >
> > Clean version is below. He must have sent it to me separately
> > because it worked for me.
>
> Yeah this worked fine ... strange.
>
Probably there is some outlook server in the path :(
Guenter
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2013-08-16 15:59 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-08-09 15:33 [PATCH] gpio: Fix bit masking in Kontron PLD GPIO driver Michael Brunner
2013-08-09 19:25 ` Guenter Roeck
2013-08-10 22:12 ` Kevin Strasser
2013-08-16 15:09 ` Linus Walleij
2013-08-16 15:51 ` Guenter Roeck
2013-08-16 15:58 ` Linus Walleij
2013-08-16 15:59 ` Guenter Roeck
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).