linux-input.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Peter Hutterer <peter.hutterer@who-t.net>
To: Matthew Helsley <matt.helsley@gmail.com>
Cc: linux-input@vger.kernel.org, linux-usb@vger.kernel.org,
	xorg@lists.freedesktop.org
Subject: Re: evdev and Trust TB-5300 tablet: wrong axis labels
Date: Tue, 15 Dec 2009 20:32:06 +1000	[thread overview]
Message-ID: <20091215103204.GA2144@salty> (raw)
In-Reply-To: <6a12d2f30912142343j7d012bc0x91275c14630f83df@mail.gmail.com>

On Mon, Dec 14, 2009 at 11:43:21PM -0800, Matthew Helsley wrote:
> > <match key="input.product" contains="U-LOGIC">
> 
> Mine reports "UC-Logic Technology Corp." as the usb.vendor string. My
> "input.product" for that device is exactly "    Tablet PF1209" (space
> included). So the rule would have to check the usb.vendor_id of the
> parent "node".
> 
> For my tablet I chose to be quite specific:
> 
> <match key="info.capabilities" contains="input.touchpad">
> <match key="info.product" contains="Tablet PF1209">
> <match key="/org/freedesktop/Hal/devices/computer:system.kernel.name"
> string="Linux">
> 
> (I chose the last since evdev is a Linux driver and I didn't know
> whether my changes to these .fdi files might eventually be useful on a
> *BSD.)
> 
> >       <merge key="input.x11_driver" type="string">evdev</merge>
> > </match>
> 
> Yes, I've hit this problem and I keep forgetting about it because it's
> hidden in the .fdi files, waiting for whenever my distro "upgrades"
> them. Sorry, Daniil, I completely forgot to mention this problem :(.

the distro fdi files are usually in /usr/share/hal, the user-specific ones
in /etc/hal. hence once you drop your custom fdi into /etc/hal, it'll stay
there and override the distro ones.
 
> <tangent>
> The .fdi file that assigns the synaptic driver to these devices based
> solely on the "input.touchpad" capability seems quite wrong to me. My
> guess is most tablets that rely on evdev will report absolute
> coordinates. If anything, based on their comparably-small physical
> size, I'd expect "touchpads" would report relative coordinates. Plus
> synaptic can't be the only touchpad vendor/whatnot, can it? Why should
> its driver try to claim them all?

> 
> Perhaps it should have it's own match key:
> 
> <match key="info.product" contains="Synaptics TouchPad">
> 
> (which works for my touchpad at least) rather than:
> 
> <match key="info.capabilities" contains="input.touchpad">
> 
> Sorry, I don't know: Who maintains the .fdi files -- the driver
> developer, the distro, or HAL developers? In my distro they're in
> /usr/share/hal/policy and the way its packaged suggests the driver
> developers are responsible.
> 
> Peter am I way off here?

maybe partly. the fdi file is shipped with the synaptics tarballs, though
distros are of course free to create their own. we used to have a similar
match rule but synaptics is just the driver name, the driver supports alps,
appletouch, bcm5974, elantech and synaptics touchpads. that is on linux
anyway - the hardware-specific parts are handled by the kernel.
so the synaptics driver is really more of a generic touchpad driver.

on BSD the evdev backend doesn't come into play so I guess synaptics is
overly eager claiming devices it shouldn't. If you have any suggestions for
improving the fdi, don't hesitate to send in a patch. 
In a separate thread maybe, no need to spam the kernel lists for this.

Cheers,
  Peter

      reply	other threads:[~2009-12-15 10:32 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <200912130058.46220.dan@riga.lv>
2009-12-13  1:27 ` evdev and Trust TB-5300 tablet: wrong axis labels Matthew Helsley
2009-12-13 16:25   ` Daniil V. Kolpakov
2009-12-13 21:23     ` Daniil V. Kolpakov
2009-12-13 21:36       ` Daniil V. Kolpakov
     [not found]     ` <200912131825.03872.dan-A/X3QuVk8WQ@public.gmane.org>
2009-12-14  2:35       ` Matthew Helsley
2009-12-14 23:06         ` Daniil V. Kolpakov
2009-12-14 23:11           ` Daniil V. Kolpakov
2009-12-14 23:46           ` Peter Hutterer
     [not found]             ` <20091214234652.GB23029-FGgOdbPLo1kAicBL8TP8PQ@public.gmane.org>
2009-12-15  7:43               ` Matthew Helsley
2009-12-15 10:32                 ` Peter Hutterer [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20091215103204.GA2144@salty \
    --to=peter.hutterer@who-t.net \
    --cc=linux-input@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=matt.helsley@gmail.com \
    --cc=xorg@lists.freedesktop.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).