| Message ID | Zhp5QlMGazcW+RV5@black (mailing list archive) |
|---|---|
| State | New |
| Delegated to: | Jiri Kosina |
| Headers | show |
| Series | HID: hid-debug: more informative output for EV_KEY | expand |
On Sat, 13 Apr 2024, Thomas Kuehne wrote: > Currently hid-debug's hid_resolv_event prints questions marks for > all entries without explicit mapping information. This makes > debugging unnecessarily complicated as multiple different > keys may simply result in the same uninformative output. > > Some common event codes are deliberately not defined in > input-event-codes.h. For example the 16th gamepad key. > > Instead, print the hexadecimal codes for all events without symbolic > names. Applied, thanks.
diff --git a/drivers/hid/hid-debug.c b/drivers/hid/hid-debug.c index add353a17853..0a0c435a1284 100644 --- a/drivers/hid/hid-debug.c +++ b/drivers/hid/hid-debug.c @@ -3582,8 +3582,15 @@ static const char **names[EV_MAX + 1] = { static void hid_resolv_event(__u8 type, __u16 code, struct seq_file *f) { - seq_printf(f, "%s.%s", events[type] ? events[type] : "?", - names[type] ? (names[type][code] ? names[type][code] : "?") : "?"); + if (events[type]) + seq_printf(f, "%s.", events[type]); + else + seq_printf(f, "%02x.", type); + + if (names[type] && names[type][code]) + seq_printf(f, "%s", names[type][code]); + else + seq_printf(f, "%04x", code); } static void hid_dump_input_mapping(struct hid_device *hid, struct seq_file *f)
Currently hid-debug's hid_resolv_event prints questions marks for all entries without explicit mapping information. This makes debugging unnecessarily complicated as multiple different keys may simply result in the same uninformative output. Some common event codes are deliberately not defined in input-event-codes.h. For example the 16th gamepad key. Instead, print the hexadecimal codes for all events without symbolic names. Signed-off-by: Thomas Kuehne <thomas.kuehne@gmx.li> --- drivers/hid/hid-debug.c | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) base-commit: 9ed46da14b9b9b2ad4edb3b0c545b6dbe5c00d39 -- 2.40.1