Initial Configuration for Wedge800CACT Platform Manager by brandonchuang · Pull Request #648 · facebook/fboss

brandonchuang · 2025-11-11T03:23:22Z

Summary:

Add the initial platform_manager.json configuration for the Wedge800CACT platform manager.

Description:

Included all platform devices in the platform_manager.json configuration for Wedge800CACT. The configuration was validated on actual Wedge800CACT hardware.

Test Plan:

Executed platform_manager with the configuration on Wedge800CACT.
w800c_20251125_platform_manager.txt
Reviewed generated logs to validate correct functionality.
Loaded and unloaded platform_manager and related kernel modules 10 times on kernel 6.11.1 20251111_run_platform_manager_10_times.py_PASSED.txt
Executed platform_manager_hw_test; all items passed
w800c_20251125_platform_manager_hw_test.txt
Verified all sysfs attributes under /run/devmap/cplds/{MCB,SCM,SMB,FAN}_CPLD against i2cget bit values 20251111_run_devmap_cplds_sysfs_test.py_PASSED.txt
20251111_run_devmap_sensors_fan_cpld_test.py_PASSED.txt
Triggered the FAN watchdog and confirmed PWM behavior after timeout
20251111_run_devmap_watchdog_test.py_PASSED.txt
Accessed fan module LEDs via /sys/class/leds/fan1 and confirmed correctness via i2cget 20251111_led_fan_module_led_test.py_PASSED.txt
Accessed front panel LEDs via sysfs and confirmed correctness via i2cget
20251111_led_sys_led_test.py_PASSED.txt
Read/write front port LEDs via sysfs and verified expected output. Compared front port LED values with raw devmem reads. Sequentially programmed port LEDs and verified ordering and color correctness.
20251111_port_led_test.py_PASSED.txt
Verified that all sensors are detected
20251111_run_devmap_sensors_test_strict_version.py_PASSED.txt
Read xcvr_ctl via sysfs and compared against raw devmem values 20251111_run_devmap_xcvr_ctrl_sysfs_get_test.py_PASSED.txt
Triggered the xcvr_reset attribute and verified EEPROM presence/absence behavior. 20251111_run_devmap_xcvr_ctrl_reset_test.py_PASSED.txt

somasun · 2025-11-20T19:55:14Z

+        "address": "0x54",
+        "kernelDeviceName": "24c02"
+      },
+      "pmUnitName": "MCB"


Please remove the "pmUnitName" names from all slotTypeConfigs. Since the EEPROM's ProductName field is programmed correctly, this should not be required.

Please remove the "pmUnitName" names from all slotTypeConfigs. Since the EEPROM's ProductName field is programmed correctly, this should not be required.

@somasun
Could we keep that part? We need pmUnitName when the EEPROM is present but still empty (before the ProductName is programmed). Without pmUnitName, the system cannot bring up the associated CPLD device, and therefore cannot disable the CPLD write-protect. This makes the initial EEPROM provisioning process very difficult.

somasun · 2025-11-20T19:55:21Z

+        "address": "0x54",
+        "kernelDeviceName": "24c64"
+      },
+      "pmUnitName": "SCM"


somasun · 2025-11-20T19:55:30Z

+        "address": "0x51",
+        "kernelDeviceName": "24c64"
+      },
+      "pmUnitName": "BMC"


somasun · 2025-11-20T19:55:36Z

+        "address": "0x50",
+        "kernelDeviceName": "24c64"
+      },
+      "pmUnitName": "SMB"


somasun · 2025-11-20T19:55:44Z

