-
Notifications
You must be signed in to change notification settings - Fork 6.5k
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.
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
samples: subsys: usb: webusb: Fix Win10 detection #59758
samples: subsys: usb: webusb: Fix Win10 detection #59758
Conversation
789ee76
to
36616c5
Compare
36616c5
to
ac97f5d
Compare
.wLength = sizeof(struct ms_os_20_guids_property_desc), | ||
.wDescriptorType = MS_OS_20_FEATURE_REG_PROPERTY, | ||
.wPropertyDataType = MS_OS_20_PROPERTY_DATA_REG_MULTI_SZ, | ||
.wPropertyNameLength = 42, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wish zephyr had sizeof_field()
macro like Linux kernel does.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Confirmed to work on nrf52840dk_nrf52840. Windows 10 automatically installed WinUSB driver and Zadig shows device as WCID compatible.
ac97f5d
to
f3be566
Compare
e420eb5
to
e0b6d95
Compare
This patch refactors the usage of MS OS 2.0 descriptors in the WebUSB sample. The function subset header was removed since it is not allowed for non-composite devices. Also, a new random GUID was added for automatic driver installation. Signed-off-by: Maximilian Deubel <[email protected]>
e0b6d95
to
675c7e0
Compare
This patch refactors the usage of MS OS 2.0 descriptors in the WebUSB sample. The function subset header was removed since it is not allowed for non-composite devices.
Also, a new random GUID was added for automatic driver installation.