Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1436612 > unrolled thread
| Started by | Paul Gortmaker <paul.gortmaker@windriver.com> |
|---|---|
| First post | 2016-07-05 00:00 +0200 |
| Last post | 2016-07-05 20:50 +0200 |
| Articles | 2 — 2 participants |
Back to article view | Back to linux.kernel
[PATCH -next] connector: make cn_proc explicitly non-modular Paul Gortmaker <paul.gortmaker@windriver.com> - 2016-07-05 00:00 +0200
Re: [PATCH -next] connector: make cn_proc explicitly non-modular David Miller <davem@davemloft.net> - 2016-07-05 20:50 +0200
| From | Paul Gortmaker <paul.gortmaker@windriver.com> |
|---|---|
| Date | 2016-07-05 00:00 +0200 |
| Subject | [PATCH -next] connector: make cn_proc explicitly non-modular |
| Message-ID | <rRk4i-2wk-3@gated-at.bofh.it> |
The Kconfig controlling build of this code is currently: drivers/connector/Kconfig:config PROC_EVENTS drivers/connector/Kconfig: bool "Report process events to userspace" ...meaning that it currently is not being built as a module by anyone. Lets remove the two modular references, so that when reading the driver there is no doubt it is builtin-only. Since module_init translates to device_initcall in the non-modular case, the init ordering remains unchanged with this commit. Cc: Evgeniy Polyakov <zbr@ioremap.net> Cc: netdev@vger.kernel.org Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com> --- drivers/connector/cn_proc.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/connector/cn_proc.c b/drivers/connector/cn_proc.c index b02f9c606e0b..a782ce87715c 100644 --- a/drivers/connector/cn_proc.c +++ b/drivers/connector/cn_proc.c @@ -22,7 +22,6 @@ * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA */ -#include <linux/module.h> #include <linux/kernel.h> #include <linux/ktime.h> #include <linux/init.h> @@ -390,5 +389,4 @@ static int __init cn_proc_init(void) } return 0; } - -module_init(cn_proc_init); +device_initcall(cn_proc_init); -- 2.8.4
[toc] | [next] | [standalone]
| From | David Miller <davem@davemloft.net> |
|---|---|
| Date | 2016-07-05 20:50 +0200 |
| Message-ID | <rRDzX-6xV-5@gated-at.bofh.it> |
| In reply to | #1436612 |
From: Paul Gortmaker <paul.gortmaker@windriver.com> Date: Mon, 4 Jul 2016 17:50:58 -0400 > The Kconfig controlling build of this code is currently: > > drivers/connector/Kconfig:config PROC_EVENTS > drivers/connector/Kconfig: bool "Report process events to userspace" > > ...meaning that it currently is not being built as a module by anyone. > Lets remove the two modular references, so that when reading the driver > there is no doubt it is builtin-only. > > Since module_init translates to device_initcall in the non-modular > case, the init ordering remains unchanged with this commit. > > Cc: Evgeniy Polyakov <zbr@ioremap.net> > Cc: netdev@vger.kernel.org > Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com> Applied, thanks.
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web