Page MenuHomeFreeBSD

Add backlight(8)
ClosedPublic

Authored by manu on Aug 31 2020, 2:53 PM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 24 2024, 10:22 AM
Unknown Object (File)
Feb 24 2024, 10:22 AM
Unknown Object (File)
Feb 24 2024, 10:22 AM
Unknown Object (File)
Feb 24 2024, 10:21 AM
Unknown Object (File)
Feb 24 2024, 10:21 AM
Unknown Object (File)
Feb 24 2024, 10:04 AM
Unknown Object (File)
Jan 13 2024, 2:56 AM
Unknown Object (File)
Dec 20 2023, 7:10 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.