Skip to content

Conversation

crawfxrd
Copy link
Member

@crawfxrd crawfxrd commented Sep 4, 2025

Sync changes from CB:75286 and CB:76584.

Need to test on:

  • bonw15-b (Barlow Ridge)
  • addw3, bonw15, oryp12, or serw13 (Maple Ridge)

@crawfxrd crawfxrd requested review from a team and jackpot51 September 4, 2025 17:37
@crawfxrd crawfxrd force-pushed the dtbt-sync branch 2 times, most recently from 04a63cd to eb87d4a Compare September 4, 2025 18:03
Copy link
Member

@jacobgkau jacobgkau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems to break Thunderbolt on the oryp12 (Maple Ridge). On currently released firmware, an HP dock works with both USB and Ethernet devices plugged into the dock. On this branch, the system still detects the dock as a Thunderbolt device, but the downstream USB and Ethernet devices don't show up.

@jacobgkau
Copy link
Member

Confirmed Thunderbolt works on master firmware (with coreboot 6982eaf), so the regression is specific to this PR. Here's dmesg after plugging in the dock with the working and non-working firmware:

dock-working-6982eaf.txt
dock-not-working-eb87d4a.txt

@crawfxrd
Copy link
Member Author

It's only allocating a single downstream bus for the TBT device itself, and failing to allocate any memory resources.

Does the dock work if the it's connected before booting?

@jacobgkau
Copy link
Member

jacobgkau commented Oct 14, 2025

Does the dock work if the it's connected before booting?

Yes, just tried rebooting while it was plugged in, and it started working on the next boot. Seems to then work if I unplug and re-plug on the same boot, too.

@crawfxrd
Copy link
Member Author

Can you get the output from cbmem -c as well with/without dock attached before boot?

@jacobgkau
Copy link
Member

Here you go:
booted-without-afterplug.txt
booted-with.txt

(I collected output both before and after plugging the dock in on the non-plugged boot, but they were identical.)

Sync changes from CB:75286 and CB:76584.

Change-Id: I0544ecb26869fa137f5bf59f0b510280611deaa1
Link: https://review.coreboot.org/c/coreboot/+/75286
Link: https://review.coreboot.org/c/coreboot/+/76584
Signed-off-by: Tim Crawford <[email protected]>
Change-Id: I2ce092ee8344034fd6e6a3493d3be17ac4c471f3
Signed-off-by: Tim Crawford <[email protected]>
@crawfxrd
Copy link
Member Author

Re-added the devicetree entries, which should hopefully fix hotplug.

@crawfxrd crawfxrd requested a review from jacobgkau October 15, 2025 15:05
@jacobgkau
Copy link
Member

Just built & flashed with 876b0e1, and it's still not seeing the downstream devices on hotplug, only after rebooting.

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.

3 participants