lkml.org 
[lkml]   [2010]   [Feb]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 1/1] Staging: hv: Add proper versioning to HV drivers-Corrected.
Minor nits below:

On Thu, Feb 11, 2010 at 11:02:42PM +0000, Hank Janssen wrote:
>
> Provide proper versioning information for all HV drivers.
> With removal of build time/date/and Minor number as requested
> By Greg KH
>
> Cc: Greg Kroah-Hartman <gregkh@suse.de>
> Signed-off-by: Hank Janssen <hjanssen@microsoft.com>
> Signed-off-by: Haiyang Zhang <haiyang@microsoft.com>

Don't line up the <email@domain>, just put only one space, there are a
few hundred thousand examples to look at for the correct formatting :)

>
> drivers/staging/hv/VersionInfo.h | 22 +++++++++++++++++++---

You forgot a "---" between the signed-off-by and the diffstat. If you
use either git or quilt, it will get this correct automagically. That
infers you used something else, which is quite scary. What are you
using instead to create patches?

Don't worry, I can edit this by hand when I apply it, and will do so in
a few days when I get to my "to-apply" queue, but in the future, please
don't do this.

thanks,

greg k-h


\
 
 \ /
  Last update: 2010-02-12 00:41    [W:1.405 / U:0.316 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site