* hp_accel: add two more axis information
@ 2009-03-08 21:03 Pavel Machek
2009-03-08 21:06 ` Daniel Mack
0 siblings, 1 reply; 5+ messages in thread
From: Pavel Machek @ 2009-03-08 21:03 UTC (permalink / raw)
To: Eric Piel, Andrew Morton, kernel list, Vladimir Botka, mmarek
Add two more laptops to whitelist.
Signed-off-by: Michal Marek <mmarek@suse.cz>
Signed-off-by: Pavel Machek <pavel@ucw.cz>
diff --git a/drivers/hwmon/hp_accel.c b/drivers/hwmon/hp_accel.c
index 29c83b5..448a462 100644
--- a/drivers/hwmon/hp_accel.c
+++ b/drivers/hwmon/hp_accel.c
@@ -187,6 +187,7 @@ static struct dmi_system_id lis3lv02d_dmi_ids[] = {
AXIS_DMI_MATCH("NC2510", "HP Compaq 2510", y_inverted),
AXIS_DMI_MATCH("NC8510", "HP Compaq 8510", xy_swap_inverted),
AXIS_DMI_MATCH("HP2133", "HP 2133", xy_rotated_left),
+ AXIS_DMI_MATCH("HP2140", "HP 2140", xy_swap_inverted),
AXIS_DMI_MATCH("NC653x", "HP Compaq 653", xy_rotated_left_usd),
AXIS_DMI_MATCH("NC673x", "HP Compaq 673", xy_rotated_left_usd),
AXIS_DMI_MATCH("NC651xx", "HP Compaq 651", xy_rotated_right),
@@ -201,6 +202,7 @@ static struct dmi_system_id lis3lv02d_dmi_ids[] = {
PRODUCT_NAME, "HP Pavilion dv5",
BOARD_NAME, "3600",
y_inverted),
+ AXIS_DMI_MATCH("DV7", "HP Pavilion dv7", x_inverted),
{ NULL, }
/* Laptop models without axis info (yet):
* "NC6910" "HP Compaq 6910"
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
^ permalink raw reply related [flat|nested] 5+ messages in thread* Re: hp_accel: add two more axis information
2009-03-08 21:03 hp_accel: add two more axis information Pavel Machek
@ 2009-03-08 21:06 ` Daniel Mack
2009-03-08 21:07 ` Pavel Machek
0 siblings, 1 reply; 5+ messages in thread
From: Daniel Mack @ 2009-03-08 21:06 UTC (permalink / raw)
To: Pavel Machek
Cc: Eric Piel, Andrew Morton, kernel list, Vladimir Botka, mmarek
On Sun, Mar 08, 2009 at 10:03:30PM +0100, Pavel Machek wrote:
> Add two more laptops to whitelist.
>
> Signed-off-by: Michal Marek <mmarek@suse.cz>
> Signed-off-by: Pavel Machek <pavel@ucw.cz>
>
>
> diff --git a/drivers/hwmon/hp_accel.c b/drivers/hwmon/hp_accel.c
> index 29c83b5..448a462 100644
> --- a/drivers/hwmon/hp_accel.c
> +++ b/drivers/hwmon/hp_accel.c
> @@ -187,6 +187,7 @@ static struct dmi_system_id lis3lv02d_dmi_ids[] = {
> AXIS_DMI_MATCH("NC2510", "HP Compaq 2510", y_inverted),
> AXIS_DMI_MATCH("NC8510", "HP Compaq 8510", xy_swap_inverted),
> AXIS_DMI_MATCH("HP2133", "HP 2133", xy_rotated_left),
> + AXIS_DMI_MATCH("HP2140", "HP 2140", xy_swap_inverted),
> AXIS_DMI_MATCH("NC653x", "HP Compaq 653", xy_rotated_left_usd),
> AXIS_DMI_MATCH("NC673x", "HP Compaq 673", xy_rotated_left_usd),
> AXIS_DMI_MATCH("NC651xx", "HP Compaq 651", xy_rotated_right),
> @@ -201,6 +202,7 @@ static struct dmi_system_id lis3lv02d_dmi_ids[] = {
> PRODUCT_NAME, "HP Pavilion dv5",
> BOARD_NAME, "3600",
> y_inverted),
> + AXIS_DMI_MATCH("DV7", "HP Pavilion dv7", x_inverted),
> { NULL, }
> /* Laptop models without axis info (yet):
> * "NC6910" "HP Compaq 6910"
Does this apply on top of my recent ACPI resolve patches? Could you test
the latest versions already?
Daniel
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: hp_accel: add two more axis information
2009-03-08 21:06 ` Daniel Mack
@ 2009-03-08 21:07 ` Pavel Machek
2009-03-23 0:11 ` Andrew Morton
0 siblings, 1 reply; 5+ messages in thread
From: Pavel Machek @ 2009-03-08 21:07 UTC (permalink / raw)
To: Daniel Mack; +Cc: Eric Piel, Andrew Morton, kernel list, Vladimir Botka, mmarek
On Sun 2009-03-08 22:06:25, Daniel Mack wrote:
> On Sun, Mar 08, 2009 at 10:03:30PM +0100, Pavel Machek wrote:
> > Add two more laptops to whitelist.
> >
> > Signed-off-by: Michal Marek <mmarek@suse.cz>
> > Signed-off-by: Pavel Machek <pavel@ucw.cz>
> >
> >
> > diff --git a/drivers/hwmon/hp_accel.c b/drivers/hwmon/hp_accel.c
> > index 29c83b5..448a462 100644
> > --- a/drivers/hwmon/hp_accel.c
> > +++ b/drivers/hwmon/hp_accel.c
> > @@ -187,6 +187,7 @@ static struct dmi_system_id lis3lv02d_dmi_ids[] = {
> > AXIS_DMI_MATCH("NC2510", "HP Compaq 2510", y_inverted),
> > AXIS_DMI_MATCH("NC8510", "HP Compaq 8510", xy_swap_inverted),
> > AXIS_DMI_MATCH("HP2133", "HP 2133", xy_rotated_left),
> > + AXIS_DMI_MATCH("HP2140", "HP 2140", xy_swap_inverted),
> > AXIS_DMI_MATCH("NC653x", "HP Compaq 653", xy_rotated_left_usd),
> > AXIS_DMI_MATCH("NC673x", "HP Compaq 673", xy_rotated_left_usd),
> > AXIS_DMI_MATCH("NC651xx", "HP Compaq 651", xy_rotated_right),
> > @@ -201,6 +202,7 @@ static struct dmi_system_id lis3lv02d_dmi_ids[] = {
> > PRODUCT_NAME, "HP Pavilion dv5",
> > BOARD_NAME, "3600",
> > y_inverted),
> > + AXIS_DMI_MATCH("DV7", "HP Pavilion dv7", x_inverted),
> > { NULL, }
> > /* Laptop models without axis info (yet):
> > * "NC6910" "HP Compaq 6910"
>
> Does this apply on top of my recent ACPI resolve patches? Could you test
> the latest versions already?
I guess it should; you did not change anything in the
whitelist. Unfortunately, I no longer have hardware to do testing.
Pavel
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: hp_accel: add two more axis information
2009-03-08 21:07 ` Pavel Machek
@ 2009-03-23 0:11 ` Andrew Morton
2009-03-23 10:07 ` Pavel Machek
0 siblings, 1 reply; 5+ messages in thread
From: Andrew Morton @ 2009-03-23 0:11 UTC (permalink / raw)
To: Pavel Machek; +Cc: Daniel Mack, Eric Piel, kernel list, Vladimir Botka, mmarek
On Sun, 8 Mar 2009 22:07:57 +0100 Pavel Machek <pavel@ucw.cz> wrote:
> On Sun 2009-03-08 22:06:25, Daniel Mack wrote:
> > On Sun, Mar 08, 2009 at 10:03:30PM +0100, Pavel Machek wrote:
> > > Add two more laptops to whitelist.
> > >
> > > Signed-off-by: Michal Marek <mmarek@suse.cz>
> > > Signed-off-by: Pavel Machek <pavel@ucw.cz>
> > >
> > >
> > > diff --git a/drivers/hwmon/hp_accel.c b/drivers/hwmon/hp_accel.c
> > > index 29c83b5..448a462 100644
> > > --- a/drivers/hwmon/hp_accel.c
> > > +++ b/drivers/hwmon/hp_accel.c
> > > @@ -187,6 +187,7 @@ static struct dmi_system_id lis3lv02d_dmi_ids[] = {
> > > AXIS_DMI_MATCH("NC2510", "HP Compaq 2510", y_inverted),
> > > AXIS_DMI_MATCH("NC8510", "HP Compaq 8510", xy_swap_inverted),
> > > AXIS_DMI_MATCH("HP2133", "HP 2133", xy_rotated_left),
> > > + AXIS_DMI_MATCH("HP2140", "HP 2140", xy_swap_inverted),
> > > AXIS_DMI_MATCH("NC653x", "HP Compaq 653", xy_rotated_left_usd),
> > > AXIS_DMI_MATCH("NC673x", "HP Compaq 673", xy_rotated_left_usd),
> > > AXIS_DMI_MATCH("NC651xx", "HP Compaq 651", xy_rotated_right),
> > > @@ -201,6 +202,7 @@ static struct dmi_system_id lis3lv02d_dmi_ids[] = {
> > > PRODUCT_NAME, "HP Pavilion dv5",
> > > BOARD_NAME, "3600",
> > > y_inverted),
> > > + AXIS_DMI_MATCH("DV7", "HP Pavilion dv7", x_inverted),
> > > { NULL, }
> > > /* Laptop models without axis info (yet):
> > > * "NC6910" "HP Compaq 6910"
> >
> > Does this apply on top of my recent ACPI resolve patches? Could you test
> > the latest versions already?
>
> I guess it should; you did not change anything in the
> whitelist. Unfortunately, I no longer have hardware to do testing.
I'm all confused and don't know what to do with this patch.
I guess I'll just apply it and see if I get shouted at.
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: hp_accel: add two more axis information
2009-03-23 0:11 ` Andrew Morton
@ 2009-03-23 10:07 ` Pavel Machek
0 siblings, 0 replies; 5+ messages in thread
From: Pavel Machek @ 2009-03-23 10:07 UTC (permalink / raw)
To: Andrew Morton; +Cc: Daniel Mack, Eric Piel, kernel list, Vladimir Botka, mmarek
On Sun 2009-03-22 17:11:28, Andrew Morton wrote:
> On Sun, 8 Mar 2009 22:07:57 +0100 Pavel Machek <pavel@ucw.cz> wrote:
>
> > On Sun 2009-03-08 22:06:25, Daniel Mack wrote:
> > > On Sun, Mar 08, 2009 at 10:03:30PM +0100, Pavel Machek wrote:
> > > > Add two more laptops to whitelist.
> > > >
> > > > Signed-off-by: Michal Marek <mmarek@suse.cz>
> > > > Signed-off-by: Pavel Machek <pavel@ucw.cz>
> > > >
> > > >
> > > > diff --git a/drivers/hwmon/hp_accel.c b/drivers/hwmon/hp_accel.c
> > > > index 29c83b5..448a462 100644
> > > > --- a/drivers/hwmon/hp_accel.c
> > > > +++ b/drivers/hwmon/hp_accel.c
> > > > @@ -187,6 +187,7 @@ static struct dmi_system_id lis3lv02d_dmi_ids[] = {
> > > > AXIS_DMI_MATCH("NC2510", "HP Compaq 2510", y_inverted),
> > > > AXIS_DMI_MATCH("NC8510", "HP Compaq 8510", xy_swap_inverted),
> > > > AXIS_DMI_MATCH("HP2133", "HP 2133", xy_rotated_left),
> > > > + AXIS_DMI_MATCH("HP2140", "HP 2140", xy_swap_inverted),
> > > > AXIS_DMI_MATCH("NC653x", "HP Compaq 653", xy_rotated_left_usd),
> > > > AXIS_DMI_MATCH("NC673x", "HP Compaq 673", xy_rotated_left_usd),
> > > > AXIS_DMI_MATCH("NC651xx", "HP Compaq 651", xy_rotated_right),
> > > > @@ -201,6 +202,7 @@ static struct dmi_system_id lis3lv02d_dmi_ids[] = {
> > > > PRODUCT_NAME, "HP Pavilion dv5",
> > > > BOARD_NAME, "3600",
> > > > y_inverted),
> > > > + AXIS_DMI_MATCH("DV7", "HP Pavilion dv7", x_inverted),
> > > > { NULL, }
> > > > /* Laptop models without axis info (yet):
> > > > * "NC6910" "HP Compaq 6910"
> > >
> > > Does this apply on top of my recent ACPI resolve patches? Could you test
> > > the latest versions already?
> >
> > I guess it should; you did not change anything in the
> > whitelist. Unfortunately, I no longer have hardware to do testing.
>
> I'm all confused and don't know what to do with this patch.
>
> I guess I'll just apply it and see if I get shouted at.
That sounds like a plan :-). And I don't think you'll get shouted at,
the patch should be very safe.
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2009-03-23 10:04 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-03-08 21:03 hp_accel: add two more axis information Pavel Machek
2009-03-08 21:06 ` Daniel Mack
2009-03-08 21:07 ` Pavel Machek
2009-03-23 0:11 ` Andrew Morton
2009-03-23 10:07 ` Pavel Machek
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox