Project

General

Profile

Actions

Bug #256

closed

X201: Not probing for dock on wakeup

Added by Stefan Ott about 4 years ago. Updated almost 3 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
-
Target version:
-
Start date:
03/19/2020
Due date:
% Done:

100%

Estimated time:
Affected versions:
Needs backport to:
Affected hardware:
Affected OS:

Description

With stock BIOS and older versions of coreboot (4.9-7f520c8f), the Thinkpad X201 re-attaches itself to the docking station if it was docked while suspended; with the current master (and 4.11 release) this does not work anymore, i.e. it only detects the dock if the docking event occurs while the machine is awake.


Files

coreboot-4.9-powerup.log (6.98 KB) coreboot-4.9-powerup.log Stefan Ott, 03/19/2020 05:59 PM
coreboot-4.9-redock.log (2.75 KB) coreboot-4.9-redock.log Stefan Ott, 03/19/2020 05:59 PM
coreboot-4.9-wakeup.log (2.75 KB) coreboot-4.9-wakeup.log Stefan Ott, 03/19/2020 05:59 PM
coreboot-4.11-powerup.log (27.1 KB) coreboot-4.11-powerup.log Stefan Ott, 03/19/2020 05:59 PM
coreboot-4.11-redock.log (17.5 KB) coreboot-4.11-redock.log Stefan Ott, 03/19/2020 05:59 PM
coreboot-4.11-wakeup.log (17.5 KB) coreboot-4.11-wakeup.log Stefan Ott, 03/19/2020 05:59 PM
coreboot-x201-dock-attach.patch (1.96 KB) coreboot-x201-dock-attach.patch Stefan Ott, 03/20/2020 12:44 PM
Actions #1

Updated by Paul Menzel about 4 years ago

Please attach the logs cbmem -1 with the old-working and regressed coreboot versions.

I believe your issue has the higher chances to be fixed, if you bisect the commit causing the regression (git bisect).

Actions #2

Updated by Stefan Ott about 4 years ago

Paul Menzel wrote:

Please attach the logs cbmem -1 with the old-working and regressed coreboot versions.

I believe your issue has the higher chances to be fixed, if you bisect the commit causing the regression (git bisect).

I did some debugging with cbmem and noticed that coreboot does detect the dock (it says "dock is connected") on resume, it just seems to fail to tell the OS about it; the logs are attached.

Updated by Stefan Ott about 4 years ago

Here we go, I made a bunch of logs. For coreboot 4.9 (works) and 4.11 (does not work) I took a cbmem -1 log after bootup (docked), wakeup (docked) and after a docking procedure (while awake).

Actions #4

Updated by Stefan Ott about 4 years ago

I found the commit that broke it with git bisect, it appears to be 72f13e5 ("Make dock init in ramstage fully mainboard-specific").

Actions #5

Updated by Stefan Ott about 4 years ago

And indeed, if I revert that commit, it works. I'll attach a crude patch to show what I had to change.

Actions #6

Updated by Angel Pons almost 3 years ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 100
Actions

Also available in: Atom PDF