thunderbolt: Support writing to router config space
Needs ReviewPublic
Actions

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

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.Mon, Jul 13, 8:47 AM

Herald added a subscriber: imp. · View Herald TranscriptMon, Jul 13, 8:47 AM

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

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

ngie added inline comments.Mon, Jul 13, 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?