Page MenuHomeFreeBSD

Add backlight(8)
ClosedPublic

Authored by manu on Aug 31 2020, 2:53 PM.
Tags
None
Referenced Files
F137010257: D26251.id76595.diff
Thu, Nov 20, 11:42 PM
F137009444: D26251.id.diff
Thu, Nov 20, 11:40 PM
F137009017: D26251.id77103.diff
Thu, Nov 20, 11:39 PM
F137008463: D26251.id76402.diff
Thu, Nov 20, 11:38 PM
F137007916: D26251.id77792.diff
Thu, Nov 20, 11:36 PM
F137007129: D26251.diff
Thu, Nov 20, 11:34 PM
F136919786: D26251.id76595.diff
Thu, Nov 20, 4:11 PM
Unknown Object (File)
Tue, Nov 18, 10:19 AM

Details

Reviewers
None
Group Reviewers
manpages
Commits
rS366370: Add backlight(8)
Summary

This tool is used to configure registered backlights.
It can incr/decr (default to 10%) or accept a percentage value directly.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

manu requested review of this revision.Aug 31 2020, 2:53 PM
manu added a parent revision: D26250: Add backlight subsystem.
usr.bin/backlight/backlight.8
81

Should we xref acpi_video as well, since it can be used for the same thing (amongts other things). You should probably add the xref in acpi_video to this tool as well.

usr.bin/backlight/backlight.8
81

I don't think so, for me acpi_video is useless.
It doesn't work on all (minus one not tested) my laptops without loading the drm driver and sometimes doesn't even work after.

usr.bin/backlight/backlight.8
81

Since a lot of people and guides mention acpi_video, and specifically the brightness sysctl, I thought it was a good idea, to draw attention to this tool.

usr.bin/backlight/backlight.8
81

I plan to send a mail to current/x11 after commiting all this, I think this will draw more attention than in a man page that people won't read if they know about the sysctl :)

usr.bin/backlight/backlight.8
81

ok :)

val_packett.cool added inline comments.
share/man/man9/Makefile
13 ↗(On Diff #76402)

I guess this patch needs to change the man8 makefile? This man9 thing was already in D26250

Don't add the man9 page here

gbe added a subscriber: gbe.

LGTM!

Add support for -i to query backlight info.

This revision was not accepted when it landed; it landed in state Needs Review.Oct 2 2020, 6:21 PM
Closed by commit rS366370: Add backlight(8) (authored by manu). · Explain Why
This revision was automatically updated to reflect the committed changes.