Thread (22 messages) 22 messages, 3 authors, 2014-04-18

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

From: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Date: 2014-04-14 13:42:23
Also in: lkml

On Fri, 2014-04-11 at 16:33 +0800, Hongbo Zhang wrote:
quoted
quoted
quoted
+         * 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 [off-list ref]
Intel Finland Oy
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help