On Fri, Jul 11, 2008 at 4:36 AM, Maciej Sosnowski
checkpatch had some valid suggestions, please give it a run.
A general comment about this change it seems to add a bit of
complexity. The driver now has two workqueue routines and a tasklet.
Might some of these be combined? In other words, do we need a free
running watchdog when no descriptors are in flight? It seems that
using a timer to kick the cleanup tasklet could be used in place of
the ioat_dma_chan_watchdog workqueue.
This should be broken out into a separate patch. Perhaps with an
explanation of why it needs extra time for this particular