On Mon, Sep 27, 2010 at 10:59:22AM +0200, Linus Walleij wrote: > 2010/9/27 Ben Dooks <ben-i2c@xxxxxxxxx>: > > > On Thu, Sep 23, 2010 at 09:03:40AM +0200, Linus Walleij wrote: > >> This fixes some kerneldoc and assorted documenatation in the > >> Nomadik I2C driver without semantic impact. > >> > >> Acked-by: Srinidhi Kasagar <srinidhi.kasagar@xxxxxxxxxxxxxx> > >> Signed-off-by: Sundar R Iyer <sundar.iyer@xxxxxxxxxxxxxx> > >> Signed-off-by: Linus Walleij <linus.walleij@xxxxxxxxxxxxxx> > > > > You've correctly got yourself in the signed-off-by line, but why is there > > a reference to 'Sundar R Iye' in there? If this is the original patch > > authour then you need to have a From: line at the top as it looks like > > you (Linus Walleij) are re-sending the patch. > > Actually there are several original authors, we're squashing internal > company history into this patch set (with more on its way). The > changes are not split this way in the original git, they are split this > way because of your request to split the patch set. I would much prefer to see less squashing, it makes life difficult for the upstream. I'll accept your signed-off as a you have made sure that this is work that is yours to submit. I do not like having authours removed from patches, it is all too easy if you go about squashing two different authours work together and then not being able to re-split them if you have a problem. It means that if your tree is handling external patches, then you could end up with the situation where it is difficult or almost impossible to undo a change which the original sender either did not have the authority to send, or has something in that should have never been sent (i've seen several issues where companies have had desperate back-peddling because someone forgot to remove some $secret from the repository before allowing it to be uploaded somewhere). The other issue is that if you start squashing different fixes and updates together, it makes it difficult to go through examining the effects of a change. When these are merged upstream it is occasionally useful to be able to bisect these change in case one fix or change breaks something for another user. Basically, if possible, please avoid too much squashing and also avoid squashing two different authours of patches together. > Sundar is one author, I'm delivering so we're doing this with a > double-SoB, couldn't think of anything better. It would have been helpful to add a 'From:' line as well, git can be made to re-write the authour of a patch, but you'll have to search for it. Please find a method for future reference. > > I'll apply these once this is sorted out. > > OK I hope Sundar is happy with this, else he'll tell us. > > Yours, > Linus Walleij -- Ben Q: What's a light-year? A: One-third less calories than a regular year. -- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
