Re: [PATCH] regulator: tps65910: wire up sleep control configuration

From: Michał Mirosław
Date: Wed Jun 14 2017 - 14:55:09 EST


On Wed, Jun 14, 2017 at 06:15:18PM +0100, Mark Brown wrote:
> On Wed, Jun 14, 2017 at 05:13:29PM +0200, Michał Mirosław wrote:
> > On Wed, Jun 14, 2017 at 03:42:08PM +0100, Mark Brown wrote:
> > > On Wed, Jun 14, 2017 at 02:25:13PM +0200, Michał Mirosław wrote:
> > > > References: 201cf052810d20814a77ca0e0045a2c1a3508a1f
>
> > > I have no idea what you mean by this, sorry.
>
> > This is a commit id which introduced the feature:
>
> Please include human readable descriptions of things like commits and
> issues being discussed in e-mail in your mails, this makes them much
> easier for humans to read especially when they have no internet access.
> I do frequently catch up on my mail on flights or while otherwise
> travelling so this is even more pressing for me than just being about
> making things a bit easier to read.
>
> I also still don't entirely know what you're trying to tell me with
> this... as far as I can tell it's just noise?

As the patch only modifies device-tree binding, I included reference
to a patch that implements the feature. I guess this is not of much
use to you if you don't have access to the git tree while reviewing.

Since this is rather easy to find in git history, I will just remove the
References line.

Best Regards,
Michał Mirosław