On 06/01/15 06:02, Sudip Mukherjee wrote:
On Mon, Jan 05, 2015 at 08:24:06PM +0100, Piotr Kubus wrote:
On Sun, Jan 04, 2015 at 11:31:29PM +0100, Konrad Zapalowicz wrote:
On 01/04, Piotr Kubus wrote:
This is a patch to the dmm32at.c file that fixes up a code indent error found by the checkpatch.pl tool.
Nice however improve your commit message. The rule is that the lines
should wrap at 72nd column except for quoted material such as compiler
output, etc...
The 72-character columns are important for allowing quoting and they
play nicely with standard indentation from git log.
Hey,
I couldn't find that rule in Kernel documentation. Besides there were no
complaints from checkpatch.pl tool.
Please check SubmittingPatches in Documentation. It says:
'the "summary" must be no more than 70-75 characters'
In this case, it's not the "summary" that's at issue, it's the body of
the commit message, as Konrad mentioned above.
--
-=( Ian Abbott @ MEV Ltd. E-mail: <abbotti@xxxxxxxxx> )=-
-=( Web: http://www.mev.co.uk/ )=-
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
