apps/netcfg: show Broadcom for vendor id 14E4 #708

Merged
Burer merged 1 commits from igorsh/kolibrios:bcm57xx into bcm57xx 2026-09-20 09:45:11 +00:00
Contributor
No description provided.
igorsh added 1 commit 2026-09-19 19:57:42 +00:00
The Company column is looked up in VendorsTab by PCI vendor id with a
linear scan, and the scan's terminator carries the fallback string, so an
id that is missing from the table prints the terminator's 'Unknown'. The
table had 0x166D (a different Broadcom id) but not 0x14E4, which is what
the NetXtreme 57xx parts report, so netcfg showed no vendor for the card
the bcm57xx driver drives.

Add 0x14E4 in the ascending run, with the same wording the 0x166D entry
and programs/testing/pcidev/vendors.inc already use.

Verified by assembling with fasm and walking VendorsTab in the resulting
binary: 31 entries, 0x14E4 resolves to 'Broadcom Corp.', 0x166D and
0x10B7 are unchanged, and ids still absent from the table fall back to
'Unknown' as before.

Assisted-by: ZCode:deepseek-flash
CODEOWNERS rules requested review from network 2026-09-19 19:57:42 +00:00
igorsh closed this pull request 2026-09-19 19:58:45 +00:00
igorsh reopened this pull request 2026-09-19 20:08:44 +00:00
Leency approved these changes 2026-09-19 23:46:01 +00:00
Doczom requested review from hidnplayr 2026-09-20 04:37:39 +00:00
hidnplayr approved these changes 2026-09-20 08:37:11 +00:00
Burer changed title from netcfg: show Broadcom for vendor id 14E4 to apps/netcfg: show Broadcom for vendor id 14E4 2026-09-20 09:44:46 +00:00
Burer merged commit d98755374f into bcm57xx 2026-09-20 09:45:11 +00:00
Burer deleted branch bcm57xx 2026-09-20 09:45:11 +00:00
Sign in to join this conversation.
No Reviewers
No labels
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: KolibriOS/kolibrios#708