Add Daisy Chain Protocol Documentation and instructions on how to build and flash#154
Merged
Conversation
Contributor
Author
|
@elsevers, there is documentation on the Daisy Chain Protocol along with programming the AMDS. can you review this? |
elsevers
requested changes
May 13, 2026
Contributor
There was a problem hiding this comment.
To do:
- Update benchmark data for single AMDC-AMDS
- Tracked here: #165
- On daisy chain page:
- add daisy chain benchmark data (3x AMDS's with 8 cards each, 3x 2S boards with 2 cards each)
- This comment provides links to where to find timing data and Saleae captures: Severson-Group/AMDS#102 (comment)
- Tracked here: #167
- remove discussion that reads like a diff
- Explain a bit more on how it works (yes, discussion of DMA is good)
- Cabling! Pinouts to wire it all up. What pins go to what pins.
- Tracked here: #166
- Explain how the header packets from each AMDS are adjusted as they progress through the chain and then what registers in the AMDC Firmware (
AMDS_CH_1_DATA_REG_OFFSET-AMDS_CH_24_DATA_REG_OFFSET) they ultimately appear in.- Tracked here: #158
- add daisy chain benchmark data (3x AMDS's with 8 cards each, 3x 2S boards with 2 cards each)
| ``` | ||
|
|
||
| ```{warning} | ||
| Ensure that you |
This was referenced May 15, 2026
Restructure the AMDS docs to better incorporate daisy chain and firmware build info
This was referenced May 17, 2026
- further information on theory of operation - architecture details - move comments that apply to non-daisy-chain AMDC mode to firmware article
Removed warning block from profiling tasks tutorial.
elsevers
approved these changes
May 17, 2026
Contributor
elsevers
left a comment
There was a problem hiding this comment.
I have gone through this thoroughly, completing the minimum required to release this and splitting out other items into new issue. Let's merge this now into our next-release staging area.
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
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.
Closes #153 and #163
Notes
Anything reviewers should be aware of when reviewing? Other related issues? Known problems? Future work?
Self-Review
Approachsection of the issue(s) it is closing? YesReviewer Instructions
Reviewers, please copy and paste a suitable review checklist into your review and answer all questions.
Appendix
This section should be the same for all PRs. Do not edit this section when creating a PR.
Review Checklists
Checklists maintained by the eLev lab for research repositories include:
Standard checklist