[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20090330155533.ee3e6dae.akpm@linux-foundation.org>
Date: Mon, 30 Mar 2009 15:55:33 -0700
From: Andrew Morton <akpm@...ux-foundation.org>
To: Pavel Machek <pavel@....cz>
Cc: linux-kernel@...r.kernel.org, eric.piel@...mplin-utc.net,
Luca Cappa <lcappa@...il.com>
Subject: Re: axes conversion for hp compaq 8710w for HP_ACCEL linux kernel
module (fwd)
On Sun, 22 Mar 2009 00:08:19 +0100
Pavel Machek <pavel@....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@...il.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.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists