On 9/2/2013 12:25 PM, Kuninori Morimoto wrote:
Hi Sorry for my many response.
No Issues. and thanks for the review.
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. George Cherian (3): gpio: pcf857x: change to devm_request_threaded_irq gpio: pcf857x: remove the irq_demux_work gpio: pcf857x: call the gpio user handler iff gpio_to_irq is done drivers/gpio/gpio-pcf857x.c | 51 +++++++++++++++++++++++---------------------- 1 file changed, 26 insertions(+), 25 deletions(-)Basically, I have no objection about these patches, but I have 2 opinions - I guess we can merge #1 and #2 patches ? ("as is" is very OK for me though) - we don't need gpio->irq any more ? (additional patch is very OK for me)
I will sent one patch removing gpio->irq too.
Best regards --- Kuninori Morimoto
-- -George -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
