[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Date: Wed, 9 Sep 2020 08:39:24 +0200 (CEST)
From: Jiri Kosina <jikos@...nel.org>
To: Douglas Anderson <dianders@...omium.org>
cc: Hans de Goede <hdegoede@...hat.com>,
Andrea Borgia <andrea@...gia.bo.it>,
Kai-Heng Feng <kai.heng.feng@...onical.com>,
Dmitry Torokhov <dmitry.torokhov@...il.com>,
Aaron Ma <aaron.ma@...onical.com>,
Benjamin Tissoires <benjamin.tissoires@...hat.com>,
Daniel Playfair Cal <daniel.playfair.cal@...il.com>,
Pavel Balan <admin@...ma.net>,
You-Sheng Yang <vicamo.yang@...onical.com>,
linux-input@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] HID: i2c-hid: Prefer asynchronous probe
On Wed, 2 Sep 2020, Douglas Anderson wrote:
> Adding printouts to the i2c_hid_probe() function shows that it takes
> quite some time. It used to take about 70 ms, but after commit
> eef4016243e9 ("HID: i2c-hid: Always sleep 60ms after I2C_HID_PWR_ON
> commands") it takes about 190 ms. This is not tons of time but it's
> not trivial. Because we haven't yet specified that we'd prefer
> asynchronous probe for this driver then, if the driver is builtin to
> the kernel, we'll wait for this driver to finish before we start
> probes for more drivers. Let's set the flag to enable asynchronous
> for this driver so that other drivers aren't blocked from probing
> until we finish.
>
> Since this driver can be configured as a module and modules are
> always asynchronously probed this is quite a safe change and will
> benefit anyone who has a reason to build this driver into the kernel
> instead of using it as a module.
>
> Signed-off-by: Douglas Anderson <dianders@...omium.org>
Applied, thanks.
--
Jiri Kosina
SUSE Labs
Powered by blists - more mailing lists