[PATCH v3 6/8] DMA: Freescale: change descriptor release process for supporting async_tx

Andy Shevchenko andriy.shevchenko at linux.intel.com
Mon Apr 14 23:41:57 EST 2014

On Fri, 2014-04-11 at 16:33 +0800, Hongbo Zhang wrote:

> >>> +         * hardware channel, subsequent descriptors are either in
> >>> +         * process or have not been submitted
> >> Dot at the eol. Check in all comments.
> >
> > Even though I saw there are other comments without the dots, I think 
> > it is better to have it.
> > Thanks, all.
> >
> Hmm... think it again, it it really necessary to have it?
> Even I have it in my patch, there are already so many comments exists 
> without it.

For my opinion is better to keep style in your patches. Better if it
commonly used style in the driver. But comment against comments is
really minor thing.

Andy Shevchenko <andriy.shevchenko at linux.intel.com>
Intel Finland Oy

More information about the Linuxppc-dev mailing list