All of lore.kernel.org
 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.