Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751438AbcJ2Ujr (ORCPT ); Sat, 29 Oct 2016 16:39:47 -0400 Received: from mail1.windriver.com ([147.11.146.13]:53924 "EHLO mail1.windriver.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750806AbcJ2Ujq (ORCPT ); Sat, 29 Oct 2016 16:39:46 -0400 From: Paul Gortmaker To: , Greg Kroah-Hartman CC: Paul Gortmaker , Matias Bjorling Subject: [PATCH 1/3] lightnvm: make core.c explicitly non-modular Date: Sat, 29 Oct 2016 16:38:41 -0400 Message-ID: <20161029203843.515-2-paul.gortmaker@windriver.com> X-Mailer: git-send-email 2.10.1 In-Reply-To: <20161029203843.515-1-paul.gortmaker@windriver.com> References: <20161029203843.515-1-paul.gortmaker@windriver.com> MIME-Version: 1.0 Content-Type: text/plain Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2120 Lines: 65 The Kconfig currently controlling compilation of this code is: drivers/lightnvm/Kconfig:menuconfig NVM drivers/lightnvm/Kconfig: bool "Open-Channel SSD target support" ...meaning that it currently is not being built as a module by anyone. Lets remove the modular code that is essentially orphaned, so that when reading the driver there is no doubt it is builtin-only. Since module_misc_driver translates to device_initcall in the non-modular case, the init ordering remains unchanged with this commit. We also delete the MODULE_LICENSE tag etc. since all that information is already contained at the top of the file in the comments. We replace module.h with moduleparam.h because this file still uses module params to control behaviour. Also note that MODULE_ALIAS is a no-op for non-modular code. Cc: Matias Bjorling Signed-off-by: Paul Gortmaker --- drivers/lightnvm/core.c | 14 ++++++-------- 1 file changed, 6 insertions(+), 8 deletions(-) diff --git a/drivers/lightnvm/core.c b/drivers/lightnvm/core.c index 1cac0f8bc0dc..2329f050ff82 100644 --- a/drivers/lightnvm/core.c +++ b/drivers/lightnvm/core.c @@ -22,7 +22,7 @@ #include #include #include -#include +#include #include #include #include @@ -889,6 +889,10 @@ static const struct kernel_param_ops nvm_configure_by_str_event_param_ops = { .get = nvm_configure_get, }; +/* + * Not available as modular, but easiest way to remain compatible with + * existing boot arg behaviour is to continue using module param here. + */ #undef MODULE_PARAM_PREFIX #define MODULE_PARAM_PREFIX "lnvm." @@ -1162,10 +1166,4 @@ static struct miscdevice _nvm_misc = { .nodename = "lightnvm/control", .fops = &_ctl_fops, }; -module_misc_device(_nvm_misc); - -MODULE_ALIAS_MISCDEV(MISC_DYNAMIC_MINOR); - -MODULE_AUTHOR("Matias Bjorling "); -MODULE_LICENSE("GPL v2"); -MODULE_VERSION("0.1"); +builtin_misc_device(_nvm_misc); -- 2.10.1