+    },
+    "FAN_SLOT": {
+      "numOutgoingI2cBuses": 0,
+      "pmUnitName": "FAN"


somasun · 2025-11-20T19:57:50Z

+          "address": "0x37",
+          "kernelDeviceName": "adc128d818",
+          "pmUnitScopedName": "MCB_VOLTAGE_MONITOR5",
+          "initRegSettings": [


Please add a __comment__ on why this i2cset is needed for this device.

MCB_VOLTAGE_MONITOR5 only uses IN0–IN5, so Mode 1 is not needed. I’ve removed the initRegSettings for this device. Please help review, thanks.

somasun · 2025-11-20T19:58:31Z

+          "address": "0x35",
+          "kernelDeviceName": "w800_scmcpld",
+          "pmUnitScopedName": "SCM_CPLD",
+          "initRegSettings": [


Please add a __comment__ on why this i2cset is needed for this device.

I’ve added the requested comment. Please help review, thanks.

somasun · 2025-11-20T19:58:39Z

+          "address": "0x37",
+          "kernelDeviceName": "adc128d818",
+          "pmUnitScopedName": "SCM_VOLTAGE_MONITOR1",
+          "initRegSettings": [


The requested comment has been added here as well.

somasun · 2025-11-20T19:58:48Z

+          "address": "0x1f",
+          "kernelDeviceName": "adc128d818",
+          "pmUnitScopedName": "SMB_VOLTAGE_MONITOR1",
+          "initRegSettings": [


The requested comment has been added here as well.

somasun · 2025-11-20T19:58:55Z

+          "address": "0x35",
+          "kernelDeviceName": "adc128d818",
+          "pmUnitScopedName": "SMB_VOLTAGE_MONITOR2",
+          "initRegSettings": [


The requested comment has been added here as well.

meta-codesync · 2025-11-20T19:59:53Z

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

somasun · 2025-11-20T21:43:05Z

Can you also please fix wedge800bact config with the above changes.

brandonchuang · 2025-11-24T07:21:11Z

Can you also please fix wedge800bact config with the above changes.

Yes, I will update the wedge800bact configuration accordingly and submit a PR with the changes.

facebook-github-bot · 2025-11-25T08:08:20Z

@brandonchuang has updated the pull request. You must reimport the pull request before landing.

tao-ren · 2025-12-22T19:36:57Z

+  "chassisEepromDevicePath": "/[CHASSIS_EEPROM]",
+  "numXcvrs": 33,
+  "bspKmodsRpmName": "fboss_bsp_kmods",
+  "bspKmodsRpmVersion": "3.4.0-1",


@brandonchuang , let's bump BSP version to 4.0.0-1, like what we did for wedge800b. Thanks.

@brandonchuang , let's bump BSP version to 4.0.0-1, like what we did for wedge800b. Thanks.

@tao-ren @RomanChoporov-Meta
Done. The BSP version has been bumped to 4.0.0-1.

ghost · 2025-12-22T21:25:45Z

+  "chassisEepromDevicePath": "/[CHASSIS_EEPROM]",
+  "numXcvrs": 33,
+  "bspKmodsRpmName": "fboss_bsp_kmods",
+  "bspKmodsRpmVersion": "3.4.0-1",


ghost · 2025-12-22T22:19:26Z

Waiting for a decision regarding pmUnitName in #696 (comment)

facebook-github-bot · 2025-12-23T03:19:02Z

@brandonchuang has updated the pull request. You must reimport the pull request before landing.

brandonchuang · 2025-12-23T03:28:32Z

Waiting for a decision regarding pmUnitName in #696 (comment)

@RomanChoporov-Meta
Please help check the comment in #696.
I will update the PR based on the conclusion. Thanks.

Summary: Add the initial platform_manager.json configuration for the Wedge800CACT platform manager. Description: Included all platform devices in the platform_manager.json configuration for Wedge800CACT. The configuration was validated on actual Wedge800CACT hardware. Test Plan: - Executed platform_manager with the configuration on Wedge800CACT. [w800c_20251125_platform_manager.txt](https://github.com/user-attachments/files/23741725/w800c_20251125_platform_manager.txt) - Reviewed generated logs to validate correct functionality. - Loaded and unloaded platform_manager and related kernel modules 10 times on kernel 6.11.1 [20251111_run_platform_manager_10_times.py_PASSED.txt](https://github.com/user-attachments/files/23468562/20251111_run_platform_manager_10_times.py_PASSED.txt) - Executed platform_manager_hw_test; all items passed [w800c_20251125_platform_manager_hw_test.txt](https://github.com/user-attachments/files/23741726/w800c_20251125_platform_manager_hw_test.txt) - Verified all sysfs attributes under /run/devmap/cplds/{MCB,SCM,SMB,FAN}_CPLD against i2cget bit values [20251111_run_devmap_cplds_sysfs_test.py_PASSED.txt](https://github.com/user-attachments/files/23468556/20251111_run_devmap_cplds_sysfs_test.py_PASSED.txt) [20251111_run_devmap_sensors_fan_cpld_test.py_PASSED.txt](https://github.com/user-attachments/files/23468557/20251111_run_devmap_sensors_fan_cpld_test.py_PASSED.txt) - Triggered the FAN watchdog and confirmed PWM behavior after timeout [20251111_run_devmap_watchdog_test.py_PASSED.txt](https://github.com/user-attachments/files/23468559/20251111_run_devmap_watchdog_test.py_PASSED.txt) - Accessed fan module LEDs via /sys/class/leds/fan1 and confirmed correctness via i2cget [20251111_led_fan_module_led_test.py_PASSED.txt](https://github.com/user-attachments/files/23468550/20251111_led_fan_module_led_test.py_PASSED.txt) - Accessed front panel LEDs via sysfs and confirmed correctness via i2cget [20251111_led_sys_led_test.py_PASSED.txt](https://github.com/user-attachments/files/23468552/20251111_led_sys_led_test.py_PASSED.txt) - Read/write front port LEDs via sysfs and verified expected output. Compared front port LED values with raw devmem reads. Sequentially programmed port LEDs and verified ordering and color correctness. [20251111_port_led_test.py_PASSED.txt](https://github.com/user-attachments/files/23468555/20251111_port_led_test.py_PASSED.txt) - Verified that all sensors are detected [20251111_run_devmap_sensors_test_strict_version.py_PASSED.txt](https://github.com/user-attachments/files/23468558/20251111_run_devmap_sensors_test_strict_version.py_PASSED.txt) - Read xcvr_ctl via sysfs and compared against raw devmem values [20251111_run_devmap_xcvr_ctrl_sysfs_get_test.py_PASSED.txt](https://github.com/user-attachments/files/23468561/20251111_run_devmap_xcvr_ctrl_sysfs_get_test.py_PASSED.txt) - Triggered the xcvr_reset attribute and verified EEPROM presence/absence behavior. [20251111_run_devmap_xcvr_ctrl_reset_test.py_PASSED.txt](https://github.com/user-attachments/files/23468560/20251111_run_devmap_xcvr_ctrl_reset_test.py_PASSED.txt)

facebook-github-bot · 2025-12-23T08:43:21Z

@brandonchuang has updated the pull request. You must reimport the pull request before landing.

meta-codesync · 2026-01-20T17:16:22Z

@RomanChoporov-Meta merged this pull request in c6be5b0.

meta-cla Bot added the CLA Signed label Nov 11, 2025

somasun requested changes Nov 20, 2025

View reviewed changes

brandonchuang force-pushed the wedge800c_pmgr branch from 843db50 to d17ff20 Compare November 25, 2025 08:08

tao-ren reviewed Dec 22, 2025

View reviewed changes

ghost suggested changes Dec 22, 2025

View reviewed changes

brandonchuang force-pushed the wedge800c_pmgr branch from d17ff20 to 031cdd0 Compare December 23, 2025 03:18

brandonchuang force-pushed the wedge800c_pmgr branch from 031cdd0 to 2bcb903 Compare December 23, 2025 08:43

meta-codesync Bot closed this in c6be5b0 Jan 20, 2026

facebook-github-bot added the Merged label Jan 20, 2026

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Initial Configuration for Wedge800CACT Platform Manager#648

Initial Configuration for Wedge800CACT Platform Manager#648
brandonchuang wants to merge 1 commit into
facebook:mainfrom
brandonchuang:wedge800c_pmgr

brandonchuang commented Nov 11, 2025 •

edited

Loading

somasun Nov 20, 2025

brandonchuang Nov 24, 2025

somasun Nov 20, 2025

somasun Nov 20, 2025

somasun Nov 20, 2025

somasun Nov 20, 2025

somasun Nov 20, 2025

brandonchuang Nov 25, 2025

somasun Nov 20, 2025

brandonchuang Nov 25, 2025

somasun Nov 20, 2025

brandonchuang Nov 25, 2025

somasun Nov 20, 2025

brandonchuang Nov 25, 2025

somasun Nov 20, 2025

brandonchuang Nov 25, 2025

meta-codesync Bot commented Nov 20, 2025

somasun commented Nov 20, 2025

brandonchuang commented Nov 24, 2025

facebook-github-bot commented Nov 25, 2025

tao-ren Dec 22, 2025

ghost Dec 22, 2025

brandonchuang Dec 23, 2025

ghost Dec 22, 2025

ghost commented Dec 22, 2025

facebook-github-bot commented Dec 23, 2025

brandonchuang commented Dec 23, 2025

facebook-github-bot commented Dec 23, 2025

meta-codesync Bot commented Jan 20, 2026

Labels

4 participants

Uh oh!

Conversation

brandonchuang commented Nov 11, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary:

Description:

Test Plan:

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

meta-codesync Bot commented Nov 20, 2025

somasun commented Nov 20, 2025

brandonchuang commented Nov 24, 2025

facebook-github-bot commented Nov 25, 2025

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

ghost commented Dec 22, 2025

facebook-github-bot commented Dec 23, 2025

brandonchuang commented Dec 23, 2025

facebook-github-bot commented Dec 23, 2025

meta-codesync Bot commented Jan 20, 2026

Labels

4 participants

brandonchuang commented Nov 11, 2025 •

edited

Loading