Re: [PATCH v3] platform/x86: ideapad-laptop: do not poke keyboard controller

From: Dmitry Torokhov

Date: Mon Oct 26 2026 - 23:07:06 EST


Hi Rong,

On Sun, Oct 25, 2026 at 10:25:46AM +0800, Rong Zhang wrote:
> > +static const struct input_device_id ideapad_tpswitch_ids[] = {
> > + {
> > + .flags = INPUT_DEVICE_ID_MATCH_EVBIT |
> > + INPUT_DEVICE_ID_MATCH_KEYBIT |
> > + INPUT_DEVICE_ID_MATCH_ABSBIT,
> > + .bustype = BUS_I8042,
> > + .vendor = 0x0002,
>
> Missing INPUT_DEVICE_ID_MATCH_BUS and INPUT_DEVICE_ID_MATCH_VENDOR?
>
> Other than that and the module parameter's name, the patch looks good to
> me. With them fixed:
>
> Reviewed-by: Rong Zhang <i@xxxxxxxx>
> Tested-by: Rong Zhang <i@xxxxxxxx> # not matching other touchpads

Indeed, well spotted! I have added the missing match flags and your tags
in v4.

Thanks.

--
Dmitry