On Mon, Jun 17, 2019 at 08:40:48PM +0200, Enrico Weigelt, metux IT consult wrote:
> From: Enrico Weigelt <info@xxxxxxxxx>
>
> Reduce driver init boilerplate by using the new
> subsys_platform_driver() macro.
> --- a/drivers/gpio/gpio-msic.c
> +++ b/drivers/gpio/gpio-msic.c
> @@ -306,9 +306,4 @@ static int platform_msic_gpio_probe(struct platform_device *pdev)
> },
> .probe = platform_msic_gpio_probe,
> };
> -
> -static int __init platform_msic_gpio_init(void)
> -{
> - return platform_driver_register(&platform_msic_gpio_driver);
> -}
> -subsys_initcall(platform_msic_gpio_init);
> +subsys_platform_driver(platform_msic_gpio_driver);
How did you test this? Had you even compiled it?
P.S. Perhaps it makes #include <linux/init.h> redundant.
--
With Best Regards,
Andy Shevchenko
