Skip to content

[Nexthop][m4340nhp] Wire qsfp_service, led_service and wedge_qsfp_util - #1875

Open
marif-nexthop wants to merge 6 commits into
meta:mainfrom
nexthop-ai:marif.fboss-m4340nhp-qsfp-led-wiring
Open

marif-nexthop wants to merge 6 commits into
meta:mainfrom
nexthop-ai:marif.fboss-m4340nhp-qsfp-led-wiring

Conversation

@marif-nexthop

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

Summary

PR commit stack:

  1. [Nexthop][m4340nhp] Add the platform_manager config #1871
  2. This (top of stack)

Add the m4340nhp BSP platform mapping, which buildFromXcvrLib builds from the platform's platform_manager.json, and use it in qsfp_service, led_service and wedge_qsfp_util, as for m4061nhp. BspPlatformMapTest gets an m4340nhp case.

Test Plan

On a m4340nhp device, qsfp_service and led_service start and fboss2 show transceiver lists all 66 transceivers. The changed targets build in the OSS build.

@marif-nexthop
marif-nexthop requested review from a team as code owners October 30, 2026 23:27
@meta-cla meta-cla Bot added the CLA Signed label Oct 30, 2026
@github-actions github-actions Bot added nexthop project:m4340nhp FBOSS project: m4340nhp labels Oct 30, 2026
@marif-nexthop
marif-nexthop force-pushed the marif.fboss-m4340nhp-qsfp-led-wiring branch from ddb04e0 to a99ffe5 Compare October 31, 2026 02:16

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant