-
Notifications
You must be signed in to change notification settings - Fork 72
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
WinUSB bulk in/out device #49
Comments
@rundekugel are you able to trace the |
@pabs3 sorry, I found no time to trace the issue. |
Please let us know if you find time to trace the issue. I think you have provided enough info about WinUSB, thanks. For the sake of completeness, which versions of the Linux kernel are you finding the error on? Have you tried the most recent ones (5.15-rc1 right now or 5.14.5 stable)? Unfortunately I don't have any physical USB UDC gadget devices nor do I have a Windows install, so I won't be able to debug this either. |
I'm sorry, I don't find time to trace the issue. |
I created an small embedded WinUSB device which offers 2 bulk endpoints. This device can communicate with Linux and with Windows10 without installing driver, or a .inf file. And it's not recognized as serial com port.
Now I try to build an embedded Linux gadget with your libusbgx. It must respond to OS_DESC (0xee) as WinUSB device. And must not be recognized as serial com port. It shall behave like a g_serial with param "use_acm=no"
If I build it with filesystem commands. Then the os_desc setting in Window10 it's recognized in registry settings
But not recognized as WinUSB device
It looks like, the os_desc feature does only fully work for RNDIS.
This is the respond to vendor request, which works when I send it from my embedded WinUSB to Windows:
I tried with a modified example of your code:
myWinusbGadget.zip
diff:
winusb-diff.zip
I get error when trying to usbg_set_interf_os_desc() :
How can I make it working for gser ? Is it possible with changing your src? Or do I have to change the kernel gadget files?
Can you help me to add this feature to your lib?
The text was updated successfully, but these errors were encountered: