public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Dmitry Torokhov <dtor_core@ameritech.net>
To: walt <wa1ter@myrealbox.com>
Cc: linux-kernel@vger.kernel.org, Andrew Morton <akpm@osdl.org>
Subject: Re: PS2 mouse changes for 2.6
Date: Mon, 29 Dec 2003 23:21:53 -0500	[thread overview]
Message-ID: <200312292321.55287.dtor_core@ameritech.net> (raw)
In-Reply-To: <3FF0CC02.9000508@myrealbox.com>

On Monday 29 December 2003 07:51 pm, walt wrote:
> Dmitry Torokhov wrote:
> > On Monday 29 December 2003 09:01 am, walt wrote:
> >>I see no deprecation warnings when starting the kernel with
> >>psmouse_noext, which I was expecting to see.
> >
> > It is emitted with KERN_WARNING severity and is not necessary seen on
> > the console. Check your dmesg.
>
> No warnings in dmesg on two different machines booted with
> psmouse_noext.
>
> I am NOT complaining -- my mouse works better today than yesterday :0)
>
> Is there anything I can do to help with this problem (?)

OK, when I switched the parameter processing to the new technique I missed
the fact that it requires specifying prefix (module name + '.') when the
module is compiled directly into the kernel. Therefore in the lastest -bk
you have to pass "psmouse.psmouse_noext=1" to the kernel for it to be.
recognized.

Since we already have an unique prefix (psmouse) for all parameters and
changing parameter names at this time is not desirable (IMHO) I propose
the patch below to restore the old behavior.

Dmitry

===================================================================


ChangeSet@1.1517, 2003-12-29 22:53:14-05:00, dtor_core@ameritech.net
  Input: Suppress prefix generation for psmouse parameter names
         regardless of whether psmouse is built as a module or
         compiled into the kernel image


 psmouse-base.c |    9 +++++++++
 1 files changed, 9 insertions(+)


===================================================================



diff -Nru a/drivers/input/mouse/psmouse-base.c b/drivers/input/mouse/psmouse-base.c
--- a/drivers/input/mouse/psmouse-base.c	Mon Dec 29 23:20:41 2003
+++ b/drivers/input/mouse/psmouse-base.c	Mon Dec 29 23:20:41 2003
@@ -22,6 +22,15 @@
 #include "synaptics.h"
 #include "logips2pp.h"
 
+/*
+ * Reset module param prefix regardless of whether we build psmouse as
+ * a module or directly into kernel, otherwise for build-in case
+ * parameters will have to be specified as psmouse.psmouse_proto which
+ * is unsightly
+ */
+#undef MODULE_PARAM_PREFIX
+#define MODULE_PARAM_PREFIX /* empty */
+
 MODULE_AUTHOR("Vojtech Pavlik <vojtech@suse.cz>");
 MODULE_DESCRIPTION("PS/2 mouse driver");
 MODULE_LICENSE("GPL");

  reply	other threads:[~2003-12-30  4:22 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <fa.fl0st45.t3auq7@ifi.uio.no>
     [not found] ` <fa.jm00pl0.5nsn3g@ifi.uio.no>
2003-12-30  0:51   ` PS2 mouse changes for 2.6 walt
2003-12-30  4:21     ` Dmitry Torokhov [this message]
2003-12-29 14:01 walt
2003-12-29 23:43 ` 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=200312292321.55287.dtor_core@ameritech.net \
    --to=dtor_core@ameritech.net \
    --cc=akpm@osdl.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=wa1ter@myrealbox.com \
    /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