Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758548Ab0BXW7A (ORCPT ); Wed, 24 Feb 2010 17:59:00 -0500 Received: from smtp1.linux-foundation.org ([140.211.169.13]:40394 "EHLO smtp1.linux-foundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1758042Ab0BXW66 (ORCPT ); Wed, 24 Feb 2010 17:58:58 -0500 Date: Wed, 24 Feb 2010 14:58:22 -0800 From: Andrew Morton To: Ben Gardner Cc: Andres Salomon , linux-kernel@vger.kernel.org Subject: Re: [PATCH] cs5535_gpio: gpio_chip.get should return the input value Message-Id: <20100224145822.803e86ed.akpm@linux-foundation.org> In-Reply-To: <808c8e9d1002231455q379b26acq4b4a50188c5d73be@mail.gmail.com> References: <808c8e9d1002231455q379b26acq4b4a50188c5d73be@mail.gmail.com> X-Mailer: Sylpheed 2.4.8 (GTK+ 2.12.5; x86_64-redhat-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1024 Lines: 28 On Tue, 23 Feb 2010 16:55:17 -0600 Ben Gardner wrote: > The gpio_chip.get() function for the CS5535 GPIO driver currently returns the > output value instead of the input value. > This patch changes it to return the input value. > > Signed-off-by: Ben Gardner > --- > --- linux-2.6.33-rc8.orig/drivers/gpio/cs5535-gpio.c > +++ linux-2.6.33-rc8/drivers/gpio/cs5535-gpio.c > @@ -154,7 +154,7 @@ > > static int chip_gpio_get(struct gpio_chip *chip, unsigned offset) > { > - return cs5535_gpio_isset(offset, GPIO_OUTPUT_VAL); > + return cs5535_gpio_isset(offset, GPIO_READ_BACK); > } > > static void chip_gpio_set(struct gpio_chip *chip, unsigned offset, int val) What were the user-visible effects of this bug? -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/