Hi Leon, On 17.01.2017 12:24, Leon Romanovsky wrote: > On Tue, Jan 17, 2017 at 10:11:12AM +0100, Jack Wang wrote: >> >> >> As Jason suggested, we have 4 elements for per port arrays, >> it's better to have a separate structure to represent them. >> >> It simplifies code a bit, ~ 30 lines of code less :) > > The commit message should be descriptive. > You really NEED to read SubmittingPatches before sending patches. > http://lxr.free-electrons.com/source/Documentation/SubmittingPatches#L106 Okay, I can improve the commit message. > > 106 2) Describe your changes > 107 ------------------------ > 108 > 109 Describe your problem. Whether your patch is a one-line bug fix or > 110 5000 lines of a new feature, there must be an underlying problem that > 111 motivated you to do this work. Convince the reviewer that there is a > 112 problem worth fixing and that it makes sense for them to read past the > 113 first paragraph. > > And if I read email headers correctly, you still didn't use "git send-email" to send the patch. Yes, you're right. But I thought I can have my own preference, or only git send-email is allowed? Thanks, > > Thanks > -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html