Page MenuHomeFreeBSD

Add backlight(8)
ClosedPublic

Authored by manu on Aug 31 2020, 2:53 PM.

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

Repository
rS FreeBSD src repository
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

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
80 ↗(On Diff #76402)

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
80 ↗(On Diff #76402)

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
80 ↗(On Diff #76402)

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
80 ↗(On Diff #76402)

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
80 ↗(On Diff #76402)

ok :)

greg_unrelenting.technology 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.Fri, Oct 2, 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.