diff options
| author |   Matthijs Kooijman <matthijs@stdin.nl> | 2013-04-25 23:39:14 +0200 |
|---|---|---|
| committer |   Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-05-16 15:22:38 -0700 |
| commit | 6aafb003852a8d0c7b96705bab330cb8d51a3c0a (patch) | |
| tree | 59935951a821a4aa94d4da03a92177298128d34b /drivers/staging/dwc2/hcd.h | |
| parent | staging: dwc2: do not use IRQF_DISABLED (diff) | |
| download | linux-dev-6aafb003852a8d0c7b96705bab330cb8d51a3c0a.tar.xz linux-dev-6aafb003852a8d0c7b96705bab330cb8d51a3c0a.zip | |
staging: dwc2: use irq_return_t for interrupt handlers
The top-level hcd interrupt handlers already used irq_return_t, but the
functions to which it delegates the actual work and the common irq
handler returned plain ints. In addition, they used the IRQ_RETVAL in
the wrong way (but because of the values of the various constants, this
didn't result in wrong behaviour).
Signed-off-by: Stephen Warren <swarren@wwwdotorg.org>
[matthijs@stdin.nl: Split patch from bigger patch and added commit message]
Signed-off-by: Matthijs Kooijman <matthijs@stdin.nl>
Acked-by: Paul Zimmerman <paulz@synopsys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/dwc2/hcd.h')
| -rw-r--r-- | drivers/staging/dwc2/hcd.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/staging/dwc2/hcd.h b/drivers/staging/dwc2/hcd.h index d071f1a05df1..8595c31b36ec 100644 --- a/drivers/staging/dwc2/hcd.h +++ b/drivers/staging/dwc2/hcd.h @@ -650,10 +650,10 @@ extern void dwc2_hcd_save_data_toggle(struct dwc2_hsotg *hsotg, * * @hsotg: The DWC2 HCD * - * Returns non zero if interrupt is handled - * Return 0 if interrupt is not handled + * Returns IRQ_HANDLED if interrupt is handled + * Return IRQ_NONE if interrupt is not handled */ -extern int dwc2_hcd_intr(struct dwc2_hsotg *hsotg); +extern irqreturn_t dwc2_hcd_intr(struct dwc2_hsotg *hsotg); /** * dwc2_hcd_stop() - Halts the DWC_otg host mode operation |