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

Update deck pin allocation table and compatibility matrix #165

Merged
merged 5 commits into from
Mar 1, 2024

Conversation

gemenerik
Copy link
Member

It's easiest to review individual commits, total diff shows a completely replaced compatibility matrix.

- Flow deck reset and motion on UART2
- Micro SD card deck alternative SPI connection on UART1+IO4
As indicated in the description above the deck pin allocation table, parentheses should indicate that pins are physically unconnected. However, UART1 and UART2 on the AI-deck are not physically unconnected. They may not be required for most funcitonalities (although UART2 has become important by defaultwith the introduction of CPX), but that should be indicated otherwise, if desired.
- With CPX, AI-deck is no longer by default compatible with bigquad and buzzer deck. Add note explaining.
- Improve incompatibility indication consistency (remove "no")
@gemenerik
Copy link
Member Author

gemenerik commented Feb 29, 2024

As can be found on the product page's development tab, BigQuad deck rev. C requires a workaround to be compatible with the Loco and MicroSD decks. For the current revision (C1) no such modification is required for compatibility. The current compatibility matrix shows these decks as compatible without note. A discussion should be had about whether or not to limit the compatibility matrix to the latest (n) revisions of the decks.

Either revision of the BigQuad deck can work with the flow deck, but it requires a pretty serious modification of the flow deck. Again, this mod can currently be found on the BigQuad deck product page. The current compatibility matrix does not include a note on this mod and marks the decks as incompatible. A discussion should be had about to what limit to suggest modifications.

Another point of discussion is whether or not it is logical to show these notes on compatibility on the BigQuad deck page.

In order to close this pull request I moved this discussion into a new issue #166

@gemenerik gemenerik marked this pull request as ready for review February 29, 2024 15:12
Copy link
Contributor

@knmcguire knmcguire left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good! Let's do it

@knmcguire knmcguire merged commit 26dfd97 into master Mar 1, 2024
1 check passed
@knmcguire knmcguire deleted the rik/compatibility branch March 1, 2024 08:28
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