[PATCH] arm64: Change misleading function names in dma-mapping

Catalin Marinas catalin.marinas at arm.com
Thu Feb 6 05:29:09 EST 2014


On Thu, Feb 06, 2014 at 10:09:09AM +0000, Will Deacon wrote:
> On Thu, Feb 06, 2014 at 06:28:22AM +0000, Ritesh Harjani wrote:
> > 
> > From: Ritesh Harjani <ritesh.harjani at gmail.com<mailto:ritesh.harjani at gmail.com>>
> > 
> > arm64_swiotlb_alloc/free_coherent name can be misleading
> > somtimes with CMA support being enabled after this
> > patch (c2104debc235b745265b64d610237a6833fd53)
> > 
> > Change this name to be more generic:
> > arm64_dma_alloc/free_coherent
> 
> I'm fine with this cosmetic change, but the (pre-existing) arm64 prefix looks
> a little odd for a static function.

Good point. I'm happy with a simpler __dma_alloc_coherent().

-- 
Catalin



More information about the linux-arm-kernel mailing list