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

MAX5970 alerts are ignored #1546

Open
wesolows opened this issue Oct 4, 2023 · 0 comments
Open

MAX5970 alerts are ignored #1546

wesolows opened this issue Oct 4, 2023 · 0 comments
Labels
gimlet hardware Related to hardware components. service processor Related to the service processor.

Comments

@wesolows
Copy link
Contributor

wesolows commented Oct 4, 2023

The MAX5970 hotplug controllers in Gimlet have their ALERT_L and I2C lines routed to the SP, but the SP doesn't do anything with the ALERT_L signal nor does it currently act on any of the status registers other than Register::adc_chx_{cs,mon}_XX which implement the ability to obtain the present readings of voltage and current sensors. Information that may be useful to acquire on an alert interrupt includes the contents of the status0, status3 and fault[2:0] registers and the historical peak detectors {max,min}_chx_{cs,mon}_{lsb,msb}, in addition to the circular buffer contents covered by #1545. This could modeled as additional sensors, gathered continuously, and made available upstack in the same manner as the near-real-time samples, but the intent of this ticket is to cover one-shot collection of this data in response to and shortly after an alert.

@wesolows wesolows added hardware Related to hardware components. service processor Related to the service processor. gimlet labels Oct 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
gimlet hardware Related to hardware components. service processor Related to the service processor.
Projects
None yet
Development

No branches or pull requests

1 participant