Skip to content

Conversation

dzarista
Copy link
Contributor

Loads the adc rail driver for meru800bia scm and smb and meru800bfa scm.

Tested that platform manager successfully loads the driver for all applicable devices and that all voltage readings are correctly showing up in sensors.

@facebook-github-bot
Copy link
Contributor

@tao-ren has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@somasun
Copy link
Contributor

somasun commented Jun 11, 2025

Can you please add more to the motivation of this PR? Will we be using any of the sensors enabled by this driver in sensor_service?

@dzarista
Copy link
Contributor Author

Can you please add more to the motivation of this PR? Will we be using any of the sensors enabled by this driver in sensor_service?

These additional monitorings could be helpful in debugging future escalations especially on DPM-less systems. This won't be monitored by sensor_service, the info will be in sensors and showtech.

@@ -4734,7 +4741,7 @@
},
"chassisEepromDevicePath": "/SMB_SLOT@0/[IDPROM]",
"bspKmodsRpmName": "arista_bsp_kmods",
"bspKmodsRpmVersion": "0.7.9-1",
"bspKmodsRpmVersion": "0.7.10-1",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dzarista, we will need to hold the PR for now, because BSP version 0.7.10 is not ready in Meta environment. @Scott8440 is preparing the release internally, and I will drop you a note once it's ready. Thanks.

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

Successfully merging this pull request may close these issues.

4 participants