From nobody Sun Oct 5 14:12:40 2025 Received: from mail-il1-f202.google.com (mail-il1-f202.google.com [209.85.166.202]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id B300B26FA58 for ; Mon, 4 Aug 2025 14:11:31 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.166.202 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1754316693; cv=none; b=dZTaSaEePOO5qAzvfqnlMbh92miXJZXPpqaC3t0XzuLouqqObBzfUMbCALU+aXnZ49eFBlhEtkCZeW2qXTb+GXwMYCZfjzVZXTxMjVZp+R2OfggRmwKT5Y79mNzJvNKORubmA2TFlku4bDg6IRLIHK55B6ktqin6w4HCGcjfCiU= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1754316693; c=relaxed/simple; bh=md9UePRU22vt6Mwv4wXF5OgaKfWWEer98Wt06d08/OQ=; h=Date:In-Reply-To:Mime-Version:References:Message-ID:Subject:From: To:Cc:Content-Type; b=DWkTP5qOJUHkTojbIlELpGh8W7LCF2A8etK1RloDTHV/ItkbZXQVojt/eMC+J42r7KGdUhCkzd1HaJt38wYgRoTUwROxIvltunKBnqmu++J1jhPmbVYMu9J2mtp7Apma6rQcY871OGPxSEjLgjG56fBFOqLe+6DKCeTgRkB1uh8= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com; spf=pass smtp.mailfrom=flex--jdenose.bounces.google.com; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b=OO06CBDX; arc=none smtp.client-ip=209.85.166.202 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=flex--jdenose.bounces.google.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b="OO06CBDX" Received: by mail-il1-f202.google.com with SMTP id e9e14a558f8ab-3e401ca74fbso33745215ab.1 for ; Mon, 04 Aug 2025 07:11:31 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20230601; t=1754316691; x=1754921491; darn=vger.kernel.org; h=cc:to:from:subject:message-id:references:mime-version:in-reply-to :date:from:to:cc:subject:date:message-id:reply-to; bh=F6NmZhEmKGYBR+PNmHw027JfaNjka2TCkl4+s269vJA=; b=OO06CBDXdIOcsR1QEVz0Q7qaeNDtCjUX0uzj+ZP40aVvf+bXbO8O5KiDnNqDV5NR3A qNeSx52fXUGVbYgDl2MXvst4uNrMg166lG3bfPLjll5lVYvZ/fLfJSlWLxFzV24xO28r q0mEXjUsimziIf+nfAq1oATW3Ruifj2Nkt0K/pH9uUqz87ia/oldC/Pii9EJL+LTPCba A+pxG0PMs3/EH6WP+YdA63MyB2U0d+ZgagjNEDndUwBs0yKqLAPxEG20u0k06sG/Hf3D wZJhxwf2L9IiYP0FXttKmJvEaZzGsNzsbEViULqFrdN7VtWjNUKgzFAjykrx8n5m/6Eg TTHQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1754316691; x=1754921491; h=cc:to:from:subject:message-id:references:mime-version:in-reply-to :date:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=F6NmZhEmKGYBR+PNmHw027JfaNjka2TCkl4+s269vJA=; b=R7wqdA8lOQKl0HV3j4itWJzRnlfxFkdOk7BC6gqPMPEth5qU7DBTcwac8QF0aLKA4K PLhcZ3Az7+El5bwr8RW/C5JxyISkjz3LwbOENlGciYp1LzqMLBUDAPY2Jkai3Xk7g1x4 uDJr4O6fXNP9yftzAhMUb4iMgOQoDhQRL7GhPL1zasHPeDlOIXig6gkcmgEu0MPkVjLI hYYbeEPdR0G41fBmmRKaSN/xSgJWnjLo1gW3NPw81AA2XaGxuBYSyBUJEVEXA7aBZjdP NzDVX4wUenxx7i6YTj1wJNo8GLC71uqERDcGjVqAB874GkIo2MsQj1i9ccMUCen1C8B1 aFdg== X-Forwarded-Encrypted: i=1; AJvYcCUkIK6MUvOy7mRqomZv/D8FOKZLxN6jj5ITPDmlSiY9h1l+E1NJFxIjOOJcs4h3HZMbCkYDwEo4WZsqaCQ=@vger.kernel.org X-Gm-Message-State: AOJu0YwdlYdWs7+ysM9W9VjCTb6XtnexsIJZvkG957/lzbgo+jC2+oM9 MxnGowkEhXAS5+yMYviJqMC/F6n59ZzlAOXLFz229H1T6HKgj3/d9LxCuwdk5Rk6A6m9ShcDKuO vTIhWkVclrQ== X-Google-Smtp-Source: AGHT+IFnashLwLqi+8ZIUULRCqBQ8xPAkOpNzmANHD0VchzkUehGr33uHUOQJBilDkrgfGR3jajFwfoMscpG X-Received: from ilbeh24.prod.google.com ([2002:a05:6e02:4c18:b0:3e3:cbd4:ca12]) (user=jdenose job=prod-delivery.src-stubby-dispatcher) by 2002:a05:6e02:1f85:b0:3e3:e78e:4e0a with SMTP id e9e14a558f8ab-3e41610d65cmr191359165ab.1.1754316690876; Mon, 04 Aug 2025 07:11:30 -0700 (PDT) Date: Mon, 04 Aug 2025 14:11:26 +0000 In-Reply-To: <20250804-support-forcepads-v2-0-138ca980261d@google.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 References: <20250804-support-forcepads-v2-0-138ca980261d@google.com> X-Mailer: b4 0.14.2 Message-ID: <20250804-support-forcepads-v2-11-138ca980261d@google.com> Subject: [PATCH v2 11/11] HID: multitouch: add haptic multitouch support From: Jonathan Denose To: Jiri Kosina , Benjamin Tissoires , Dmitry Torokhov , Jonathan Corbet , Henrik Rydberg Cc: linux-input@vger.kernel.org, linux-kernel@vger.kernel.org, linux-doc@vger.kernel.org, Angela Czubak , "Sean O'Brien" , Jonathan Denose Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable From: Angela Czubak Add new option (MULTITOUCH_HAPTIC) to mark whether hid-multitouch should try and configure simple haptic device. Once this option is configured, and the device is recognized to have simple haptic capabilities, check input frames for pressure and handle it using hid_haptic_* API. Signed-off-by: Angela Czubak Co-developed-by: Jonathan Denose Signed-off-by: Jonathan Denose --- drivers/hid/Kconfig | 11 ++++ drivers/hid/Makefile | 2 +- drivers/hid/hid-haptic.h | 52 +++++++++++++++++ drivers/hid/hid-multitouch.c | 136 +++++++++++++++++++++++++++++++++++++++= +++- 4 files changed, 199 insertions(+), 2 deletions(-) diff --git a/drivers/hid/Kconfig b/drivers/hid/Kconfig index ad6bcc4248cc111705d7cfde2b1481b46353e2d7..b7452f11a4f914f92af582ed054= d42ecbcd6cb9e 100644 --- a/drivers/hid/Kconfig +++ b/drivers/hid/Kconfig @@ -817,6 +817,17 @@ config HID_MULTITOUCH To compile this driver as a module, choose M here: the module will be called hid-multitouch. =20 +config MULTITOUCH_HAPTIC + bool "Simple haptic multitouch support" + depends on HID_MULTITOUCH + select HID_HAPTIC + default n + help + Support for simple multitouch haptic devices. + Adds extra parsing and FF device for the hid multitouch driver. + It can be used for Elan 2703 haptic touchpad. + To enable, say Y. + config HID_NINTENDO tristate "Nintendo Joy-Con, NSO, and Pro Controller support" depends on NEW_LEDS diff --git a/drivers/hid/Makefile b/drivers/hid/Makefile index 361a7daedeb85454114def8afb5f58caeab58a00..be09b4f13b2058a0a1d7eab79f3= 5def758120fc4 100644 --- a/drivers/hid/Makefile +++ b/drivers/hid/Makefile @@ -4,7 +4,7 @@ # hid-y :=3D hid-core.o hid-input.o hid-quirks.o hid-$(CONFIG_DEBUG_FS) +=3D hid-debug.o -hid-$(CONFIG_HID_HAPTIC) +=3D hid-haptic.o +hid-$(CONFIG_MULTITOUCH_HAPTIC) +=3D hid-haptic.o =20 obj-$(CONFIG_HID_BPF) +=3D bpf/ =20 diff --git a/drivers/hid/hid-haptic.h b/drivers/hid/hid-haptic.h index 0a34b0c6d706a985630962acc41f7a8eb73cd343..808cec0b4e51eba1f58b839f3e5= 52493655b7899 100644 --- a/drivers/hid/hid-haptic.h +++ b/drivers/hid/hid-haptic.h @@ -58,6 +58,7 @@ struct hid_haptic_device { struct hid_haptic_effect stop_effect; }; =20 +#ifdef CONFIG_MULTITOUCH_HAPTIC void hid_haptic_feature_mapping(struct hid_device *hdev, struct hid_haptic_device *haptic, struct hid_field *field, struct hid_usage @@ -77,3 +78,54 @@ void hid_haptic_handle_press_release(struct hid_haptic_d= evice *haptic); void hid_haptic_pressure_reset(struct hid_haptic_device *haptic); void hid_haptic_pressure_increase(struct hid_haptic_device *haptic, __s32 pressure); +#else +static inline +void hid_haptic_feature_mapping(struct hid_device *hdev, + struct hid_haptic_device *haptic, + struct hid_field *field, struct hid_usage + *usage) +{} +static inline +bool hid_haptic_check_pressure_unit(struct hid_haptic_device *haptic, + struct hid_input *hi, struct hid_field *field) +{ + return false; +} +static inline +int hid_haptic_input_mapping(struct hid_device *hdev, + struct hid_haptic_device *haptic, + struct hid_input *hi, + struct hid_field *field, struct hid_usage *usage, + unsigned long **bit, int *max) +{ + return 0; +} +static inline +int hid_haptic_input_configured(struct hid_device *hdev, + struct hid_haptic_device *haptic, + struct hid_input *hi) +{ + return 0; +} +static inline +void hid_haptic_reset(struct hid_device *hdev, struct hid_haptic_device *h= aptic) +{} +static inline +int hid_haptic_init(struct hid_device *hdev, struct hid_haptic_device **ha= ptic_ptr) +{ + return 0; +} +static inline +void hid_haptic_handle_press_release(struct hid_haptic_device *haptic) {} +static inline +bool hid_haptic_handle_input(struct hid_haptic_device *haptic) +{ + return false; +} +static inline +void hid_haptic_pressure_reset(struct hid_haptic_device *haptic) {} +static inline +void hid_haptic_pressure_increase(struct hid_haptic_device *haptic, + __s32 pressure) +{} +#endif diff --git a/drivers/hid/hid-multitouch.c b/drivers/hid/hid-multitouch.c index b41001e02da7e02d492bd85743b359ed7ec16e7f..4ff9ac5022b13a0739dbc7ae5f6= ebd84f0114a73 100644 --- a/drivers/hid/hid-multitouch.c +++ b/drivers/hid/hid-multitouch.c @@ -49,6 +49,8 @@ MODULE_LICENSE("GPL"); =20 #include "hid-ids.h" =20 +#include "hid-haptic.h" + /* quirks to control the device */ #define MT_QUIRK_NOT_SEEN_MEANS_UP BIT(0) #define MT_QUIRK_SLOT_IS_CONTACTID BIT(1) @@ -167,11 +169,13 @@ struct mt_report_data { struct mt_device { struct mt_class mtclass; /* our mt device class */ struct timer_list release_timer; /* to release sticky fingers */ + struct hid_haptic_device *haptic; /* haptic related configuration */ struct hid_device *hdev; /* hid_device we're attached to */ unsigned long mt_io_flags; /* mt flags (MT_IO_FLAGS_*) */ __u8 inputmode_value; /* InputMode HID feature value */ __u8 maxcontacts; bool is_buttonpad; /* is this device a button pad? */ + bool is_haptic_touchpad; /* is this device a haptic touchpad? */ bool serial_maybe; /* need to check for serial protocol */ =20 struct list_head applications; @@ -490,6 +494,95 @@ static void mt_get_feature(struct hid_device *hdev, st= ruct hid_report *report) kfree(buf); } =20 +#if defined(CONFIG_MULTITOUCH_HAPTIC) +static int mt_haptic_init(struct hid_device *hdev, + struct hid_haptic_device **haptic_ptr) +{ + return hid_haptic_init(hdev, haptic_ptr); +} + +static void mt_haptic_feature_mapping(struct hid_device *hdev, + struct hid_haptic_device *haptic, + struct hid_field *field, struct hid_usage *usage) +{ + return hid_haptic_feature_mapping(hdev, haptic, field, usage); +} + +static bool mt_haptic_check_pressure_unit(struct hid_haptic_device *haptic, + struct hid_input *hi, struct hid_field *field) +{ + return hid_haptic_check_pressure_unit(haptic, hi, field); +} + +static void mt_haptic_pressure_reset(struct hid_haptic_device *haptic) +{ + return hid_haptic_pressure_reset(haptic); +} + +static void mt_haptic_pressure_increase(struct hid_haptic_device *haptic, + __s32 pressure) +{ + return hid_haptic_pressure_increase(haptic, pressure); +} + +static int mt_haptic_input_mapping(struct hid_device *hdev, + struct hid_haptic_device *haptic, + struct hid_input *hi, + struct hid_field *field, struct hid_usage *usage, + unsigned long **bit, int *max) +{ + return hid_haptic_input_mapping(hdev, haptic, hi, field, usage, bit, max); +} + +static int mt_haptic_input_configured(struct hid_device *hdev, + struct hid_haptic_device *haptic, + struct hid_input *hi) +{ + return hid_haptic_input_configured(hdev, haptic, hi); +} +#else +static int mt_haptic_init(struct hid_device *hdev, + struct hid_haptic_device **haptic_ptr) +{ + return 0; +} + +static void mt_haptic_feature_mapping(struct hid_device *hdev, + struct hid_haptic_device *haptic, + struct hid_field *field, struct hid_usage *usage) +{} + +static bool mt_haptic_check_pressure_unit(struct hid_haptic_device *haptic, + struct hid_input *hi, struct hid_field *field) +{ + return 0; +} + +static void mt_haptic_pressure_reset(struct hid_haptic_device *haptic) +{} + +static void mt_haptic_pressure_increase(struct hid_haptic_device *haptic, + __s32 pressure) +{} + +static int mt_haptic_input_mapping(struct hid_device *hdev, + struct hid_haptic_device *haptic, + struct hid_input *hi, + struct hid_field *field, struct hid_usage *usage, + unsigned long **bit, int *max) +{ + return 0; +} + +static int mt_haptic_input_configured(struct hid_device *hdev, + struct hid_haptic_device *haptic, + struct hid_input *hi) +{ + return 0; +} +#endif + + static void mt_feature_mapping(struct hid_device *hdev, struct hid_field *field, struct hid_usage *usage) { @@ -525,6 +618,8 @@ static void mt_feature_mapping(struct hid_device *hdev, mt_get_feature(hdev, field->report); break; } + + mt_haptic_feature_mapping(hdev, td->haptic, field, usage); } =20 static void set_abs(struct input_dev *input, unsigned int code, @@ -856,6 +951,9 @@ static int mt_touch_input_mapping(struct hid_device *hd= ev, struct hid_input *hi, case HID_DG_TIPPRESSURE: set_abs(hi->input, ABS_MT_PRESSURE, field, cls->sn_pressure); + td->is_haptic_touchpad =3D + mt_haptic_check_pressure_unit(td->haptic, + hi, field); MT_STORE_FIELD(p); return 1; case HID_DG_SCANTIME: @@ -980,6 +1078,8 @@ static void mt_sync_frame(struct mt_device *td, struct= mt_application *app, =20 app->num_received =3D 0; app->left_button_state =3D 0; + if (td->is_haptic_touchpad) + mt_haptic_pressure_reset(td->haptic); =20 if (test_bit(MT_IO_FLAGS_ACTIVE_SLOTS, &td->mt_io_flags)) set_bit(MT_IO_FLAGS_PENDING_SLOTS, &td->mt_io_flags); @@ -1137,6 +1237,9 @@ static int mt_process_slot(struct mt_device *td, stru= ct input_dev *input, minor =3D minor >> 1; } =20 + if (td->is_haptic_touchpad) + mt_haptic_pressure_increase(td->haptic, *slot->p); + x =3D hdev->quirks & HID_QUIRK_X_INVERT ? input_abs_get_max(input, ABS_MT_POSITION_X) - *slot->x : *slot->x; @@ -1324,6 +1427,9 @@ static int mt_touch_input_configured(struct hid_devic= e *hdev, if (cls->is_indirect) app->mt_flags |=3D INPUT_MT_POINTER; =20 + if (td->is_haptic_touchpad) + app->mt_flags |=3D INPUT_MT_TOTAL_FORCE; + if (app->quirks & MT_QUIRK_NOT_SEEN_MEANS_UP) app->mt_flags |=3D INPUT_MT_DROP_UNUSED; =20 @@ -1359,6 +1465,7 @@ static int mt_input_mapping(struct hid_device *hdev, = struct hid_input *hi, struct mt_device *td =3D hid_get_drvdata(hdev); struct mt_application *application; struct mt_report_data *rdata; + int ret; =20 rdata =3D mt_find_report_data(td, field->report); if (!rdata) { @@ -1421,6 +1528,11 @@ static int mt_input_mapping(struct hid_device *hdev,= struct hid_input *hi, if (field->physical =3D=3D HID_DG_STYLUS) hi->application =3D HID_DG_STYLUS; =20 + ret =3D mt_haptic_input_mapping(hdev, td->haptic, hi, field, usage, bit, + max); + if (ret !=3D 0) + return ret; + /* let hid-core decide for the others */ return 0; } @@ -1635,6 +1747,14 @@ static int mt_input_configured(struct hid_device *hd= ev, struct hid_input *hi) struct hid_report *report; int ret; =20 + if (td->is_haptic_touchpad && (td->mtclass.name =3D=3D MT_CLS_WIN_8 || + td->mtclass.name =3D=3D MT_CLS_WIN_8_FORCE_MULTI_INPUT)) { + if (mt_haptic_input_configured(hdev, td->haptic, hi) =3D=3D 0) + td->is_haptic_touchpad =3D false; + } else { + td->is_haptic_touchpad =3D false; + } + list_for_each_entry(report, &hi->reports, hidinput_list) { rdata =3D mt_find_report_data(td, report); if (!rdata) { @@ -1764,7 +1884,6 @@ static int mt_probe(struct hid_device *hdev, const st= ruct hid_device_id *id) int ret, i; struct mt_device *td; const struct mt_class *mtclass =3D mt_classes; /* MT_CLS_DEFAULT */ - for (i =3D 0; mt_classes[i].name ; i++) { if (id->driver_data =3D=3D mt_classes[i].name) { mtclass =3D &(mt_classes[i]); @@ -1777,6 +1896,10 @@ static int mt_probe(struct hid_device *hdev, const s= truct hid_device_id *id) dev_err(&hdev->dev, "cannot allocate multitouch data\n"); return -ENOMEM; } + td->haptic =3D kzalloc(sizeof(*(td->haptic)), GFP_KERNEL); + if (!td->haptic) + return -ENOMEM; + td->haptic->hdev =3D hdev; td->hdev =3D hdev; td->mtclass =3D *mtclass; td->inputmode_value =3D MT_INPUTMODE_TOUCHSCREEN; @@ -1840,6 +1963,17 @@ static int mt_probe(struct hid_device *hdev, const s= truct hid_device_id *id) =20 mt_set_modes(hdev, HID_LATENCY_NORMAL, TOUCHPAD_REPORT_ALL); =20 + if (td->is_haptic_touchpad) { + if (mt_haptic_init(hdev, &td->haptic)) { + dev_warn(&hdev->dev, "Cannot allocate haptic for %s\n", + hdev->name); + td->is_haptic_touchpad =3D false; + kfree(td->haptic); + } + } else { + kfree(td->haptic); + } + return 0; } =20 --=20 2.50.1.565.gc32cd1483b-goog