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

[Helge Deller <deller@gmx.de>]

On 10/8/24 08:58, Uwe Kleine-König wrote:
> On Tue, Oct 08, 2024 at 08:30:40AM +0200, Thomas Zimmermann wrote:
>> Hi
>>
>> Am 07.10.24 um 18:35 schrieb Uwe Kleine-König:
>>> 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/dma after the previous
>>> conversion commits apart from the wireless drivers 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.
>>
>> This paragraph refers to dma and wireless code. I don't understand how that
>> is related.
>
> Ah, too many patches and missing automation in some parts.
>
> @Helge, can you replace this paragraph by:

will do.
Thanks!
Helge


> 	Convert all platform drivers below drivers/video/fbdev 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.
>
> ? Or should I resend?
>
> Thanks for noticing,
> Uwe