From: "Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>
To: Borislav Petkov <bp@alien8.de>, Tony Luck <tony.luck@intel.com>
Cc: Dinh Nguyen <dinguyen@kernel.org>,
James Morse <james.morse@arm.com>,
Mauro Carvalho Chehab <mchehab@kernel.org>,
Robert Richter <rric@kernel.org>,
linux-edac@vger.kernel.org, kernel@pengutronix.de,
Jan Luebbe <jlu@pengutronix.de>,
Stefan Schaeckeler <sschaeck@cisco.com>,
Joel Stanley <joel@jms.id.au>, Andrew Jeffery <andrew@aj.id.au>,
linux-arm-kernel@lists.infradead.org,
linux-aspeed@lists.ozlabs.org,
Shravan Kumar Ramani <shravankr@nvidia.com>,
Lei Wang <lewan@microsoft.com>,
Andre Przywara <andre.przywara@arm.com>,
Johannes Thumshirn <morbidrsa@gmail.com>,
Avi Fishman <avifishman70@gmail.com>,
Tomer Maimon <tmaimon77@gmail.com>,
Tali Perry <tali.perry1@gmail.com>,
Patrick Venture <venture@google.com>,
Nancy Yuen <yuenn@google.com>,
Benjamin Fair <benjaminfair@google.com>,
Marvin Lin <kflin@nuvoton.com>, Stanley Chu <yschu@nuvoton.com>,
openbmc@lists.ozlabs.org, Ralf Baechle <ralf@linux-mips.org>,
linux-mips@vger.kernel.org,
Manivannan Sadhasivam <mani@kernel.org>,
Andy Gross <agross@kernel.org>,
Bjorn Andersson <andersson@kernel.org>,
Konrad Dybcio <konrad.dybcio@linaro.org>,
linux-arm-msm@vger.kernel.org,
Michal Simek <michal.simek@amd.com>,
Tero Kristo <kristo@kernel.org>,
Khuong Dinh <khuong@os.amperecomputing.com>,
Shubhrajyoti Datta <shubhrajyoti.datta@amd.com>,
Sai Krishna Potthuri <sai.krishna.potthuri@amd.com>
Subject: [PATCH 00/21] EDAC: Convert to platform remove callback returning void
Date: Wed, 4 Oct 2023 15:12:33 +0200 [thread overview]
Message-ID: <20231004131254.2673842-1-u.kleine-koenig@pengutronix.de> (raw)
Hello,
this series converts all platform drivers below drivers/edac 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. However none of the edac drivers suffered from the easy
to make bug, so all drivers are converted in a trivial way.
See commit 5c5a7680e67b ("platform: Provide a remove callback that
returns no value") for an extended explanation and the eventual goal.
The patch for npcm was already sent back in June
(https://lore.kernel.org/linux-edac/20230628071354.665300-1-u.kleine-koenig@pengutronix.de)
but didn't result in enthusiastic review comments and it wasn't picked
up.
There are no interdependencies between the patches. As there are still
quite a few drivers to convert, I'm happy about every patch that makes
it in. So even if there is a merge conflict with one patch until you
apply, please apply the remainder of this series anyhow. I'll come back
to the part that you (maybe) skipped at a later point.
Best regards
Uwe
Uwe Kleine-König (21):
EDAC/altera: Convert to platform remove callback returning void
EDAC/armada_xp: Convert to platform remove callback returning void
EDAC/aspeed: Convert to platform remove callback returning void
EDAC/bluefield: Convert to platform remove callback returning void
EDAC/cell: Convert to platform remove callback returning void
EDAC/cpc925: Convert to platform remove callback returning void
EDAC/dmc520: Convert to platform remove callback returning void
EDAC/highbank_l2: Convert to platform remove callback returning void
EDAC/highbank_mc: Convert to platform remove callback returning void
EDAC/mpc85xx: Convert to platform remove callback returning void
EDAC/npcm: Convert to platform remove callback returning void
EDAC/octeon-l2c: Convert to platform remove callback returning void
EDAC/octeon-lmc: Convert to platform remove callback returning void
EDAC/octeon-pc: Convert to platform remove callback returning void
EDAC/octeon-pci: Convert to platform remove callback returning void
EDAC/ppc4xx: Convert to platform remove callback returning void
EDAC/qcom: Convert to platform remove callback returning void
EDAC/synopsys: Convert to platform remove callback returning void
EDAC/ti: Convert to platform remove callback returning void
EDAC/xgene: Convert to platform remove callback returning void
EDAC/zynqmp: Convert to platform remove callback returning void
drivers/edac/altera_edac.c | 12 ++++--------
drivers/edac/armada_xp_edac.c | 12 ++++--------
drivers/edac/aspeed_edac.c | 6 ++----
drivers/edac/bluefield_edac.c | 6 ++----
drivers/edac/cell_edac.c | 5 ++---
drivers/edac/cpc925_edac.c | 6 ++----
drivers/edac/dmc520_edac.c | 6 ++----
drivers/edac/highbank_l2_edac.c | 5 ++---
drivers/edac/highbank_mc_edac.c | 5 ++---
drivers/edac/mpc85xx_edac.c | 11 ++++-------
drivers/edac/npcm_edac.c | 6 ++----
drivers/edac/octeon_edac-l2c.c | 6 ++----
drivers/edac/octeon_edac-lmc.c | 5 ++---
drivers/edac/octeon_edac-pc.c | 5 ++---
drivers/edac/octeon_edac-pci.c | 6 ++----
drivers/edac/ppc4xx_edac.c | 7 ++-----
drivers/edac/qcom_edac.c | 6 ++----
drivers/edac/synopsys_edac.c | 6 ++----
drivers/edac/ti_edac.c | 6 ++----
drivers/edac/xgene_edac.c | 6 ++----
drivers/edac/zynqmp_edac.c | 6 ++----
21 files changed, 48 insertions(+), 91 deletions(-)
base-commit: 0bb80ecc33a8fb5a682236443c1e740d5c917d1d
--
2.40.1
next reply other threads:[~2023-10-04 13:13 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-10-04 13:12 Uwe Kleine-König [this message]
2023-10-04 13:12 ` [PATCH 12/21] EDAC/octeon-l2c: Convert to platform remove callback returning void Uwe Kleine-König
2023-10-10 14:03 ` Philippe Mathieu-Daudé
2023-10-04 13:12 ` [PATCH 13/21] EDAC/octeon-lmc: " Uwe Kleine-König
2023-10-10 14:02 ` Philippe Mathieu-Daudé
2023-10-04 13:12 ` [PATCH 14/21] EDAC/octeon-pc: " Uwe Kleine-König
2023-10-10 14:03 ` Philippe Mathieu-Daudé
2023-10-04 13:12 ` [PATCH 15/21] EDAC/octeon-pci: " Uwe Kleine-König
2023-10-10 14:03 ` Philippe Mathieu-Daudé
2023-11-20 16:20 ` [PATCH 00/21] EDAC: " Uwe Kleine-König
2023-11-20 17:21 ` Borislav Petkov
2023-11-20 22:36 ` Borislav Petkov
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=20231004131254.2673842-1-u.kleine-koenig@pengutronix.de \
--to=u.kleine-koenig@pengutronix.de \
--cc=agross@kernel.org \
--cc=andersson@kernel.org \
--cc=andre.przywara@arm.com \
--cc=andrew@aj.id.au \
--cc=avifishman70@gmail.com \
--cc=benjaminfair@google.com \
--cc=bp@alien8.de \
--cc=dinguyen@kernel.org \
--cc=james.morse@arm.com \
--cc=jlu@pengutronix.de \
--cc=joel@jms.id.au \
--cc=kernel@pengutronix.de \
--cc=kflin@nuvoton.com \
--cc=khuong@os.amperecomputing.com \
--cc=konrad.dybcio@linaro.org \
--cc=kristo@kernel.org \
--cc=lewan@microsoft.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-aspeed@lists.ozlabs.org \
--cc=linux-edac@vger.kernel.org \
--cc=linux-mips@vger.kernel.org \
--cc=mani@kernel.org \
--cc=mchehab@kernel.org \
--cc=michal.simek@amd.com \
--cc=morbidrsa@gmail.com \
--cc=openbmc@lists.ozlabs.org \
--cc=ralf@linux-mips.org \
--cc=rric@kernel.org \
--cc=sai.krishna.potthuri@amd.com \
--cc=shravankr@nvidia.com \
--cc=shubhrajyoti.datta@amd.com \
--cc=sschaeck@cisco.com \
--cc=tali.perry1@gmail.com \
--cc=tmaimon77@gmail.com \
--cc=tony.luck@intel.com \
--cc=venture@google.com \
--cc=yschu@nuvoton.com \
--cc=yuenn@google.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 NNTP newsgroup(s).