[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20160330170053.GK2350@sirena.org.uk>
Date: Wed, 30 Mar 2016 10:00:53 -0700
From: Mark Brown <broonie@...nel.org>
To: Jon Hunter <jonathanh@...dia.com>
Cc: Liam Girdwood <lgirdwood@...il.com>,
Javier Martinez Canillas <javier@....samsung.com>,
linux-kernel@...r.kernel.org, linux-tegra@...r.kernel.org
Subject: Re: [PATCH] regulator: Fix deadlock during regulator registration
On Wed, Mar 30, 2016 at 05:46:05PM +0100, Jon Hunter wrote:
Please fix your mail client to word wrap within paragraphs at something
substantially less than 80 columns. Doing this makes your messages much
easier to read and reply to.
> > Please don't paste entire backlogs into changelogs, they're *enormous*,
> > mostly noise and obscure any actual content that's in there through
> > denial of service. If they're useful then include edited subsets that
> > highlight the relevant sections of the backtrace so your changelog is
> > more legible.
> Ok, no problem. Are you happy with the patch otherwise? If so, do you
> want me to resend or do you wish to trim the backlog? I think that
I haven't reviwed it yet.
> this part is interesting ...
I'm really getting nothing at all from this, all it really does is point
to the call site but that's already much more clearly identified in the
changelog. We certainly don't need to know which driver called this.
Download attachment "signature.asc" of type "application/pgp-signature" (474 bytes)
Powered by blists - more mailing lists