Skip to content

[Celestica] Leh800bcls: Config: Update sensors according to the latest HW spec - #1531

Open
chee-zhang wants to merge 2 commits into
facebook:mainfrom
chee-zhang:leh800bcls_update_sensors
Open

[Celestica] Leh800bcls: Config: Update sensors according to the latest HW spec#1531
chee-zhang wants to merge 2 commits into
facebook:mainfrom
chee-zhang:leh800bcls_update_sensors

Conversation

@chee-zhang

@chee-zhang chee-zhang commented Aug 19, 2026

Copy link
Copy Markdown
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
  clang-format.........................................(no files to check)Skipped
  shellcheck...........................................(no files to check)Skipped
  shfmt................................................(no files to check)Skipped
  trim trailing whitespace.................................................Passed
  fix end of files.........................................................Passed
  check yaml...........................................(no files to check)Skipped
  check json...............................................................Passed
  check for merge conflicts................................................Passed
  ruff check...........................................(no files to check)Skipped
  ruff format..........................................(no files to check)Skipped
  Prevent sai_impl in fboss manifest.......................................Passed

Summary

This PR is to update sensors according to the latest HW spec.

  1. Update INLET and OUTLET sensors of SMB
  2. Update MCB_UP15_XP12R0V_COME_IIN thresholds for Netlake2.0
  3. Update compute coefficient of HSCB_UP1_XP48R0V_HOTSWAP_IOUT according to the HW design

Test Plan

  • SMB
| SMB_L_U17_LM75B_INLET3                  | 32.75 | PASS   | [-25.00, 90.00]  | [-20.00, 85.00]  | 2s ago      | /run/devmap/sensors/SMB_L_INLET_TSENSOR3/temp1_input      |
| SMB_L_U7_LM75B_OUTLET2                  | 38.50 | PASS   | [-25.00, 90.00]  | [-20.00, 85.00]  | 2s ago      | /run/devmap/sensors/SMB_L_OUTLET_TSENSOR2/temp1_input     |
| SMB_L_U8_LM75B_INLET1                   | 35.12 | PASS   | [-25.00, 90.00]  | [-20.00, 85.00]  | 2s ago      | /run/devmap/sensors/SMB_L_INLET_TSENSOR1/temp1_input      |
| SMB_R_U17_LM75B_INLET3                  | 33.88 | PASS   | [-25.00, 90.00]  | [-20.00, 85.00]  | 2s ago      | /run/devmap/sensors/SMB_R_INLET_TSENSOR3/temp1_input      |
| SMB_R_U7_LM75B_OUTLET2                  | 37.00 | PASS   | [-25.00, 95.00]  | [-20.00, 90.00]  | 2s ago      | /run/devmap/sensors/SMB_R_OUTLET_TSENSOR2/temp1_input     |
| SMB_R_U8_LM75B_INLET1                   | 29.50 | PASS   | [-25.00, 95.00]  | [-20.00, 90.00]  | 2s ago      | /run/devmap/sensors/SMB_R_INLET_TSENSOR1/temp1_input      |
  • MCB_UP15_XP12R0V_COME_IIN

Netlake1.0

| MCB_UP15_XP12R0V_COME_IIN     | 2.56    | PASS   | [, 15.00]        | [, 10.00]        | 4s ago      | /run/devmap/sensors/LOAD_SWITCH_MONITOR/curr1_input       |

Netlake2.0

| MCB_UP15_XP12R0V_COME_IIN     | 1.41   | PASS   | [, 20.00]        | [, 16.00]        | 3s ago      | /run/devmap/sensors/LOAD_SWITCH_MONITOR/curr1_input       |
  • HSCB_UP1_XP48R0V_HOTSWAP_IOUT

before

V0824 06:02:18.575778 226048 SensorServiceImpl.cpp:360] HSCB_UP1_XP48R0V_HOTSWAP_IOUT (/run/devmap/sensors/48V_HSC_MONITOR/curr1_input) : 26.8584

after

V0824 10:28:39.470780 234082 SensorServiceImpl.cpp:360] HSCB_UP1_XP48R0V_HOTSWAP_IOUT (/run/devmap/sensors/48V_HSC_MONITOR/curr1_input) : 33.591

@meta-cla meta-cla Bot added the CLA Signed label Aug 19, 2026
@chee-zhang
chee-zhang marked this pull request as ready for review August 19, 2026 05:07
@chee-zhang
chee-zhang requested a review from a team as a code owner August 19, 2026 05:07
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.

1 participant