Skip to content

Commit

Permalink
Enable voltage and current sensors for AFCv4
Browse files Browse the repository at this point in the history
Previously, the voltage and current sensors were disabled because it
was causing the MCH to crash with a lot of 'AMC +12V Curr' assertion /
deassertion messages and IPMB errors. The root cause for this
behavior is not yet understood, but it is somewhat related to the
voltage sensors SDR struct not been populated with the thresholds
(they were all commented). I've updated all thresholds (voltages and
currents) to reasonable defaults, but they might be wrong, so a more
in depth study of the AFCv4 power rails should be conducted to obtain
the correct values.

It works for now, and the MCH doesn't crash, neither complain about
out of range voltages or currents.
  • Loading branch information
augustofg committed Jul 17, 2024
1 parent ba4cab6 commit fd76e22
Show file tree
Hide file tree
Showing 2 changed files with 134 additions and 133 deletions.
4 changes: 2 additions & 2 deletions port/board/afc-v4/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -19,8 +19,8 @@ set(TARGET_MODULES
"EEPROM_AT24MAC"
"LM75"
"MAX6642"
# "INA3221_VOLTAGE"
# "INA3221_CURRENT"
"INA3221_VOLTAGE"
"INA3221_CURRENT"
"UART_DEBUG"
"MCP23016"
"ADC"
Expand Down
Loading

0 comments on commit fd76e22

Please sign in to comment.