From: "Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>
To: Arnd Bergmann <arnd@arndb.de>, soc@kernel.org
Cc: Geert Uytterhoeven <geert+renesas@glider.be>,
Tony Lindgren <tony@atomide.com>, Yuan Can <yuancan@huawei.com>,
Thierry Reding <thierry.reding@gmail.com>,
Rob Herring <robh@kernel.org>,
Samuel Holland <samuel@sholland.org>,
Jernej Skrabec <jernej.skrabec@gmail.com>,
Jon Hunter <jonathanh@nvidia.com>, Chen-Yu Tsai <wens@csie.org>,
Andy Gross <agross@kernel.org>,
linux-sunxi@lists.linux.dev, Nick Alcock <nick.alcock@oracle.com>,
Yangtao Li <frank.li@vivo.com>, Liu Ying <victor.liu@nxp.com>,
linux-arm-msm@vger.kernel.org, Jay Fang <f.fangjian@huawei.com>,
Hans de Goede <hdegoede@redhat.com>,
linux-tegra@vger.kernel.org,
Linux-OMAP <linux-omap@vger.kernel.org>,
linux-arm-kernel@lists.infradead.org,
Laurentiu Tudor <laurentiu.tudor@nxp.com>,
"Sicelo A. Mhlongo" <absicsz@gmail.com>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Bjorn Andersson <andersson@kernel.org>,
linux-kernel@vger.kernel.org,
Konrad Dybcio <konrad.dybcio@linaro.org>,
Luis Chamberlain <mcgrof@kernel.org>,
Pengutronix Kernel Team <kernel@pengutronix.de>
Subject: [PULL v2] bus: Convert to platform remove callback returning void
Date: Tue, 28 Nov 2023 18:49:27 +0100 [thread overview]
Message-ID: <20231128174927.m46dgp4juig2omci@pengutronix.de> (raw)
In-Reply-To: <20231116115307.32rovgcej2s5pe4r@pengutronix.de>
[-- Attachment #1: Type: text/plain, Size: 4392 bytes --]
Hello Arnd,
On Thu, Nov 16, 2023 at 12:53:07PM +0100, Uwe Kleine-König wrote:
> On Fri, Nov 10, 2023 at 07:32:01AM +0100, Arnd Bergmann wrote:
> > On Thu, Nov 9, 2023, at 21:28, Uwe Kleine-König wrote:
> > > this series converts all drivers below drivers/bus to struct
> > > platform_driver::remove_new(). See commit 5c5a7680e67b ("platform:
> > > Provide a remove callback that returns no value") for an extended
> > > explanation and the eventual goal.
> > >
> > > drivers/bus/fsl-mc was already addressed earlier with a separate
> > > series[1].
> > >
> > > All conversations are trivial, because all .remove() callbacks returned
> > > zero unconditionally.
> > >
> > > Some of the drivers touched here don't have a maintainer and there is no
> > > maintainer for all of drivers/bus. It would be great if someone could pick up
> > > the whole series, maybe Arnd might do that?
> >
> > Sure, please send a pull request to soc@kernel.org if there
> > are no further comments.
>
> here it comes:
>
> The following changes since commit b85ea95d086471afb4ad062012a4d73cd328fa86:
>
> Linux 6.7-rc1 (2023-11-12 16:19:07 -0800)
Here comes an updated PR. The only change is that it also includes two
Reviewed-by tags for Jernej Skrabec that came in after I sent the PR.
Obviously that changed the commit hash which made it necessary to redo
the PR. I hope it doesn't disturb your tooling that there are now two
similar pull requests.
Best regards
Uwe
The following changes since commit b85ea95d086471afb4ad062012a4d73cd328fa86:
Linux 6.7-rc1 (2023-11-12 16:19:07 -0800)
are available in the Git repository at:
https://git.pengutronix.de/git/ukl/linux tags/bus-platform-remove-void
for you to fetch changes up to fc540426f7baa0c7d4b477e80435d075659092a2:
bus: ts-nbus: Convert to platform remove callback returning void (2023-11-28 18:43:27 +0100)
----------------------------------------------------------------
This series converts all drivers below drivers/bus to struct
platform_driver::remove_new(). See commit 5c5a7680e67b ("platform:
Provide a remove callback that returns no value") for an extended
explanation and the eventual goal.
After the initial simplification in commit 864acca58000 ("bus: fsl-mc:
Drop if block with always false condition") all conversations are
trivial because the remove callbacks all return zero unconditionally.
----------------------------------------------------------------
Uwe Kleine-König (14):
bus: fsl-mc: Drop if block with always false condition
bus: fsl-mc: Convert to platform remove callback returning void
bus: hisi_lpc: Convert to platform remove callback returning void
bus: omap-ocp2scp: Convert to platform remove callback returning void
bus: omap_l3_smx: Convert to platform remove callback returning void
bus: qcom-ssc-block-bus: Convert to platform remove callback returning void
bus: simple-pm-bus: Convert to platform remove callback returning void
bus: sun50i-de2: Convert to platform remove callback returning void
bus: sunxi-rsb: Convert to platform remove callback returning void
bus: tegra-aconnect: Convert to platform remove callback returning void
bus: tegra-gmi: Convert to platform remove callback returning void
bus: ti-pwmss: Convert to platform remove callback returning void
bus: ti-sysc: Convert to platform remove callback returning void
bus: ts-nbus: Convert to platform remove callback returning void
drivers/bus/fsl-mc/fsl-mc-bus.c | 16 +++-------------
drivers/bus/hisi_lpc.c | 6 ++----
drivers/bus/omap-ocp2scp.c | 6 ++----
drivers/bus/omap_l3_smx.c | 6 ++----
drivers/bus/qcom-ssc-block-bus.c | 6 ++----
drivers/bus/simple-pm-bus.c | 7 +++----
drivers/bus/sun50i-de2.c | 5 ++---
drivers/bus/sunxi-rsb.c | 6 ++----
drivers/bus/tegra-aconnect.c | 6 ++----
drivers/bus/tegra-gmi.c | 6 ++----
drivers/bus/ti-pwmss.c | 5 ++---
drivers/bus/ti-sysc.c | 6 ++----
drivers/bus/ts-nbus.c | 6 ++----
13 files changed, 28 insertions(+), 59 deletions(-)
--
Pengutronix e.K. | Uwe Kleine-König |
Industrial Linux Solutions | https://www.pengutronix.de/ |
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
prev parent reply other threads:[~2023-11-28 17:50 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-11-09 20:28 [PATCH 00/12] bus: Convert to platform remove callback returning void Uwe Kleine-König
2023-11-09 20:28 ` [PATCH 11/12] bus: ti-sysc: " Uwe Kleine-König
2023-11-10 6:59 ` Tony Lindgren
2023-11-10 6:32 ` [PATCH 00/12] bus: " Arnd Bergmann
2023-11-16 11:53 ` [PULL] " Uwe Kleine-König
2023-11-28 17:49 ` Uwe Kleine-König [this message]
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=20231128174927.m46dgp4juig2omci@pengutronix.de \
--to=u.kleine-koenig@pengutronix.de \
--cc=absicsz@gmail.com \
--cc=agross@kernel.org \
--cc=andersson@kernel.org \
--cc=arnd@arndb.de \
--cc=f.fangjian@huawei.com \
--cc=frank.li@vivo.com \
--cc=geert+renesas@glider.be \
--cc=gregkh@linuxfoundation.org \
--cc=hdegoede@redhat.com \
--cc=jernej.skrabec@gmail.com \
--cc=jonathanh@nvidia.com \
--cc=kernel@pengutronix.de \
--cc=konrad.dybcio@linaro.org \
--cc=laurentiu.tudor@nxp.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-omap@vger.kernel.org \
--cc=linux-sunxi@lists.linux.dev \
--cc=linux-tegra@vger.kernel.org \
--cc=mcgrof@kernel.org \
--cc=nick.alcock@oracle.com \
--cc=robh@kernel.org \
--cc=samuel@sholland.org \
--cc=soc@kernel.org \
--cc=thierry.reding@gmail.com \
--cc=tony@atomide.com \
--cc=victor.liu@nxp.com \
--cc=wens@csie.org \
--cc=yuancan@huawei.com \
/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 read-only IMAP folder(s) and NNTP newsgroup(s).