linux-kernel - RE: [PATCH] hp_accel: Add accelerometer quirk for HP ProBook 440 G4

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Mon, 30 Oct 2017 06:10:15 +0000
From:   Osama Khan <osama.khan@...csson.com>
To:     Andy Shevchenko <andy.shevchenko@...il.com>
CC:     "eric.piel@...mplin-utc.net" <eric.piel@...mplin-utc.net>,
        "platform-driver-x86@...r.kernel.org" 
        <platform-driver-x86@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: RE: [PATCH] hp_accel: Add accelerometer quirk for HP ProBook 440 G4

Thanks for reviewing the patch. I'll keep your feedback in mind for future submissions.

-----Original Message-----
From: platform-driver-x86-owner@...r.kernel.org [mailto:platform-driver-x86-owner@...r.kernel.org] On Behalf Of Andy Shevchenko
Sent: Friday, October 27, 2017 9:31 PM
To: Osama Khan <osama.khan@...csson.com>
Cc: eric.piel@...mplin-utc.net; platform-driver-x86@...r.kernel.org; linux-kernel@...r.kernel.org
Subject: Re: [PATCH] hp_accel: Add accelerometer quirk for HP ProBook 440 G4

On Sat, Oct 21, 2017 at 1:42 PM, Osama Khan <osama.khan@...csson.com> 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@...csson.com>
> ---
> 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