Thread (24 messages) 24 messages, 3 authors, 2017-09-20

Re: [PATCH v7 2/8] mfd: wm97xx-core: core support for wm97xx Codec

flat view

From: Mark Brown <broonie@kernel.org>
Date: 2017-09-20 11:35:34
Also in: alsa-devel, linux-arm-kernel, lkml

On Wed, Sep 20, 2017 at 09:24:43AM +0100, Lee Jones wrote:
On Tue, 19 Sep 2017, Mark Brown wrote:
quoted
It really helps if we can get the earlier parts of the series moving
even if the rest of it's in review still - I know I back off on
reviewing things if it looks like their dependencies aren't making
progress.
Just for your FYI, I tend to review my parts then wait for the
rest of the series to be reviewed, then discuss how the set should be
handled.
Holding back on reviewing your parts is not a good idea.  If everyone
did that we'd end up in a circular Maintainer dependency and nothing
would ever get reviewed.
I get fed up with having to re-review the same patches over and over and
over again while people work out what's going on with the earlier parts
of the series, that gets very time consuming and is error prone.  If
we're getting churn early on in the series that's causing things to
change then it's better to let that settle down and look properly at the
results than glaze over looking at each minor change as it comes in and
miss other things that get tweaked.  Once the dependencies are stable
then it becomes worthwhile again.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help