From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-il1-f201.google.com (mail-il1-f201.google.com [209.85.166.201]) (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 723ED2749F9 for ; Mon, 4 Aug 2025 14:11:31 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.166.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1754316693; cv=none; b=emDsJkqxybFQ63QE5oNZvEtcZPMUHP/JW/6AjEul1iP9O0L2Adv6VUktWWlGbUPmWz2Pfty/Hj7GxFIGNCEyz8+qqXcLzp9aq8wGIcZnHOn0NuIobsogae8SSM6ODYjOPJ+mEuJUJEBBsxO/SdaKyevirG/svoICqIrgWYcirTs= 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.201 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-f201.google.com with SMTP id e9e14a558f8ab-3e40bc54f89so36545155ab.0 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=vw0VABAFNzhGQlnoeKl7S5Imzh9h5ykjmC9z/8PBn578xUUEx3gzE983KA+SZZvsJw 7bS3msNrUSg7P8nqIJX5ZF6q/uXwSQjkOCu3EoB3jjq6EyDzOcLfUNb034WhPjmsgnKC qjG6FQ/HpDpE1Tjpbs2AdqWi29MbbboQKEHJ9jBzxsfW/vBGhzrO/JHvpbos4iQQ2HIo OOViMaTV31i+xYua2kpGtGJiUlj/1vZk4gmwnM+eczuGvHU3W0lkYY8Zlspshz7JAzgh UcUWVrFHBS51NkD1oSFrhqUudUq71gsrs35qwt/PQc7YuiwyXpMMWbqQKb2c3Xw+VQvz NDlQ== X-Forwarded-Encrypted: i=1; AJvYcCVVlvh7OAVXgFqyJJnCGfjsVPjb7WYTn32zF4wd/zXrD8lg4thRXmbrCTk+23J7auU7ieL+h0AEe6Q=@vger.kernel.org X-Gm-Message-State: AOJu0YxiqnqH1A+xYmX4O/jBX5E/isamupokGR/Sr8z10uN/8z/MSeo9 cH3jVNFYz4kk2LmQYaY+Z0XRsTVm5lu32Mcr5xFyBo6rXuk1MwNHa1B8x6nPmLhpUvjRiwdflW2 5oQ7cyP5LsQ== 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-doc@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" 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..b7452f11a4f914f92af582ed054d42ecbcd6cb9e 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. +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..be09b4f13b2058a0a1d7eab79f35def758120fc4 100644 --- a/drivers/hid/Makefile +++ b/drivers/hid/Makefile @@ -4,7 +4,7 @@ # hid-y := hid-core.o hid-input.o hid-quirks.o hid-$(CONFIG_DEBUG_FS) += hid-debug.o -hid-$(CONFIG_HID_HAPTIC) += hid-haptic.o +hid-$(CONFIG_MULTITOUCH_HAPTIC) += hid-haptic.o obj-$(CONFIG_HID_BPF) += bpf/ diff --git a/drivers/hid/hid-haptic.h b/drivers/hid/hid-haptic.h index 0a34b0c6d706a985630962acc41f7a8eb73cd343..808cec0b4e51eba1f58b839f3e552493655b7899 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; }; +#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_device *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 *haptic) +{} +static inline +int hid_haptic_init(struct hid_device *hdev, struct hid_haptic_device **haptic_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..4ff9ac5022b13a0739dbc7ae5f6ebd84f0114a73 100644 --- a/drivers/hid/hid-multitouch.c +++ b/drivers/hid/hid-multitouch.c @@ -49,6 +49,8 @@ MODULE_LICENSE("GPL"); #include "hid-ids.h" +#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 */ struct list_head applications; @@ -490,6 +494,95 @@ static void mt_get_feature(struct hid_device *hdev, struct hid_report *report) kfree(buf); } +#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); } static void set_abs(struct input_dev *input, unsigned int code, @@ -856,6 +951,9 @@ static int mt_touch_input_mapping(struct hid_device *hdev, struct hid_input *hi, case HID_DG_TIPPRESSURE: set_abs(hi->input, ABS_MT_PRESSURE, field, cls->sn_pressure); + td->is_haptic_touchpad = + 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, app->num_received = 0; app->left_button_state = 0; + if (td->is_haptic_touchpad) + mt_haptic_pressure_reset(td->haptic); 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, struct input_dev *input, minor = minor >> 1; } + if (td->is_haptic_touchpad) + mt_haptic_pressure_increase(td->haptic, *slot->p); + x = 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_device *hdev, if (cls->is_indirect) app->mt_flags |= INPUT_MT_POINTER; + if (td->is_haptic_touchpad) + app->mt_flags |= INPUT_MT_TOTAL_FORCE; + if (app->quirks & MT_QUIRK_NOT_SEEN_MEANS_UP) app->mt_flags |= INPUT_MT_DROP_UNUSED; @@ -1359,6 +1465,7 @@ static int mt_input_mapping(struct hid_device *hdev, struct hid_input *hi, struct mt_device *td = hid_get_drvdata(hdev); struct mt_application *application; struct mt_report_data *rdata; + int ret; rdata = 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 == HID_DG_STYLUS) hi->application = HID_DG_STYLUS; + ret = mt_haptic_input_mapping(hdev, td->haptic, hi, field, usage, bit, + max); + if (ret != 0) + return ret; + /* let hid-core decide for the others */ return 0; } @@ -1635,6 +1747,14 @@ static int mt_input_configured(struct hid_device *hdev, struct hid_input *hi) struct hid_report *report; int ret; + if (td->is_haptic_touchpad && (td->mtclass.name == MT_CLS_WIN_8 || + td->mtclass.name == MT_CLS_WIN_8_FORCE_MULTI_INPUT)) { + if (mt_haptic_input_configured(hdev, td->haptic, hi) == 0) + td->is_haptic_touchpad = false; + } else { + td->is_haptic_touchpad = false; + } + list_for_each_entry(report, &hi->reports, hidinput_list) { rdata = mt_find_report_data(td, report); if (!rdata) { @@ -1764,7 +1884,6 @@ static int mt_probe(struct hid_device *hdev, const struct hid_device_id *id) int ret, i; struct mt_device *td; const struct mt_class *mtclass = mt_classes; /* MT_CLS_DEFAULT */ - for (i = 0; mt_classes[i].name ; i++) { if (id->driver_data == mt_classes[i].name) { mtclass = &(mt_classes[i]); @@ -1777,6 +1896,10 @@ static int mt_probe(struct hid_device *hdev, const struct hid_device_id *id) dev_err(&hdev->dev, "cannot allocate multitouch data\n"); return -ENOMEM; } + td->haptic = kzalloc(sizeof(*(td->haptic)), GFP_KERNEL); + if (!td->haptic) + return -ENOMEM; + td->haptic->hdev = hdev; td->hdev = hdev; td->mtclass = *mtclass; td->inputmode_value = MT_INPUTMODE_TOUCHSCREEN; @@ -1840,6 +1963,17 @@ static int mt_probe(struct hid_device *hdev, const struct hid_device_id *id) mt_set_modes(hdev, HID_LATENCY_NORMAL, TOUCHPAD_REPORT_ALL); + 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 = false; + kfree(td->haptic); + } + } else { + kfree(td->haptic); + } + return 0; } -- 2.50.1.565.gc32cd1483b-goog