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

Enhancements: Utilize "event" bits on i2c devices #143

Open
2 of 4 tasks
vthakur7f opened this issue Oct 4, 2018 · 0 comments
Open
2 of 4 tasks

Enhancements: Utilize "event" bits on i2c devices #143

vthakur7f opened this issue Oct 4, 2018 · 0 comments

Comments

@vthakur7f
Copy link
Collaborator

Product: (OC-SDR, OC-LTE, OC-LTE, OC-Power, others)

OC-SDR

Component: (PSU, GBC, BBU, FEM, LED, ME)

GBC/FE/SDR

Category: (Hardware, Software/stack, Firmware/bootloader, Documentation)

Firmware/OCWare
Describe the solution you'd like
These devices provide a register that we can read to check whether or not the device is asserting this line and what the reason is.

For example, the SE98ATP provides the ESTAT bit in the configuration register and some bits in the temperatureregister to figure out the cause.

Devices to change:

  • SE98ATP
  • ADT7481
  • INA226
  • LTC4274 (we check multiple registers when 0x00 would tell us everything we need)

Is your feature request related to a problem? Please describe.
The code currently infers whether or not our temperature/current sensors are asserting the alert/event pin by reading their current value and comparing to the configured value. While this technically works, it's hacky and definitely isn't how the devices are intended to be used.

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

No branches or pull requests

1 participant