On Wed, Nov 30, 2011 at 02:29:11PM -0800, Andrew Morton wrote: > On Mon, 28 Nov 2011 14:36:59 -0800 > "Darrick J. Wong" <djw...@us.ibm.com> wrote: > > > This patchset (re)uses Bob Pearson's crc32 slice-by-8 code to stamp out a > > software crc32c implementation. > > I think the attributions here are all messed up. As the patches stand, > it appears that you wrote all of them. But I don't think that is the > case. > > If Bob wrote a particular patch then that patch should be sent with a > From: Bob Pearson <rpear...@systemfabricworks.com> right at the start > of the changelog so that he is recorded as the primary author. If the > email sender (ie: you) was the primary author then this attribution can > be omitted and we fall back to using the From: from the email headers. > > Also, every one of these patches should have you own signed-off-by, > regardless of its authorship. For reasons explained in > Documentation/SubmittingPatches, section 12. > > Please fix these things up and resend. > > Also, it would be conventional and useful if each patch title was > prefixed by its subsystem identifier. ie, "removed two instances of > trailing whitespaces" should be titled "crc32: remove two instances of > trailing whitespace" or "lib/crc32.c: remove two instances of trailing > whitespaces".
Okay, I'll massage the changelogs to give them more descriptive subjects, and fix the attribution chain. Thank you for the feedback. --D > > -- > To unsubscribe from this list: send the line "unsubscribe linux-ext4" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe linux-crypto" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
