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

[IO] - Implement PDU 24A #205

Open
Tracked by #201
jr1221 opened this issue Sep 25, 2024 · 1 comment · May be fixed by #221 or Northeastern-Electric-Racing/Embedded-Base#194
Open
Tracked by #201

[IO] - Implement PDU 24A #205

jr1221 opened this issue Sep 25, 2024 · 1 comment · May be fixed by #221 or Northeastern-Electric-Racing/Embedded-Base#194
Assignees

Comments

@jr1221
Copy link
Contributor

jr1221 commented Sep 25, 2024

Description

Implement the 24A PDU as it is a little different.

Blocked by Northeastern-Electric-Racing/Embedded-Base#151, Northeastern-Electric-Racing/Embedded-Base#152

Acceptance Criteria

  • 24A PDU can control IO and can read fuse and shutdown state
  • Reading of fuse and shutdown is 1 or 2 register calls each, NOT one per line
  • CAN YAML for new data is added

Proposed Solution

  • Update pca9539 --> tca9539
  • Add a current sensor read command x4 and send over CAN, corresponding YAML
  • Update pinout according to schematic.
  • Update fuse and shutdown extraction and repackaging as bools in YAML, use correct pins (note: low refresh rate)
  • Update PDU CTRL functions to correct pins.
  • Update PDU TSMS sense to correct pins
  • Update driver interaction layer as needed
@jr1221
Copy link
Contributor Author

jr1221 commented Nov 4, 2024

Here is the order of how I would tackle this:

  1. Add current sensing reading and configuring to pdu.c. Use altium to find the max current and shunt resistors for each chip, as well as the device address.
  2. Add to monitor.c the use of these functions in vNonFunctionalDataCollection
  3. Come up with the CAN format for sending this current data (I can discuss that with you)
  4. Add the sending of the current in this can format.

Once these are done I can review, then we can discuss other tasks related to this ticket.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment