Bug 15129

Summary: elographics and InitValuatorAxisStruct problem
Product: xorg Reporter: Thomas <xorg>
Component: Input/elographicsAssignee: Xorg Project Team <xorg-team>
Status: RESOLVED WONTFIX QA Contact: Xorg Project Team <xorg-team>
Severity: major    
Priority: highest Keywords: NEEDINFO
Version: git   
Hardware: x86 (IA32)   
OS: Linux (All)   
Whiteboard:
i915 platform: i915 features:

Description Thomas 2008-03-19 07:28:10 UTC
Hello,

elographics drivers doesn't work on my system (xorg 7.3). The events are not correctly sent.

If I change the InitValuatorAxisStruct that correct my problem

<         /* X valuator */
<         xf86InitValuatorAxisStruct(dev, 0, -1, -1, 1, 0, 1);
<         xf86InitValuatorDefaults(dev, 0);
<         /* Y valuator */
<         xf86InitValuatorAxisStruct(dev, 1, -1, -1, 1, 0, 1);
<         xf86InitValuatorDefaults(dev, 1);
---
> 	InitValuatorAxisStruct(dev, 0, -1, -1,
> 			       9500,
> 			       0     /* min_res */,
> 			       9500  /* max_res */);
> 	InitValuatorAxisStruct(dev, 1, -1, -1,
> 			       10500,
> 			       0     /* min_res */,
> 			       10500 /* max_res */);
Comment 1 Peter Hutterer 2008-05-01 06:15:21 UTC
(In reply to comment #0)
> Hello,
> 
> elographics drivers doesn't work on my system (xorg 7.3). The events are not
> correctly sent.
> 
> If I change the InitValuatorAxisStruct that correct my problem
> 
> <         /* X valuator */
> <         xf86InitValuatorAxisStruct(dev, 0, -1, -1, 1, 0, 1);
> <         xf86InitValuatorDefaults(dev, 0);
> <         /* Y valuator */
> <         xf86InitValuatorAxisStruct(dev, 1, -1, -1, 1, 0, 1);
> <         xf86InitValuatorDefaults(dev, 1);
> ---
> > 	InitValuatorAxisStruct(dev, 0, -1, -1,
> > 			       9500,
> > 			       0     /* min_res */,
> > 			       9500  /* max_res */);
> > 	InitValuatorAxisStruct(dev, 1, -1, -1,
> > 			       10500,
> > 			       0     /* min_res */,
> > 			       10500 /* max_res */);


Thomas:
is there any way to get the axis range from the device rather than hardcoding it?

(also, please create patches either by git diff or diff -u. it makes them a lot easier to read)

Comment 2 Peter Hutterer 2008-07-16 05:42:46 UTC
WONTFIX, no reply for over two months and a patch that most likely breaks
other devices due to hardcoded valuator ranges.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.