openFPGALoader icon indicating copy to clipboard operation
openFPGALoader copied to clipboard

WIP: bmp: Implement. Requires firmware from https://github.com/UweBonnes/b…

Open UweBonnes opened this issue 3 years ago • 6 comments

…lackmagic/tree/fixes

Hello,

if you find time, please have a look at this PR. It works only for Linux, maybe for Mac, but windows device detection is not implemented. Code from BMP needs porting and testing. C++ Coding style and general coding style need rework. Speed is probably not good, as shiftIR|DR need up to 5 usb round trips. Please comment.

UweBonnes avatar Sep 04 '21 21:09 UweBonnes

Thanks for this PR. Will study this one and add feedback

trabucayre avatar Sep 07 '21 11:09 trabucayre

Sorry, fixes isnot enough after yesterdays changes. Stand By.

UweBonnes avatar Sep 07 '21 11:09 UweBonnes

Use https://github.com/UweBonnes/blackmagic/tree/fixes1 or better avr.

UweBonnes avatar Sep 07 '21 12:09 UweBonnes

BMP has code to work on windows. Somebody with windows needs to port and test.

UweBonnes avatar Sep 29 '21 09:09 UweBonnes

Maybe in a first time it make sense to add this driver only for linux devices, and add a note to request for someone with a windows machine to check/add/fix support.

trabucayre avatar Sep 29 '21 17:09 trabucayre

LGTM. The only thing not clear to me is about adding GPL code (bmd_remote.hpp) into a projet using Apache2 license?

trabucayre avatar Jul 24 '23 06:07 trabucayre