> >> Signed-off-by: Pantelis Antoniou <panto@xxxxxxxxxxxxxxxxxxxxxxx> > >> Signed-off-by: Patil, Rachna <rachna@xxxxxx> > >> Signed-off-by: Felipe Balbi <balbi@xxxxxx> > >> [bigeasy: module alias, rename to ti,am3359-tscadc as it was tested on > >> AM3359] > > I honestly can't tell if this is a change from the last version of your > > patchset or a description of this patch changes in general. > > This is cluttering your commit logs, please remove this as well. > > I took the original patch. Every change I made to it because people > asked to merge changes into the patch where the problem occurred I > added it here before my sign-off. > > In the end I would like not to post a patch with "From: != me" and > don't make change which the original author did not do. Also dropping > their authorship isn't nice. What could we agree on? Generally speaking, if it is necessary to merge various author's patches into one, then you can the merger will tend to take authorship of the commit. Note that just because you are the author of the commit, it doesn't mean you authored the patch. I also use the rule of thumb that if you make significant changes to a patch, then you can also assume authorship too. I'll leave the 'how much is significant' to your own good judgement. If you're just making a few fixups, then just add your SOB in the normal way. That should be enough reward for a mere few patch fixes. Adding little 'I-did-this' notes to the commit log should mostly be avoided IMO. -- Lee Jones Linaro ST-Ericsson Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line "unsubscribe linux-iio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html