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

Reply via email to