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
| ||
|
Date: Tue, 11 Jun 2019 10:07:07 -0700 From: "dmitry.torokhov@...il.com" <dmitry.torokhov@...il.com> To: Pali Rohár <pali.rohar@...il.com> Cc: Hui Wang <hui.wang@...onical.com>, Xiaoxiao Liu <xiaoxiao.liu-1@...alps.com>, XiaoXiao Liu <sliuuxiaonxiao@...il.com>, "peter.hutterer@...-t.net" <peter.hutterer@...-t.net>, "linux-input@...r.kernel.org" <linux-input@...r.kernel.org>, "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>, Xiaojian Cao <xiaojian.cao@...alps.com>, "zhangfp1@...ovo.com" <zhangfp1@...ovo.com>, Naoki Saito <naoki.saito@...salpine.com>, Hideo Kawase <hideo.kawase@...salpine.com> Subject: Re: 答复: 答复: 答复: [PATCH] input: alps-fix the issue alps cs19 trackstick do not work. On Tue, Jun 11, 2019 at 09:23:33AM +0200, Pali Rohár wrote: > On Tuesday 11 June 2019 12:32:33 Hui Wang wrote: > > On 2019/6/11 上午11:23, Hui Wang wrote: > > > On 2019/6/11 上午11:05, Xiaoxiao Liu wrote: > > > > Hi Pali, > > > > > > > > I discussed with our FW team about this problem. > > > > We think the V8 method means a touchpad feature and does not fit > > > > the CS19 trackpoint device. > > > > CS19 TrackPoint needn't use any Absolute (Raw) mode and is usually > > > > use standard mouse data. > > > > CS19 TrackPoint device is a completely different device with > > > > DualPoint device of Dell/HP. > > > > CS19 TrackPoint device is independent of Touchpad. (Touchpad is > > > > connecting by I2C, TrackPoint is directly connecting with PS2 port.) > > > > And it has completely another FW. > > > > > > > > So we think it is better to use the mouse mode for CS19 trackpoint. > > > > > > Maybe here is some mis-understanding, the mouse mode here doesn't mean > > > we use psmouse-base.c for cs19 (bare ps/2 mouse), we plan to use > > > trackpoint.c to drive this HW, so this trackpoint has all features a > > > trackpoint should have. > > > > > And I sent a patch one month ago to let the the trackpoint.c to drive this > > HW: https://www.spinics.net/lists/linux-input/msg61341.html, maybe that > > patch is reference. > > So instead of creating blacklist, you should check for TP_VARIANT_ALPS > in alps.c and disallow its usage. > > Or maybe better, move trackpoint.c detect code before alsp.c detect code > in psmouse-base. And no changes in alps.c are needed. I'd be very cautions of moving around the protocol detection. It is very fragile, so if we can detect trackpoint-only case in alps.c and skip on to trackpoint I would prefer it. Thanks. -- Dmitry
Powered by blists - more mailing lists