Bug 51953 - broken keyboard on Asus Vivobook X1504VAP
Summary: broken keyboard on Asus Vivobook X1504VAP
Status: CLOSED WORKSFORME
Alias: None
Product: Sisyphus
Classification: Development
Component: kernel-image-6.12 (show other bugs)
Version: unstable
Hardware: x86_64 Linux
: P5 normal
Assignee: kernelbot@altlinux.org
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-11-05 16:09 MSK by Sergey Bolshakov
Modified: 2024-11-18 11:59 MSK (History)
4 users (show)

See Also:


Attachments
патч (843 bytes, patch)
2024-11-05 16:09 MSK, Sergey Bolshakov
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Sergey Bolshakov 2024-11-05 16:09:31 MSK
Created attachment 17126 [details]
патч

Чиним клавиатуру на ASUS X1504VA Vivobook 15
Comment 1 Sergey Bolshakov 2024-11-07 18:27:59 MSK
Также хорошо было бы иметь
CONFIG_PINCTRL_TIGERLAKE=y

поскольку иначе на этом ноуте не проходит должным образом инициализация тачпада.
Comment 2 Vitaly Chikunov 2024-11-11 18:34:55 MSK
На большинстве дистрибутивов PINCTRL_TIGERLAKE=m, как и у нас. Может лучше как то добавлять его в initrd?


 suse | config/i386/pae:CONFIG_PINCTRL_TIGERLAKE=m
 suse | config/x86_64/default:CONFIG_PINCTRL_TIGERLAKE=m
 redhat/kernel-ark | redhat/configs/common/generic/x86/CONFIG_PINCTRL_TIGERLAKE:CONFIG_PINCTRL_TIGERLAKE=m
 ubuntu/linux | debian.master/config/annotations:CONFIG_PINCTRL_TIGERLAKE                        
 policy<{'amd64': 'm'}>    
 ubuntu/linux | kevinbecker/setup/realtime-6.11:debian.master/config/annotations:CONFIG_PINCTRL_TIGERLAKE                        
 policy<{'amd64': 'm'}>                                                                                              
 debian/linux | debian/config/kernelarch-x86/config:CONFIG_PINCTRL_TIGERLAKE=y                                             
 clearlinux | config:CONFIG_PINCTRL_TIGERLAKE=m                                                                            
 archlinux | config:CONFIG_PINCTRL_TIGERLAKE=m                                                                             
 openmandriva | i386-omv-defconfig:CONFIG_PINCTRL_TIGERLAKE=m                                                              
 openmandriva | x86-omv-defconfig:CONFIG_PINCTRL_TIGERLAKE=m                                                               
 rosa | kernel-i686.config:CONFIG_PINCTRL_TIGERLAKE=m
 rosa | kernel-x86_64.config:CONFIG_PINCTRL_TIGERLAKE=m
 azurelinux | SPECS-EXTENDED/kernel-rt/config:# CONFIG_PINCTRL_TIGERLAKE is not set
 azurelinux | SPECS/kernel-mshv/config:# CONFIG_PINCTRL_TIGERLAKE is not set
 azurelinux | SPECS/kernel/config:# CONFIG_PINCTRL_TIGERLAKE is not set

В принципе это +44 К ядру.
Comment 3 Vitaly Chikunov 2024-11-11 18:40:20 MSK
На счет патча - пожалуйста, шлите его в апстрим для review.
Comment 4 Sergey Bolshakov 2024-11-12 17:02:04 MSK
(In reply to Vitaly Chikunov from comment #2)
> На большинстве дистрибутивов PINCTRL_TIGERLAKE=m, как и у нас. Может лучше
> как то добавлять его в initrd?
> 
> 
>  suse | config/i386/pae:CONFIG_PINCTRL_TIGERLAKE=m
>  suse | config/x86_64/default:CONFIG_PINCTRL_TIGERLAKE=m
>  redhat/kernel-ark |
> redhat/configs/common/generic/x86/CONFIG_PINCTRL_TIGERLAKE:
> CONFIG_PINCTRL_TIGERLAKE=m
>  ubuntu/linux | debian.master/config/annotations:CONFIG_PINCTRL_TIGERLAKE   
> 
>  policy<{'amd64': 'm'}>    
>  ubuntu/linux |
> kevinbecker/setup/realtime-6.11:debian.master/config/annotations:
> CONFIG_PINCTRL_TIGERLAKE                        
>  policy<{'amd64': 'm'}>                                                     
> 
>  debian/linux |
> debian/config/kernelarch-x86/config:CONFIG_PINCTRL_TIGERLAKE=y              
> 
>  clearlinux | config:CONFIG_PINCTRL_TIGERLAKE=m                             
> 
>  archlinux | config:CONFIG_PINCTRL_TIGERLAKE=m                              
> 
>  openmandriva | i386-omv-defconfig:CONFIG_PINCTRL_TIGERLAKE=m               
> 
>  openmandriva | x86-omv-defconfig:CONFIG_PINCTRL_TIGERLAKE=m                
> 
>  rosa | kernel-i686.config:CONFIG_PINCTRL_TIGERLAKE=m
>  rosa | kernel-x86_64.config:CONFIG_PINCTRL_TIGERLAKE=m
>  azurelinux | SPECS-EXTENDED/kernel-rt/config:# CONFIG_PINCTRL_TIGERLAKE is
> not set
>  azurelinux | SPECS/kernel-mshv/config:# CONFIG_PINCTRL_TIGERLAKE is not set
>  azurelinux | SPECS/kernel/config:# CONFIG_PINCTRL_TIGERLAKE is not set
> 
> В принципе это +44 К ядру.

Там где-то гонка, похоже, и висящий на i2c тачпад инициализируется как-то не так.
Специально изготовленный initrd, где этот (pinctrl) модуль грузится c MODULE_PRELOAD, до 
старта udev, тоже решает проблему, да, но тогда нужно устраивать угадайку в make-initrd,
тестировать и т.п -- и чего ради ?
Comment 5 Sergey Bolshakov 2024-11-12 17:03:19 MSK
(In reply to Vitaly Chikunov from comment #3)
> На счет патча - пожалуйста, шлите его в апстрим для review.

Шутите, должно быть.
Comment 6 Aleksandr Sysoev 2024-11-13 10:43:23 MSK
После обновления до актуальной версии bios: X1504VAP-305 клавиатура заработала и без патча. Ошибка не актуальна.
Comment 7 Sergey Bolshakov 2024-11-18 11:59:12 MSK
(In reply to Aleksandr Sysoev from comment #6)
> После обновления до актуальной версии bios: X1504VAP-305 клавиатура
> заработала и без патча. Ошибка не актуальна.

Тем не менее, как видно из патча, в ядре уже есть вхождение для Asus Vivobook X1704VAP,
для которого, я не сомневаюсь, точно так же есть обновление bios.
наверное, дело в том, что полезно иметь работающую клавиатуру
и для того, чтобы установить обновление bios, правда ?