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

Support open_path() and update EEPROM for Python 3.0 #17

Open
wants to merge 13 commits into
base: master
Choose a base branch
from

Conversation

rdpoor
Copy link

@rdpoor rdpoor commented Nov 12, 2018

Building upon Lukas K's fork at https://github.com/carrotIndustries:

Larger changes:

  • The MCP2210() constructor accepts keyword arguments vid, pid and path. With no args given, vid and pid default to MCP2210_VID = 0x04d8 and MCP2210_PID = 0x00de, i.e. the values for the MCP2210. If you have multiple MCP2210 devices on your USB chain, the path keyword lets you name a specific MCP2210 on the chain.
  • dev.eeprom now works under Python3, reading and writing unsigned 8-bit ints (and arrays of ints). Have not tested for compatibility under Python2.x (but that's becoming increasingly irrelevant).

Smaller changes:

  • Added a unittest test file to cover the above changes.
  • Touched up ReadMe.md to reflect the changes above and updated print() syntax for Python3.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants