Re: [PULL REQUEST] i2c for 4.11

From: Wolfram Sang
Date: Sat Mar 11 2017 - 11:10:15 EST


Linus,

> You added an extra revert, only to pull in the *exact* same patch. WTF?

The patch I reverted had a broken, typoed Signed-off. My rationale was
to have only a proper one, and not two different ones coming from
different trees. Looks like I misjudged, sorry.

> Please stop this kind of stupidity. Now I have that unnecessary revert
> with a misleading commit log in my tree, in addition to that stupid
> merge that has no explanation and just reinstates the exact same thing
> you reverted.

I see your point.

> And talking about that merge I will repeat this one more time: merges
> are commits too. You had damn well add explanations for what a merge
> merges and why it is done.

Understood. Peter joined as co-maintainer kind of recently, and I guess
it shows that I didn't have to pass on merges a lot so far. It is not
that I ignored your preferences for merges (which make total sense to
me), I just missed to check how to properly do it. Will improve!

Regards,

Wolfram

Attachment: signature.asc
Description: PGP signature