[PATCH] perf: Switch back to struct platform_driver::remove()

Will Deacon will at kernel.org
Tue Oct 29 06:32:38 PDT 2024


On Sun, Oct 27, 2024 at 07:03:14PM +0100, Uwe Kleine-König wrote:
> After commit 0edb555a65d1 ("platform: Make platform_driver::remove()
> return void") .remove() is (again) the right callback to implement for
> platform drivers.
> 
> Convert all platform drivers below drivers/perf to use .remove(), with
> the eventual goal to drop struct platform_driver::remove_new(). As
> .remove() and .remove_new() have the same prototypes, conversion is done
> by just changing the structure member name in the driver initializer.
> 
> Signed-off-by: Uwe Kleine-König <u.kleine-koenig at baylibre.com>
> ---
> Hello
> 
> I did a single patch for all of drivers/perf. While I usually prefer to
> do one logical change per patch, this seems to be overengineering here
> as the individual changes are really trivial and shouldn't be much in
> the way for stable backports. But I'll happily split the patch if you
> prefer it split.
> 
> Note I didn't Cc: the maintainers of each driver as this would hit
> sending limits.
> 
> This is based on Friday's next, if conflicts arise when you apply it at
> some later time and don't want to resolve them, feel free to just drop
> the changes to the conflicting files. I'll notice and followup at a
> later time then. Or ask me for a fixed resend.

Alternatively, if you share whatever script you used to generate the
diff (did you use coccinelle?) I can just run it right before I tag the
branch for 6.13 and use your commit message.

Will



More information about the linux-arm-kernel mailing list