hwrng: cctrng - don't open code init and exit functions
authorMartin Kaiser <martin@kaiser.cx>
Wed, 19 Jul 2023 07:18:03 +0000 (09:18 +0200)
committerHerbert Xu <herbert@gondor.apana.org.au>
Fri, 28 Jul 2023 10:20:25 +0000 (18:20 +0800)
Do not open code the init and exit functions of the cctrng driver.

If we move the BUILD_BUG_ON checks into the probe function, we can use
module_platform_driver and make the code shorter.

Signed-off-by: Martin Kaiser <martin@kaiser.cx>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
drivers/char/hw_random/cctrng.c

index 302ffa3..241da7e 100644 (file)
@@ -492,6 +492,10 @@ static int cctrng_probe(struct platform_device *pdev)
        u32 val;
        int irq;
 
+       /* Compile time assertion checks */
+       BUILD_BUG_ON(CCTRNG_DATA_BUF_WORDS < 6);
+       BUILD_BUG_ON((CCTRNG_DATA_BUF_WORDS & (CCTRNG_DATA_BUF_WORDS-1)) != 0);
+
        drvdata = devm_kzalloc(dev, sizeof(*drvdata), GFP_KERNEL);
        if (!drvdata)
                return -ENOMEM;
@@ -698,21 +702,7 @@ static struct platform_driver cctrng_driver = {
        .remove = cctrng_remove,
 };
 
-static int __init cctrng_mod_init(void)
-{
-       /* Compile time assertion checks */
-       BUILD_BUG_ON(CCTRNG_DATA_BUF_WORDS < 6);
-       BUILD_BUG_ON((CCTRNG_DATA_BUF_WORDS & (CCTRNG_DATA_BUF_WORDS-1)) != 0);
-
-       return platform_driver_register(&cctrng_driver);
-}
-module_init(cctrng_mod_init);
-
-static void __exit cctrng_mod_exit(void)
-{
-       platform_driver_unregister(&cctrng_driver);
-}
-module_exit(cctrng_mod_exit);
+module_platform_driver(cctrng_driver);
 
 /* Module description */
 MODULE_DESCRIPTION("ARM CryptoCell TRNG Driver");