Re: [PATCH 1/1] Input: trackpoint - Add new ELAN trackpoint device variant id 7

[Yao Yuan]

On Mon, Nov 09, 2026 at 06:36:15PM -0800, Dmitry Torokhov wrote:
> Hi Yuan,
>
> On Mon, Nov 02, 2026 at 09:08:36PM +0800, Yuan Yao wrote:
> > The new ELAN trackpoint variant id 7 is found on thinkpad
> > X14 Gen1 laptop(Product Name: 21XMA006CD), it's now
> > incorrectly probed as Generic or Logitech PS/2 mouse.
> >
> > Add ELAN variant id 7 to match the TP_READ_ID command
> > return value, probe the device as trackpoint correctly.
> >
> > BEFORE:
> > input: PS/2 Generic Mouse as /devices/platform/i8042/serio1/input/input63
> >
> > AFTER:
> > psmouse serio1: trackpoint: Elan_variant7 TrackPoint firmware: 0xa1, buttons: 3/3
> > input: TPPS/2 Elan_variant7 TrackPoint as /devices/platform/i8042/serio1/input/input65
> >
> > Signed-off-by: Yuan Yao <yaoyuan0329os@xxxxxxxxx>
> > ---
> >  drivers/input/mouse/trackpoint.c | 2 ++
> >  drivers/input/mouse/trackpoint.h | 1 +
> >  2 files changed, 3 insertions(+)
> >
> > diff --git a/drivers/input/mouse/trackpoint.c b/drivers/input/mouse/trackpoint.c
> > index 3bd8fdf56cd3..f46fc1665c96 100644
> > --- a/drivers/input/mouse/trackpoint.c
> > +++ b/drivers/input/mouse/trackpoint.c
> > @@ -25,6 +25,7 @@ static const char * const trackpoint_variants[] = {
> >  	[TP_VARIANT_NXP]		= "NXP",
> >  	[TP_VARIANT_JYT_SYNAPTICS]	= "JYT_Synaptics",
> >  	[TP_VARIANT_SYNAPTICS]		= "Synaptics",
> > +	[TP_VARIANT_ELAN_ID_7]		= "Elan_variant7"
>

Hi Dmitry,

> I wonder if we really need to say "Elan_variant7" or if we should simply
> say "Elan" here as well.

Thanks for your reply.

I'm fine to just "Elan" if it's ok to you. I just want to
highlight the device has id 7 from other ELAN devices which
has 3, so I give it "Elan_variant7".

>
> Jingle, is this your part? What is the difference from Elan trackpoint
> reporting 0x03 as its variant?
>
> Thanks.
>
> --
> Dmitry