Re: hp_accel: add two more axis information

From: Andrew Morton
Date: Sun Mar 22 2009 - 20:19:49 EST


On Sun, 8 Mar 2009 22:07:57 +0100 Pavel Machek <pavel@xxxxxx> 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@xxxxxxx>
> > > Signed-off-by: Pavel Machek <pavel@xxxxxx>
> > >
> > >
> > > 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.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/