Re: [PATCH v4 0/3] cleanup of gpio_pcf857x.c
Hi Linus, On Tuesday 17 September 2013 14:51:48 Linus Walleij wrote: > On Wed, Sep 4, 2013 at 9:03 AM, George Cherian wrote: > > This patch series > > > > - removes the irq_demux_work > > - Uses devm_request_threaded_irq > > - Call the user handler iff gpio_to_irq is done. > > > > v1 --> v2 > > > > Split v1 to 3 patches > > > > v2 --> v3 > > > > Remove the unnecessary dts patches. > > > > v3 --> v4 > > > > Remove gpio->irq (in patch 2) > > Applied all three patches with Kuninoro's ACK. > > However your last iteration was still labeled "v3" in the subject > so I was a bit confused, but I took all those sent on sep 4. > > Please check the result in my GPIO tree or linux-next and > make sure it works. > > > Note: these patches were made after applying [1]. > > [1] - [PATCH v5] gpio: pcf857x: Add OF support - > > https://lkml.org/lkml/2013/8/27/70 > > Well that patch is not finished, but by rebasing patch 2 with > patch -p1 < patch2.patch it applied anyway. > > Now Laurent has to finalize his DT bindings on top of your patches instead. I will do so. At first sight I don't see any conflict issue. -- Regards, Laurent Pinchart -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v4 0/3] cleanup of gpio_pcf857x.c
On Wed, Sep 4, 2013 at 9:03 AM, George Cherian wrote: > This patch series > - removes the irq_demux_work > - Uses devm_request_threaded_irq > - Call the user handler iff gpio_to_irq is done. > > v1 --> v2 > Split v1 to 3 patches > v2 --> v3 > Remove the unnecessary dts patches. > v3 --> v4 > Remove gpio->irq (in patch 2) Applied all three patches with Kuninoro's ACK. However your last iteration was still labeled "v3" in the subject so I was a bit confused, but I took all those sent on sep 4. Please check the result in my GPIO tree or linux-next and make sure it works. > Note: these patches were made after applying [1]. > [1] - [PATCH v5] gpio: pcf857x: Add OF support - > https://lkml.org/lkml/2013/8/27/70 Well that patch is not finished, but by rebasing patch 2 with patch -p1 < patch2.patch it applied anyway. Now Laurent has to finalize his DT bindings on top of your patches instead. Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v4 0/3] cleanup of gpio_pcf857x.c
Hi > This patch series > - removes the irq_demux_work > - Uses devm_request_threaded_irq > - Call the user handler iff gpio_to_irq is done. > > v1 --> v2 > Split v1 to 3 patches > v2 --> v3 > Remove the unnecessary dts patches. > v3 --> v4 > Remove gpio->irq (in patch 2) > > Note: these patches were made after applying [1]. > [1] - [PATCH v5] gpio: pcf857x: Add OF support - > https://lkml.org/lkml/2013/8/27/70 > > George Cherian (3): > gpio: pcf857x: change to devm_request_threaded_irq > gpio: pcf857x: remove the irq_demux_work and gpio->irq > gpio: pcf857x: call the gpio user handler iff gpio_to_irq is done > > drivers/gpio/gpio-pcf857x.c | 53 > ++--- > 1 file changed, 26 insertions(+), 27 deletions(-) For all patches Acked-by: Kuninori Morimoto Best regards --- Kuninori Morimoto -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCH v4 0/3] cleanup of gpio_pcf857x.c
This patch series - removes the irq_demux_work - Uses devm_request_threaded_irq - Call the user handler iff gpio_to_irq is done. v1 --> v2 Split v1 to 3 patches v2 --> v3 Remove the unnecessary dts patches. v3 --> v4 Remove gpio->irq (in patch 2) Note: these patches were made after applying [1]. [1] - [PATCH v5] gpio: pcf857x: Add OF support - https://lkml.org/lkml/2013/8/27/70 George Cherian (3): gpio: pcf857x: change to devm_request_threaded_irq gpio: pcf857x: remove the irq_demux_work and gpio->irq gpio: pcf857x: call the gpio user handler iff gpio_to_irq is done drivers/gpio/gpio-pcf857x.c | 53 ++--- 1 file changed, 26 insertions(+), 27 deletions(-) -- 1.8.1.4 -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html