Re: [PATCH 3/4] gpio: Kontron PLD gpio driver

2013-04-12 Thread Linus Walleij
On Fri, Apr 12, 2013 at 1:09 PM, Michael Brunner wrote: >> (...) >> > +struct kempld_gpio_data { >> > + struct gpio_chipchip; >> > + int irq; >> > + struct kempld_device_data *pld; >> > + uint16_tmas

Re: [PATCH 3/4] gpio: Kontron PLD gpio driver

2013-04-12 Thread Michael Brunner
Hi Linus, As this code is from me I will comment on your review. On Wed, 10 Apr 2013 22:45:51 +0200 Linus Walleij wrote: (...) > Trying to do some real review... > > (...) > > +++ b/drivers/gpio/gpio-kempld.c > > +#include > > Is this used? Actually not, this can be removed. > > +#inc

Re: [PATCH 3/4] gpio: Kontron PLD gpio driver

2013-04-10 Thread Linus Walleij
On Mon, Apr 8, 2013 at 7:15 PM, Kevin Strasser wrote: > From: Michael Brunner > > Add gpio support for the on-board PLD found on some Kontron embedded > modules. > > Signed-off-by: Michael Brunner > Signed-off-by: Kevin Strasser Trying to do some real review... (...) > +++ b/drivers/gpio/gpi

Re: [PATCH 3/4] gpio: Kontron PLD gpio driver

2013-04-10 Thread Linus Walleij
On Tue, Apr 9, 2013 at 6:41 PM, Guenter Roeck wrote: > On Tue, Apr 09, 2013 at 10:46:15AM +0200, Linus Walleij wrote: >> On Mon, Apr 8, 2013 at 7:15 PM, Kevin Strasser >> wrote: >> >> > From: Michael Brunner >> > >> > Add gpio support for the on-board PLD found on some Kontron embedded >> > modu

Re: [PATCH 3/4] gpio: Kontron PLD gpio driver

2013-04-09 Thread Guenter Roeck
On Tue, Apr 09, 2013 at 10:46:15AM +0200, Linus Walleij wrote: > On Mon, Apr 8, 2013 at 7:15 PM, Kevin Strasser > wrote: > > > From: Michael Brunner > > > > Add gpio support for the on-board PLD found on some Kontron embedded > > modules. > > > > Signed-off-by: Michael Brunner > > Signed-off-by

Re: [PATCH 3/4] gpio: Kontron PLD gpio driver

2013-04-09 Thread Linus Walleij
On Mon, Apr 8, 2013 at 7:15 PM, Kevin Strasser wrote: > From: Michael Brunner > > Add gpio support for the on-board PLD found on some Kontron embedded > modules. > > Signed-off-by: Michael Brunner > Signed-off-by: Kevin Strasser This looks very generic, setting and clearing bits in bytesized

[PATCH 3/4] gpio: Kontron PLD gpio driver

2013-04-08 Thread Kevin Strasser
From: Michael Brunner Add gpio support for the on-board PLD found on some Kontron embedded modules. Signed-off-by: Michael Brunner Signed-off-by: Kevin Strasser --- drivers/gpio/Kconfig| 22 ++ drivers/gpio/Makefile |2 + drivers/gpio/gpio-kempld.c | 476