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 majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html