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

MCP23017 some features not working since 2018 #810

Closed
gen2thomas opened this issue Mar 15, 2022 · 0 comments
Closed

MCP23017 some features not working since 2018 #810

gen2thomas opened this issue Mar 15, 2022 · 0 comments

Comments

@gen2thomas
Copy link
Collaborator

I'm trying to implement a driver for Adafruit RGB Positive 16x2 LCD+Keypad Kit for Raspberry Pi.
Therefore I started writing a wrapper around MCP23017 (port expander) and HD44780 (LCD driver).

Unfortunately the MCP driver lost some features with this commit:

  • SetPullUp() not working anymore
  • SetGPIOPolarity() not working anymore

Although the originated problem was caused only by a small bug in the read function, there was a huge change done in the write function, which was responsible to set or clear bit in the read register and write the new value, which was perfect before. This old behavior can be seen by the incomplete cleanup in the parameters of write() - the parameter "pin" is not used anymore.

Also the subsequent commit has only fixed new problems, introduced by the first commit.

I will start with re-insert the old lines at write function and try to improve the unit tests to reflect the problem.

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

No branches or pull requests

1 participant