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

Changes from bringup with daisy-chain bus #3

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

nickyivyca
Copy link

To make this build in the first place for daisy-chain, changed how the templating is implemented to make the code buildable.
See here for my main reference: https://stackoverflow.com/questions/495021/why-can-templates-only-be-implemented-in-the-header-file

Added Mute/Unmute discharge commands.

Switched to non-blocking wait for conversion, since conversion times are always greater than 1ms. I only changed this for the daisy chain files, but this may be desirable for addressed.

Wake up for daisy goes for an extra cycle, if I remember correctly this was necessary to wake up the whole bus.

Fixed the hex value for StartCombinedCellVoltageGpioConversion.

Reversed the data order for writing to the bus, since when writing, the first data that gets sent goes to the final chip, as opposed to reading, where the first chip's data comes out first.

@nickyivyca nickyivyca requested a review from colescott May 30, 2022 04:09
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.

1 participant