Page MenuHomeFreeBSD
Differential D53897

xhci: Add header values for Debug Capability (DbC)
ClosedPublic
Actions

Authored by thj on Nov 24 2025, 1:19 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Aug 25, 6:35 PM
Unknown Object (File)
Tue, Aug 25, 1:50 PM
Unknown Object (File)
Mon, Aug 24, 7:10 PM
Unknown Object (File)
Sun, Aug 23, 1:10 PM
Unknown Object (File)
Sun, Aug 23, 12:02 AM
Unknown Object (File)
Fri, Aug 21, 12:20 PM
Unknown Object (File)
Thu, Aug 13, 12:13 PM
Unknown Object (File)
Thu, Aug 13, 1:28 AM
View All Files
Subscribers
imp

Details

Reviewers
aokblast
adrian
bz
emaste
imp
Group Reviewers
USB
Commits
rGa0dd6022391a: xhci: Add header values for Debug Capability (DbC)

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

thj created this revision.Nov 24 2025, 1:19 PM
Herald added a subscriber: imp. · View Herald TranscriptNov 24 2025, 1:19 PM
thj requested review of this revision.Nov 24 2025, 1:19 PM
Harbormaster completed remote builds in B68822: Diff 167017.Nov 24 2025, 1:19 PM
emaste added inline comments.Nov 24 2025, 2:03 PM
sys/dev/usb/controller/xhcireg.h
221–224

Why both the "extract bits 15:8" macro and shifted #defines for in and out? Also what's special about 0xFF? 0x02-0xFE are equally reserved afaics.

252

SBR is bit 1, not 2?

thj updated this revision to Diff 167122.Nov 26 2025, 10:56 AM
  • Correct DC SBR bit definition
  • Change doorbell field definition and remove get method
Harbormaster completed remote builds in B68866: Diff 167122.Nov 26 2025, 10:56 AM
thj marked 2 inline comments as done.Nov 26 2025, 10:58 AM
thj added inline comments.
sys/dev/usb/controller/xhcireg.h
221–224

Thanks for the close read. Looking futher at the implementation and reading the bits is pointless, they should ways read as 0. I've changed the mask, but this isn't a bit field it, instead it is either the value 0 or 1.

252

Thanks for the close read!

imp accepted this revision.Nov 29 2025, 10:36 AM
This revision is now accepted and ready to land.Nov 29 2025, 10:36 AM
emaste added a comment.Nov 29 2025, 5:27 PM

FYI I didn't look closely at the ones after XHCI_DCST

thj updated this revision to Diff 167466.Dec 3 2025, 9:56 AM
thj marked 2 inline comments as done.
  • Add a comment labelling the Doorbell
  • The Device address field is 7 bits wide rather than 3 bits
This revision now requires review to proceed.Dec 3 2025, 9:56 AM
Harbormaster completed remote builds in B69009: Diff 167466.Dec 3 2025, 9:56 AM
imp accepted this revision.Dec 3 2025, 8:22 PM
This revision is now accepted and ready to land.Dec 3 2025, 8:22 PM
Closed by commit rGa0dd6022391a: xhci: Add header values for Debug Capability (DbC) (authored by thj). · Explain WhyDec 5 2025, 10:19 AM
This revision was automatically updated to reflect the committed changes.
thj added a commit: rGa0dd6022391a: xhci: Add header values for Debug Capability (DbC).

Revision Contents
Changeset List

PathSize
sys/
dev/
usb/
controller/
121 lines

Diff 167571

View Options

sys/dev/usb/controller/xhcireg.h

Loading...