Re: [Linuxwacom-devel] [PATCH 0/4] ISDV4 cleanup stage 2

2010-03-18 Thread Timo Aaltonen
On Thu, 18 Mar 2010, Peter Hutterer wrote:


 This patch series includes the parsing patches that I had sent earlier
 already, though reworked and this time even tested (I have an lenovo x201 on
 my desk ATM).
 It's no functional change, the only real thing here is that the protocol
 parsing has moved into a separate function, making it (hopefully) easier to
 debug and add new features in the future as new devices appear.

 Available for testing on the isdv4-cleanup-stage2 branch of
 git://people.freedesktop.org/~whot/xf86-input-wacom

 Please give this a quick spin, if it works for you I'd like to merge it and
 push out 0.10.5 - it's long overdue. If it breaks something, I'll probably
 push out 0.10.5 without it and let this simmer for longer.

Seems to work just fine with Thinkpad X61 tablet which has a serial 
device.

Tested-by: Timo Aaltonen tjaal...@ubuntu.com


t

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
___
Linuxwacom-devel mailing list
Linuxwacom-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxwacom-devel


Re: [Linuxwacom-devel] [PATCH 0/4] ISDV4 cleanup stage 2

2010-03-18 Thread Chris Bagwell
On Thu, Mar 18, 2010 at 12:32 AM, Peter Hutterer
peter.hutte...@who-t.net wrote:
 Please give this a quick spin, if it works for you I'd like to merge it and
 push out 0.10.5 - it's long overdue. If it breaks something, I'll probably
 push out 0.10.5 without it and let this simmer for longer.

Peter, before you release 0.10.5, I'd really like to get a Bamboo
specific work around in.  Linuxwacom has already deleted the logic in
question but it can't be totally deleted in xf86-input-wacom until a
large patch related to gestures is brought over.

I'll send a patch in tonight but in case your working fast, here is
the code in question.

In wcmUSB.c - wcmParseChannel() - Line 999 - This logic is meant to
only execute on touch screens.  It doesn't make sense to execute on
touch pads and is source of common complaint that Bamboo users are
getting unwanted mouse clicks:

if (common-wcmCapacityDefault  0)
MOD_BUTTONS (0, event-value);

I wrapped it with following for time being (from my MT patch though):

/* FIXME: This conflicts with gesture logic
 * in wcmEvent().  It should be centralized
 * there.  Also, touchpads should not be
 * calling this.   We do not have the
 * correct priv pointer handy to check
 * for ABSOLUTE mode so doing it the easy
 * way until gesture logic is centralized.
 */
if (!(common-tablet_id = 0xd0 
common-tablet_id = 0xd3))

Chris

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
___
Linuxwacom-devel mailing list
Linuxwacom-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxwacom-devel


Re: [Linuxwacom-devel] [PATCH 0/4] ISDV4 cleanup stage 2

2010-03-18 Thread Bryan Hundven
On Thu, Mar 18, 2010 at 3:57 PM, Peter Hutterer
peter.hutte...@who-t.net wrote:
 Hi Bryan,
 On Thu, Mar 18, 2010 at 11:31:16AM -0700, Bryan Hundven wrote:
 In my previous testing, I had my dock connected to my x61, and I was
 able to use my tablet. The dock also has a serial port.

 I am not using my dock now and found that the tablet does not work.
 Attached is my Xorg.0.log

 Just one question though - is this a new error resulting from the branch
 below or is this the case with the master branch already too?

Yes, I get the same issue with master.

 From your logfile, it looks like the device has some other IO issues though.
 (EE) Serial Wacom Tablet eraser: wcmWriteWait error : Input/output error
 Which may be explained by the switch from dock - non-dock, but search me
 what exactly happens here.

In a previous thread, Chris pointed to a possible issue with my bios version.
I checked that and was already running the newest version.
But, I have issues in other places, for instance I have problems with
MTRR with my intel video driver. It could be that something is up with
my serial port as well.

 To get more info as to why I'm not able to use it, would I need to
 build Xorg with debug info?

 Debuginfo is useful when you're debugging the driver with gdb (you'll
 need two machines for that, see
 ahttp://wiki.x.org/wiki/Development/Documentation/ServerDebugging).
 I recommend to use --enable-debug at configure time and start with the
 highest DebugLevel and CommonDebug options for the device. This should shed
 some more light onto the situation.

 Cheers,
  Peter

Luckly, I have a jhbuild of xorg already done, I'll just need to
rebuild with --enable-debug.

