Re: [PATCH RESEND 04/16] staging: rtl8188eu: remove unused macro GEN_DRV_CMD_HANDLER

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Dear Larry,

Larry Finger wrote:
On 08/28/2015 03:45 PM, Luca Ceresoli wrote:
Signed-off-by: Luca Ceresoli <luca@xxxxxxxxxxxxxxxx>
Cc: Larry Finger <Larry.Finger@xxxxxxxxxxxx>
Cc: Greg Kroah-Hartman <gregkh@xxxxxxxxx>

Empty commit messages are generally not accepted.

Meaning something like this?

  commit 96ac283d6490932ff5097259c18a4d03bdddb010
  Author: Chaehyun Lim <chaehyun.lim@xxxxxxxxx>
  Date:   Tue Aug 11 10:32:39 2015 +0900

      staging: wilc1000: wilc_memory.h: remove unused define

      Remove unused define macro that is never used.

      Signed-off-by: Chaehyun Lim <chaehyun.lim@xxxxxxxxx>
      Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>

There are several examples like this in the kernel git history.

Fine, I'll expand my commit messages accordingly.

But I don't see what added value this brings to the commit title.
I really would like to understand why this is a good practice. Can you
point me to an explanation (links are welcome)?

Thanks.

--
Luca
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel



[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux