Messages in this thread | | | Date | Thu, 11 Feb 2010 15:36:12 -0800 | From | Greg KH <> | Subject | Re: [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
| |