After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 782155 - Some PCs still turn off the backlight by zero brightness even when the maximum brightness level is less than 100.
Some PCs still turn off the backlight by zero brightness even when the maximu...
Status: RESOLVED OBSOLETE
Product: gnome-settings-daemon
Classification: Core
Component: power
unspecified
Other Linux
: Normal enhancement
: ---
Assigned To: gnome-settings-daemon-maint
gnome-settings-daemon-maint
Depends on:
Blocks:
 
 
Reported: 2017-05-04 09:50 UTC by Shih-Yuan Lee (FourDollars)
Modified: 2019-03-20 11:42 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
power: Provide a customizable minimum brightness setting (3.49 KB, patch)
2017-05-04 09:54 UTC, Shih-Yuan Lee (FourDollars)
none Details | Review
Avoid using zero brightness by settings. (3.38 KB, patch)
2017-05-04 13:47 UTC, Shih-Yuan Lee (FourDollars)
rejected Details | Review

Description Shih-Yuan Lee (FourDollars) 2017-05-04 09:50:32 UTC
The minimum brightness of the laptop screen panel may not be realistic sometimes especially when having a problematic screen panel.

For example, Dell Precision 5720 AIO only has 9 levels in /sys/class/backlight/intel_backlight/max_brightness because the wrong settings in VBIOS. It is an all-in-one desktop PC and there is no brightness hotkey to control the backlight. When we set 0 to /sys/class/backlight/intel_backlight/brightness, it turns off the panel's backlight and there is no way to brightness up the backlight again so we can only avoid using 0 as the minimum backlight level.
Comment 1 Shih-Yuan Lee (FourDollars) 2017-05-04 09:54:38 UTC
Created attachment 351039 [details] [review]
power: Provide a customizable minimum brightness setting

I have tested this patch on Fedora 25 with jhbuild and it seems to work fine as what I expected.
Comment 2 Bastien Nocera 2017-05-04 10:00:49 UTC
Review of attachment 351039 [details] [review]:

That's out of the question.

See https://bugzilla.gnome.org/show_bug.cgi?id=744278 for how we'd want it to be solved.
Comment 3 Shih-Yuan Lee (FourDollars) 2017-05-04 12:45:37 UTC
I can still reduplicate this issue on Dell Precision 5720 AIO. It only has 9 levels in /sys/class/backlight/intel_backlight/max_brightness because the wrong settings in VBIOS. How can we fix this problem? Any idea?
Comment 4 Shih-Yuan Lee (FourDollars) 2017-05-04 13:47:36 UTC
Created attachment 351052 [details] [review]
Avoid using zero brightness by settings.
Comment 5 Bastien Nocera 2017-05-04 13:59:49 UTC
Review of attachment 351052 [details] [review]:

No, this shouldn't need a configuration option. If this requires fixing the kernel, the kernel be fixed.
Comment 6 Iain Lane 2017-06-20 10:09:23 UTC
(In reply to Bastien Nocera from comment #5)
> Review of attachment 351052 [details] [review] [review]:
> 
> No, this shouldn't need a configuration option. If this requires fixing the
> kernel, the kernel be fixed.

Sorry for being dense - what kind of fix do you have in mind?
Comment 7 GNOME Infrastructure Team 2019-03-20 11:42:26 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to GNOME's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.gnome.org/GNOME/gnome-settings-daemon/issues/331.