Skip to content

linux: Parse HID_FIRMWARE_VERSION from uevent for release_number #844

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
Youw wants to merge 2 commits into from
Closed

linux: Parse HID_FIRMWARE_VERSION from uevent for release_number #844

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
65f8797
linux: Parse HID_FIRMWARE_VERSION from uevent for release_number
JohnAZoidberg Jan 24, 2026
31c0ef2
linux: note the kernel version that provides HID_FIRMWARE_VERSION
Youw Nov 8, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 11 additions & 3 deletions linux/hid.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -571,7 +571,8 @@ static int get_hid_report_descriptor_from_hidraw(hid_device *dev, struct hidraw_
*/
static int parse_uevent_info(const char *uevent, unsigned *bus_type,
unsigned short *vendor_id, unsigned short *product_id,
char **serial_number_utf8, char **product_name_utf8)
char **serial_number_utf8, char **product_name_utf8,
unsigned short *version)
{
char tmp[1024];

Expand Down Expand Up @@ -622,6 +623,11 @@ static int parse_uevent_info(const char *uevent, unsigned *bus_type,
/* The caller has to free the serial number */
*serial_number_utf8 = strdup(value);
found_serial = 1;
} else if (strcmp(key, "HID_FIRMWARE_VERSION") == 0) {
/* Device version from kernel, introduced in Linux v7.1:
https://github.com/torvalds/linux/commit/6ca9029c823b7853e980585e757343e0e84227cd
*/
*version = (unsigned short)strtol(value, NULL, 16);
}

next_line:
Expand All @@ -648,6 +654,7 @@ static struct hid_device_info * create_device_info_for_device(struct udev_device
char *serial_number_utf8 = NULL;
char *product_name_utf8 = NULL;
unsigned bus_type;
unsigned short hid_version = 0;
int result;
struct hidraw_report_descriptor report_desc;

Expand All @@ -670,7 +677,8 @@ static struct hid_device_info * create_device_info_for_device(struct udev_device
&dev_vid,
&dev_pid,
&serial_number_utf8,
&product_name_utf8);
&product_name_utf8,
&hid_version);

if (!result) {
/* parse_uevent_info() failed for at least one field. */
Expand Down Expand Up @@ -709,7 +717,7 @@ static struct hid_device_info * create_device_info_for_device(struct udev_device
cur_dev->serial_number = utf8_to_wchar_t(serial_number_utf8);

/* Release Number */
cur_dev->release_number = 0x0;
cur_dev->release_number = hid_version;

/* Interface Number */
cur_dev->interface_number = -1;
Expand Down
Loading