thunderbolt: Support writing to router config space
Needs ReviewPublic
Actions

Authored by obiwac on Jul 13 2026, 8:47 AM.

Tags

None

Referenced Files

	F170652932: D58209.id181891.diff
	Sat, Sep 5, 8:09 PM

	Unknown Object (File)
	Sat, Sep 5, 2:14 PM

	Unknown Object (File)
	Sat, Sep 5, 4:10 AM

	Unknown Object (File)
	Fri, Sep 4, 10:26 PM

	Unknown Object (File)
	Fri, Sep 4, 4:42 PM

	Unknown Object (File)
	Thu, Sep 3, 6:02 PM

	Unknown Object (File)
	Thu, Sep 3, 5:52 PM

	Unknown Object (File)
	Thu, Sep 3, 5:49 PM

View All 40 Files

Subscribers

imp

Details

Reviewers

olce
jhb
ngie

Summary

Not to be confused with PCI config space; we write to a router's config
space by preparing a write command and enqueuing it on our NHI's
transmit ring.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 74763
Build 71646: arc lint + arc unit

Event Timeline

obiwac created this revision.Jul 13 2026, 8:47 AM

Herald added a subscriber: imp. · View Herald TranscriptJul 13 2026, 8:47 AM

obiwac requested review of this revision.Jul 13 2026, 8:47 AM

Harbormaster completed remote builds in B74763: Diff 181891.Jul 13 2026, 8:47 AM

ngie added inline comments.Jul 13 2026, 10:31 PM

sys/dev/thunderbolt/router.c
643	`size_t` instead of `int` seems more appropropriate, but I see that this type is passed through a bunch of callers.
651–659	Would you please replace `4` with a more usable constant / reference to some documentation that describes where this constraint is defined (I realize `router_prepare_read` needs similar treatment)?
sys/dev/thunderbolt/tbcfg_reg.h
67	Why remove this comment?