Hi, > > - (void)ide_ack_intr(hwif); > + if (hwif->ack_intr(hwif)) > + hwif->ack_intr(hwif); > > Shouldn't that be: > > - (void)ide_ack_intr(hwif); > + if (hwif->ack_intr) > + hwif->ack_intr(hwif);
Seconded. No point in ack'ing the int twice (not to mention the other side
effects).
Michael
--
To unsubscribe from this list: send the line "unsubscribe linux-m68k" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
