On Sat, Feb 19, 2022 at 08:52:08AM +0800, Baoquan He wrote: > dma_alloc_coherent() allocates dma buffer with device's addressing > limitation in mind. It's redundent to specify GFP_DMA when calling > dma_alloc_coherent(). replace it with GFP_KERNEL. Plase avoid the overly long line. The rest looks good.