[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20190611072333.nd4va4q2m5epmukc@pali>
Date: Tue, 11 Jun 2019 09:23:33 +0200
From: Pali Rohár <pali.rohar@...il.com>
To: Hui Wang <hui.wang@...onical.com>
Cc: Xiaoxiao Liu <xiaoxiao.liu-1@...alps.com>,
XiaoXiao Liu <sliuuxiaonxiao@...il.com>,
"dmitry.torokhov@...il.com" <dmitry.torokhov@...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 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.
> > Regards,
> >
> > Hui.
> >
> > >
> > > Best Regards
> > > Shona
> > > -----邮件原件-----
> > > 发件人: Pali Rohár <pali.rohar@...il.com>
> > > 发送时间: Monday, June 10, 2019 6:43 PM
> > > 收件人: 劉 曉曉 Xiaoxiao Liu <xiaoxiao.liu-1@...alps.com>
> > > 抄送: XiaoXiao Liu <sliuuxiaonxiao@...il.com>;
> > > dmitry.torokhov@...il.com; peter.hutterer@...-t.net;
> > > hui.wang@...onical.com; linux-input@...r.kernel.org;
> > > linux-kernel@...r.kernel.org; 曹 曉建 Xiaojian Cao
> > > <xiaojian.cao@...alps.com>; zhangfp1@...ovo.com; 斉藤 直樹 Naoki
> > > Saito <naoki.saito@...salpine.com>; 川瀬 英夫 Hideo Kawase
> > > <hideo.kawase@...salpine.com>
> > > 主题: Re: 答复: 答复: [PATCH] input: alps-fix the issue alps cs19
> > > trackstick do not work.
> > >
> > > On Monday 10 June 2019 10:03:51 Xiaoxiao Liu wrote:
> > > > Hi Pali,
> > > Hi!
> > >
> > > > We register our CS19 device as ALPS_ONLY_TRACKSTICK device.
> > > > And let the V8 protocol function support the process of
> > > > ALPS_ONLY_TRACKSTICK device.
> > > >
> > > > I want to confirm if this solution OK?
> > > Yes, it is fine. Just make sure that touchapad input device is not
> > > registered when this ALPS_ONLY_TRACKSTICK flag is set. So userspace
> > > would not see any fake/unavailable touchpad input device.
> > >
> > > > Xiaoxiao.Liu
> > > --
> > > Pali Rohár
> > > pali.rohar@...il.com
--
Pali Rohár
pali.rohar@...il.com
Powered by blists - more mailing lists