linux-kernel - Re: elantech touchpad driver miss-recognising logitech mice

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <20090130152330.4ff2cbb2.akpm@linux-foundation.org>
Date:	Fri, 30 Jan 2009 15:23:30 -0800
From:	Andrew Morton <akpm@...ux-foundation.org>
To:	Arjan Opmeer <arjan@...eer.net>
Cc:	apw@...onical.com, patrakov@...il.com, dmitry.torokhov@...il.com,
	linux-kernel@...r.kernel.org
Subject: Re: elantech touchpad driver miss-recognising logitech mice

On Mon, 26 Jan 2009 17:23:03 +0100
Arjan Opmeer <arjan@...eer.net> wrote:

> 
> On Mon, Jan 26, 2009 at 03:08:21PM +0000, Andy Whitcroft wrote:
> > 
> > Failing that I did build a modified patch (below) based on that from the
> > thread plus the suggested improvements.  This has been tested by people
> > with the Logitech mice successfully, and also by people with elantech
> > touchpads who report no regressions.
> 
> Good! This means the suggested fix is working :)
> 
> However, my original proposal:
> 
>    http://patchwork.kernel.org/patch/595/
> 
> also removed printing the version query result in elantech_init() to keep
> the amount of log messages the same.
> 
> > If there is no later patches then perhaps you could test this one, and
> > then I can push it upstream.
> 
> Dmitry suggested moving the Elantech detection down in psmouse_extensions()
> in psmouse-base.c to make the mouse detection less chatty:
> 
>    http://lkml.org/lkml/2009/1/20/55
> 
> However, Alexander pointed out he did not move it far enough down:
> 
>    http://lkml.org/lkml/2009/1/22/18
> 
> It should have been moved all the way down below the call to
> im_explorer_detect() for him to not see the extra Elantech detection noise.
> 
> Now, it is Dmitry's call again to determine whether he really wants to move
> the call to elantech_detect() that far down or can live with the extra log
> messages... :)
> 

blah.  Please propose a final patch for me to stash away, lest the
issue get forgotten about.

--
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/