Hi, > On Jan 11, 2014, at 11:31 AM, Tejun Heo <tj@xxxxxxxxxx> wrote: > > Hello, David. > >> On Fri, Jan 10, 2014 at 02:07:21PM -0600, David Milburn wrote: >> Tejun, to avoid some of the interrupt handling code duplication, would >> it better to have a AHCI_HFLAG_FLUSH and then change >> >> this driver to set flush >> >> hpriv->hpriv.flush = (void *) xgene_ahci_iob_flush; >> >> and then standard code flush when necessary >> >> ahci_handle_port_interrupt >> >> /* board-specific flush before servicing completed cmds */ >> if (hpriv-> flags & AHCI_HFLAG_FLUSH) >> hpriv->flush(hpriv); > > And this would be the only spot that this would be necessary? If so, > why not just wrap the interrupt handler? The flush has to occurred immediately after reading the CI register. It can not wrap around the isr routine and issue the flush after or before the library ahci isr routine. -Loc -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
