Re: [PATCH 1/3] hexagon: remove the sync_single_for_cpu DMA operation

From: Richard Kuo
Date: Tue Jul 24 2018 - 23:29:54 EST


On Thu, Jul 19, 2018 at 05:56:33AM -0700, Christoph Hellwig wrote:
> hexagon does all the required cache maintainance at dma map time, and none
> at unmap time. It thus has to implement sync_single_for_device to match
> the map cace for buffer reuse, but there is no point in doing another
> invalidation in the sync_single_cpu_case, which in terms of cache
> maintainance is equivalent to the unmap case.
>
> Signed-off-by: Christoph Hellwig <hch@xxxxxx>
> ---
> arch/hexagon/kernel/dma.c | 8 --------
> 1 file changed, 8 deletions(-)
>

Patch series looks good. Definitely appreciate the cleanup.

I can take it through my tree, or if not:

Acked-by: Richard Kuo <rkuo@xxxxxxxxxxxxxx>

Thanks!

--
Employee of Qualcomm Innovation Center, Inc.
Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project