Re: [PATCH 1/1] Staging: hv: Add proper versioning to HVdrivers-Corrected.

[Greg KH]

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@xxxxxxx>
> Signed-off-by: Hank Janssen    <hjanssen@xxxxxxxxxxxxx>
> Signed-off-by: Haiyang Zhang   <haiyang@xxxxxxxxxxxxx>

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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/