[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20081230142903.GA3449@adopmeer.homeip.net>
Date: Tue, 30 Dec 2008 15:29:03 +0100
From: Arjan Opmeer <arjan@...eer.net>
To: Dmitry Torokhov <dmitry.torokhov@...il.com>
Cc: "Alexander E. Patrakov" <patrakov@...il.com>,
linux-kernel@...r.kernel.org, linux-input@...r.kernel.org
Subject: Re: 2.6.28 thinks that my PS/2 mouse is a touchpad
On Tue, Dec 30, 2008 at 01:15:01AM -0800, Dmitry Torokhov wrote:
> On Monday 29 December 2008 22:03:35 Alexander E. Patrakov wrote:
> >
> > This is still slightly wrong. The mouse does work, but is misnamed in
> > the dmesg output and in /proc/bus/input/devices:
> >
> > psmouse serio1: ID: 10 00 64<6>elantech.c: Elantech version query result
> > 0x00, 0x01, 0x64.
> > elantech.c: Probably not a real Elantech touchpad. Aborting.
> > input: ImExPS/2 Elantech Touchpad as /class/input/input8
>
> Yeah, elantech_init() is too late, we need to do something in
> elantech_detect()...
Well, we could also move that Elantech version query and test for sane
values into elantech_detect(). However, because we have no private data
structure at that time it would mean we have to do the same query again in
elantech_init() to store fw_version_maj and fw_version_min.
> Maybe we need to pull part of Logitech check (number of buttons for
> example) from logips2pp.c.
I think the solution above is simpler although maybe not as pretty... :)
Arjan
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists