Removed TLC and NodeID #638
Merged
Conversation
This file contains 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
balazsracz
added a commit
that referenced
this pull request
Aug 9, 2022
* 'lull-bigio' of github.com:bakerstu/openmrn: Removed TLC and NodeID fro mlull-bigio branch (#638)
balazsracz
added a commit
that referenced
this pull request
Aug 11, 2022
…s for a total of 128 lines. (#641) * Fixes for turnout control and PortD LED/SNAP conflict * Adding main.cxx. Apparently git doesn't follow sym links? Huh. * Updates to changes: Moved notes around use of snap switch define to config.hxx from main.cxx Removed unnecessary inline comments in main.cxx Changed from using if to ifdef Updated how port D consumers are instantiated depending on port D use (snap/not snap) * Additional fixes around snap switches. Tested and verified on F303RE boards. * updated config.hxx for Nucleo F091 board to reflect Port D and E changes for Snap Switch or LED options to make existing changes for F303 board. * pushing changes up for remote development. * Migration to use NUM_MCPIOs from NUM_EXTBOARDS to better reflect the hardware options. Setup code to support 2 to 6 MCPs. * Setup for 8 MCPs completed. Tested build for 4, 6 and 8 MCPs successfully. Live testing on Nodes is next stage. Potentially we may have to break out the CDI configuration to make it easier to use. * Additional cleanup and code validation for max MCPs. * additional changes. still troubleshooting crash on startup with 6 or 8 MCPs. * Fixed array sizing. Working on breaking external boards CDI into two groups. * vApplication stack overflow crash. For Balazs to exaimine. * Finalized and tested MCP configuration on F303. Support for up to 8 MCP chips on DevKit exposed I2C bus. * Fixed 1 typo from previous commit. * Removed TLC and NodeID so pull request will be clean * Removed TLC and NodeID fro mlull-bigio branch (#638) Co-authored-by: Rick Lull <[email protected]> * restore node IDs files from master. * fix node id again * Fixes bug that prevents NUM_MCPIOS==0 from working. * Reduces maximum memory size to ensure we have enough spare to not wear out flash. * Removes duplication of declarations over the different MCP counts. * Simplifies the definition of multiplied mcpios. * reverts an unnecessary change * fix some more code duplication * fix whitespace * makes changing num mcpios clear the eeprom Co-authored-by: Rick Lull <[email protected]> Co-authored-by: Rick Lull <[email protected]> Co-authored-by: Balazs Racz <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
BigIO ready.