]> git.baikalelectronics.ru Git - kernel.git/commit
pinctrl: qcom: Use return value from irq_set_wake() call
authorMaulik Shah <mkshah@codeaurora.org>
Mon, 28 Sep 2020 04:32:00 +0000 (10:02 +0530)
committerMarc Zyngier <maz@kernel.org>
Tue, 6 Oct 2020 10:23:14 +0000 (11:23 +0100)
commitdd0e5161f79ffdec9c99c11e7e1dc3251c2467c8
tree6ef6a9d4ed07b1f2b85dbf124b9926b149ada383
parent9a200ccb858f41a97d637a7cb0adf0c00f1e0af1
pinctrl: qcom: Use return value from irq_set_wake() call

msmgpio irqchip was not using return value of irq_set_irq_wake() callback
since previously GIC-v3 irqchip neither had IRQCHIP_SKIP_SET_WAKE flag nor
it implemented .irq_set_wake callback. This lead to irq_set_irq_wake()
return error -ENXIO.

However from 'commit 093aa003672c ("irqchip/gic-v3: Allow interrupt to be
configured as wake-up sources")' GIC irqchip has IRQCHIP_SKIP_SET_WAKE
flag.

Use return value from irq_set_irq_wake() and irq_chip_set_wake_parent()
instead of always returning success.

Fixes: 063c6ff428fb ("pinctrl/msm: Setup GPIO chip in hierarchy")
Signed-off-by: Maulik Shah <mkshah@codeaurora.org>
Signed-off-by: Marc Zyngier <maz@kernel.org>
Tested-by: Stephen Boyd <swboyd@chromium.org>
Reviewed-by: Douglas Anderson <dianders@chromium.org>
Reviewed-by: Stephen Boyd <swboyd@chromium.org>
Acked-by: Bjorn Andersson <bjorn.andersson@linaro.org>
Acked-by: Linus Walleij <linus.walleij@linaro.org>
Link: https://lore.kernel.org/r/1601267524-20199-3-git-send-email-mkshah@codeaurora.org
drivers/pinctrl/qcom/pinctrl-msm.c