RE: [PATCH] staging: dwc2: do not clear pending interrupts twice

From: Paul Zimmerman
Date: Fri Nov 15 2013 - 14:59:43 EST


> From: Julien DELACOU [mailto:julien.delacou@xxxxxx]
> Sent: Friday, November 15, 2013 2:40 AM
>
> Pending interrupts clearing is done in dwc2_enable_common_interrupts
> so we don't need to do it twice.
>
> Signed-off-by: Julien Delacou <julien.delacou@xxxxxx>
> ---
> drivers/staging/dwc2/core.c | 3 ---
> 1 file changed, 3 deletions(-)
>
> diff --git a/drivers/staging/dwc2/core.c b/drivers/staging/dwc2/core.c
> index c8ff668..8374ec3 100644
> --- a/drivers/staging/dwc2/core.c
> +++ b/drivers/staging/dwc2/core.c
> @@ -533,9 +533,6 @@ void dwc2_enable_host_interrupts(struct dwc2_hsotg *hsotg)
> writel(0, hsotg->regs + GINTMSK);
> writel(0, hsotg->regs + HAINTMSK);
>
> - /* Clear any pending interrupts */
> - writel(0xffffffff, hsotg->regs + GINTSTS);
> -
> /* Enable the common interrupts */
> dwc2_enable_common_interrupts(hsotg);
>
> --

Acked-by: Paul Zimmerman <paulz@xxxxxxxxxxxx>

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/