Re: [d-kernel] [PATCH 2/3] Revert "Input: clear BTN_RIGHT/MIDDLE on buttonpads"

[Vitaly Chikunov <vt@altlinux.org>]

On Thu, Mar 24, 2022 at 11:57:46AM +0300, Nikolai Kostrigin wrote:
> 
> 24.03.2022 11:16, Vitaly Chikunov пишет:
> > On Thu, Mar 24, 2022 at 10:51:24AM +0300, nickel@altlinux.org wrote:
> > > From: Nikolai Kostrigin <nickel@altlinux.org>
> > > 
> > > This commit breaks touchpad right button operation on ICL Si1516 laptop
> > 
> > Как я понимаю, вы сделали выбор в пользу ICL Si1516 laptop, но против:
> > 
> >       - 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
> > 
> > Почему почему этот laptop должен наладиться, а эти сломаться?
> > 
> 
> Свою позицию я уже высказывал в упомянутой баге. Они не сломаются, а на
> некоторое время задержится их починка, т.к. они, по идее, изначально
> "сломаны". До появления ломающего коммита никто не заявил о наличии проблем
> с этими моделями, т.е. никто не ожидал его с нетерпением.
> В противоположность этому, для ICL Si1516 наблюдается очевидный регресс,
> подтвержденный тестированием, что блокирует обновление ядер в стабильных
> бранчах.
> Мы возьмем эту проблему в работу ASAP и, впоследствии, планируем "откатить
> откат".

Хорошо. Добавьте пожалуйста в коммит (как тэг Link:) ссылку на баг[1] или багрепорт
письмо в рассылке апстрима[2] по этой проблеме. Или просто пришлите её и я добавлю
сам в коммит. (Это нужно для того чтобы можно было потом возвращаться к этой
проблеме и отслеживать её решение).

[1] https://bugzilla.kernel.org/
[2] https://lore.kernel.org/

> 
> > > 
> > > Refer to ALT#42123
> > > 
> > > This reverts commit 6279c5263dedc6af67c194ec8e0f91bc2849cdf8.
> > > 
> > > Reported-by: Sergey Ivanov <zagagyka@basealt.ru>
> > > Signed-off-by: Nikolai Kostrigin <nickel@altlinux.org>
> > > ---
> > >   drivers/input/input.c | 6 ------
> > >   1 file changed, 6 deletions(-)
> > > 
> > > diff --git a/drivers/input/input.c b/drivers/input/input.c
> > > index c3139bc2aa0d..ccaeb2426385 100644
> > > --- a/drivers/input/input.c
> > > +++ b/drivers/input/input.c
> > > @@ -2285,12 +2285,6 @@ int input_register_device(struct input_dev *dev)
> > >   	/* KEY_RESERVED is not supposed to be transmitted to userspace. */
> > >   	__clear_bit(KEY_RESERVED, dev->keybit);
> > > -	/* Buttonpads should not map BTN_RIGHT and/or BTN_MIDDLE. */
> > > -	if (test_bit(INPUT_PROP_BUTTONPAD, dev->propbit)) {
> > > -		__clear_bit(BTN_RIGHT, dev->keybit);
> > > -		__clear_bit(BTN_MIDDLE, dev->keybit);
> > > -	}
> > > -
> > >   	/* Make sure that bitmasks not mentioned in dev->evbit are clean. */
> > >   	input_cleanse_bitmasks(dev);
> > > -- 
> > > 2.33.0
> > > 
> > > _______________________________________________
> > > devel-kernel mailing list
> > > devel-kernel@lists.altlinux.org
> > > https://lists.altlinux.org/mailman/listinfo/devel-kernel
> > _______________________________________________
> > devel-kernel mailing list
> > devel-kernel@lists.altlinux.org
> > https://lists.altlinux.org/mailman/listinfo/devel-kernel
> 
> -- 
> Best regards,
> Nikolai Kostrigin
> _______________________________________________
> devel-kernel mailing list
> devel-kernel@lists.altlinux.org
> https://lists.altlinux.org/mailman/listinfo/devel-kernel