clk: s2mps11: initialize driver via module_platform_driver
authorKrzysztof Kozlowski <krzk@kernel.org>
Mon, 21 Sep 2020 20:35:57 +0000 (22:35 +0200)
committerStephen Boyd <sboyd@kernel.org>
Tue, 22 Sep 2020 19:30:52 +0000 (12:30 -0700)
The driver was using subsys_initcall() because in old times deferred
probe was not supported everywhere and specific ordering was needed.
Since probe deferral works fine and specific ordering is discouraged
(hides dependencies between drivers and couples their boot order), the
driver can be converted to regular module_platform_driver.

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
Link: https://lore.kernel.org/r/20200921203558.19554-1-krzk@kernel.org
Tested-by: Marek Szyprowski <m.szyprowski@samsung.com>
Signed-off-by: Stephen Boyd <sboyd@kernel.org>
drivers/clk/clk-s2mps11.c

index 2ce370c804aae736a7a596997972d916326ab740..aa21371f9104c9dde800b48770b050d80e137b69 100644 (file)
@@ -267,18 +267,7 @@ static struct platform_driver s2mps11_clk_driver = {
        .remove = s2mps11_clk_remove,
        .id_table = s2mps11_clk_id,
 };
-
-static int __init s2mps11_clk_init(void)
-{
-       return platform_driver_register(&s2mps11_clk_driver);
-}
-subsys_initcall(s2mps11_clk_init);
-
-static void __exit s2mps11_clk_cleanup(void)
-{
-       platform_driver_unregister(&s2mps11_clk_driver);
-}
-module_exit(s2mps11_clk_cleanup);
+module_platform_driver(s2mps11_clk_driver);
 
 MODULE_DESCRIPTION("S2MPS11 Clock Driver");
 MODULE_AUTHOR("Yadwinder Singh Brar <yadi.brar@samsung.com>");