diff options
| author | 2022-08-23 16:29:10 +0000 | |
|---|---|---|
| committer | 2022-08-23 16:35:00 +0000 | |
| commit | 5189478291e79cfe9bb5aba30cbb7de856b6bfcc (patch) | |
| tree | 3d816c3750af2c4ea230bcfb89c06a5c6e938843 /libs/input/KeyLayoutMap.cpp | |
| parent | b4ef6ab1e7c0f6c5b924a5812eaa40c8eeaab617 (diff) | |
EventHub: Associate AsociatedDevice using sysfs path, not descriptor
EventHub previously attempted to link an EventHub device to an
AssociatedDevice - which contains info about the EventHub device
obtained through sysfs - through the descriptor. Since each EventHub
device has a unique descriptor whose uniqueness is guaranteed by the
use of the nonce value, using the descriptor to link the
AssociatedDevice to device means that no two devices will share the same
AssociatedDevice even if they have the same sysfs paths. This is clearly
not working as intended.
We now link the AssociatedDevice to the EventHub device by the sysfs
path so that separate EventHub devices that share the same sysfs path
also share the same AssociatedDevice.
Unfortunately, we don't have good code coverage for reading from sysfs,
and there are no existing test cases.
Bug: 243005009
Test: Manual, check input dump
Change-Id: Iea2bb7fd0493baa6e03df15f876c5d895c997b14
Diffstat (limited to 'libs/input/KeyLayoutMap.cpp')
0 files changed, 0 insertions, 0 deletions