Re: [PATCH] Apple Motion Sensor driver

[Jean Delvare <khali@linux-fr.org>]

On Mon, 13 Nov 2006 17:20:26 +0100, Johannes Berg wrote:
> On Mon, 2006-11-13 at 17:14 +0100, Stelian Pop wrote:
> > 
> > +               input_report_abs(ams_info.idev, ABS_X, x - ams_info.xcalib);
> > +               input_report_abs(ams_info.idev, ABS_Y, y - ams_info.ycalib);
> > +               input_report_abs(ams_info.idev, ABS_Z, z - ams_info.zcalib); 
> 
> Sorry about chiming in so late. When I tried to use this with neverball,
> ams_info.xcalib - x (and similar for the others) was more useful because
> of the way things are oriented. If I tilt my powerbook to the left then
> with this original code the mouse cursor moves to the right which is
> contrary to what neverball expects.
> 
> Not sure if we want to change this or not, it sort of boils down to a
> userspace issue and we could just patch neverball to have a direction
> inversion :)

For what it's worth, the hdaps driver offers a parameter to invert the
axes at the driver level. It sounds sane to me, as ideally we should be
able to detect it from the hardware, and user-space should not need to
care about hardware specific details.

-- 
Jean Delvare