Skip to content

Conversation

@JunjunGu2
Copy link
Contributor

Pre-submission checklist

  • I've ran the linters locally and fixed lint errors related to the files I modified in this PR. You can install the linters by running pip install -r requirements-dev.txt && pre-commit install
  • pre-commit run
    pre_commit_run

Summary

This PR updates the sensor configurations for SMB_3V3_L_VRM_VOUT and SMB_3V3_R_VRM_VOUT per hardware team requirements.
The update accounts for a hardware change between the main and second source components: the second source requires a multiplication factor of 3 to report correct voltage values.

Test Plan

Run platform_manager, sensor_service and sensor_service_client to verify the sensor config for SMB_3V3_L_VRM_VOUT and SMB_3V3_R_VRM_VOUT.
main_source:
sensor_service_client-3 0 20
pm-3.0.20.txt
sensors-3.0.20.txt
second source:
sensor_service_client-2 3 10
pm-2.3.10.txt
sensors-2.3.10.txt

…3V3_R_VRM_VOUT

Signed-off-by: Junjun Gu <junjun.gu@celestica.com>
@meta-cla meta-cla bot added the CLA Signed label Dec 19, 2025
@JunjunGu2 JunjunGu2 marked this pull request as ready for review December 19, 2025 05:54
@meta-codesync
Copy link

meta-codesync bot commented Dec 23, 2025

@mikechoifb has imported this pull request. If you are a Meta employee, you can view this in D89735307.

@meta-codesync meta-codesync bot closed this in e94ec9d Jan 8, 2026
@meta-codesync
Copy link

meta-codesync bot commented Jan 8, 2026

@mikechoifb merged this pull request in e94ec9d.

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.

2 participants