Received-SPF: pass (google.com: domain of linux-kernel+bounces-202202-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1;
Date: Wed, 5 Jun 2024 12:33:11 +0200
From: Uwe =?utf-8?Q?Kleine-K=C3=B6nig?= <ukleinek@kernel.org>
To: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: "Rafael J. Wysocki" <rafael@kernel.org>, 
	Mark Brown <broonie@kernel.org>, Takashi Iwai <tiwai@suse.de>, David Gow <davidgow@google.com>, 
	linux-kernel@vger.kernel.org, kernel@pengutronix.de, Philipp Zabel <p.zabel@pengutronix.de>, 
	Nishanth Menon <nm@ti.com>, Jeremy Kerr <jk@ozlabs.org>, 
	Rodolfo Giometti <giometti@enneenne.com>, Thierry Reding <treding@nvidia.com>, 
	Matt Coster <matt.coster@imgtec.com>, Thomas Zimmermann <tzimmermann@suse.de>, 
	Dave Jiang <dave.jiang@intel.com>, Dan Williams <dan.j.williams@intel.com>, 
	Vishal Verma <vishal.l.verma@intel.com>, Bjorn Andersson <andersson@kernel.org>
Subject: [GIT PULL] platform: Make platform_driver::remove() return void
Message-ID: <acwqbvhf3h5z4shzwdwwwfkvxvmchlpbh5k6kwcubhpe3ncvdf@v5gai33oolhn>
References: <20240527083416.1177106-2-u.kleine-koenig@pengutronix.de>
 <2024060432-relieving-yonder-85ae@gregkh>
Precedence: bulk
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
	protocol="application/pgp-signature"; boundary="k4kblf7m2zlxrzw2"
Content-Disposition: inline
In-Reply-To: <2024060432-relieving-yonder-85ae@gregkh>


--k4kblf7m2zlxrzw2
Content-Type: text/plain; charset=iso-8859-1
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

Hello Greg

On Tue, Jun 04, 2024 at 06:13:54PM +0200, Greg Kroah-Hartman wrote:
> On Mon, May 27, 2024 at 10:34:15AM +0200, Uwe Kleine-K=F6nig wrote:
> > there are only very little platform drivers left in v6.10-rc1 that need
> > to be changed to .remove_new() before this patch can be applied. They
> > were all sent out to the respective maintainers, most of them suggested
> > to apply the patches together with this one.
> >=20
> > You can fetch this patch together with all necessary commits from:
> >=20
> > 	https://git.kernel.org/pub/scm/linux/kernel/git/ukleinek/linux.git pla=
tform-remove-void
> >=20
> > If you have no concerns, I can also provide you a signed tag for
> > pulling. I think that's easier than indiviually applying them, but I can
> > also send out the complete series if you prefer.
>=20
> A signed tag is good, I can just pull from that, thanks!

here it comes:

The following changes since commit 1613e604df0cd359cf2a7fbd9be7a0bcfacfabd0:

  Linux 6.10-rc1 (2024-05-26 15:20:12 -0700)

are available in the Git repository at:

  https://git.kernel.org/pub/scm/linux/kernel/git/ukleinek/linux.git tags/p=
latform-remove-void-step-b

for you to fetch changes up to 0edb555a65d1ef047a9805051c36922b52a38a9d:

  platform: Make platform_driver::remove() return void (2024-05-27 10:34:35=
 +0200)

----------------------------------------------------------------
Change struct platform_driver::remove() to return void

This is step b) of the plan outlined in commit 5c5a7680e67b ("platform:
Provide a remove callback that returns no value"), which completes the
first major step of making the remove callback return no value. Up to
now it returned an int which however was mostly ignored by the driver
core and lured driver authors to believe there is some error handling.

Note that the Linux driver model assumes that removing a device cannot
fail, so this isn't about being lazy and not implementing error handling
in the core and so making .remove return void is the right thing to do.

----------------------------------------------------------------
Uwe Kleine-K=F6nig (18):
      reset: meson-audio-arb: Convert to platform remove callback returning=
 void
      reset: rzg2l-usbphy-ctrl: Convert to platform remove callback returni=
ng void
      reset: ti-sci: Convert to platform remove callback returning void
      Merge branch 'reset/next' of git://git.pengutronix.de/pza/linux
      fsi: master-aspeed: Convert to platform remove callback returning void
      fsi: master-ast-cf: Convert to platform remove callback returning void
      fsi: master-gpio: Convert to platform remove callback returning void
      fsi: occ: Convert to platform remove callback returning void
      pps: clients: gpio: Convert to platform remove callback returning void
      gpu: host1x: mipi: Benefit from devm_clk_get_prepared()
      drm/imagination: Convert to platform remove callback returning void
      drm/mediatek: Convert to platform remove callback returning void
      gpu: host1x: Convert to platform remove callback returning void
      gpu: ipu-v3: Convert to platform remove callback returning void
      nvdimm/e820: Convert to platform remove callback returning void
      nvdimm/of_pmem: Convert to platform remove callback returning void
      samples: qmi: Convert to platform remove callback returning void
      platform: Make platform_driver::remove() return void

 drivers/base/platform.c                 | 10 ++--------
 drivers/fsi/fsi-master-aspeed.c         |  6 ++----
 drivers/fsi/fsi-master-ast-cf.c         |  6 ++----
 drivers/fsi/fsi-master-gpio.c           |  6 ++----
 drivers/fsi/fsi-occ.c                   |  6 ++----
 drivers/gpu/drm/imagination/pvr_drv.c   |  7 ++-----
 drivers/gpu/drm/mediatek/mtk_padding.c  |  5 ++---
 drivers/gpu/host1x/dev.c                |  6 ++----
 drivers/gpu/host1x/mipi.c               | 17 +----------------
 drivers/gpu/ipu-v3/ipu-common.c         |  6 ++----
 drivers/gpu/ipu-v3/ipu-pre.c            |  5 ++---
 drivers/gpu/ipu-v3/ipu-prg.c            |  6 ++----
 drivers/nvdimm/e820.c                   |  5 ++---
 drivers/nvdimm/of_pmem.c                |  6 ++----
 drivers/pps/clients/pps-gpio.c          |  5 ++---
 drivers/reset/reset-meson-audio-arb.c   |  6 ++----
 drivers/reset/reset-rzg2l-usbphy-ctrl.c |  6 ++----
 drivers/reset/reset-ti-sci.c            |  6 ++----
 include/linux/platform_device.h         | 15 +++++++--------
 samples/qmi/qmi_sample_client.c         |  6 ++----
 20 files changed, 44 insertions(+), 97 deletions(-)

Thanks for considering this,
Uwe

--k4kblf7m2zlxrzw2
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQEzBAABCgAdFiEEP4GsaTp6HlmJrf7Tj4D7WH0S/k4FAmZgPuYACgkQj4D7WH0S
/k6BOQf+IQTl+EBUsW7y5sIUElNQb299hOQfaxdDWvFu6im/T12dNTzp5k9dM0C2
EdH5038yJQtmlFLoYLZlIUKEm2hYN/qL5ofgPBfp5VjPzxQ7tfbSqg316aL9aLh1
1SoKlEW6XSUyC4m9xWPTTBudfO8OcitK5IZidvIwAJFQFlFCF4YfETRVeUYORK6i
LlSgnwp2SGHURnsyW837BkadpzoV3FgThMH9BS4qtkli1QQWxExATSEIO3FGSU8l
s4kYA6W9S5fSLB8JYlOcDC6hIHJ397DL3Py7m/oi6VaauLojCiJmZI4WeesKBCrG
qC/6iDPo4X92MDkHjKDi7Dv5A3M/Bw==
=EYSE
-----END PGP SIGNATURE-----

--k4kblf7m2zlxrzw2--