Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:5 as permitted sender) client-ip=2620:137:e000::3:5;
Feedback-ID: i56a14606:Fastmail
User-Agent: Cyrus-JMAP/3.9.0-alpha0-957-ga1ccdb4cff-fm-20230919.001-ga1ccdb4c
MIME-Version: 1.0
Message-Id: <1b2fddf8-c0a6-4afa-8ad0-f280dea1607f@app.fastmail.com>
In-Reply-To: <CACPK8XeROYz_XaB3TvUhdXm7Vm8fjC8yU+mfvA58=_FiDrBy-g@mail.gmail.com>
References: <20230925095532.1984344-1-u.kleine-koenig@pengutronix.de>
 <CACPK8XeROYz_XaB3TvUhdXm7Vm8fjC8yU+mfvA58=_FiDrBy-g@mail.gmail.com>
Date:   Wed, 27 Sep 2023 10:43:16 +0200
From:   "Arnd Bergmann" <arnd@arndb.de>
To:     "Joel Stanley" <joel@jms.id.au>,
        =?UTF-8?Q?Uwe_Kleine-K=C3=B6nig?= <u.kleine-koenig@pengutronix.de>
Cc:     "Leo Li" <leoyang.li@nxp.com>,
        "Herve Codina" <herve.codina@bootlin.com>,
        "Qiang Zhao" <qiang.zhao@nxp.com>,
        "Hitomi Hasegawa" <hasegawa-hitomi@fujitsu.com>,
        "Huisong Li" <lihuisong@huawei.com>,
        "Krzysztof Halasa" <khalasa@piap.pl>,
        "Karol Gugala" <kgugala@antmicro.com>,
        "Mateusz Holenko" <mholenko@antmicro.com>,
        "Gabriel Somlo" <gsomlo@gmail.com>,
        "Yinbo Zhu" <zhuyinbo@loongson.cn>,
        "Matthias Brugger" <matthias.bgg@gmail.com>,
        "Conor.Dooley" <conor.dooley@microchip.com>,
        "Daire McNamara" <daire.mcnamara@microchip.com>,
        "Krzysztof Kozlowski" <krzysztof.kozlowski@linaro.org>,
        "Andy Gross" <agross@kernel.org>,
        "Bjorn Andersson" <andersson@kernel.org>,
        "Konrad Dybcio" <konrad.dybcio@linaro.org>,
        =?UTF-8?Q?Heiko_St=C3=BCbner?= <heiko@sntech.de>,
        "Thierry Reding" <thierry.reding@gmail.com>,
        "Jon Hunter" <jonathanh@nvidia.com>,
        "Sumit Gupta" <sumitg@nvidia.com>,
        "Shang XiaoJing" <shangxiaojing@huawei.com>,
        "Muhammad Usama Anjum" <usama.anjum@collabora.com>,
        "Rob Herring" <robh@kernel.org>, "Nishanth Menon" <nm@ti.com>,
        "Santosh Shilimkar" <ssantosh@kernel.org>,
        "Michal Simek" <michal.simek@amd.com>,
        "Andrew Jeffery" <andrew@aj.id.au>,
        linux-arm-kernel@lists.infradead.org,
        linux-aspeed@lists.ozlabs.org, linux-kernel@vger.kernel.org,
        "Zev Weiss" <zev@bewilderbeest.net>, linuxppc-dev@lists.ozlabs.org,
        loongarch@lists.linux.dev,
        "AngeloGioacchino Del Regno" 
        <angelogioacchino.delregno@collabora.com>,
        linux-mediatek@lists.infradead.org,
        linux-riscv@lists.infradead.org,
        "zhang.songyi" <zhang.songyi@zte.com.cn>,
        "Lubomir Rintel" <lkundrak@v3.sk>, linux-arm-msm@vger.kernel.org,
        linux-rockchip@lists.infradead.org,
        "Alim Akhtar" <alim.akhtar@samsung.com>,
        linux-samsung-soc@vger.kernel.org,
        "Nick Alcock" <nick.alcock@oracle.com>,
        "Yang Yingliang" <yangyingliang@huawei.com>,
        linux-tegra@vger.kernel.org, linux-pm@vger.kernel.org,
        "Ruan Jinjie" <ruanjinjie@huawei.com>,
        "Pengutronix Kernel Team" <kernel@pengutronix.de>
Subject: Re: [PATCH 00/40] soc: Convert to platform remove callback returning void
Content-Type: text/plain;charset=utf-8
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

On Wed, Sep 27, 2023, at 04:25, Joel Stanley wrote:
> On Mon, 25 Sept 2023 at 09:55, Uwe Kleine-K=C3=B6nig <u.kleine-koenig@=
pengutronix.de> wrote:
>>
>> this series converts all platform drivers below drivers/soc to use
>> .remove_new(). The motivation is to get rid of an integer return code
>> that is (mostly) ignored by the platform driver core and error prone =
on
>> the driver side.
>>
>> See commit 5c5a7680e67b ("platform: Provide a remove callback that
>> returns no value") for an extended explanation and the eventual goal.
>>
>> As there is no single maintainer team for drivers/soc, I suggest the
>> individual maintainers to pick up "their" patches.
>
> I'd be happy if Arnd merged the lot at once. Arnd, what do you think?
>
> If that will be too messy then I understand. I have queued the aspeed
> ones locally and will push that out if we decide that's the best way
> to go.

The main downside of merging it all at once through the soc tree
is that there may be patches that conflict with other work going on
in individual drivers.

What I'd suggest doing here is:

- have platform maintainers pick up patches for their drivers
  if that is their preference for any reason

- get a pull request from Uwe for the soc tree for anything that has
  not been picked up in one or two weeks from now

      Arnd