devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Lee Jones <lee@kernel.org>
To: Mark Brown <broonie@kernel.org>
Cc: Okan Sahin <okan.sahin@analog.com>,
	Rob Herring <robh+dt@kernel.org>,
	Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
	Liam Girdwood <lgirdwood@gmail.com>,
	Jonathan Cameron <jic23@kernel.org>,
	Lars-Peter Clausen <lars@metafoo.de>,
	Andy Shevchenko <andriy.shevchenko@linux.intel.com>,
	Cosmin Tanislav <demonsingur@gmail.com>,
	Stephen Boyd <sboyd@kernel.org>,
	Ulf Hansson <ulf.hansson@linaro.org>,
	Caleb Connolly <caleb.connolly@linaro.org>,
	Marcus Folkesson <marcus.folkesson@gmail.com>,
	Ramona Bolboaca <ramona.bolboaca@analog.com>,
	ChiYuan Huang <cy_huang@richtek.com>,
	Ibrahim Tilki <Ibrahim.Tilki@analog.com>,
	William Breathitt Gray <william.gray@linaro.org>,
	Arnd Bergmann <arnd@arndb.de>,
	Hugo Villeneuve <hvilleneuve@dimonoff.com>,
	ChiaEn Wu <chiaen_wu@richtek.com>,
	Haibo Chen <haibo.chen@nxp.com>,
	devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-iio@vger.kernel.org
Subject: Re: [PATCH v7 5/5] mfd: max77541: Add ADI MAX77541/MAX77540 PMIC Support
Date: Fri, 21 Apr 2023 08:39:38 +0100	[thread overview]
Message-ID: <20230421073938.GO996918@google.com> (raw)
In-Reply-To: <09eb8e4c-3e73-41f0-bf42-8ddf3c4254ec@sirena.org.uk>

On Thu, 20 Apr 2023, Mark Brown wrote:

> On Thu, Apr 20, 2023 at 11:34:38AM +0100, Lee Jones wrote:
> 
> > Once the regulator driver has been reviewed, I can take the set.
> 
> > Please apply this if you have to resubmit:
> 
> > For my own reference (apply this as-is to your sign-off block):
> 
> > Acked-for-MFD-by: Lee Jones <lee@kernel.org>
> 
> For situations like this where there's a depends on to the MFD it'd be
> great if you could just apply the MFD rather than waiting, the
> individual drivers can either get applied on top or just go via the
> subsystem and have everything sort itself out in the merge window.  It'd
> help things move along faster and be less confusing.
> 
> These serieses tend to get so many resends that I'm often just not
> looking at them, previously I'd have just applied the function driver
> when it's ready but with the complaints when the core ends up missing
> the merge window but function drivers are going in I stopped.  In the 
> past I've ended up missing things because either there's multiple
> serieses for similarly named devices out at once or (less often) some
> change results in a repeat review being needed so it's easier to just
> wait for things to settle down.

I'll try anything once!

Fair warning, I think this is going to massively complicate things. 

Either we're going to be left with a situation where child-driver
maintainers are scrabbling around looking for previous versions for the
MFD pull-request or contributors being forced to wait a full cycle for
their dependencies to arrive in the maintainer's base.

I'm not sure why simply providing your Ack when you're happy with the
driver and forgetting about the set until the pull-request arrives, like
we've been doing for nearly a decade now, isn't working for you anymore
but I'm mostly sure this method will be a regression.

-- 
Lee Jones [李琼斯]

  reply	other threads:[~2023-04-21  7:40 UTC|newest]

Thread overview: 40+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-04-12 11:12 [PATCH v7 0/5] Add MAX77541/MAX77540 PMIC Support Okan Sahin
2023-04-12 11:12 ` [PATCH v7 1/5] dt-bindings: regulator: max77541: Add ADI MAX77541/MAX77540 Regulator Okan Sahin
2023-06-28 14:17   ` Lee Jones
2023-04-12 11:12 ` [PATCH v7 2/5] regulator: max77541: Add ADI MAX77541/MAX77540 Regulator Support Okan Sahin
2023-04-21 12:41   ` Mark Brown
2023-06-28 14:17   ` Lee Jones
2023-04-12 11:12 ` [PATCH v7 3/5] iio: adc: max77541: Add ADI MAX77541 ADC Support Okan Sahin
2023-04-12 13:41   ` Andy Shevchenko
2023-06-28 14:18   ` Lee Jones
2023-04-12 11:12 ` [PATCH v7 4/5] dt-bindings: mfd: max77541: Add ADI MAX77541/MAX77540 Okan Sahin
2023-06-28 14:18   ` Lee Jones
2023-04-12 11:12 ` [PATCH v7 5/5] mfd: max77541: Add ADI MAX77541/MAX77540 PMIC Support Okan Sahin
2023-04-12 13:41   ` Andy Shevchenko
2023-04-20 10:34   ` Lee Jones
2023-04-20 16:52     ` Mark Brown
2023-04-21  7:39       ` Lee Jones [this message]
2023-04-21 13:33         ` Mark Brown
2023-06-13  7:34           ` Sahin, Okan
2023-06-21 17:13             ` Lee Jones
2023-06-26 17:54               ` Rob Herring
2023-06-27 13:56                 ` Lee Jones
2023-06-27 14:10                   ` Rob Herring
2023-06-27 14:32                     ` William Breathitt Gray
2023-06-27 16:33                       ` Lee Jones
2023-06-27 18:21                         ` Rob Herring
2023-06-28 13:40                           ` Lee Jones
2023-06-28 19:20                             ` Mark Brown
2023-06-29  7:25                               ` Lee Jones
2023-06-29 10:38                                 ` Mark Brown
2023-06-29 15:51                                   ` Lee Jones
2023-06-29 16:00                                     ` Mark Brown
2023-06-29 17:48                                       ` Rob Herring
2023-06-29 17:58                                         ` Mark Brown
2023-06-29 18:14                                           ` Rob Herring
2023-06-29 18:22                                             ` Mark Brown
2023-06-30  7:17                                               ` Lee Jones
2023-06-30 11:58                                                 ` Mark Brown
2023-06-27 14:39                     ` Mark Brown
2023-06-27 14:25                   ` Mark Brown
2023-06-21 17:14   ` Lee Jones

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20230421073938.GO996918@google.com \
    --to=lee@kernel.org \
    --cc=Ibrahim.Tilki@analog.com \
    --cc=andriy.shevchenko@linux.intel.com \
    --cc=arnd@arndb.de \
    --cc=broonie@kernel.org \
    --cc=caleb.connolly@linaro.org \
    --cc=chiaen_wu@richtek.com \
    --cc=cy_huang@richtek.com \
    --cc=demonsingur@gmail.com \
    --cc=devicetree@vger.kernel.org \
    --cc=haibo.chen@nxp.com \
    --cc=hvilleneuve@dimonoff.com \
    --cc=jic23@kernel.org \
    --cc=krzysztof.kozlowski+dt@linaro.org \
    --cc=lars@metafoo.de \
    --cc=lgirdwood@gmail.com \
    --cc=linux-iio@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=marcus.folkesson@gmail.com \
    --cc=okan.sahin@analog.com \
    --cc=ramona.bolboaca@analog.com \
    --cc=robh+dt@kernel.org \
    --cc=sboyd@kernel.org \
    --cc=ulf.hansson@linaro.org \
    --cc=william.gray@linaro.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).