Northeastern-Electric-Racing / Northeastern-Electric-Racing/Embedded-Base
[Driver] - Make m24c32 driver platform agnostic
@Sabramz is already working on this.
Since Feb 3, 2025.
- Dominant language
- C
- Stars
- 2
- Forks
- 2
- PR merge metrics
- No merged PRs in 30d
Description
Current Features
this driver depends on stm32 HAL functions
Desired Additional Features
We want these to work with all potential hardware. This involves replacing HAL specific code with generalized code that utilized function pointers to use HAL functionality. The way in which we want to handle this is outlined in this doc under the "Driver Generalization" header.
Also a part of this ticket, find all references to this driver in Shepherd and Cerberus, and fix them to accommodate the new driver structure. A good way to check this is to update your embedded-base submodule to reflect the changes made to this driver, and then try building shepherd and cerberus. If they relied on this device, they will likely fail to build unless you make the appropriate changes
Screenshots (as needed)
No response
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.
Assessment
This issue has not been assessed yet.