public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Adrian Bunk <bunk@stusta.de>
To: Dmitry Torokhov <dtor_core@ameritech.net>
Cc: Martin Michlmayr <tbm@cyrius.com>,
	Al Viro <viro@ftp.linux.org.uk>, Dave Jones <davej@redhat.com>,
	linux-kernel@vger.kernel.org, Vojtech Pavlik <vojtech@suse.cz>
Subject: [2.6 patch] make INPUT a bool
Date: Fri, 3 Feb 2006 23:36:15 +0100	[thread overview]
Message-ID: <20060203223615.GU4408@stusta.de> (raw)
In-Reply-To: <200601250004.06543.dtor_core@ameritech.net>

On Wed, Jan 25, 2006 at 12:04:06AM -0500, Dmitry Torokhov wrote:
> On Tuesday 24 January 2006 18:14, Martin Michlmayr wrote:
> > * Dmitry Torokhov <dmitry.torokhov@gmail.com> [2006-01-24 18:08]:
> > > > More interesting question: is pis^H^H^Hsysfs interaction in there safe for
> > > > modular code?
> > > 
> > > The core should be safe, at least I was trying to make it this way, so
> > > if you see something wrong - shout. Locking is another question
> > > though...
> > 
> > So do you want an updated patch using _GPL to export the symbols or to
> > change CONFIG_INPUT to boolean?
> 
> I guess having input core as a module does not make much sense, so
> we should change CONFIG_INPUT to be boolean _and_ clean up the core
> code removing module unloading support.

Is the patch below what you were thinking of?

> Dmitry

cu
Adrian


<--  snip  -->


Make INPUT a bool.

INPUT!=y is only possible if EMBEDDED=y, and in such cases it doesn't 
make that much sense to make it modular.


Signed-off-by: Adrian Bunk <bunk@stusta.de>

---

 drivers/input/Kconfig |    2 +-
 drivers/input/input.c |    8 --------
 2 files changed, 1 insertion(+), 9 deletions(-)

--- linux-2.6.16-rc1-mm5-full/drivers/input/Kconfig.old	2006-02-03 22:42:18.000000000 +0100
+++ linux-2.6.16-rc1-mm5-full/drivers/input/Kconfig	2006-02-03 22:42:29.000000000 +0100
@@ -5,7 +5,7 @@
 menu "Input device support"
 
 config INPUT
-	tristate "Generic input layer (needed for keyboard, mouse, ...)" if EMBEDDED
+	bool "Generic input layer (needed for keyboard, mouse, ...)" if EMBEDDED
 	default y
 	---help---
 	  Say Y here if you have any input device (mouse, keyboard, tablet,
--- linux-2.6.16-rc1-mm5-full/drivers/input/input.c.old	2006-02-03 22:42:41.000000000 +0100
+++ linux-2.6.16-rc1-mm5-full/drivers/input/input.c	2006-02-03 22:47:44.000000000 +0100
@@ -984,12 +984,4 @@
 	return err;
 }
 
-static void __exit input_exit(void)
-{
-	input_proc_exit();
-	unregister_chrdev(INPUT_MAJOR, "input");
-	class_unregister(&input_class);
-}
-
 subsys_initcall(input_init);
-module_exit(input_exit);


  parent reply	other threads:[~2006-02-03 22:36 UTC|newest]

Thread overview: 46+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-01-24 18:19 [PATCH] Export symbols so CONFIG_INPUT works as a module Martin Michlmayr
2006-01-24 18:31 ` Arjan van de Ven
2006-01-24 18:34 ` Dave Jones
2006-01-24 18:41   ` Al Viro
2006-01-24 23:08     ` Dmitry Torokhov
2006-01-24 23:14       ` Martin Michlmayr
2006-01-25  5:04         ` Dmitry Torokhov
2006-01-25  7:51           ` Vojtech Pavlik
2006-01-25 10:54             ` Martin Michlmayr
2006-01-26 22:17             ` Adrian Bunk
2006-01-25 14:20           ` Jan Engelhardt
2006-02-03 22:36           ` Adrian Bunk [this message]
2006-01-24 19:08   ` Martin Michlmayr
2006-01-24 19:20     ` Dave Jones
  -- strict thread matches above, loose matches on Subject: below --
2006-02-14 15:22 [2.6 patch] make INPUT a bool Adrian Bunk
2006-02-14 18:14 ` Jan Engelhardt
2006-02-14 18:22   ` Adrian Bunk
2006-02-14 23:47     ` Adrian Bunk
2006-02-17 15:56     ` Jan Engelhardt
2006-02-17 16:38       ` Adrian Bunk
2006-02-20  3:33         ` Samuel Masham
2006-02-20 13:28           ` Adrian Bunk
2006-02-22  1:34             ` Herbert Poetzl
2006-02-22  2:31               ` Adrian Bunk
2006-02-22  2:44                 ` Herbert Poetzl
2006-02-22  3:10                   ` Adrian Bunk
2006-02-22  3:20                     ` Dmitry Torokhov
2006-02-22 12:01                     ` Herbert Poetzl
2006-02-22 12:15                       ` Adrian Bunk
2006-02-22 18:09                         ` Herbert Poetzl
2006-02-25 11:58                 ` Geert Uytterhoeven
2006-02-25 12:46                   ` Adrian Bunk
2006-02-25 14:22                     ` Jan Engelhardt
2006-02-25 14:50                       ` Adrian Bunk
2006-02-25 15:29                         ` Jan Engelhardt
2006-02-25 15:40                           ` Adrian Bunk
2006-02-25 22:01                             ` Jan Engelhardt
2006-02-25 22:07                               ` Adrian Bunk
2006-02-25 22:23                                 ` Dmitry Torokhov
2006-02-26 18:13                                   ` Adrian Bunk
2006-02-27  7:11                                   ` Vojtech Pavlik
2006-02-27 12:59                                 ` Vojtech Pavlik
2006-02-25 14:57                       ` Jesper Juhl
2006-02-15  6:20 ` Dmitry Torokhov
2006-02-16 23:22   ` Adrian Bunk
2006-02-17  1:47     ` Dmitry Torokhov

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20060203223615.GU4408@stusta.de \
    --to=bunk@stusta.de \
    --cc=davej@redhat.com \
    --cc=dtor_core@ameritech.net \
    --cc=linux-kernel@vger.kernel.org \
    --cc=tbm@cyrius.com \
    --cc=viro@ftp.linux.org.uk \
    --cc=vojtech@suse.cz \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox