https://bugzilla.kernel.org/show_bug.cgi?id=216516 --- Comment #10 from kolAflash (kolAflash@xxxxxxxxxxxx) --- Created attachment 301870 --> https://bugzilla.kernel.org/attachment.cgi?id=301870&action=edit kernel log for s2idle: v6.0-rc6 with CONFIG_AMD_PMC enabled (In reply to Mario Limonciello (AMD) from comment #9) > You're missing CONFIG_AMD_PMC. Compiled with CONFIG_AMD_PMC. Going to s2idle two times. (first time just for a few seconds) The good news: With CONFIG_AMD_PMC in s2idle the screen doesn't freeze. The bad news: After less than 8 hours in s2idle the battery went from 60 % to 0 % !!! That's why there's no second wakeup. (system was down because battery was drained) With S3 the battery will loose less than 5 % charge in the same time. And the notebook is brand new. So there should be no battery degradation. ... just did another s2idle test and battery went from 67 % to 58 % in one hour. (In reply to Mario Limonciello (AMD) from comment #6) > [...] > As you're patching the firmware this is not a regression. It takes more to > properly enable S3 in Pre-OS code than changing a variable evaluated by the > Linux kernel's ACPI interpreter.. > > Thorsten - this is not a regression, patching the BIOS like this is > completely off the beating path. We can't possibly support S3 in Linux if > the OEM doesn't offer it in their BIOS. I think you exaggerate a little. I'm just enabling existing OEM code! Also I wouldn't say this is patching the firmware, because it's just the ACPI data, not the BIOS software. Just changing the existing "if" statement. See: comment #5 I'm pretty sure there's a way to enable this OEM code without recompiling the ACPI data. Probably by setting the MSCE variable with a hidden BIOS option or something else I haven't found yet. -- You may reply to this email to add a comment. You are receiving this mail because: You are watching the assignee of the bug.
