| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <20130816155156.GA6953@roeck-us.net> Date: Fri, 16 Aug 2013 08:51:56 -0700 From: Guenter Roeck <linux@...ck-us.net> To: Linus Walleij <linus.walleij@...aro.org> Cc: Michael Brunner <mibru@....de>, "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>, Kevin Strasser <strassek@...r.orst.edu>, Darren Hart <dvhart@...ux.intel.com>, Samuel Ortiz <sameo@...ux.intel.com>, Chris Healy <chealy@...co-us.com>, "linux-gpio@...r.kernel.org" <linux-gpio@...r.kernel.org> Subject: Re: [PATCH] gpio: Fix bit masking in Kontron PLD GPIO driver 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@....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@...tron.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@...tron.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@...tron.com> Signed-off-by: Guenter Roeck <linux@...ck-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-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists