diff options
author | Kamalesh Babulal <kamalesh@linux.vnet.ibm.com> | 2008-04-05 21:00:57 +0800 |
---|---|---|
committer | Herbert Xu <herbert@gondor.apana.org.au> | 2008-04-21 10:19:34 +0800 |
commit | 3af5b90bde5000abc739996cb03fd718e753d053 (patch) | |
tree | 579ab3154c56dfa49e83acb463f04daed8135988 /crypto/crc32c.c | |
parent | 7dc748e4e720c1a98185363096ad7582e9113092 (diff) | |
download | op-kernel-dev-3af5b90bde5000abc739996cb03fd718e753d053.zip op-kernel-dev-3af5b90bde5000abc739996cb03fd718e753d053.tar.gz |
[CRYPTO] all: Clean up init()/fini()
On Thu, Mar 27, 2008 at 03:40:36PM +0100, Bodo Eggert wrote:
> Kamalesh Babulal <kamalesh@linux.vnet.ibm.com> wrote:
>
> > This patch cleanups the crypto code, replaces the init() and fini()
> > with the <algorithm name>_init/_fini
>
> This part ist OK.
>
> > or init/fini_<algorithm name> (if the
> > <algorithm name>_init/_fini exist)
>
> Having init_foo and foo_init won't be a good thing, will it? I'd start
> confusing them.
>
> What about foo_modinit instead?
Thanks for the suggestion, the init() is replaced with
<algorithm name>_mod_init ()
and fini () is replaced with <algorithm name>_mod_fini.
Signed-off-by: Kamalesh Babulal <kamalesh@linux.vnet.ibm.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Diffstat (limited to 'crypto/crc32c.c')
-rw-r--r-- | crypto/crc32c.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/crypto/crc32c.c b/crypto/crc32c.c index 0fa7443..0dcf64a 100644 --- a/crypto/crc32c.c +++ b/crypto/crc32c.c @@ -98,18 +98,18 @@ static struct crypto_alg alg = { } }; -static int __init init(void) +static int __init crc32c_mod_init(void) { return crypto_register_alg(&alg); } -static void __exit fini(void) +static void __exit crc32c_mod_fini(void) { crypto_unregister_alg(&alg); } -module_init(init); -module_exit(fini); +module_init(crc32c_mod_init); +module_exit(crc32c_mod_fini); MODULE_AUTHOR("Clay Haapala <chaapala@cisco.com>"); MODULE_DESCRIPTION("CRC32c (Castagnoli) calculations wrapper for lib/crc32c"); |