Skip to content

curate: add NXP MCU-Link CMSIS-DAP PID 1fc9:0143 (fbuild#722)#44

Merged
zackees merged 2 commits into
otherfrom
chore/add-mcu-link-pid
Jul 9, 2026
Merged

curate: add NXP MCU-Link CMSIS-DAP PID 1fc9:0143 (fbuild#722)#44
zackees merged 2 commits into
otherfrom
chore/add-mcu-link-pid

Conversation

@zackees

@zackees zackees commented Jul 9, 2026

Copy link
Copy Markdown
Member

Adds the NXP MCU-Link debug-probe VID:PID (1fc9:0143) to the other curation layer — companion to the existing LPC-Link2 (1fc9:0132). fbuild deploys through MCU-Link via CMSIS-DAP; this lets fbuild port scan resolve it offline from the embedded archive instead of a hardcoded table (FastLED/fbuild#722 / #959).

🤖 Generated with Claude Code

Companion to the existing LPC-Link2 (1fc9:0132) entry. Onboard debug probe
on LPCXpresso / MCX / i.MX RT eval boards; fbuild deploys through it via
CMSIS-DAP. Sourced from NXP MCU-Link firmware VID:PID allocation.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Jul 9, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 8d487427-606c-46d7-bb91-b79a708ba493

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/add-mcu-link-pid

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

…d#722)

The board-attribution pipeline mislabels shared USB-UART bridge chips with
whatever board first claimed them (10c4->Arduino, 0403->Digilent, 1a86->OLIMEX).
Add unconditional vid_overrides pinning the real chip vendor:
  10c4 Silicon Labs (CP210x), 0403 FTDI (FT232), 1a86 QinHeng/WCH (CH340),
  067b Prolific (PL2303).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@zackees
zackees merged commit e09fcaf into other Jul 9, 2026
1 check passed
zackees added a commit to FastLED/fbuild that referenced this pull request Jul 9, 2026
…table (#722) (#1013)

The per-vendor VID:PID ingest sub-issues (#723#740) are all closed, so the
FastLED/boards pipeline now carries far richer per-board data than fbuild's
stale 745-byte embedded proto. Regenerate `usb-vids.proto.zstd` from the
current boards data (745 B → 10 KB, 36 VIDs / 1369 PID rows) so offline
resolution names real boards/probes instead of `Device 0xPPPP` placeholders:

  303A:1001 → WEMOS LOLIN S3     1FC9:0132 → NXP LPC-Link2 (CMSIS-DAP)
  1FC9:0143 → NXP MCU-Link       16C0:0483 → Teensy 4.0

Curated on the boards `other` branch (FastLED/boards#44): added the MCU-Link
PID (1fc9:0143) and authoritative vendor overrides for shared UART-bridge VIDs
that the board-attribution pipeline otherwise mislabels (10c4 Silicon Labs,
0403 FTDI, 1a86 QinHeng/WCH, 067b Prolific — previously Arduino/Digilent/OLIMEX).

Delete the hardcoded `FRIENDLY_PRODUCTS` supplement table from `port_scan`:
friendly product names are now owned by the boards VID:PID data and embedded at
build time. A missing name is a boards-data gap, not an fbuild source edit.

Verified offline on real hardware: `port scan --offline` shows
`Silicon Labs / …`, `NXP LPC-Link2 …`, `PJRC (Teensy) / Teensy 4.0`,
`Espressif Systems / WEMOS LOLIN S3` with no network and no hardcoded table.

Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant