Northeastern-Electric-Racing / Northeastern-Electric-Racing/Odysseus
[Cleanup] - Take in interrupt line as a parameter in the overlay
Nobody has claimed this yet.
- Dominant language
- Shell
- Stars
- 5
- Forks
- 1
- Avg merge
- 3d 17h
- Merged PRs (30d)
- 1
Description
Description
Right now, the newracom overlay file in the board/tpu directory has a hardcoded interrupt.
Acceptance Criteria
- can change interrupt without recompiling overlay
Proposed Solution
It should instead be a parameter like mcp2515-can0-spi4 so the config.txt can dynamically change the interupt pin. The default on config.txt should be that of TPU v1, but the other interrupt for stock ALFA hats should be noted in a comment.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Read the overlay files in board/tpu and the related config.txt entries to find where the interrupt is hardcoded and how parameters such as mcp2515-can0-spi4 are represented. Done means the interrupt can be changed through config.txt without recompiling the overlay, with the TPU v1 default retained and the stock ALFA interrupt noted in a comment.
Written by the indexing model from the issue text.
Assessment
- Domain
- embedded-iot
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100