[PATCH 0/8] ARM: Convert to platform remove callback returning void

Arnd Bergmann arnd at arndb.de
Wed Mar 8 07:06:47 PST 2023


On Wed, Mar 8, 2023, at 11:00, Uwe Kleine-König wrote:
> Hello,
>
> this patch series adapts the platform drivers below arch/arm 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 core
> 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 warning. The
> device is removed anyhow and an early return from .remove() usually yields a
> resource leak.
>
> By changing the remove callback to return void driver authors cannot
> reasonably assume any more that there is some kind of cleanup later.
>
> All drivers in arch/arm returned zero unconditionally in their remove callback,
> so they could all be converted trivially to .remove_new().
>
> Note that this series depends on commit 5c5a7680e67b ("platform: Provide
> a remove callback that returns no value") which is included in v6.3-rc1.

Looks good to me,

Reviewed-by: Arnd Bergmann <arnd at arndb.de>

> I'm unsure who will pick up this series. Will it go as a whole via arm-soc? Or
> will the individual maintainers pick it up?

I can take it through the soc tree, please send it to
soc at kernel.org once you are done getting Acks.

Since all eight patches in the series have the exact same
changelog text and are all trivial, I'd prefer them to be
folded into a single patch if that works for you.

      Arnd



More information about the linux-arm-kernel mailing list