Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

darp8: PLTRST# virtual wire not de-asserted on S3 resume #507

Open
crawfxrd opened this issue Jan 17, 2024 · 0 comments
Open

darp8: PLTRST# virtual wire not de-asserted on S3 resume #507

crawfxrd opened this issue Jan 17, 2024 · 0 comments

Comments

@crawfxrd
Copy link
Member

  • Model: darp8
  • BIOS version: 2024-01-10_6c402c3
  • EC version: 2024-01-10_6c402c3
  • OS:
  • Kernel:

The last batch of darp8 do not de-assert the PLTRST# virtual wire on S3 resume, causing the system to hang on wake.

Steps to reproduce

  • Switch to S3 suspend: echo deep | sudo tee /sys/power/mem_sleep
  • Suspend the system: sudo systemctl suspend
  • Wake the system: Press the power button

Expected behavior

System resumes to desktop.

Actual behavior

System hangs with solid green LED and black screen.

Additional info

This is seemingly an unidentified BOM change from Clevo, as the engineering units we originally received and first batch of production units that customers received (which also had an unidentified change that broke on our firmware) do not have the issue.

Ref: #469

crawfxrd added a commit that referenced this issue Mar 1, 2024
CSME should be enabled when S0ix is used for power savings during
suspend. TGL-U is just one case where we default as our TGL-U models
do not support S3. Other models may use S0ix as well, such as darp8 and
darp9 due to new batches just not working with S3 anymore [1].

[1]: #507

Signed-off-by: Tim Crawford <[email protected]>
jackpot51 pushed a commit that referenced this issue Mar 1, 2024
CSME should be enabled when S0ix is used for power savings during
suspend. TGL-U is just one case where we default as our TGL-U models
do not support S3. Other models may use S0ix as well, such as darp8 and
darp9 due to new batches just not working with S3 anymore [1].

[1]: #507

Signed-off-by: Tim Crawford <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant