Re: [PATCH] Revert "regulator: core: Add support for active-discharge configuration"

[Mark Brown]

On Fri, Mar 25, 2016 at 07:06:21PM +0200, Vladimir Zapolskiy wrote:
> On 25.03.2016 17:45, Mark Brown wrote:

> > Nope, they're patches like other patches.  A big part of what the
> > subject line does is help people work out what they need to look at,
> > something that doesn't pattern match doesn't fulfil that need.

> regarding pattern matching just remove ^ from the regexp and people
> be aware of this "regulator: core" change.

The pattern matching in this case is typically being done by human
beings.

> Please look at 1 hour old 1701f680407c ("Revert "ppdev: use new parport
> device model" ") --- that's IMHO the proper way to create and apply
> reverts.

The fact that a change happens to have been generated with a revert has
no technical impact, you're not communicating with machines here but
rather with people.  Yes, some people don't mind especially if they're
just directly publishing the commit (where it matters much less) but
that doesn't override all other considerations.

> And I disagree with the modified subject and commit message done by you
> and without a notice mentioning this your change in the commit message.

I already published it so it's a bit late now.

Attachment: signature.asc
Description: PGP signature