wifi: ath10k: sdio: simplify module initialization
authorKrzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Wed, 3 Apr 2024 14:16:52 +0000 (16:16 +0200)
committerUlf Hansson <ulf.hansson@linaro.org>
Thu, 4 Apr 2024 09:09:12 +0000 (11:09 +0200)
This driver's initialization functions do not perform any custom code,
except printing messages.  Printing messages on modules
loading/unloading is discouraged because it pollutes the dmesg
regardless whether user actually has this device.  Core kernel code
already gives tools to investigate whether module was loaded or not.

Drop the printing messages which allows to replace open-coded
module_sdio_driver().

Acked-by: Jeff Johnson <quic_jjohnson@quicinc.com>
Acked-by: Kalle Valo <kvalo@kernel.org>
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Link: https://lore.kernel.org/r/20240403-module-owner-sdio-v2-3-ae46d6b955eb@linaro.org
Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
drivers/net/wireless/ath/ath10k/sdio.c

index 0ab5433f6cf6f261cde6347edf38a219e979843d..1acb9fba9a8e13edb31bb861b2b9b41d548f6a2f 100644 (file)
@@ -2671,25 +2671,7 @@ static struct sdio_driver ath10k_sdio_driver = {
                .pm = ATH10K_SDIO_PM_OPS,
        },
 };
-
-static int __init ath10k_sdio_init(void)
-{
-       int ret;
-
-       ret = sdio_register_driver(&ath10k_sdio_driver);
-       if (ret)
-               pr_err("sdio driver registration failed: %d\n", ret);
-
-       return ret;
-}
-
-static void __exit ath10k_sdio_exit(void)
-{
-       sdio_unregister_driver(&ath10k_sdio_driver);
-}
-
-module_init(ath10k_sdio_init);
-module_exit(ath10k_sdio_exit);
+module_sdio_driver(ath10k_sdio_driver);
 
 MODULE_AUTHOR("Qualcomm Atheros");
 MODULE_DESCRIPTION("Driver support for Qualcomm Atheros 802.11ac WLAN SDIO devices");