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

Patch fails when installing ACPI driver #55

Closed
virtualritz opened this issue Jan 7, 2023 · 3 comments
Closed

Patch fails when installing ACPI driver #55

virtualritz opened this issue Jan 7, 2023 · 3 comments

Comments

@virtualritz
Copy link

After updating to the latest version of the repo (with the RMARB5B0P0E0E BIOS version which matches my laptop's), when running:

sudo /bin/sh install.sh acpi,

I get

[snip...]
Parsing completed
Disassembly completed
ASL Output:    /home/user/modern_laptop/fixes/acpi/dsdt.dsl - 410204 bytes
patching file dsdt.dsl
Reversed (or previously applied) patch detected!  Assume -R? [n] 

Desktop

  • OS: Ubuntu 22.04.1 LTS
  • Kernel: Linux 5.17.0-1025-oem #26-Ubuntu SMP PREEMPT
@vrolife
Copy link
Owner

vrolife commented Jan 8, 2023

You may have lost the original ACPI backup (Folder ACPI_BACKUP_RMARB5B0P0E0E after first installation)。
In this case, install linux 6.0 by command sudo apt install linux-oem-22.04b to get keyboard working without ACPI patch, then uninstall the ACPI patch sudo /bin/sh uninstall.sh acpi and delete folder ACPI_BACKUP_RMARB5B0P0E0E. After a reboot, your laptop will boot with original ACPI tables, and just install the new ACPI patch.

@virtualritz
Copy link
Author

When I first installed I was on the previous BIOS, RMARB5B0P0C0C. I guess that's why.

This worked, thanks.

Maybe these update instructions for ACPI after a BIOS update should be added to the README?

And on that note: it seems there is no need for using the 5.17 kernel any more? 6.0 seems to work just fine?

@vrolife
Copy link
Owner

vrolife commented Jan 8, 2023

When I first installed I was on the previous BIOS, RMARB5B0P0C0C. I guess that's why.

This worked, thanks.

Maybe these update instructions for ACPI after a BIOS update should be added to the README?

And on that note: it seems there is no need for using the 5.17 kernel any more? 6.0 seems to work just fine?

Linux 6.0 is recommended because GPU drivers are more stable (less than 6.0 will encounter random crashes with a small probability) and PSR capabilities are available. I'll update README tomorrow.

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

No branches or pull requests

2 participants