Skip to content
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

Windows: Add isochronous support for libusbK and WinUSB #46

Closed
piedar opened this issue Dec 18, 2014 · 12 comments
Closed

Windows: Add isochronous support for libusbK and WinUSB #46

piedar opened this issue Dec 18, 2014 · 12 comments
Milestone

Comments

@piedar
Copy link

@piedar piedar commented Dec 18, 2014

@JoshBlake proposed this patch to libusbx, but it was never merged. A cherry-pick still applies cleanly to libusb HEAD.

git clone https://github.com/libusb/libusb.git
cd libusb
git remote add jblake https://github.com/JoshBlake/libusbx.git
git fetch jblake
git cherry-pick c5b0af4 1c74211

I tested the patch and it appears to work with libusbK v3.0.7.0. Any chance this can be merged into libusb? It would make libfreenect a lot easier to build on Windows by replacing our emulation layer that is woefully out of date.

@jjqq2013
Copy link

@jjqq2013 jjqq2013 commented Jan 16, 2015

from Windows 8.1, WinUSB also support isochronous mode. Hope this be implemented in future.

@mcuee
Copy link
Contributor

@mcuee mcuee commented Aug 17, 2015

It will be good to add WinUSB isoc support (directly to libusb WIndows backend or through libusbK). Travis mentioned he intended to add WinUSB isoc support to libusbK recently but I am not so sure when he will start that work.

@mcuee
Copy link
Contributor

@mcuee mcuee commented Aug 17, 2015

Another option is to look at usbdk backend which has not yet been integrated.

https://github.com/dmitryfleytman/libusb/tree/usbdk-backend-v3

@mcuee mcuee changed the title Windows: Add isochronous support via libusbk backend Windows: Add isochronous support Aug 17, 2015
@mcuee mcuee added the enhancement label Aug 17, 2015
@mkassner
Copy link

@mkassner mkassner commented Aug 20, 2015

https://github.com/ktossell/libuvc would finally work on windows if isochronous transfers where to be implemented with winusb8.1: libuvc/libuvc#12

Any chance for this to happen in the near future?

@xlz
Copy link

@xlz xlz commented Oct 11, 2015

@mcuee
What is the plan now?
Is the usbdk backend favored over the libusbk patch? WinUSB? What about Windows 7 users?

@mcuee
Copy link
Contributor

@mcuee mcuee commented Oct 17, 2015

@diabolo38
Copy link

@diabolo38 diabolo38 commented Oct 18, 2015

with new Microsoft policy on driver signing i'm scared no new libusbk release will ever happen.
So usbdk and winusb seam the only possible candidate to enhance support on windows.

I will not be able to participate actively on development but i can participate and provide some punctual help with testing on win 7 (uvc cam, gadget 0 target) and bus level debug if needed.

@xlz
Copy link

@xlz xlz commented Jan 8, 2016

I just tested usbdk. It failed to enumerate Kinect v2 sensor (no other tests could follow). It also has some incomplete support for important features daynix/UsbDk#10, which is specially required by libfreenect2.

The isochronous patch for libusbK backend just works, and has been extensively tested. What is the reason not to merge it?

@hjelmn hjelmn added the windows label Mar 5, 2016
@mcuee mcuee changed the title Windows: Add isochronous support Windows: Add isochronous support for libusbK and WinUSB Mar 6, 2016
@mcuee
Copy link
Contributor

@mcuee mcuee commented Mar 6, 2016

Ref: #246

@philemonf
Copy link
Contributor

@philemonf philemonf commented Jul 18, 2017

Ref: #284

@mcuee
Copy link
Contributor

@mcuee mcuee commented Mar 6, 2018

Take note #246 is now closed since it has been merged.

@piedar
Copy link
Author

@piedar piedar commented Feb 24, 2019

Excellent! 👍

@piedar piedar closed this Feb 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
8 participants
You can’t perform that action at this time.