2010-04-07 20:52:14

by Takashi Iwai

Subject: [PATCH] lis3: add support for HP ProBook 432x/442x/452x/522x

Correct axis-mappings for new HP ProBook laptops.

Signed-off-by: Takashi Iwai <[email protected]>
---
drivers/hwmon/hp_accel.c | 4 ++++
1 files changed, 4 insertions(+), 0 deletions(-)

diff --git a/drivers/hwmon/hp_accel.c b/drivers/hwmon/hp_accel.c
index be475e8..c8ab505 100644
--- a/drivers/hwmon/hp_accel.c
+++ b/drivers/hwmon/hp_accel.c
@@ -217,6 +217,10 @@ static struct dmi_system_id lis3lv02d_dmi_ids[] = {
AXIS_DMI_MATCH("DV7", "HP Pavilion dv7", x_inverted),
AXIS_DMI_MATCH("HP8710", "HP Compaq 8710", y_inverted),
AXIS_DMI_MATCH("HDX18", "HP HDX 18", x_inverted),
+ AXIS_DMI_MATCH("HPB432x", "HP ProBook 432", xy_rotated_left),
+ AXIS_DMI_MATCH("HPB442x", "HP ProBook 442", xy_rotated_left),
+ AXIS_DMI_MATCH("HPB452x", "HP ProBook 452", y_inverted),
+ AXIS_DMI_MATCH("HPB522x", "HP ProBook 522", xy_swap),
{ NULL, }
/* Laptop models without axis info (yet):
* "NC6910" "HP Compaq 6910"
--
1.7.0.3

2010-04-07 21:23:18

by Éric Piel

Subject: Re: [PATCH] lis3: add support for HP ProBook 432x/442x/452x/522x

Op 07-04-10 22:52, Takashi Iwai schreef:
> Correct axis-mappings for new HP ProBook laptops.
>
> Signed-off-by: Takashi Iwai <[email protected]>
Looks fine. Thanks Takashi!

Acked-by: Eric Piel <[email protected]>

Eric
> ---
> drivers/hwmon/hp_accel.c | 4 ++++
> 1 files changed, 4 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/hwmon/hp_accel.c b/drivers/hwmon/hp_accel.c
> index be475e8..c8ab505 100644
> --- a/drivers/hwmon/hp_accel.c
> +++ b/drivers/hwmon/hp_accel.c
> @@ -217,6 +217,10 @@ static struct dmi_system_id lis3lv02d_dmi_ids[] = {
> AXIS_DMI_MATCH("DV7", "HP Pavilion dv7", x_inverted),
> AXIS_DMI_MATCH("HP8710", "HP Compaq 8710", y_inverted),
> AXIS_DMI_MATCH("HDX18", "HP HDX 18", x_inverted),
> + AXIS_DMI_MATCH("HPB432x", "HP ProBook 432", xy_rotated_left),
> + AXIS_DMI_MATCH("HPB442x", "HP ProBook 442", xy_rotated_left),
> + AXIS_DMI_MATCH("HPB452x", "HP ProBook 452", y_inverted),
> + AXIS_DMI_MATCH("HPB522x", "HP ProBook 522", xy_swap),
> { NULL, }
> /* Laptop models without axis info (yet):
> * "NC6910" "HP Compaq 6910"