Page MenuHomeFreeBSD
Differential D48983

acpi_asus_wmi(4): Improve keyboard backlight support.
AcceptedPublic
Actions

Authored by wulf on Thu, Feb 13, 8:02 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Feb 18, 3:49 AM
Unknown Object (File)
Thu, Feb 13, 9:18 AM
Subscribers
imp
ziaee

Details

Reviewers
mav
Summary

acpi_asus_wmi(4): Fix maximal keyboard backlight level
Although Linux source code comment says that backlight level is encoded
in 3 bits of data, actual code limits maximum level to 3.

acpi_asus_wmi(4): Add backlight(9) support for keyboard

acpi_asus_wmi: turn off/on keyboard backlight on suspend/resume

Sponsored by: Future Crew LLC
MFC after: 1 month

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 62484
Build 59368: arc lint + arc unit

Event Timeline

wulf created this revision.Thu, Feb 13, 8:02 AM
Herald added subscribers: ziaee, imp. · View Herald TranscriptThu, Feb 13, 8:02 AM
wulf requested review of this revision.Thu, Feb 13, 8:02 AM
Harbormaster completed remote builds in B62416: Diff 150944.Thu, Feb 13, 8:02 AM
mav added a comment.Fri, Feb 14, 2:11 AM

I have no objections, but I wonder whether maximum of 3 vs 7 might be model-specific?

sys/dev/acpi_support/acpi_asus_wmi.c
413

Why random mix of UINT32 and uint32_t?

419–421

Braces?

ziaee added inline comments.Fri, Feb 14, 3:44 AM
share/man/man4/acpi_asus_wmi.4
76
wulf updated this revision to Diff 151107.Mon, Feb 17, 8:24 AM

Fix manpage and braces

Harbormaster completed remote builds in B62484: Diff 151107.Mon, Feb 17, 8:24 AM
wulf marked 2 inline comments as done.Mon, Feb 17, 8:29 AM
In D48983#1117257, @mav wrote:

I have no objections, but I wonder whether maximum of 3 vs 7 might be model-specific?

Linux driver defaults it to 3 and I was not able to find any quirks changing that. May be I just missed something.

sys/dev/acpi_support/acpi_asus_wmi.c
413

UINT32 is ACPI style and uint32_t is FreeBSD. This function converts backlight value from FreeBSD to ACPI

wulf added inline comments.Mon, Feb 17, 8:30 AM
share/man/man4/acpi_asus_wmi.4
76

Fixed

sys/dev/acpi_support/acpi_asus_wmi.c
419–421

Fixed

mav accepted this revision.Tue, Feb 18, 6:11 PM
This revision is now accepted and ready to land.Tue, Feb 18, 6:11 PM

Revision Contents
Changeset List

PathSize
share/
man/
man4/
10 lines
sys/
dev/
acpi_support/
133 lines
modules/
acpi/
acpi_asus_wmi/
2 lines

Diff 151107

View Options

share/man/man4/acpi_asus_wmi.4

Loading...
View Options

sys/dev/acpi_support/acpi_asus_wmi.c

Loading...
View Options

sys/modules/acpi/acpi_asus_wmi/Makefile

Loading...