Project

General

Profile

Actions

Bug #147

closed

Backlight level setting is not restored when the display wakes up from sleep on ThinkPad X230

Added by Tobis Greer over 6 years ago. Updated about 2 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
-
Target version:
-
Start date:
11/03/2017
Due date:
% Done:

100%

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

Description

I am using Ubuntu 17.10, it also occurred on 17.04.

Whenever the display is sleeping (the laptop is still awake, just the display is off), and when it wakes up again (by moving the mouse, or issuing a keypress); then the backlight setting is at 100% regardless of what it was set at before the display went to sleep.

If I press the Fn+F8 key to set it again, one press is enough for the backlight level to go down several increments; back to where it was before the display turned off.

This didn't happen with the same Ubuntu install but running the stock BIOS.

For context, here is an excerpt from the mailing list:

this is a known issue. It actually was there all the time but only
got more visible with the introduction of ACPI/OpRegion for the inte-
grated GFX. Here is what I found out so far (on my ArchLinux):

w/o OpRegion:
  1. `acpi_video` driver gets loaded, systemd restores brightness from
     last shutdown.
  2. `i915` driver gets loaded, reads the current brightness.
  3. `i915` exposes backlight as `intel_backlight`, systemd restores
     brightness from last shutdown here as well.

Every time `i915` power cycles the backlight, it restores the value it
has seen when it was loaded.

w/ Opregion the order somehow changes to 2. 1. 3., now `i915` always
restores to 100% that it read before systemd restored the brightness.

A sane solution would be to extend XBCM in `src/drivers/intel/gma/acpi/
configure_brightness_levels.asl` to always notify `i915` when we change
the backlight through OpRegion mailbox 3.

Happy to provide more logs or details as required. Thanks!

Actions #1

Updated by Warrick Baijius over 5 years ago

This also occurs for me on a T420 running Debian 9.4. Has there been any progress? Is there anything I/we can do to help? I do not have coding knowledge, so my ability to assist will be limited, but I am willing to try.

Actions #2

Updated by Alexander Couzens over 5 years ago

  • Status changed from New to Feedback
  • % Done changed from 0 to 100

Hi Tobias,
Hi Warrick,

thanks for your bug report.
This bug has been fixed in https://review.coreboot.org/27711.
However, it introduced another bug, please wait until https://review.coreboot.org/c/coreboot/+/28345 has been merged.

Best Regards,
lynxis

Actions #3

Updated by Felix Singer about 2 years ago

  • Status changed from Feedback to Closed

Closed. No response since 3 years.

Actions

Also available in: Atom PDF