Re: [PATCH v7 2/8] mfd: wm97xx-core: core support for wm97xx Codec
From: Mark Brown
Date: Wed Sep 20 2017 - 07:35:38 EST
On Wed, Sep 20, 2017 at 09:24:43AM +0100, Lee Jones wrote:
> On Tue, 19 Sep 2017, Mark Brown wrote:
> > 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
> 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.
Description: PGP signature