Re: [PATCH 00/49] mtd: nand: Convert to platform remove callback returning void

Hi Uwe,

[email protected] wrote on Sat,  1 Apr 2023 18:18:49 +0200:

> Hello,
>=20
> this series adapts the platform drivers below drivers/mtd/nand to use the
> .remove_new() callback. Compared to the traditional .remove() callback
> .remove_new() returns no value. This is a good thing because the driver c=
ore
> doesn't (and cannot) cope for errors during remove. The only effect of a
> non-zero return value in .remove() is that the driver core emits a warnin=
g. The
> device is removed anyhow and an early return from .remove() usually yield=
s a
> resource leak.
>=20
> By changing the remove callback to return void driver authors cannot
> reasonably assume any more that there is some kind of cleanup later.
>=20
> As all drivers already return 0 in their .remove callback, they can be
> converted trivially.

I've looked at the different patches, they look good to me but as they
are all trivial and exactly identical, would you mind sending this
again all squashed in a single patch? A subsystem-wide conversion seems
appropriate. In all cases I plan to take this for the next merge
window.

I've collected the tags received so far if you want:

Acked-by: Nicolas Ferre <[email protected]> # atmel
Reviewed-by: Paul Cercueil <[email protected]> # ingenic
Reviewed-by: Philippe Mathieu-Daud=C3=A9 <[email protected]> # ingenic
Acked-by: Martin Blumenstingl <[email protected]> # intel
Reviewed-by: Martin Blumenstingl <[email protected]> # mes=
on
Acked-by: Roger Quadros <[email protected]> # omap
Reviewed-by: Geert Uytterhoeven <[email protected]> # renesas
Reviewed-by: Heiko Stuebner <[email protected]> # rockchip
Acked-by: Jernej Skrabec <[email protected]> # sunxi
Acked-by: Thierry Reding <[email protected]> # tegra

BTW, thanks to all of you who reviewed and answered to this thread!
This is highly appreciated.=20

Thanks,
Miqu=C3=A8l