linux-kernel - Re: [PATCH v8 4/4] HID: i2c-hid: Introduce goodix-i2c-hid using i2c-hid core

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] [thread-next>] [day] [month] [year] [list]

Date:   Wed, 6 Jan 2021 08:44:05 -0800
From:   Doug Anderson <dianders@...omium.org>
To:     Jiri Kosina <jkosina@...e.cz>,
        Benjamin Tissoires <benjamin.tissoires@...hat.com>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        Dmitry Torokhov <dmitry.torokhov@...il.com>
Cc:     Hans de Goede <hdegoede@...hat.com>,
        "open list:HID CORE LAYER" <linux-input@...r.kernel.org>,
        Kai-Heng Feng <kai.heng.feng@...onical.com>,
        Rob Herring <robh+dt@...nel.org>,
        Stephen Boyd <swboyd@...omium.org>,
        Andrea Borgia <andrea@...gia.bo.it>,
        Jiri Kosina <jikos@...nel.org>,
        Masahiro Yamada <masahiroy@...nel.org>,
        LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v8 4/4] HID: i2c-hid: Introduce goodix-i2c-hid using
 i2c-hid core

Hi,

On Fri, Dec 11, 2020 at 2:25 PM Douglas Anderson <dianders@...omium.org> wrote:
>
> +static const struct goodix_i2c_hid_timing_data goodix_gt7375p_timing_data = {
> +       .post_power_delay_ms = 10,
> +       .post_gpio_reset_delay_ms = 120,

As I've been going through my holiday email I found that Goodix has
released a new datasheet (datasheet rev 0.6 from 2020-12-11) where
this moves from 120 ms to 180 ms.  Sigh.

I'm hoping that we can just land it with the 120 though and then I'll
send a follow-up patch to change to 180.  This will avoid spamming
everyone with a v9 of the patch series.  Please yell if there are any
objections to that plan.  I'd also be fine if you wanted to just
change it to 180 when landing it.

Thanks much!

-Doug