Re: [PATCH] hp_accel: Add accelerometer quirk for HP ProBook 440 G4

[Andy Shevchenko]

On Sat, Oct 21, 2017 at 1:42 PM, Osama Khan <osama.khan@xxxxxxxxxxxx> wrote:
> Added support for HP ProBook 440 G4 laptops by including the accelerometer
> orientation quirk for that device. Testing was performed based on the
> axis orientation guidelines here:
> https://www.kernel.org/doc/Documentation/misc-devices/lis3lv02d
> which states "If the left side is elevated, X increases (becomes positive)"
>
> When tested, on lifting the left edge, x values became increasingly negative
> thus indicating an inverted x-axis on the installed lis3lv02d chip.
> This was compensated by adding an entry for this device in hp_accel.c
> specifying the quirk as x_inverted. The patch was tested on a
> ProBook 440 G4 device and x-axis as well as y and z-axis values are now
> generated as per spec.
>

Thanks for the patch.

First of all, indentation is mangled and it doesn't clearly applies.
Second, we use a prefix platform/x86 for patches against PDx86 subsystem.

P.S. No need to resend this time, just check your mail client and be
aware about the prefix in the future.

> Signed-off-by: Osama Khan <osama.khan@xxxxxxxxxxxx>
> ---
> diff --git a/drivers/platform/x86/hp_accel.c b/drivers/platform/x86/hp_accel.c
> index 493d8910a74e..7b12abe86b94 100644
> --- a/drivers/platform/x86/hp_accel.c
> +++ b/drivers/platform/x86/hp_accel.c
> @@ -240,6 +240,7 @@ static const struct dmi_system_id lis3lv02d_dmi_ids[] = {
>         AXIS_DMI_MATCH("HDX18", "HP HDX 18", x_inverted),
>         AXIS_DMI_MATCH("HPB432x", "HP ProBook 432", xy_rotated_left),
>         AXIS_DMI_MATCH("HPB440G3", "HP ProBook 440 G3", x_inverted_usd),
> +       AXIS_DMI_MATCH("HPB440G4", "HP ProBook 440 G4", x_inverted),
>         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),



-- 
With Best Regards,
Andy Shevchenko