Re: [PATCH 2/2] Revert "ASoC: ux500: drop platform DAI assignments"
From: Mark Brown
Date: Tue Jul 18 2017 - 08:59:40 EST
On Tue, Jul 18, 2017 at 12:36:29PM +0200, Johan Hovold wrote:
> On Tue, Jul 18, 2017 at 11:06:55AM +0100, Mark Brown wrote:
> > Reverts shouldn't be special - they're just regular patches and should
> > have sensible changelogs like any others.
> Stating that you're reverting a commit and which commit that is is in
> the summary is arguable sensible (of course, you still also need further
> details in the commit message body itself describing why it was needed).
> Check the logs and you'll see that we have a ton of "Revert <reverted
> commit summary>" for various subsystems. In fact, it seems to be by far
> the most common summary for direct reverts.
The easily findable ones are, and it doesn't mean it's good practice -
reverts seem to attract particularly bad commit messages in general, not
just the subject lines, and I happen to have a pre-canned response for
this so...
Attachment:
signature.asc
Description: PGP signature