[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <nx3huvqw5b3kvoklqyy6ryv7g7uh66vwp5abyw44ppm7vtts7j@rmq6oswnunu6>
Date: Sun, 28 Dec 2025 15:15:20 -0800
From: Dmitry Torokhov <dmitry.torokhov@...il.com>
To: Andy Shevchenko <andriy.shevchenko@...ux.intel.com>
Cc: duoming@....edu.cn, linux-input@...r.kernel.org,
linux-kernel@...r.kernel.org, pali@...nel.org, kuba@...nel.org, alexander.deucher@....com,
akpm@...ux-foundation.org, johannes.berg@...el.com, pkshih@...ltek.com,
hverkuil+cisco@...nel.org, tglx@...utronix.de, mingo@...nel.org
Subject: Re: [PATCH v2 RESEND 2/2] Input: psmouse - Replace flush_workqueue()
with disable_delayed_work_sync()
On Sat, Dec 27, 2025 at 05:58:05PM +0200, Andy Shevchenko wrote:
> On Thu, Dec 18, 2025 at 10:25:27PM +0800, duoming@....edu.cn wrote:
> > On Wed, 17 Dec 2025 22:58:33 -0800 Dmitry Torokhov wrote:
>
> ...
>
> > > What do you mean? If you enable OLPC support the option to enable hgpk
> > > protocol driver should become available, right?
> >
> > The hgpk_init() function is never called by any other function in the
> > kernel and is therefore dead code. Since the delayed work item recalib_wq
> > is initialized within this function, it is consequently never scheduled
> > during runtime.
>
> Actually I confirm this as of v6.19-rc2. Perhaps some patch removed some
> functionality in the past and we may remove more now?
Yeah, it looks like I broke it with c378b5119eb0 ("Input: psmouse -
factor out common protocol probing code") in 2015. While it should be
easy to fix (by adding hgkp_init() to the protocol definition) I think
we should drop hgkp extended support since nobody complained in 10
years.
Thanks.
--
Dmitry
Powered by blists - more mailing lists