Re: [PATCH] staging:vt6655: fix coding style violations for lines over 80 characters

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

 



On Fri, Mar 11, 2016 at 01:13:58PM -0800, Kathryn Hampton wrote:

Please turn off html email, as the mail servers will reject it.

> Hi, I'm still working through the steps to get this right. The first mail was
> my first ever attempt at sending to this list, doing a patch, getting mail
> working on my system and a lot more. It didn't look like anything was sent so I
> tried again, resending to myself. I saw a complaint about the length of my
> commit description (76) so I amended that and sent out the second time using
> your example with git sendmail (thinking the first one with Mutt never went
> out). The code and patch were unchanged, and the second email was the correct
> one. (I know how not to do duplicates in the future now).
> 
> Can you use the most recent mail I sent?

I don't know what "recent mail" you are referring to here, sorry.  I get
a lot of patches to review, and can't remember any of them.

> If not, can someone please tell me the
> correct protocol for sending out the same file? Or do I need to create another
> patch file to send? Is this where RESEND should get added into the subject?
> Should I amend the commit to change the subject wording a bit? Please let me
> know what works the best for you. Thanks!

Add a 'version' to the subject when resending the patch, and describe
what you did differently.  The file, Documentation/SubmittingPatches in
the kernel source tree should explain this pretty well.

thanks,

greg k-h
_______________________________________________
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