Thanks again Peter!

 Here is the output from udevadm:
 br...@bryan-x61t:~$ udevadm info -a -p  $(udevadm info -q path -n /dev/ttyS0)

 Udevadm info starts with the device specified by the devpath and then
 walks up the chain of parent devices. It prints for every device
 found, all possible attributes in the udev rules key format.
 A rule to match, can be composed by the attributes of the device
 and the attributes from one single parent device.

   looking at device '/devices/pnp0/00:0a/tty/ttyS0':
     KERNEL==ttyS0
     SUBSYSTEM==tty
     DRIVER==

   looking at parent device '/devices/pnp0/00:0a':
     KERNELS==00:0a
     SUBSYSTEMS==pnp
     DRIVERS==serial
     ATTRS{id}==WACf008

   looking at parent device '/devices/pnp0':
     KERNELS==pnp0
     SUBSYSTEMS==
     DRIVERS==

 On Wed, Mar 17, 2010 at 10:32 PM, Peter Hutterer
 peter.hutte...@who-t.net wrote:
 
  This patch series includes the parsing patches that I had sent earlier
  already, though reworked and this time even tested (I have an lenovo x201 
  on
  my desk ATM).
  It's no functional change, the only real thing here is that the protocol
  parsing has moved into a separate function, making it (hopefully) easier to
  debug and add new features in the future as new devices appear.
 
  Available for testing on the isdv4-cleanup-stage2 branch of
  git://people.freedesktop.org/~whot/xf86-input-wacom
 
  Please give this a quick spin, if it works for you I'd like to merge it and
  push out 0.10.5 - it's long overdue. If it breaks something, I'll probably
  push out 0.10.5 without it and let this simmer for longer.
 
  Cheers,
   Peter
 

 --
 Bryan Hundven
 bryanhund...@gmail.com


 X.Org X Server 1.7.5
 Release Date: 2010-02-16
 X Protocol Version 11, Revision 0
 Build Operating System: Linux 2.6.24-23-server i686 Ubuntu
 Current Operating System: Linux bryan-x61t 2.6.34-rc1 #4 SMP PREEMPT Sat Mar 
 13 19:12:37 PST 2010 i686
 Kernel command line: BOOT_IMAGE=/boot/vmlinuz-2.6.34-rc1 
 root=UUID=4e2189a6-19f6-4caf-92dc-dd0260adc487 ro quiet splash
 Build Date: 12 March 2010  07:37:07AM
 xorg-server 2:1.7.5-1ubuntu3 (buildd@)
 Current version of pixman: 0.16.4
       Before reporting problems, check http://wiki.x.org
       to make sure that you have the latest version.
 Markers: (--) probed, (**) from config file, (==) default setting,
       (++) from command line, (!!) notice, (II) informational,
       (WW) warning, (EE) error, (NI) not implemented, (??) unknown.
 (==) Log file: /var/log/Xorg.0.log, Time: Thu Mar 18 11:24:51 2010
 (==) Using config file: /etc/X11/xorg.conf
 (==) ServerLayout X.org Configured
 (**) |--Screen Screen0 (0)
 (**) |   |--Monitor Monitor0
 (**) |   |--Device Card0
 (**) |--Input Device Mouse0
 (**) |--Input Device Keyboard0
 (**) |--Input Device stylus
 (**) |--Input Device eraser
 (**) |--Input Device cursor
 (**) |--Input Device touch
 (==) Automatically adding devices
 (==) Automatically enabling devices
 (WW) The directory /usr/share/fonts/X11/cyrillic does not exist.
       Entry deleted from font path.
 (WW) The directory /usr/share/fonts/X11/cyrillic does not exist.
       Entry deleted from font path.
 (**) FontPath set to:
       /usr/share/fonts/X11/misc,
       /usr/share/fonts/X11/100dpi/:unscaled,
       /usr/share/fonts/X11/75dpi/:unscaled,
   

[Linuxwacom-devel] [PATCH 0/4] ISDV4 cleanup stage 2

2010-03-17 Thread Peter Hutterer

This patch series includes the parsing patches that I had sent earlier
already, though reworked and this time even tested (I have an lenovo x201 on
my desk ATM).
It's no functional change, the only real thing here is that the protocol
parsing has moved into a separate function, making it (hopefully) easier to
debug and add new features in the future as new devices appear.

Available for testing on the isdv4-cleanup-stage2 branch of 
git://people.freedesktop.org/~whot/xf86-input-wacom

Please give this a quick spin, if it works for you I'd like to merge it and
push out 0.10.5 - it's long overdue. If it breaks something, I'll probably
push out 0.10.5 without it and let this simmer for longer.

Cheers,
  Peter

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
___
Linuxwacom-devel mailing list
Linuxwacom-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxwacom-devel