completed mcp23017. Will push and merge. Once merged another PR to …#146
Merged
Conversation
…ndense common code and delete utilities no longer needed
taartspi
commented
Jun 27, 2026
| // create an I2C to the <MCP23008. | ||
| I2C mcpDev = createI2cDevice(pi4j, busNum, address); | ||
| // Create the Mcp23008Driver passing the MCP23008 I2C device | ||
| Mcp23017Driver mcpDriver = new Mcp23017Driver(mcpDev); |
Collaborator
Author
There was a problem hiding this comment.
jumped the gun. needs the mcp23017Driver update in the driver snapshot
Updated Mcp23017Driver instantiation to include a second parameter.
Collaborator
Author
|
will merge using the older mcp23017driver ctor |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
…condense common code and delete utilities no longer needed