On Tue, 27 Mar 2012, Marek Szyprowski wrote: > From: Andrzej Pietrasiewicz <andrzej.p@xxxxxxxxxxx> > -static inline void *dma_alloc_coherent(struct device *dev, size_t size, > - dma_addr_t *daddr, gfp_t gfp) > +#define dma_alloc_coherent(d,s,h,f) dma_alloc_attrs(d,s,h,f,NULL) Please make this an inline function. > > +#define dma_alloc_coherent(d,s,h,f) dma_alloc_attrs(d,s,h,f,NULL) Inline please. > -static inline void dma_free_coherent(struct device *dev, size_t size, > - void *vaddr, dma_addr_t bus) > +#define dma_free_coherent(d,s,c,h) dma_free_attrs(d,s,c,h,NULL) Ditto Reviewed-by: Thomas Gleixner <tglx@xxxxxxxxxxxxx> Thanks, tglx