Re: [PATCH v3 15/24] media: exynos4-is: Remove call to memset after dma_alloc_coherent

2019-07-15 Thread Sylwester Nawrocki
On 7/15/19 05:18, Fuqian Huang wrote: > In commit 518a2f1925c3 > ("dma-mapping: zero memory returned from dma_alloc_*"), > dma_alloc_coherent has already zeroed the memory. > So memset is not needed > > Signed-off-by: Fuqian Huang Reviewed-by: Sylwester Nawrocki

Re: [PATCH v3 15/24] media: exynos4-is: Remove call to memset after dma_alloc_coherent

2019-07-15 Thread Sylwester Nawrocki
On 7/15/19 11:43, Fuqian Huang wrote: > Should I rewrite the commit log? Just mention that dma_alloc_coherent > has already zeroed the memory and not to reference the commit? I don't think it is really needed, since at hash 518a2f1925c3 dma_alloc_coherent() already zeroes the memory, so in fact

Re: [PATCH v3 15/24] media: exynos4-is: Remove call to memset after dma_alloc_coherent

2019-07-15 Thread Fuqian Huang
Sylwester Nawrocki 於 2019年7月15日週一 下午5:00寫道: > > On 7/15/19 05:18, Fuqian Huang wrote: > > In commit 518a2f1925c3 > > ("dma-mapping: zero memory returned from dma_alloc_*"), > > dma_alloc_coherent has already zeroed the memory. > > So memset is not needed > > I can't see any changes in the above

Re: [PATCH v3 15/24] media: exynos4-is: Remove call to memset after dma_alloc_coherent

2019-07-15 Thread Sylwester Nawrocki
On 7/15/19 05:18, Fuqian Huang wrote: > In commit 518a2f1925c3 > ("dma-mapping: zero memory returned from dma_alloc_*"), > dma_alloc_coherent has already zeroed the memory. > So memset is not needed I can't see any changes in the above mentioned commit that would make dma_alloc_coherent()

[PATCH v3 15/24] media: exynos4-is: Remove call to memset after dma_alloc_coherent

2019-07-14 Thread Fuqian Huang
In commit 518a2f1925c3 ("dma-mapping: zero memory returned from dma_alloc_*"), dma_alloc_coherent has already zeroed the memory. So memset is not needed Signed-off-by: Fuqian Huang --- Changes in v3: - Use actual commit rather than the merge commit in the commit message