Skip to content
This repository has been archived by the owner on May 27, 2020. It is now read-only.

Merge technobit (nanofury) patch to master branch #541

Closed
wants to merge 8 commits into from
Closed

Merge technobit (nanofury) patch to master branch #541

wants to merge 8 commits into from

Conversation

utdrmac
Copy link

@utdrmac utdrmac commented Feb 1, 2014

The technobit patch allows for NanoFury and YellowJacket USB miners to work with RaspberryPi without the need for hidapi. The patch, as downloaded from technobit, can only be applied successfully to cgminer 3.6.5. This pull request applies the same patch to the 3.12 main line. (Tips: 1drmacW6UYAfHNjRrxN7SYuN3Q9R9v6K6)

@ckolivas
Copy link
Owner

ckolivas commented Feb 1, 2014

There are numerous code changes in this push which are in core cgminer code for displaying, usb hotplugging and the API which are invasive and needlessly change the way the core code, display and api work. There's also a great big debug output as a comment in the patch explaining why hotplug should be disabled for the driver instead of fixing whatever problem the patch has with hotplug. There is also an official U1 driver, even if it is very simple by comparison that should be built on - without affecting other devices that use the icarus driver - instead of adding another driver for it.

@utdrmac
Copy link
Author

utdrmac commented Feb 2, 2014

I'm not the original author of the patch. All I did was make it work with latest cgminer. All I can say is that with the patch on a RaspberryPi, I have 0 issues making nanofury/yellowjacket work. Without the patch, NF/YJ are either A) not recognized or B) underperform. I can't speak for the code; I can only speak for the results and the results are better than NF/YJ running with bfgminer on MacBookPro

@ckolivas
Copy link
Owner

ckolivas commented Feb 2, 2014

I know you're not the author, I'm explaining why it's not suitable as is. I'd be happy for someone to modify it to fix the above problems and then I'd merge the code.

@ckolivas ckolivas closed this Feb 20, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants