[PATCH 2/2] usb: Remove OHCI useless masking/unmasking of WDH interrupt
Benjamin Herrenschmidt <[email protected]>
| Newsgroups | gmane.linux.usb.devel,gmane.linux.ports.ppc64.devel |
|---|---|
| Message-ID | <[email protected]> |
The OHCI driver's IRQ handler, while processing a WDH interrupt, masks and unmasks it. I believe this is both broken (no dealign with write posting) and totally useless as the IRQ cannot re-occur while we are in the handler anyway. Signed-off-by: Benjamin Herrenschmidt <[email protected]> --- It might be a leftover related to the bogus optmisation that I've removed in a separate patch. Anyway, I don't think it's any use so let's remove to useless MMIO writes. drivers/usb/host/ohci-hcd.c | 4 ---- 1 file changed, 4 deletions(-) Index: linux-work/drivers/usb/host/ohci-hcd.c =================================================================== --- linux-work.orig/drivers/usb/host/ohci-hcd.c 2007-11-26 09:17:52.000000000 +1100 +++ linux-work/drivers/usb/host/ohci-hcd.c 2007-11-26 09:17:58.000000000 +1100 @@ -807,13 +807,9 @@ static irqreturn_t ohci_irq (struct usb_ } if (ints & OHCI_INTR_WDH) { - if (HC_IS_RUNNING(hcd->state)) - ohci_writel (ohci, OHCI_INTR_WDH, ®s->intrdisable); spin_lock (&ohci->lock); dl_done_list (ohci); spin_unlock (&ohci->lock); - if (HC_IS_RUNNING(hcd->state)) - ohci_writel (ohci, OHCI_INTR_WDH, ®s->intrenable); } if (quirk_zfmicro(ohci) && (ints & OHCI_INTR_SF)) { ------------------------------------------------------------------------- This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2005. http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ _______________________________________________ [email protected] To unsubscribe, use the last form field at: https://lists.sourceforge.net/lists/listinfo/linux-usb-devel