On Wed, Jan 09, 2019 at 11:44:14AM +0100, Johan Hovold wrote:
> On Sat, Dec 22, 2018 at 08:17:02PM +0530, Nishad Kamdar wrote:
> > Convert the GPIO driver to use the GPIO irqchip library
> > GPIOLIB_IRQCHIP instead of reimplementing the same.
> > 
> > Signed-off-by: Nishad Kamdar <nishadkam...@gmail.com>
> > ---
> > Changes in v4:
> >  - Remove changes related to conversion to gpiochip_get_data() to
> >    include it as a new patch.
> >  - Remove the 'struct irq_chip' field from 'struct gb_gpio_controller'
> >    as struct gpio_chip will have an irqchip whenever
> >    CONFIG_GPIOLIB_IRQCHIP is selected.
> 
> Ok, sorry for misleading you this. It seems we cannot use the gpio-chip
> irqchip, since that will register the irqchip automatically and possibly
> in an incompatible way. This new functionality is far from well
> documented, and you basically have to review the gpiolib code to figure
> it out.
> 
Ok, I get it.
> Looks like you need to add back the struct irq_chip.
> 
Ok, I'll do that.
> Johan

Thanks for the review.

Regards,
Nishad
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to