On Fri, Oct 12, 2012 at 5:33 PM, Ulf Hansson <[email protected]> wrote:
> From: Ulf Hansson <[email protected]> > > The cookie is now used to indicate if dma_unmap_sg shall be > done in post_request. At DMA errors, the DMA job is immediately > not only terminated but also unmapped. To indicate that this > has been done the cookie is reset to zero. post_request will > thus only do dma_umap_sg for requests which has a cookie not set > to zero. > > Some corresponding duplicated code could then be removed and > moreover some corrections at DMA errors for terminating the same > DMA job twice has also been fixed. > > Signed-off-by: Ulf Hansson <[email protected]> > Signed-off-by: Per Forlin <[email protected]> It looks like it's both factoring out code and also adding some unmapping in hithereto unhandled cases, correct? It looks OK to me now atleast so Acked-by: Linus Walleij <[email protected]> Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
