Re: axes conversion for hp compaq 8710w for HP_ACCEL linux kernel module (fwd)

[Andrew Morton <akpm@linux-foundation.org>]

On Sun, 22 Mar 2009 00:08:19 +0100
Pavel Machek <pavel@ucw.cz> wrote:

> Hi!

hi to you too.

> It seems like Luca has one  more whitelist entry...
> 								Pavel
> 
> 
> Subject: axes conversion for hp compaq 8710w for HP_ACCEL linux kernel module
> From: Luca Cappa <lcappa@gmail.com>
> 
> 
> Hello,
> 
> I have a laptop HP Compaq 8710W, I compiled into my kernel the
> LIS3LV02DL and HP_ACCEL module drivers. While loading it cannot
> recognize the laptop
> model, so i am sending the necessary information to update the
> database of Axes Orientations.
> 
> >When the laptop is horizontal the position reported is about 0 for X and Y
> >and a positive value for Z
> Yes, it is about 0,0,1000, the actual reading says: (-17,-26,1018);
> 
> > If the left side is elevated, X increases (becomes positive)
> Yes, X goes toward to positive 1000.
> 
> >If the front side (where the touchpad is) is elevated, Y decreases (becomes negative)
> No, Y goes toward to positive 1000.
> 
> >If the laptop is put upside-down, Z becomes negative
> Yes, the laptop on a table Z gives 1000, and if upsidedown the Z reads
> -1000.
> 
> So in few words the Y axis is inverted,  I paste here below the very
> simple patch to configure properly the axis conversion. The patch is
> against the file hp_accel.c of 2.6.29-rc8, i.e. the one shown here:
> 
> http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=blob;f=drivers/hwmon/hp_accel.c;h=29c83b5b96974bb88cbeb67d896ba46ee71469f2;hb=HEAD
> 
> Greetings,
> Luca
> 
> 
> --- /home/luca/Desktop/hp_accel.c	2009-03-21 19:28:54.000000000 +0100
> +++ hp_accel.c	2009-03-21 19:31:05.000000000 +010
> @@ -201,6 +201,7 @@
>  			PRODUCT_NAME, "HP Pavilion dv5",
>  			BOARD_NAME, "3600",
>  			y_inverted),
> +    AXIS_DMI_MATCH("HP8710", "HP Compaq 8710", y_inverted),
>  	{ NULL, }
>  /* Laptop models witho

- changelog is a mess
- originator wasn't cc'ed
- originator's signed-off-by: is missing
- your signed-off-by: is missing
- patch isn't in `patch -p1' form
- patch is whitespace-mangled

I fixed some of that up.  It'd be nice to get some Signed-off-by:s,
please.