On Mon, Aug 17, 2020 at 05:27:33PM +0800, Bingbu Cao wrote: > On 8/15/20 12:30 AM, Andy Shevchenko wrote: > > We may use special helper macro to poll IO till condition or timeout occurs. > > + ret = readl_poll_timeout(dma, value, value & CIO2_CDMAC0_DMA_HALTED, 4000, 2000000); > > This line is too long, need a break, others look good for me. checkpatch doesn't complain, but if you insist, I'll split it in v2. -- With Best Regards, Andy Shevchenko