From mboxrd@z Thu Jan 1 00:00:00 1970 From: Peter Hutterer Subject: Re: [PATCH 3/4] input/alps: Allow touchsticks to report pressure Date: Mon, 13 Jun 2016 10:21:14 +1000 Message-ID: <20160613002114.GA6773@jelly> References: <1464436862-2649-1-git-send-email-ben@smart-cactus.org> <1464436862-2649-4-git-send-email-ben@smart-cactus.org> <625ba798-bec9-7dfe-ba9d-55a8c3ba347e@redhat.com> <871t45xs1n.fsf@smart-cactus.org> <0d0c8090-8d08-4c81-7842-02093ef2c05e@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from leo.clearchain.com ([199.73.29.74]:14929 "EHLO mail.clearchain.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932783AbcFMAVZ (ORCPT ); Sun, 12 Jun 2016 20:21:25 -0400 Content-Disposition: inline In-Reply-To: <0d0c8090-8d08-4c81-7842-02093ef2c05e@redhat.com> Sender: linux-input-owner@vger.kernel.org List-Id: linux-input@vger.kernel.org To: Hans de Goede Cc: Ben Gamari , Pali =?iso-8859-1?Q?Roh=E1r?= , linux-input@vger.kernel.org, Allen Hung , Masaki Ota , Ben Morgan , Peter Hutterer On Fri, Jun 10, 2016 at 12:57:12PM +0200, Hans de Goede wrote: > Hi, > > On 10-06-16 12:21, Ben Gamari wrote: > > Hans de Goede writes: > > > > > Hi, > > > > > > > > @@ -2996,6 +2999,10 @@ int alps_init(struct psmouse *psmouse) > > > > > > > > input_set_capability(dev2, EV_REL, REL_X); > > > > input_set_capability(dev2, EV_REL, REL_Y); > > > > + if (priv->flags & ALPS_DUALPOINT_WITH_PRESSURE) { > > > > + input_set_capability(dev2, EV_ABS, ABS_PRESSURE); > > > > + input_set_abs_params(dev2, ABS_PRESSURE, 0, 127, 0, 0); > > > > + } > > > > input_set_capability(dev2, EV_KEY, BTN_LEFT); > > > > input_set_capability(dev2, EV_KEY, BTN_RIGHT); > > > > input_set_capability(dev2, EV_KEY, BTN_MIDDLE); > > > > > > This seems wrong, reporting ABS_PRESSURE on a relative device. And yes, this > > > really is a relative device (it sends repeated delta events when you > > > keep pushing at the same force, rather then sending a single coordinate > > > value). > > > > > > Maybe we need a REL_PRESSURE for cases like this ? > > > > > Fair enough. However, if such a thing were added how would one specify > > the range of the value (which seems necessary for the pressure to have > > any meaning whatsoever)? > > Good question, adding Peter Hutterer to the Cc. > > Peter, some of the newer alps pointing sticks can also report pressure, > which is somewhat of an ill fit for the relative events interface > otherwise used with pointing sticks, do you have any suggestions on > how to deal with this ? I think ABS_PRESSURE is fine here provided the ranges are set up correctly and he device really sends an absolute value. It might confuse userspace at first but there is no real requirement for a device to stick to one interface only. REL_PRESSURE would be a lot harder to handle because we don't have a reference frame on what the permitted axis ranges could be and it would make the actual value effectively meaningless (without a MOUSE_DPI-like database) Cheers, Peter