* [PATCH v2 0/2] Do not map BTN_RIGHT/MIDDLE on buttonpads @ 2021-12-02 11:08 José Expósito 2021-12-02 11:08 ` [PATCH v2 1/2] Input: add input_set_property() José Expósito 2021-12-02 11:08 ` [PATCH v2 2/2] Input: set INPUT_PROP_BUTTONPAD using input_set_property() José Expósito 0 siblings, 2 replies; 9+ messages in thread From: José Expósito @ 2021-12-02 11:08 UTC (permalink / raw) To: jikos Cc: benjamin.tissoires, peter.hutterer, roderick.colenbrander, dmitry.torokhov, pali, rydberg, nick, linux-input, linux-kernel, José Expósito Hi all, This patchset implements the changes to buttonpads discussed in v1: https://lore.kernel.org/linux-input/YacOmYorwAIB4Q3c@quokka/T/#t I'd be great to know if maintainers are interested in migrating the other properties to use the new function (input_set_property). If so, I can send the patches :) Thanks, Jose José Expósito (2): Input: add input_set_property() Input: set INPUT_PROP_BUTTONPAD using input_set_property() drivers/hid/hid-alps.c | 2 +- drivers/hid/hid-asus.c | 3 +- drivers/hid/hid-elan.c | 3 +- drivers/hid/hid-logitech-hidpp.c | 2 +- drivers/hid/hid-magicmouse.c | 8 ++---- drivers/hid/hid-multitouch.c | 2 +- drivers/hid/hid-playstation.c | 3 +- drivers/hid/hid-sony.c | 4 +-- drivers/input/input.c | 35 ++++++++++++++++++++++++ drivers/input/keyboard/applespi.c | 2 +- drivers/input/mouse/alps.c | 3 +- drivers/input/mouse/bcm5974.c | 2 +- drivers/input/mouse/cyapa.c | 2 +- drivers/input/mouse/elan_i2c_core.c | 2 +- drivers/input/mouse/elantech.c | 6 ++-- drivers/input/mouse/focaltech.c | 4 +-- drivers/input/mouse/synaptics.c | 2 +- drivers/input/rmi4/rmi_f30.c | 2 +- drivers/input/rmi4/rmi_f3a.c | 2 +- drivers/input/touchscreen/atmel_mxt_ts.c | 2 +- include/linux/input.h | 1 + 21 files changed, 57 insertions(+), 35 deletions(-) -- 2.25.1 ^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH v2 1/2] Input: add input_set_property() 2021-12-02 11:08 [PATCH v2 0/2] Do not map BTN_RIGHT/MIDDLE on buttonpads José Expósito @ 2021-12-02 11:08 ` José Expósito 2022-01-04 6:22 ` Dmitry Torokhov 2021-12-02 11:08 ` [PATCH v2 2/2] Input: set INPUT_PROP_BUTTONPAD using input_set_property() José Expósito 1 sibling, 1 reply; 9+ messages in thread From: José Expósito @ 2021-12-02 11:08 UTC (permalink / raw) To: jikos Cc: benjamin.tissoires, peter.hutterer, roderick.colenbrander, dmitry.torokhov, pali, rydberg, nick, linux-input, linux-kernel, José Expósito Buttonpads are expected to map the INPUT_PROP_BUTTONPAD property bit and the BTN_LEFT key bit. As explained in the specification, where a device has a button type value of 0 (click-pad) or 1 (pressure-pad) there should not be discrete buttons: https://docs.microsoft.com/en-us/windows-hardware/design/component-guidelines/touchpad-windows-precision-touchpad-collection#device-capabilities-feature-report However, some drivers map the BTN_RIGHT and/or BTN_MIDDLE key bits even though the device is a buttonpad and therefore does not have those buttons. This behavior has forced userspace applications like libinput to implement different workarounds and quirks to detect buttonpads and offer to the user the right set of features and configuration options. For more information: https://gitlab.freedesktop.org/libinput/libinput/-/merge_requests/726 In order to avoid this issue add a helper function for drivers to add device properties and make sure that the conditions associated with the INPUT_PROP_BUTTONPAD property are meet. Notice that this change will not affect udev because it does not check for buttons. See systemd/src/udev/udev-builtin-input_id.c. List of known affected hardware: - Chuwi AeroBook Plus - Chuwi Gemibook - Framework Laptop - GPD Win Max - Huawei MateBook 2020 - Prestigio Smartbook 141 C2 - Purism Librem 14v1 - StarLite Mk II - AMI firmware - StarLite Mk II - Coreboot firmware - StarLite Mk III - AMI firmware - StarLite Mk III - Coreboot firmware - StarLabTop Mk IV - AMI firmware - StarLabTop Mk IV - Coreboot firmware - StarBook Mk V Signed-off-by: José Expósito <jose.exposito89@gmail.com> --- drivers/input/input.c | 35 +++++++++++++++++++++++++++++++++++ include/linux/input.h | 1 + 2 files changed, 36 insertions(+) diff --git a/drivers/input/input.c b/drivers/input/input.c index ccaeb2426385..f7e23b3b6ae5 100644 --- a/drivers/input/input.c +++ b/drivers/input/input.c @@ -2125,6 +2125,41 @@ void input_set_capability(struct input_dev *dev, unsigned int type, unsigned int } EXPORT_SYMBOL(input_set_capability); +/** + * input_set_property - add a property to the device + * @dev: device to add the property to + * @property: type of the property (INPUT_PROP_POINTER, INPUT_PROP_DIRECT...) + * + * In addition to setting up corresponding bit in dev->propbit the function + * might add or remove related capabilities. + */ +void input_set_property(struct input_dev *dev, unsigned int property) +{ + switch (property) { + case INPUT_PROP_POINTER: + case INPUT_PROP_DIRECT: + case INPUT_PROP_SEMI_MT: + case INPUT_PROP_TOPBUTTONPAD: + case INPUT_PROP_POINTING_STICK: + case INPUT_PROP_ACCELEROMETER: + break; + + case INPUT_PROP_BUTTONPAD: + input_set_capability(dev, EV_KEY, BTN_LEFT); + __clear_bit(BTN_RIGHT, dev->keybit); + __clear_bit(BTN_MIDDLE, dev->keybit); + break; + + default: + pr_err("%s: unknown property %u\n", __func__, property); + dump_stack(); + return; + } + + __set_bit(property, dev->propbit); +} +EXPORT_SYMBOL(input_set_property); + static unsigned int input_estimate_events_per_packet(struct input_dev *dev) { int mt_slots; diff --git a/include/linux/input.h b/include/linux/input.h index 0354b298d874..5f357687da42 100644 --- a/include/linux/input.h +++ b/include/linux/input.h @@ -456,6 +456,7 @@ static inline void input_mt_sync(struct input_dev *dev) } void input_set_capability(struct input_dev *dev, unsigned int type, unsigned int code); +void input_set_property(struct input_dev *dev, unsigned int property); /** * input_set_events_per_packet - tell handlers about the driver event rate -- 2.25.1 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH v2 1/2] Input: add input_set_property() 2021-12-02 11:08 ` [PATCH v2 1/2] Input: add input_set_property() José Expósito @ 2022-01-04 6:22 ` Dmitry Torokhov 2022-01-08 19:51 ` José Expósito 0 siblings, 1 reply; 9+ messages in thread From: Dmitry Torokhov @ 2022-01-04 6:22 UTC (permalink / raw) To: José Expósito Cc: jikos, benjamin.tissoires, peter.hutterer, roderick.colenbrander, pali, rydberg, nick, linux-input, linux-kernel Hi José, On Thu, Dec 02, 2021 at 12:08:06PM +0100, José Expósito wrote: > Buttonpads are expected to map the INPUT_PROP_BUTTONPAD property bit > and the BTN_LEFT key bit. > > As explained in the specification, where a device has a button type > value of 0 (click-pad) or 1 (pressure-pad) there should not be > discrete buttons: > https://docs.microsoft.com/en-us/windows-hardware/design/component-guidelines/touchpad-windows-precision-touchpad-collection#device-capabilities-feature-report > > However, some drivers map the BTN_RIGHT and/or BTN_MIDDLE key bits even > though the device is a buttonpad and therefore does not have those > buttons. > > This behavior has forced userspace applications like libinput to > implement different workarounds and quirks to detect buttonpads and > offer to the user the right set of features and configuration options. > For more information: > https://gitlab.freedesktop.org/libinput/libinput/-/merge_requests/726 > > In order to avoid this issue add a helper function for drivers to add > device properties and make sure that the conditions associated with the > INPUT_PROP_BUTTONPAD property are meet. > > Notice that this change will not affect udev because it does not check > for buttons. See systemd/src/udev/udev-builtin-input_id.c. > > List of known affected hardware: > > - Chuwi AeroBook Plus > - Chuwi Gemibook > - Framework Laptop > - GPD Win Max > - Huawei MateBook 2020 > - Prestigio Smartbook 141 C2 > - Purism Librem 14v1 > - StarLite Mk II - AMI firmware > - StarLite Mk II - Coreboot firmware > - StarLite Mk III - AMI firmware > - StarLite Mk III - Coreboot firmware > - StarLabTop Mk IV - AMI firmware > - StarLabTop Mk IV - Coreboot firmware > - StarBook Mk V > > Signed-off-by: José Expósito <jose.exposito89@gmail.com> > --- > drivers/input/input.c | 35 +++++++++++++++++++++++++++++++++++ > include/linux/input.h | 1 + > 2 files changed, 36 insertions(+) > > diff --git a/drivers/input/input.c b/drivers/input/input.c > index ccaeb2426385..f7e23b3b6ae5 100644 > --- a/drivers/input/input.c > +++ b/drivers/input/input.c > @@ -2125,6 +2125,41 @@ void input_set_capability(struct input_dev *dev, unsigned int type, unsigned int > } > EXPORT_SYMBOL(input_set_capability); > > +/** > + * input_set_property - add a property to the device > + * @dev: device to add the property to > + * @property: type of the property (INPUT_PROP_POINTER, INPUT_PROP_DIRECT...) > + * > + * In addition to setting up corresponding bit in dev->propbit the function > + * might add or remove related capabilities. > + */ > +void input_set_property(struct input_dev *dev, unsigned int property) > +{ > + switch (property) { > + case INPUT_PROP_POINTER: > + case INPUT_PROP_DIRECT: > + case INPUT_PROP_SEMI_MT: > + case INPUT_PROP_TOPBUTTONPAD: > + case INPUT_PROP_POINTING_STICK: > + case INPUT_PROP_ACCELEROMETER: > + break; > + > + case INPUT_PROP_BUTTONPAD: > + input_set_capability(dev, EV_KEY, BTN_LEFT); > + __clear_bit(BTN_RIGHT, dev->keybit); > + __clear_bit(BTN_MIDDLE, dev->keybit); I would prefer if we did this when registering input device, not when setting this property. > + break; > + > + default: > + pr_err("%s: unknown property %u\n", __func__, property); > + dump_stack(); > + return; > + } > + > + __set_bit(property, dev->propbit); > +} > +EXPORT_SYMBOL(input_set_property); > + > static unsigned int input_estimate_events_per_packet(struct input_dev *dev) > { > int mt_slots; > diff --git a/include/linux/input.h b/include/linux/input.h > index 0354b298d874..5f357687da42 100644 > --- a/include/linux/input.h > +++ b/include/linux/input.h > @@ -456,6 +456,7 @@ static inline void input_mt_sync(struct input_dev *dev) > } > > void input_set_capability(struct input_dev *dev, unsigned int type, unsigned int code); > +void input_set_property(struct input_dev *dev, unsigned int property); > > /** > * input_set_events_per_packet - tell handlers about the driver event rate > -- > 2.25.1 > Thanks. -- Dmitry ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2 1/2] Input: add input_set_property() 2022-01-04 6:22 ` Dmitry Torokhov @ 2022-01-08 19:51 ` José Expósito 0 siblings, 0 replies; 9+ messages in thread From: José Expósito @ 2022-01-08 19:51 UTC (permalink / raw) To: Dmitry Torokhov Cc: jikos, benjamin.tissoires, peter.hutterer, roderick.colenbrander, pali, rydberg, nick, linux-input, linux-kernel Hi Dmitry, On Mon, Jan 03, 2022 at 10:22:42PM -0800, Dmitry Torokhov wrote: > > +/** > > + * input_set_property - add a property to the device > > + * @dev: device to add the property to > > + * @property: type of the property (INPUT_PROP_POINTER, INPUT_PROP_DIRECT...) > > + * > > + * In addition to setting up corresponding bit in dev->propbit the function > > + * might add or remove related capabilities. > > + */ > > +void input_set_property(struct input_dev *dev, unsigned int property) > > +{ > > + switch (property) { > > + case INPUT_PROP_POINTER: > > + case INPUT_PROP_DIRECT: > > + case INPUT_PROP_SEMI_MT: > > + case INPUT_PROP_TOPBUTTONPAD: > > + case INPUT_PROP_POINTING_STICK: > > + case INPUT_PROP_ACCELEROMETER: > > + break; > > + > > + case INPUT_PROP_BUTTONPAD: > > + input_set_capability(dev, EV_KEY, BTN_LEFT); > > + __clear_bit(BTN_RIGHT, dev->keybit); > > + __clear_bit(BTN_MIDDLE, dev->keybit); > > I would prefer if we did this when registering input device, not when > setting this property. Thanks a lot for pointing me in this direction. I emailed you v3 [1] implementing the change you suggested and also including the "Acked-by" tags present in this conversation. Thanks to everyone for looking into this, appreciate it. José Expósito [1] https://lore.kernel.org/linux-input/20220108194235.14401-1-jose.exposito89@gmail.com/T/#u ^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH v2 2/2] Input: set INPUT_PROP_BUTTONPAD using input_set_property() 2021-12-02 11:08 [PATCH v2 0/2] Do not map BTN_RIGHT/MIDDLE on buttonpads José Expósito 2021-12-02 11:08 ` [PATCH v2 1/2] Input: add input_set_property() José Expósito @ 2021-12-02 11:08 ` José Expósito 2021-12-07 5:03 ` Peter Hutterer 2022-01-04 6:24 ` Dmitry Torokhov 1 sibling, 2 replies; 9+ messages in thread From: José Expósito @ 2021-12-02 11:08 UTC (permalink / raw) To: jikos Cc: benjamin.tissoires, peter.hutterer, roderick.colenbrander, dmitry.torokhov, pali, rydberg, nick, linux-input, linux-kernel, José Expósito Signed-off-by: José Expósito <jose.exposito89@gmail.com> --- drivers/hid/hid-alps.c | 2 +- drivers/hid/hid-asus.c | 3 +-- drivers/hid/hid-elan.c | 3 +-- drivers/hid/hid-logitech-hidpp.c | 2 +- drivers/hid/hid-magicmouse.c | 8 ++------ drivers/hid/hid-multitouch.c | 2 +- drivers/hid/hid-playstation.c | 3 +-- drivers/hid/hid-sony.c | 4 +--- drivers/input/keyboard/applespi.c | 2 +- drivers/input/mouse/alps.c | 3 +-- drivers/input/mouse/bcm5974.c | 2 +- drivers/input/mouse/cyapa.c | 2 +- drivers/input/mouse/elan_i2c_core.c | 2 +- drivers/input/mouse/elantech.c | 6 ++---- drivers/input/mouse/focaltech.c | 4 +--- drivers/input/mouse/synaptics.c | 2 +- drivers/input/rmi4/rmi_f30.c | 2 +- drivers/input/rmi4/rmi_f3a.c | 2 +- drivers/input/touchscreen/atmel_mxt_ts.c | 2 +- 19 files changed, 21 insertions(+), 35 deletions(-) diff --git a/drivers/hid/hid-alps.c b/drivers/hid/hid-alps.c index 2b986d0dbde4..4cbfefec949c 100644 --- a/drivers/hid/hid-alps.c +++ b/drivers/hid/hid-alps.c @@ -722,7 +722,7 @@ static int alps_input_configured(struct hid_device *hdev, struct hid_input *hi) __set_bit(EV_KEY, input->evbit); if (data->btn_cnt == 1) - __set_bit(INPUT_PROP_BUTTONPAD, input->propbit); + input_set_property(input, INPUT_PROP_BUTTONPAD); for (i = 0; i < data->btn_cnt; i++) __set_bit(BTN_LEFT + i, input->keybit); diff --git a/drivers/hid/hid-asus.c b/drivers/hid/hid-asus.c index f3ecddc519ee..99f37b636ec3 100644 --- a/drivers/hid/hid-asus.c +++ b/drivers/hid/hid-asus.c @@ -798,8 +798,7 @@ static int asus_input_configured(struct hid_device *hdev, struct hid_input *hi) MAX_PRESSURE, 0, 0); } - __set_bit(BTN_LEFT, input->keybit); - __set_bit(INPUT_PROP_BUTTONPAD, input->propbit); + input_set_property(input, INPUT_PROP_BUTTONPAD); ret = input_mt_init_slots(input, drvdata->tp->max_contacts, INPUT_MT_POINTER); diff --git a/drivers/hid/hid-elan.c b/drivers/hid/hid-elan.c index 021049805bb7..b33842b9a92f 100644 --- a/drivers/hid/hid-elan.c +++ b/drivers/hid/hid-elan.c @@ -182,8 +182,7 @@ static int elan_input_configured(struct hid_device *hdev, struct hid_input *hi) input_set_abs_params(input, ABS_MT_PRESSURE, 0, ELAN_MAX_PRESSURE, 0, 0); - __set_bit(BTN_LEFT, input->keybit); - __set_bit(INPUT_PROP_BUTTONPAD, input->propbit); + input_set_property(input, INPUT_PROP_BUTTONPAD); ret = input_mt_init_slots(input, ELAN_MAX_FINGERS, INPUT_MT_POINTER); if (ret) { diff --git a/drivers/hid/hid-logitech-hidpp.c b/drivers/hid/hid-logitech-hidpp.c index 81de88ab2ecc..d7b8e0ed0b47 100644 --- a/drivers/hid/hid-logitech-hidpp.c +++ b/drivers/hid/hid-logitech-hidpp.c @@ -2679,7 +2679,7 @@ static void wtp_populate_input(struct hidpp_device *hidpp, if (hidpp->quirks & HIDPP_QUIRK_WTP_PHYSICAL_BUTTONS) input_set_capability(input_dev, EV_KEY, BTN_RIGHT); else - __set_bit(INPUT_PROP_BUTTONPAD, input_dev->propbit); + input_set_property(input_dev, INPUT_PROP_BUTTONPAD); input_mt_init_slots(input_dev, wd->maxcontacts, INPUT_MT_POINTER | INPUT_MT_DROP_UNUSED); diff --git a/drivers/hid/hid-magicmouse.c b/drivers/hid/hid-magicmouse.c index d7687ce70614..2b28c0081362 100644 --- a/drivers/hid/hid-magicmouse.c +++ b/drivers/hid/hid-magicmouse.c @@ -545,11 +545,9 @@ static int magicmouse_setup_input(struct input_dev *input, struct hid_device *hd __clear_bit(EV_MSC, input->evbit); __clear_bit(BTN_0, input->keybit); - __clear_bit(BTN_RIGHT, input->keybit); - __clear_bit(BTN_MIDDLE, input->keybit); __set_bit(BTN_MOUSE, input->keybit); - __set_bit(INPUT_PROP_BUTTONPAD, input->propbit); __set_bit(BTN_TOOL_FINGER, input->keybit); + input_set_property(input, INPUT_PROP_BUTTONPAD); mt_flags = INPUT_MT_POINTER | INPUT_MT_DROP_UNUSED | INPUT_MT_TRACK; @@ -559,8 +557,6 @@ static int magicmouse_setup_input(struct input_dev *input, struct hid_device *hd * button (BTN_LEFT == BTN_MOUSE). Make sure we don't * advertise buttons that don't exist... */ - __clear_bit(BTN_RIGHT, input->keybit); - __clear_bit(BTN_MIDDLE, input->keybit); __set_bit(BTN_MOUSE, input->keybit); __set_bit(BTN_TOOL_FINGER, input->keybit); __set_bit(BTN_TOOL_DOUBLETAP, input->keybit); @@ -569,7 +565,7 @@ static int magicmouse_setup_input(struct input_dev *input, struct hid_device *hd __set_bit(BTN_TOOL_QUINTTAP, input->keybit); __set_bit(BTN_TOUCH, input->keybit); __set_bit(INPUT_PROP_POINTER, input->propbit); - __set_bit(INPUT_PROP_BUTTONPAD, input->propbit); + input_set_property(input, INPUT_PROP_BUTTONPAD); } diff --git a/drivers/hid/hid-multitouch.c b/drivers/hid/hid-multitouch.c index 082376a6cb3d..1bffb1787fd6 100644 --- a/drivers/hid/hid-multitouch.c +++ b/drivers/hid/hid-multitouch.c @@ -1287,7 +1287,7 @@ static int mt_touch_input_configured(struct hid_device *hdev, td->is_buttonpad = true; if (td->is_buttonpad) - __set_bit(INPUT_PROP_BUTTONPAD, input->propbit); + input_set_property(input, INPUT_PROP_BUTTONPAD); app->pending_palm_slots = devm_kcalloc(&hi->input->dev, BITS_TO_LONGS(td->maxcontacts), diff --git a/drivers/hid/hid-playstation.c b/drivers/hid/hid-playstation.c index b1b5721b5d8f..50815257ff86 100644 --- a/drivers/hid/hid-playstation.c +++ b/drivers/hid/hid-playstation.c @@ -651,8 +651,7 @@ static struct input_dev *ps_touchpad_create(struct hid_device *hdev, int width, return ERR_CAST(touchpad); /* Map button underneath touchpad to BTN_LEFT. */ - input_set_capability(touchpad, EV_KEY, BTN_LEFT); - __set_bit(INPUT_PROP_BUTTONPAD, touchpad->propbit); + input_set_property(touchpad, INPUT_PROP_BUTTONPAD); input_set_abs_params(touchpad, ABS_MT_POSITION_X, 0, width - 1, 0, 0); input_set_abs_params(touchpad, ABS_MT_POSITION_Y, 0, height - 1, 0, 0); diff --git a/drivers/hid/hid-sony.c b/drivers/hid/hid-sony.c index d1b107d547f5..302fd8fea243 100644 --- a/drivers/hid/hid-sony.c +++ b/drivers/hid/hid-sony.c @@ -1519,9 +1519,7 @@ static int sony_register_touchpad(struct sony_sc *sc, int touch_count, sc->touchpad->name = name; /* We map the button underneath the touchpad to BTN_LEFT. */ - __set_bit(EV_KEY, sc->touchpad->evbit); - __set_bit(BTN_LEFT, sc->touchpad->keybit); - __set_bit(INPUT_PROP_BUTTONPAD, sc->touchpad->propbit); + input_set_property(sc->touchpad, INPUT_PROP_BUTTONPAD); input_set_abs_params(sc->touchpad, ABS_MT_POSITION_X, 0, w, 0, 0); input_set_abs_params(sc->touchpad, ABS_MT_POSITION_Y, 0, h, 0, 0); diff --git a/drivers/input/keyboard/applespi.c b/drivers/input/keyboard/applespi.c index eda1b23002b5..9fc7608a8bbc 100644 --- a/drivers/input/keyboard/applespi.c +++ b/drivers/input/keyboard/applespi.c @@ -1289,7 +1289,7 @@ applespi_register_touchpad_device(struct applespi_data *applespi, input_set_capability(touchpad_input_dev, EV_REL, REL_Y); __set_bit(INPUT_PROP_POINTER, touchpad_input_dev->propbit); - __set_bit(INPUT_PROP_BUTTONPAD, touchpad_input_dev->propbit); + input_set_property(touchpad_input_dev, INPUT_PROP_BUTTONPAD); /* finger touch area */ input_set_abs_params(touchpad_input_dev, ABS_MT_TOUCH_MAJOR, diff --git a/drivers/input/mouse/alps.c b/drivers/input/mouse/alps.c index 4a6b33bbe7ea..d04c9553f9f2 100644 --- a/drivers/input/mouse/alps.c +++ b/drivers/input/mouse/alps.c @@ -3082,8 +3082,7 @@ int alps_init(struct psmouse *psmouse) dev1->keybit[BIT_WORD(BTN_2)] |= BIT_MASK(BTN_2); dev1->keybit[BIT_WORD(BTN_3)] |= BIT_MASK(BTN_3); } else if (priv->flags & ALPS_BUTTONPAD) { - set_bit(INPUT_PROP_BUTTONPAD, dev1->propbit); - clear_bit(BTN_RIGHT, dev1->keybit); + input_set_property(dev1, INPUT_PROP_BUTTONPAD); } else { dev1->keybit[BIT_WORD(BTN_MIDDLE)] |= BIT_MASK(BTN_MIDDLE); } diff --git a/drivers/input/mouse/bcm5974.c b/drivers/input/mouse/bcm5974.c index 59a14505b9cd..986fed0ef978 100644 --- a/drivers/input/mouse/bcm5974.c +++ b/drivers/input/mouse/bcm5974.c @@ -538,7 +538,7 @@ static void setup_events_to_report(struct input_dev *input_dev, __set_bit(BTN_LEFT, input_dev->keybit); if (cfg->caps & HAS_INTEGRATED_BUTTON) - __set_bit(INPUT_PROP_BUTTONPAD, input_dev->propbit); + input_set_property(input_dev, INPUT_PROP_BUTTONPAD); input_mt_init_slots(input_dev, MAX_FINGERS, INPUT_MT_POINTER | INPUT_MT_DROP_UNUSED | INPUT_MT_TRACK); diff --git a/drivers/input/mouse/cyapa.c b/drivers/input/mouse/cyapa.c index 77cc653edca2..364de350128b 100644 --- a/drivers/input/mouse/cyapa.c +++ b/drivers/input/mouse/cyapa.c @@ -501,7 +501,7 @@ static int cyapa_create_input_dev(struct cyapa *cyapa) __set_bit(BTN_RIGHT, input->keybit); if (cyapa->btn_capability == CAPABILITY_LEFT_BTN_MASK) - __set_bit(INPUT_PROP_BUTTONPAD, input->propbit); + input_set_property(input, INPUT_PROP_BUTTONPAD); /* Handle pointer emulation and unused slots in core */ error = input_mt_init_slots(input, CYAPA_MAX_MT_SLOTS, diff --git a/drivers/input/mouse/elan_i2c_core.c b/drivers/input/mouse/elan_i2c_core.c index 47af62c12267..b6cd6b282424 100644 --- a/drivers/input/mouse/elan_i2c_core.c +++ b/drivers/input/mouse/elan_i2c_core.c @@ -1177,7 +1177,7 @@ static int elan_setup_input_device(struct elan_tp_data *data) __set_bit(EV_ABS, input->evbit); __set_bit(INPUT_PROP_POINTER, input->propbit); if (data->clickpad) { - __set_bit(INPUT_PROP_BUTTONPAD, input->propbit); + input_set_property(input, INPUT_PROP_BUTTONPAD); } else { __set_bit(BTN_RIGHT, input->keybit); if (data->middle_button) diff --git a/drivers/input/mouse/elantech.c b/drivers/input/mouse/elantech.c index 956d9cd34796..38386a89831e 100644 --- a/drivers/input/mouse/elantech.c +++ b/drivers/input/mouse/elantech.c @@ -1122,10 +1122,8 @@ static void elantech_set_buttonpad_prop(struct psmouse *psmouse) struct input_dev *dev = psmouse->dev; struct elantech_data *etd = psmouse->private; - if (elantech_is_buttonpad(&etd->info)) { - __set_bit(INPUT_PROP_BUTTONPAD, dev->propbit); - __clear_bit(BTN_RIGHT, dev->keybit); - } + if (elantech_is_buttonpad(&etd->info)) + input_set_property(dev, INPUT_PROP_BUTTONPAD); } /* diff --git a/drivers/input/mouse/focaltech.c b/drivers/input/mouse/focaltech.c index 6fd5fff0cbff..292b931483c0 100644 --- a/drivers/input/mouse/focaltech.c +++ b/drivers/input/mouse/focaltech.c @@ -330,8 +330,6 @@ static void focaltech_set_input_params(struct psmouse *psmouse) __clear_bit(EV_REL, dev->evbit); __clear_bit(REL_X, dev->relbit); __clear_bit(REL_Y, dev->relbit); - __clear_bit(BTN_RIGHT, dev->keybit); - __clear_bit(BTN_MIDDLE, dev->keybit); /* * Now set up our capabilities. @@ -341,7 +339,7 @@ static void focaltech_set_input_params(struct psmouse *psmouse) input_set_abs_params(dev, ABS_MT_POSITION_Y, 0, priv->y_max, 0, 0); input_set_abs_params(dev, ABS_TOOL_WIDTH, 0, 15, 0, 0); input_mt_init_slots(dev, 5, INPUT_MT_POINTER); - __set_bit(INPUT_PROP_BUTTONPAD, dev->propbit); + input_set_property(dev, INPUT_PROP_BUTTONPAD); } static int focaltech_read_register(struct ps2dev *ps2dev, int reg, diff --git a/drivers/input/mouse/synaptics.c b/drivers/input/mouse/synaptics.c index ffad142801b3..1110c85dd370 100644 --- a/drivers/input/mouse/synaptics.c +++ b/drivers/input/mouse/synaptics.c @@ -1349,7 +1349,7 @@ static int set_input_params(struct psmouse *psmouse, input_set_capability(dev, EV_KEY, BTN_0 + i); if (SYN_CAP_CLICKPAD(info->ext_cap_0c)) { - __set_bit(INPUT_PROP_BUTTONPAD, dev->propbit); + input_set_property(dev, INPUT_PROP_BUTTONPAD); if (psmouse_matches_pnp_id(psmouse, topbuttonpad_pnp_ids) && !SYN_CAP_EXT_BUTTONS_STICK(info->ext_cap_10)) __set_bit(INPUT_PROP_TOPBUTTONPAD, dev->propbit); diff --git a/drivers/input/rmi4/rmi_f30.c b/drivers/input/rmi4/rmi_f30.c index 35045f161dc2..92ca5a1da1de 100644 --- a/drivers/input/rmi4/rmi_f30.c +++ b/drivers/input/rmi4/rmi_f30.c @@ -265,7 +265,7 @@ static int rmi_f30_map_gpios(struct rmi_function *fn, * mapped buttons. */ if (pdata->gpio_data.buttonpad || (button - BTN_LEFT == 1)) - __set_bit(INPUT_PROP_BUTTONPAD, input->propbit); + input_set_property(input, INPUT_PROP_BUTTONPAD); return 0; } diff --git a/drivers/input/rmi4/rmi_f3a.c b/drivers/input/rmi4/rmi_f3a.c index 0e8baed84dbb..4017deff191f 100644 --- a/drivers/input/rmi4/rmi_f3a.c +++ b/drivers/input/rmi4/rmi_f3a.c @@ -159,7 +159,7 @@ static int rmi_f3a_map_gpios(struct rmi_function *fn, struct f3a_data *f3a, input->keycodemax = f3a->gpio_count; if (pdata->gpio_data.buttonpad || (button - BTN_LEFT == 1)) - __set_bit(INPUT_PROP_BUTTONPAD, input->propbit); + input_set_property(input, INPUT_PROP_BUTTONPAD); return 0; } diff --git a/drivers/input/touchscreen/atmel_mxt_ts.c b/drivers/input/touchscreen/atmel_mxt_ts.c index 05de92c0293b..fd9b6774ee9e 100644 --- a/drivers/input/touchscreen/atmel_mxt_ts.c +++ b/drivers/input/touchscreen/atmel_mxt_ts.c @@ -2028,7 +2028,7 @@ static void mxt_set_up_as_touchpad(struct input_dev *input_dev, input_dev->name = "Atmel maXTouch Touchpad"; - __set_bit(INPUT_PROP_BUTTONPAD, input_dev->propbit); + input_set_property(input_dev, INPUT_PROP_BUTTONPAD); input_abs_set_res(input_dev, ABS_X, MXT_PIXELS_PER_MM); input_abs_set_res(input_dev, ABS_Y, MXT_PIXELS_PER_MM); -- 2.25.1 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH v2 2/2] Input: set INPUT_PROP_BUTTONPAD using input_set_property() 2021-12-02 11:08 ` [PATCH v2 2/2] Input: set INPUT_PROP_BUTTONPAD using input_set_property() José Expósito @ 2021-12-07 5:03 ` Peter Hutterer 2022-01-04 6:24 ` Dmitry Torokhov 1 sibling, 0 replies; 9+ messages in thread From: Peter Hutterer @ 2021-12-07 5:03 UTC (permalink / raw) To: José Expósito Cc: jikos, benjamin.tissoires, roderick.colenbrander, dmitry.torokhov, pali, rydberg, nick, linux-input, linux-kernel On Thu, Dec 02, 2021 at 12:08:07PM +0100, José Expósito wrote: > Signed-off-by: José Expósito <jose.exposito89@gmail.com> > --- > drivers/hid/hid-alps.c | 2 +- > drivers/hid/hid-asus.c | 3 +-- > drivers/hid/hid-elan.c | 3 +-- > drivers/hid/hid-logitech-hidpp.c | 2 +- > drivers/hid/hid-magicmouse.c | 8 ++------ > drivers/hid/hid-multitouch.c | 2 +- > drivers/hid/hid-playstation.c | 3 +-- > drivers/hid/hid-sony.c | 4 +--- > drivers/input/keyboard/applespi.c | 2 +- > drivers/input/mouse/alps.c | 3 +-- > drivers/input/mouse/bcm5974.c | 2 +- > drivers/input/mouse/cyapa.c | 2 +- > drivers/input/mouse/elan_i2c_core.c | 2 +- > drivers/input/mouse/elantech.c | 6 ++---- > drivers/input/mouse/focaltech.c | 4 +--- > drivers/input/mouse/synaptics.c | 2 +- > drivers/input/rmi4/rmi_f30.c | 2 +- > drivers/input/rmi4/rmi_f3a.c | 2 +- > drivers/input/touchscreen/atmel_mxt_ts.c | 2 +- > 19 files changed, 21 insertions(+), 35 deletions(-) series: Acked-by: Peter Hutterer <peter.hutterer@who-t.net> Cheers, Peter ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2 2/2] Input: set INPUT_PROP_BUTTONPAD using input_set_property() 2021-12-02 11:08 ` [PATCH v2 2/2] Input: set INPUT_PROP_BUTTONPAD using input_set_property() José Expósito 2021-12-07 5:03 ` Peter Hutterer @ 2022-01-04 6:24 ` Dmitry Torokhov 2022-01-04 9:18 ` Benjamin Tissoires 2022-01-06 13:09 ` Jiri Kosina 1 sibling, 2 replies; 9+ messages in thread From: Dmitry Torokhov @ 2022-01-04 6:24 UTC (permalink / raw) To: José Expósito Cc: jikos, benjamin.tissoires, peter.hutterer, roderick.colenbrander, pali, rydberg, nick, linux-input, linux-kernel Hi José, On Thu, Dec 02, 2021 at 12:08:07PM +0100, José Expósito wrote: A short description would be appreciated. Jiri, Benjamin, do you mind if I take this through my tree or you would prefer having this split? > Signed-off-by: José Expósito <jose.exposito89@gmail.com> > --- > drivers/hid/hid-alps.c | 2 +- > drivers/hid/hid-asus.c | 3 +-- > drivers/hid/hid-elan.c | 3 +-- > drivers/hid/hid-logitech-hidpp.c | 2 +- > drivers/hid/hid-magicmouse.c | 8 ++------ > drivers/hid/hid-multitouch.c | 2 +- > drivers/hid/hid-playstation.c | 3 +-- > drivers/hid/hid-sony.c | 4 +--- > drivers/input/keyboard/applespi.c | 2 +- > drivers/input/mouse/alps.c | 3 +-- > drivers/input/mouse/bcm5974.c | 2 +- > drivers/input/mouse/cyapa.c | 2 +- > drivers/input/mouse/elan_i2c_core.c | 2 +- > drivers/input/mouse/elantech.c | 6 ++---- > drivers/input/mouse/focaltech.c | 4 +--- > drivers/input/mouse/synaptics.c | 2 +- > drivers/input/rmi4/rmi_f30.c | 2 +- > drivers/input/rmi4/rmi_f3a.c | 2 +- > drivers/input/touchscreen/atmel_mxt_ts.c | 2 +- > 19 files changed, 21 insertions(+), 35 deletions(-) > > diff --git a/drivers/hid/hid-alps.c b/drivers/hid/hid-alps.c > index 2b986d0dbde4..4cbfefec949c 100644 > --- a/drivers/hid/hid-alps.c > +++ b/drivers/hid/hid-alps.c > @@ -722,7 +722,7 @@ static int alps_input_configured(struct hid_device *hdev, struct hid_input *hi) > __set_bit(EV_KEY, input->evbit); > > if (data->btn_cnt == 1) > - __set_bit(INPUT_PROP_BUTTONPAD, input->propbit); > + input_set_property(input, INPUT_PROP_BUTTONPAD); > > for (i = 0; i < data->btn_cnt; i++) > __set_bit(BTN_LEFT + i, input->keybit); > diff --git a/drivers/hid/hid-asus.c b/drivers/hid/hid-asus.c > index f3ecddc519ee..99f37b636ec3 100644 > --- a/drivers/hid/hid-asus.c > +++ b/drivers/hid/hid-asus.c > @@ -798,8 +798,7 @@ static int asus_input_configured(struct hid_device *hdev, struct hid_input *hi) > MAX_PRESSURE, 0, 0); > } > > - __set_bit(BTN_LEFT, input->keybit); > - __set_bit(INPUT_PROP_BUTTONPAD, input->propbit); > + input_set_property(input, INPUT_PROP_BUTTONPAD); > > ret = input_mt_init_slots(input, drvdata->tp->max_contacts, > INPUT_MT_POINTER); > diff --git a/drivers/hid/hid-elan.c b/drivers/hid/hid-elan.c > index 021049805bb7..b33842b9a92f 100644 > --- a/drivers/hid/hid-elan.c > +++ b/drivers/hid/hid-elan.c > @@ -182,8 +182,7 @@ static int elan_input_configured(struct hid_device *hdev, struct hid_input *hi) > input_set_abs_params(input, ABS_MT_PRESSURE, 0, ELAN_MAX_PRESSURE, > 0, 0); > > - __set_bit(BTN_LEFT, input->keybit); > - __set_bit(INPUT_PROP_BUTTONPAD, input->propbit); > + input_set_property(input, INPUT_PROP_BUTTONPAD); > > ret = input_mt_init_slots(input, ELAN_MAX_FINGERS, INPUT_MT_POINTER); > if (ret) { > diff --git a/drivers/hid/hid-logitech-hidpp.c b/drivers/hid/hid-logitech-hidpp.c > index 81de88ab2ecc..d7b8e0ed0b47 100644 > --- a/drivers/hid/hid-logitech-hidpp.c > +++ b/drivers/hid/hid-logitech-hidpp.c > @@ -2679,7 +2679,7 @@ static void wtp_populate_input(struct hidpp_device *hidpp, > if (hidpp->quirks & HIDPP_QUIRK_WTP_PHYSICAL_BUTTONS) > input_set_capability(input_dev, EV_KEY, BTN_RIGHT); > else > - __set_bit(INPUT_PROP_BUTTONPAD, input_dev->propbit); > + input_set_property(input_dev, INPUT_PROP_BUTTONPAD); > > input_mt_init_slots(input_dev, wd->maxcontacts, INPUT_MT_POINTER | > INPUT_MT_DROP_UNUSED); > diff --git a/drivers/hid/hid-magicmouse.c b/drivers/hid/hid-magicmouse.c > index d7687ce70614..2b28c0081362 100644 > --- a/drivers/hid/hid-magicmouse.c > +++ b/drivers/hid/hid-magicmouse.c > @@ -545,11 +545,9 @@ static int magicmouse_setup_input(struct input_dev *input, struct hid_device *hd > > __clear_bit(EV_MSC, input->evbit); > __clear_bit(BTN_0, input->keybit); > - __clear_bit(BTN_RIGHT, input->keybit); > - __clear_bit(BTN_MIDDLE, input->keybit); > __set_bit(BTN_MOUSE, input->keybit); > - __set_bit(INPUT_PROP_BUTTONPAD, input->propbit); > __set_bit(BTN_TOOL_FINGER, input->keybit); > + input_set_property(input, INPUT_PROP_BUTTONPAD); > > mt_flags = INPUT_MT_POINTER | INPUT_MT_DROP_UNUSED | > INPUT_MT_TRACK; > @@ -559,8 +557,6 @@ static int magicmouse_setup_input(struct input_dev *input, struct hid_device *hd > * button (BTN_LEFT == BTN_MOUSE). Make sure we don't > * advertise buttons that don't exist... > */ > - __clear_bit(BTN_RIGHT, input->keybit); > - __clear_bit(BTN_MIDDLE, input->keybit); > __set_bit(BTN_MOUSE, input->keybit); > __set_bit(BTN_TOOL_FINGER, input->keybit); > __set_bit(BTN_TOOL_DOUBLETAP, input->keybit); > @@ -569,7 +565,7 @@ static int magicmouse_setup_input(struct input_dev *input, struct hid_device *hd > __set_bit(BTN_TOOL_QUINTTAP, input->keybit); > __set_bit(BTN_TOUCH, input->keybit); > __set_bit(INPUT_PROP_POINTER, input->propbit); > - __set_bit(INPUT_PROP_BUTTONPAD, input->propbit); > + input_set_property(input, INPUT_PROP_BUTTONPAD); > } > > > diff --git a/drivers/hid/hid-multitouch.c b/drivers/hid/hid-multitouch.c > index 082376a6cb3d..1bffb1787fd6 100644 > --- a/drivers/hid/hid-multitouch.c > +++ b/drivers/hid/hid-multitouch.c > @@ -1287,7 +1287,7 @@ static int mt_touch_input_configured(struct hid_device *hdev, > td->is_buttonpad = true; > > if (td->is_buttonpad) > - __set_bit(INPUT_PROP_BUTTONPAD, input->propbit); > + input_set_property(input, INPUT_PROP_BUTTONPAD); > > app->pending_palm_slots = devm_kcalloc(&hi->input->dev, > BITS_TO_LONGS(td->maxcontacts), > diff --git a/drivers/hid/hid-playstation.c b/drivers/hid/hid-playstation.c > index b1b5721b5d8f..50815257ff86 100644 > --- a/drivers/hid/hid-playstation.c > +++ b/drivers/hid/hid-playstation.c > @@ -651,8 +651,7 @@ static struct input_dev *ps_touchpad_create(struct hid_device *hdev, int width, > return ERR_CAST(touchpad); > > /* Map button underneath touchpad to BTN_LEFT. */ > - input_set_capability(touchpad, EV_KEY, BTN_LEFT); > - __set_bit(INPUT_PROP_BUTTONPAD, touchpad->propbit); > + input_set_property(touchpad, INPUT_PROP_BUTTONPAD); > > input_set_abs_params(touchpad, ABS_MT_POSITION_X, 0, width - 1, 0, 0); > input_set_abs_params(touchpad, ABS_MT_POSITION_Y, 0, height - 1, 0, 0); > diff --git a/drivers/hid/hid-sony.c b/drivers/hid/hid-sony.c > index d1b107d547f5..302fd8fea243 100644 > --- a/drivers/hid/hid-sony.c > +++ b/drivers/hid/hid-sony.c > @@ -1519,9 +1519,7 @@ static int sony_register_touchpad(struct sony_sc *sc, int touch_count, > sc->touchpad->name = name; > > /* We map the button underneath the touchpad to BTN_LEFT. */ > - __set_bit(EV_KEY, sc->touchpad->evbit); > - __set_bit(BTN_LEFT, sc->touchpad->keybit); > - __set_bit(INPUT_PROP_BUTTONPAD, sc->touchpad->propbit); > + input_set_property(sc->touchpad, INPUT_PROP_BUTTONPAD); > > input_set_abs_params(sc->touchpad, ABS_MT_POSITION_X, 0, w, 0, 0); > input_set_abs_params(sc->touchpad, ABS_MT_POSITION_Y, 0, h, 0, 0); > diff --git a/drivers/input/keyboard/applespi.c b/drivers/input/keyboard/applespi.c > index eda1b23002b5..9fc7608a8bbc 100644 > --- a/drivers/input/keyboard/applespi.c > +++ b/drivers/input/keyboard/applespi.c > @@ -1289,7 +1289,7 @@ applespi_register_touchpad_device(struct applespi_data *applespi, > input_set_capability(touchpad_input_dev, EV_REL, REL_Y); > > __set_bit(INPUT_PROP_POINTER, touchpad_input_dev->propbit); > - __set_bit(INPUT_PROP_BUTTONPAD, touchpad_input_dev->propbit); > + input_set_property(touchpad_input_dev, INPUT_PROP_BUTTONPAD); > > /* finger touch area */ > input_set_abs_params(touchpad_input_dev, ABS_MT_TOUCH_MAJOR, > diff --git a/drivers/input/mouse/alps.c b/drivers/input/mouse/alps.c > index 4a6b33bbe7ea..d04c9553f9f2 100644 > --- a/drivers/input/mouse/alps.c > +++ b/drivers/input/mouse/alps.c > @@ -3082,8 +3082,7 @@ int alps_init(struct psmouse *psmouse) > dev1->keybit[BIT_WORD(BTN_2)] |= BIT_MASK(BTN_2); > dev1->keybit[BIT_WORD(BTN_3)] |= BIT_MASK(BTN_3); > } else if (priv->flags & ALPS_BUTTONPAD) { > - set_bit(INPUT_PROP_BUTTONPAD, dev1->propbit); > - clear_bit(BTN_RIGHT, dev1->keybit); > + input_set_property(dev1, INPUT_PROP_BUTTONPAD); > } else { > dev1->keybit[BIT_WORD(BTN_MIDDLE)] |= BIT_MASK(BTN_MIDDLE); > } > diff --git a/drivers/input/mouse/bcm5974.c b/drivers/input/mouse/bcm5974.c > index 59a14505b9cd..986fed0ef978 100644 > --- a/drivers/input/mouse/bcm5974.c > +++ b/drivers/input/mouse/bcm5974.c > @@ -538,7 +538,7 @@ static void setup_events_to_report(struct input_dev *input_dev, > __set_bit(BTN_LEFT, input_dev->keybit); > > if (cfg->caps & HAS_INTEGRATED_BUTTON) > - __set_bit(INPUT_PROP_BUTTONPAD, input_dev->propbit); > + input_set_property(input_dev, INPUT_PROP_BUTTONPAD); > > input_mt_init_slots(input_dev, MAX_FINGERS, > INPUT_MT_POINTER | INPUT_MT_DROP_UNUSED | INPUT_MT_TRACK); > diff --git a/drivers/input/mouse/cyapa.c b/drivers/input/mouse/cyapa.c > index 77cc653edca2..364de350128b 100644 > --- a/drivers/input/mouse/cyapa.c > +++ b/drivers/input/mouse/cyapa.c > @@ -501,7 +501,7 @@ static int cyapa_create_input_dev(struct cyapa *cyapa) > __set_bit(BTN_RIGHT, input->keybit); > > if (cyapa->btn_capability == CAPABILITY_LEFT_BTN_MASK) > - __set_bit(INPUT_PROP_BUTTONPAD, input->propbit); > + input_set_property(input, INPUT_PROP_BUTTONPAD); > > /* Handle pointer emulation and unused slots in core */ > error = input_mt_init_slots(input, CYAPA_MAX_MT_SLOTS, > diff --git a/drivers/input/mouse/elan_i2c_core.c b/drivers/input/mouse/elan_i2c_core.c > index 47af62c12267..b6cd6b282424 100644 > --- a/drivers/input/mouse/elan_i2c_core.c > +++ b/drivers/input/mouse/elan_i2c_core.c > @@ -1177,7 +1177,7 @@ static int elan_setup_input_device(struct elan_tp_data *data) > __set_bit(EV_ABS, input->evbit); > __set_bit(INPUT_PROP_POINTER, input->propbit); > if (data->clickpad) { > - __set_bit(INPUT_PROP_BUTTONPAD, input->propbit); > + input_set_property(input, INPUT_PROP_BUTTONPAD); > } else { > __set_bit(BTN_RIGHT, input->keybit); > if (data->middle_button) > diff --git a/drivers/input/mouse/elantech.c b/drivers/input/mouse/elantech.c > index 956d9cd34796..38386a89831e 100644 > --- a/drivers/input/mouse/elantech.c > +++ b/drivers/input/mouse/elantech.c > @@ -1122,10 +1122,8 @@ static void elantech_set_buttonpad_prop(struct psmouse *psmouse) > struct input_dev *dev = psmouse->dev; > struct elantech_data *etd = psmouse->private; > > - if (elantech_is_buttonpad(&etd->info)) { > - __set_bit(INPUT_PROP_BUTTONPAD, dev->propbit); > - __clear_bit(BTN_RIGHT, dev->keybit); > - } > + if (elantech_is_buttonpad(&etd->info)) > + input_set_property(dev, INPUT_PROP_BUTTONPAD); > } > > /* > diff --git a/drivers/input/mouse/focaltech.c b/drivers/input/mouse/focaltech.c > index 6fd5fff0cbff..292b931483c0 100644 > --- a/drivers/input/mouse/focaltech.c > +++ b/drivers/input/mouse/focaltech.c > @@ -330,8 +330,6 @@ static void focaltech_set_input_params(struct psmouse *psmouse) > __clear_bit(EV_REL, dev->evbit); > __clear_bit(REL_X, dev->relbit); > __clear_bit(REL_Y, dev->relbit); > - __clear_bit(BTN_RIGHT, dev->keybit); > - __clear_bit(BTN_MIDDLE, dev->keybit); > > /* > * Now set up our capabilities. > @@ -341,7 +339,7 @@ static void focaltech_set_input_params(struct psmouse *psmouse) > input_set_abs_params(dev, ABS_MT_POSITION_Y, 0, priv->y_max, 0, 0); > input_set_abs_params(dev, ABS_TOOL_WIDTH, 0, 15, 0, 0); > input_mt_init_slots(dev, 5, INPUT_MT_POINTER); > - __set_bit(INPUT_PROP_BUTTONPAD, dev->propbit); > + input_set_property(dev, INPUT_PROP_BUTTONPAD); > } > > static int focaltech_read_register(struct ps2dev *ps2dev, int reg, > diff --git a/drivers/input/mouse/synaptics.c b/drivers/input/mouse/synaptics.c > index ffad142801b3..1110c85dd370 100644 > --- a/drivers/input/mouse/synaptics.c > +++ b/drivers/input/mouse/synaptics.c > @@ -1349,7 +1349,7 @@ static int set_input_params(struct psmouse *psmouse, > input_set_capability(dev, EV_KEY, BTN_0 + i); > > if (SYN_CAP_CLICKPAD(info->ext_cap_0c)) { > - __set_bit(INPUT_PROP_BUTTONPAD, dev->propbit); > + input_set_property(dev, INPUT_PROP_BUTTONPAD); > if (psmouse_matches_pnp_id(psmouse, topbuttonpad_pnp_ids) && > !SYN_CAP_EXT_BUTTONS_STICK(info->ext_cap_10)) > __set_bit(INPUT_PROP_TOPBUTTONPAD, dev->propbit); > diff --git a/drivers/input/rmi4/rmi_f30.c b/drivers/input/rmi4/rmi_f30.c > index 35045f161dc2..92ca5a1da1de 100644 > --- a/drivers/input/rmi4/rmi_f30.c > +++ b/drivers/input/rmi4/rmi_f30.c > @@ -265,7 +265,7 @@ static int rmi_f30_map_gpios(struct rmi_function *fn, > * mapped buttons. > */ > if (pdata->gpio_data.buttonpad || (button - BTN_LEFT == 1)) > - __set_bit(INPUT_PROP_BUTTONPAD, input->propbit); > + input_set_property(input, INPUT_PROP_BUTTONPAD); > > return 0; > } > diff --git a/drivers/input/rmi4/rmi_f3a.c b/drivers/input/rmi4/rmi_f3a.c > index 0e8baed84dbb..4017deff191f 100644 > --- a/drivers/input/rmi4/rmi_f3a.c > +++ b/drivers/input/rmi4/rmi_f3a.c > @@ -159,7 +159,7 @@ static int rmi_f3a_map_gpios(struct rmi_function *fn, struct f3a_data *f3a, > input->keycodemax = f3a->gpio_count; > > if (pdata->gpio_data.buttonpad || (button - BTN_LEFT == 1)) > - __set_bit(INPUT_PROP_BUTTONPAD, input->propbit); > + input_set_property(input, INPUT_PROP_BUTTONPAD); > > return 0; > } > diff --git a/drivers/input/touchscreen/atmel_mxt_ts.c b/drivers/input/touchscreen/atmel_mxt_ts.c > index 05de92c0293b..fd9b6774ee9e 100644 > --- a/drivers/input/touchscreen/atmel_mxt_ts.c > +++ b/drivers/input/touchscreen/atmel_mxt_ts.c > @@ -2028,7 +2028,7 @@ static void mxt_set_up_as_touchpad(struct input_dev *input_dev, > > input_dev->name = "Atmel maXTouch Touchpad"; > > - __set_bit(INPUT_PROP_BUTTONPAD, input_dev->propbit); > + input_set_property(input_dev, INPUT_PROP_BUTTONPAD); > > input_abs_set_res(input_dev, ABS_X, MXT_PIXELS_PER_MM); > input_abs_set_res(input_dev, ABS_Y, MXT_PIXELS_PER_MM); > -- > 2.25.1 > -- Dmitry ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2 2/2] Input: set INPUT_PROP_BUTTONPAD using input_set_property() 2022-01-04 6:24 ` Dmitry Torokhov @ 2022-01-04 9:18 ` Benjamin Tissoires 2022-01-06 13:09 ` Jiri Kosina 1 sibling, 0 replies; 9+ messages in thread From: Benjamin Tissoires @ 2022-01-04 9:18 UTC (permalink / raw) To: Dmitry Torokhov Cc: José Expósito, Jiri Kosina, Peter Hutterer, Colenbrander, Roelof, pali, Henrik Rydberg, Nick Dyer, open list:HID CORE LAYER, lkml On Tue, Jan 4, 2022 at 7:24 AM Dmitry Torokhov <dmitry.torokhov@gmail.com> wrote: > > Hi José, > > On Thu, Dec 02, 2021 at 12:08:07PM +0100, José Expósito wrote: > > A short description would be appreciated. > > Jiri, Benjamin, do you mind if I take this through my tree or you would > prefer having this split? No objections from my side: Acked-by: Benjamin Tissoires <benjamin.tissoires@redhat.com> Cheers, Benjamin > > > Signed-off-by: José Expósito <jose.exposito89@gmail.com> > > --- > > drivers/hid/hid-alps.c | 2 +- > > drivers/hid/hid-asus.c | 3 +-- > > drivers/hid/hid-elan.c | 3 +-- > > drivers/hid/hid-logitech-hidpp.c | 2 +- > > drivers/hid/hid-magicmouse.c | 8 ++------ > > drivers/hid/hid-multitouch.c | 2 +- > > drivers/hid/hid-playstation.c | 3 +-- > > drivers/hid/hid-sony.c | 4 +--- > > drivers/input/keyboard/applespi.c | 2 +- > > drivers/input/mouse/alps.c | 3 +-- > > drivers/input/mouse/bcm5974.c | 2 +- > > drivers/input/mouse/cyapa.c | 2 +- > > drivers/input/mouse/elan_i2c_core.c | 2 +- > > drivers/input/mouse/elantech.c | 6 ++---- > > drivers/input/mouse/focaltech.c | 4 +--- > > drivers/input/mouse/synaptics.c | 2 +- > > drivers/input/rmi4/rmi_f30.c | 2 +- > > drivers/input/rmi4/rmi_f3a.c | 2 +- > > drivers/input/touchscreen/atmel_mxt_ts.c | 2 +- > > 19 files changed, 21 insertions(+), 35 deletions(-) > > > > diff --git a/drivers/hid/hid-alps.c b/drivers/hid/hid-alps.c > > index 2b986d0dbde4..4cbfefec949c 100644 > > --- a/drivers/hid/hid-alps.c > > +++ b/drivers/hid/hid-alps.c > > @@ -722,7 +722,7 @@ static int alps_input_configured(struct hid_device *hdev, struct hid_input *hi) > > __set_bit(EV_KEY, input->evbit); > > > > if (data->btn_cnt == 1) > > - __set_bit(INPUT_PROP_BUTTONPAD, input->propbit); > > + input_set_property(input, INPUT_PROP_BUTTONPAD); > > > > for (i = 0; i < data->btn_cnt; i++) > > __set_bit(BTN_LEFT + i, input->keybit); > > diff --git a/drivers/hid/hid-asus.c b/drivers/hid/hid-asus.c > > index f3ecddc519ee..99f37b636ec3 100644 > > --- a/drivers/hid/hid-asus.c > > +++ b/drivers/hid/hid-asus.c > > @@ -798,8 +798,7 @@ static int asus_input_configured(struct hid_device *hdev, struct hid_input *hi) > > MAX_PRESSURE, 0, 0); > > } > > > > - __set_bit(BTN_LEFT, input->keybit); > > - __set_bit(INPUT_PROP_BUTTONPAD, input->propbit); > > + input_set_property(input, INPUT_PROP_BUTTONPAD); > > > > ret = input_mt_init_slots(input, drvdata->tp->max_contacts, > > INPUT_MT_POINTER); > > diff --git a/drivers/hid/hid-elan.c b/drivers/hid/hid-elan.c > > index 021049805bb7..b33842b9a92f 100644 > > --- a/drivers/hid/hid-elan.c > > +++ b/drivers/hid/hid-elan.c > > @@ -182,8 +182,7 @@ static int elan_input_configured(struct hid_device *hdev, struct hid_input *hi) > > input_set_abs_params(input, ABS_MT_PRESSURE, 0, ELAN_MAX_PRESSURE, > > 0, 0); > > > > - __set_bit(BTN_LEFT, input->keybit); > > - __set_bit(INPUT_PROP_BUTTONPAD, input->propbit); > > + input_set_property(input, INPUT_PROP_BUTTONPAD); > > > > ret = input_mt_init_slots(input, ELAN_MAX_FINGERS, INPUT_MT_POINTER); > > if (ret) { > > diff --git a/drivers/hid/hid-logitech-hidpp.c b/drivers/hid/hid-logitech-hidpp.c > > index 81de88ab2ecc..d7b8e0ed0b47 100644 > > --- a/drivers/hid/hid-logitech-hidpp.c > > +++ b/drivers/hid/hid-logitech-hidpp.c > > @@ -2679,7 +2679,7 @@ static void wtp_populate_input(struct hidpp_device *hidpp, > > if (hidpp->quirks & HIDPP_QUIRK_WTP_PHYSICAL_BUTTONS) > > input_set_capability(input_dev, EV_KEY, BTN_RIGHT); > > else > > - __set_bit(INPUT_PROP_BUTTONPAD, input_dev->propbit); > > + input_set_property(input_dev, INPUT_PROP_BUTTONPAD); > > > > input_mt_init_slots(input_dev, wd->maxcontacts, INPUT_MT_POINTER | > > INPUT_MT_DROP_UNUSED); > > diff --git a/drivers/hid/hid-magicmouse.c b/drivers/hid/hid-magicmouse.c > > index d7687ce70614..2b28c0081362 100644 > > --- a/drivers/hid/hid-magicmouse.c > > +++ b/drivers/hid/hid-magicmouse.c > > @@ -545,11 +545,9 @@ static int magicmouse_setup_input(struct input_dev *input, struct hid_device *hd > > > > __clear_bit(EV_MSC, input->evbit); > > __clear_bit(BTN_0, input->keybit); > > - __clear_bit(BTN_RIGHT, input->keybit); > > - __clear_bit(BTN_MIDDLE, input->keybit); > > __set_bit(BTN_MOUSE, input->keybit); > > - __set_bit(INPUT_PROP_BUTTONPAD, input->propbit); > > __set_bit(BTN_TOOL_FINGER, input->keybit); > > + input_set_property(input, INPUT_PROP_BUTTONPAD); > > > > mt_flags = INPUT_MT_POINTER | INPUT_MT_DROP_UNUSED | > > INPUT_MT_TRACK; > > @@ -559,8 +557,6 @@ static int magicmouse_setup_input(struct input_dev *input, struct hid_device *hd > > * button (BTN_LEFT == BTN_MOUSE). Make sure we don't > > * advertise buttons that don't exist... > > */ > > - __clear_bit(BTN_RIGHT, input->keybit); > > - __clear_bit(BTN_MIDDLE, input->keybit); > > __set_bit(BTN_MOUSE, input->keybit); > > __set_bit(BTN_TOOL_FINGER, input->keybit); > > __set_bit(BTN_TOOL_DOUBLETAP, input->keybit); > > @@ -569,7 +565,7 @@ static int magicmouse_setup_input(struct input_dev *input, struct hid_device *hd > > __set_bit(BTN_TOOL_QUINTTAP, input->keybit); > > __set_bit(BTN_TOUCH, input->keybit); > > __set_bit(INPUT_PROP_POINTER, input->propbit); > > - __set_bit(INPUT_PROP_BUTTONPAD, input->propbit); > > + input_set_property(input, INPUT_PROP_BUTTONPAD); > > } > > > > > > diff --git a/drivers/hid/hid-multitouch.c b/drivers/hid/hid-multitouch.c > > index 082376a6cb3d..1bffb1787fd6 100644 > > --- a/drivers/hid/hid-multitouch.c > > +++ b/drivers/hid/hid-multitouch.c > > @@ -1287,7 +1287,7 @@ static int mt_touch_input_configured(struct hid_device *hdev, > > td->is_buttonpad = true; > > > > if (td->is_buttonpad) > > - __set_bit(INPUT_PROP_BUTTONPAD, input->propbit); > > + input_set_property(input, INPUT_PROP_BUTTONPAD); > > > > app->pending_palm_slots = devm_kcalloc(&hi->input->dev, > > BITS_TO_LONGS(td->maxcontacts), > > diff --git a/drivers/hid/hid-playstation.c b/drivers/hid/hid-playstation.c > > index b1b5721b5d8f..50815257ff86 100644 > > --- a/drivers/hid/hid-playstation.c > > +++ b/drivers/hid/hid-playstation.c > > @@ -651,8 +651,7 @@ static struct input_dev *ps_touchpad_create(struct hid_device *hdev, int width, > > return ERR_CAST(touchpad); > > > > /* Map button underneath touchpad to BTN_LEFT. */ > > - input_set_capability(touchpad, EV_KEY, BTN_LEFT); > > - __set_bit(INPUT_PROP_BUTTONPAD, touchpad->propbit); > > + input_set_property(touchpad, INPUT_PROP_BUTTONPAD); > > > > input_set_abs_params(touchpad, ABS_MT_POSITION_X, 0, width - 1, 0, 0); > > input_set_abs_params(touchpad, ABS_MT_POSITION_Y, 0, height - 1, 0, 0); > > diff --git a/drivers/hid/hid-sony.c b/drivers/hid/hid-sony.c > > index d1b107d547f5..302fd8fea243 100644 > > --- a/drivers/hid/hid-sony.c > > +++ b/drivers/hid/hid-sony.c > > @@ -1519,9 +1519,7 @@ static int sony_register_touchpad(struct sony_sc *sc, int touch_count, > > sc->touchpad->name = name; > > > > /* We map the button underneath the touchpad to BTN_LEFT. */ > > - __set_bit(EV_KEY, sc->touchpad->evbit); > > - __set_bit(BTN_LEFT, sc->touchpad->keybit); > > - __set_bit(INPUT_PROP_BUTTONPAD, sc->touchpad->propbit); > > + input_set_property(sc->touchpad, INPUT_PROP_BUTTONPAD); > > > > input_set_abs_params(sc->touchpad, ABS_MT_POSITION_X, 0, w, 0, 0); > > input_set_abs_params(sc->touchpad, ABS_MT_POSITION_Y, 0, h, 0, 0); > > diff --git a/drivers/input/keyboard/applespi.c b/drivers/input/keyboard/applespi.c > > index eda1b23002b5..9fc7608a8bbc 100644 > > --- a/drivers/input/keyboard/applespi.c > > +++ b/drivers/input/keyboard/applespi.c > > @@ -1289,7 +1289,7 @@ applespi_register_touchpad_device(struct applespi_data *applespi, > > input_set_capability(touchpad_input_dev, EV_REL, REL_Y); > > > > __set_bit(INPUT_PROP_POINTER, touchpad_input_dev->propbit); > > - __set_bit(INPUT_PROP_BUTTONPAD, touchpad_input_dev->propbit); > > + input_set_property(touchpad_input_dev, INPUT_PROP_BUTTONPAD); > > > > /* finger touch area */ > > input_set_abs_params(touchpad_input_dev, ABS_MT_TOUCH_MAJOR, > > diff --git a/drivers/input/mouse/alps.c b/drivers/input/mouse/alps.c > > index 4a6b33bbe7ea..d04c9553f9f2 100644 > > --- a/drivers/input/mouse/alps.c > > +++ b/drivers/input/mouse/alps.c > > @@ -3082,8 +3082,7 @@ int alps_init(struct psmouse *psmouse) > > dev1->keybit[BIT_WORD(BTN_2)] |= BIT_MASK(BTN_2); > > dev1->keybit[BIT_WORD(BTN_3)] |= BIT_MASK(BTN_3); > > } else if (priv->flags & ALPS_BUTTONPAD) { > > - set_bit(INPUT_PROP_BUTTONPAD, dev1->propbit); > > - clear_bit(BTN_RIGHT, dev1->keybit); > > + input_set_property(dev1, INPUT_PROP_BUTTONPAD); > > } else { > > dev1->keybit[BIT_WORD(BTN_MIDDLE)] |= BIT_MASK(BTN_MIDDLE); > > } > > diff --git a/drivers/input/mouse/bcm5974.c b/drivers/input/mouse/bcm5974.c > > index 59a14505b9cd..986fed0ef978 100644 > > --- a/drivers/input/mouse/bcm5974.c > > +++ b/drivers/input/mouse/bcm5974.c > > @@ -538,7 +538,7 @@ static void setup_events_to_report(struct input_dev *input_dev, > > __set_bit(BTN_LEFT, input_dev->keybit); > > > > if (cfg->caps & HAS_INTEGRATED_BUTTON) > > - __set_bit(INPUT_PROP_BUTTONPAD, input_dev->propbit); > > + input_set_property(input_dev, INPUT_PROP_BUTTONPAD); > > > > input_mt_init_slots(input_dev, MAX_FINGERS, > > INPUT_MT_POINTER | INPUT_MT_DROP_UNUSED | INPUT_MT_TRACK); > > diff --git a/drivers/input/mouse/cyapa.c b/drivers/input/mouse/cyapa.c > > index 77cc653edca2..364de350128b 100644 > > --- a/drivers/input/mouse/cyapa.c > > +++ b/drivers/input/mouse/cyapa.c > > @@ -501,7 +501,7 @@ static int cyapa_create_input_dev(struct cyapa *cyapa) > > __set_bit(BTN_RIGHT, input->keybit); > > > > if (cyapa->btn_capability == CAPABILITY_LEFT_BTN_MASK) > > - __set_bit(INPUT_PROP_BUTTONPAD, input->propbit); > > + input_set_property(input, INPUT_PROP_BUTTONPAD); > > > > /* Handle pointer emulation and unused slots in core */ > > error = input_mt_init_slots(input, CYAPA_MAX_MT_SLOTS, > > diff --git a/drivers/input/mouse/elan_i2c_core.c b/drivers/input/mouse/elan_i2c_core.c > > index 47af62c12267..b6cd6b282424 100644 > > --- a/drivers/input/mouse/elan_i2c_core.c > > +++ b/drivers/input/mouse/elan_i2c_core.c > > @@ -1177,7 +1177,7 @@ static int elan_setup_input_device(struct elan_tp_data *data) > > __set_bit(EV_ABS, input->evbit); > > __set_bit(INPUT_PROP_POINTER, input->propbit); > > if (data->clickpad) { > > - __set_bit(INPUT_PROP_BUTTONPAD, input->propbit); > > + input_set_property(input, INPUT_PROP_BUTTONPAD); > > } else { > > __set_bit(BTN_RIGHT, input->keybit); > > if (data->middle_button) > > diff --git a/drivers/input/mouse/elantech.c b/drivers/input/mouse/elantech.c > > index 956d9cd34796..38386a89831e 100644 > > --- a/drivers/input/mouse/elantech.c > > +++ b/drivers/input/mouse/elantech.c > > @@ -1122,10 +1122,8 @@ static void elantech_set_buttonpad_prop(struct psmouse *psmouse) > > struct input_dev *dev = psmouse->dev; > > struct elantech_data *etd = psmouse->private; > > > > - if (elantech_is_buttonpad(&etd->info)) { > > - __set_bit(INPUT_PROP_BUTTONPAD, dev->propbit); > > - __clear_bit(BTN_RIGHT, dev->keybit); > > - } > > + if (elantech_is_buttonpad(&etd->info)) > > + input_set_property(dev, INPUT_PROP_BUTTONPAD); > > } > > > > /* > > diff --git a/drivers/input/mouse/focaltech.c b/drivers/input/mouse/focaltech.c > > index 6fd5fff0cbff..292b931483c0 100644 > > --- a/drivers/input/mouse/focaltech.c > > +++ b/drivers/input/mouse/focaltech.c > > @@ -330,8 +330,6 @@ static void focaltech_set_input_params(struct psmouse *psmouse) > > __clear_bit(EV_REL, dev->evbit); > > __clear_bit(REL_X, dev->relbit); > > __clear_bit(REL_Y, dev->relbit); > > - __clear_bit(BTN_RIGHT, dev->keybit); > > - __clear_bit(BTN_MIDDLE, dev->keybit); > > > > /* > > * Now set up our capabilities. > > @@ -341,7 +339,7 @@ static void focaltech_set_input_params(struct psmouse *psmouse) > > input_set_abs_params(dev, ABS_MT_POSITION_Y, 0, priv->y_max, 0, 0); > > input_set_abs_params(dev, ABS_TOOL_WIDTH, 0, 15, 0, 0); > > input_mt_init_slots(dev, 5, INPUT_MT_POINTER); > > - __set_bit(INPUT_PROP_BUTTONPAD, dev->propbit); > > + input_set_property(dev, INPUT_PROP_BUTTONPAD); > > } > > > > static int focaltech_read_register(struct ps2dev *ps2dev, int reg, > > diff --git a/drivers/input/mouse/synaptics.c b/drivers/input/mouse/synaptics.c > > index ffad142801b3..1110c85dd370 100644 > > --- a/drivers/input/mouse/synaptics.c > > +++ b/drivers/input/mouse/synaptics.c > > @@ -1349,7 +1349,7 @@ static int set_input_params(struct psmouse *psmouse, > > input_set_capability(dev, EV_KEY, BTN_0 + i); > > > > if (SYN_CAP_CLICKPAD(info->ext_cap_0c)) { > > - __set_bit(INPUT_PROP_BUTTONPAD, dev->propbit); > > + input_set_property(dev, INPUT_PROP_BUTTONPAD); > > if (psmouse_matches_pnp_id(psmouse, topbuttonpad_pnp_ids) && > > !SYN_CAP_EXT_BUTTONS_STICK(info->ext_cap_10)) > > __set_bit(INPUT_PROP_TOPBUTTONPAD, dev->propbit); > > diff --git a/drivers/input/rmi4/rmi_f30.c b/drivers/input/rmi4/rmi_f30.c > > index 35045f161dc2..92ca5a1da1de 100644 > > --- a/drivers/input/rmi4/rmi_f30.c > > +++ b/drivers/input/rmi4/rmi_f30.c > > @@ -265,7 +265,7 @@ static int rmi_f30_map_gpios(struct rmi_function *fn, > > * mapped buttons. > > */ > > if (pdata->gpio_data.buttonpad || (button - BTN_LEFT == 1)) > > - __set_bit(INPUT_PROP_BUTTONPAD, input->propbit); > > + input_set_property(input, INPUT_PROP_BUTTONPAD); > > > > return 0; > > } > > diff --git a/drivers/input/rmi4/rmi_f3a.c b/drivers/input/rmi4/rmi_f3a.c > > index 0e8baed84dbb..4017deff191f 100644 > > --- a/drivers/input/rmi4/rmi_f3a.c > > +++ b/drivers/input/rmi4/rmi_f3a.c > > @@ -159,7 +159,7 @@ static int rmi_f3a_map_gpios(struct rmi_function *fn, struct f3a_data *f3a, > > input->keycodemax = f3a->gpio_count; > > > > if (pdata->gpio_data.buttonpad || (button - BTN_LEFT == 1)) > > - __set_bit(INPUT_PROP_BUTTONPAD, input->propbit); > > + input_set_property(input, INPUT_PROP_BUTTONPAD); > > > > return 0; > > } > > diff --git a/drivers/input/touchscreen/atmel_mxt_ts.c b/drivers/input/touchscreen/atmel_mxt_ts.c > > index 05de92c0293b..fd9b6774ee9e 100644 > > --- a/drivers/input/touchscreen/atmel_mxt_ts.c > > +++ b/drivers/input/touchscreen/atmel_mxt_ts.c > > @@ -2028,7 +2028,7 @@ static void mxt_set_up_as_touchpad(struct input_dev *input_dev, > > > > input_dev->name = "Atmel maXTouch Touchpad"; > > > > - __set_bit(INPUT_PROP_BUTTONPAD, input_dev->propbit); > > + input_set_property(input_dev, INPUT_PROP_BUTTONPAD); > > > > input_abs_set_res(input_dev, ABS_X, MXT_PIXELS_PER_MM); > > input_abs_set_res(input_dev, ABS_Y, MXT_PIXELS_PER_MM); > > -- > > 2.25.1 > > > > -- > Dmitry > ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2 2/2] Input: set INPUT_PROP_BUTTONPAD using input_set_property() 2022-01-04 6:24 ` Dmitry Torokhov 2022-01-04 9:18 ` Benjamin Tissoires @ 2022-01-06 13:09 ` Jiri Kosina 1 sibling, 0 replies; 9+ messages in thread From: Jiri Kosina @ 2022-01-06 13:09 UTC (permalink / raw) To: Dmitry Torokhov Cc: José Expósito, benjamin.tissoires, peter.hutterer, roderick.colenbrander, pali, rydberg, nick, linux-input, linux-kernel On Mon, 3 Jan 2022, Dmitry Torokhov wrote: > Hi José, > > On Thu, Dec 02, 2021 at 12:08:07PM +0100, José Expósito wrote: > > A short description would be appreciated. > > Jiri, Benjamin, do you mind if I take this through my tree or you would > prefer having this split? Please just take it through input.git with Acked-by: Jiri Kosina <jkosina@suse.cz> for the HID parts. Thanks! -- Jiri Kosina SUSE Labs ^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2022-01-08 19:51 UTC | newest] Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-12-02 11:08 [PATCH v2 0/2] Do not map BTN_RIGHT/MIDDLE on buttonpads José Expósito 2021-12-02 11:08 ` [PATCH v2 1/2] Input: add input_set_property() José Expósito 2022-01-04 6:22 ` Dmitry Torokhov 2022-01-08 19:51 ` José Expósito 2021-12-02 11:08 ` [PATCH v2 2/2] Input: set INPUT_PROP_BUTTONPAD using input_set_property() José Expósito 2021-12-07 5:03 ` Peter Hutterer 2022-01-04 6:24 ` Dmitry Torokhov 2022-01-04 9:18 ` Benjamin Tissoires 2022-01-06 13:09 ` Jiri Kosina
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).