Page MenuHomeFreeBSD
Files F110078666

D24653.diff
No OneTemporary
Actions

D24653.diff
View Options

Index: head/sys/dev/acpica/acpi_video.c
===================================================================
--- head/sys/dev/acpica/acpi_video.c
+++ head/sys/dev/acpica/acpi_video.c
@@ -52,6 +52,8 @@
int num;
STAILQ_ENTRY(acpi_video_output) next;
} vo_unit;
+ int vo_hasbqc; /* Query method is present. */
+ int vo_level; /* Cached level when !vo_hasbqc. */
int vo_brightness;
int vo_fullpower;
int vo_economy;
@@ -96,8 +98,8 @@
static int vid_enum_outputs(ACPI_HANDLE,
void(*)(ACPI_HANDLE, UINT32, void *), void *);
static int vo_get_brightness_levels(ACPI_HANDLE, int **);
-static int vo_get_brightness(ACPI_HANDLE);
-static void vo_set_brightness(ACPI_HANDLE, int);
+static int vo_get_brightness(struct acpi_video_output *);
+static void vo_set_brightness(struct acpi_video_output *, int);
static UINT32 vo_get_device_status(ACPI_HANDLE);
static UINT32 vo_get_graphics_state(ACPI_HANDLE);
static void vo_set_device_state(ACPI_HANDLE, UINT32);
@@ -327,9 +329,9 @@
if ((vo_get_device_status(vo->handle) & DCS_ACTIVE) == 0)
continue;
- level = vo_get_brightness(vo->handle);
+ level = vo_get_brightness(vo);
if (level != -1)
- vo_set_brightness(vo->handle, level);
+ vo_set_brightness(vo, level);
}
ACPI_SERIAL_END(video_output);
ACPI_SERIAL_END(video);
@@ -419,7 +421,7 @@
ACPI_SERIAL_BEGIN(video_output);
STAILQ_FOREACH(vo, &sc->vid_outputs, vo_next) {
if (vo->vo_levels != NULL && vo->vo_brightness == -1)
- vo_set_brightness(vo->handle,
+ vo_set_brightness(vo,
state == POWER_PROFILE_ECONOMY ?
vo->vo_economy : vo->vo_fullpower);
}
@@ -518,6 +520,8 @@
vo->handle = NULL;
vo->adr = adr;
vo->vo_unit.num = n;
+ vo->vo_hasbqc = -1;
+ vo->vo_level = -1;
vo->vo_brightness = -1;
vo->vo_fullpower = -1; /* TODO: override with tunables */
vo->vo_economy = -1;
@@ -698,7 +702,7 @@
case VID_NOTIFY_ZERO_BRN:
if (vo->vo_levels == NULL)
goto out;
- level = vo_get_brightness(handle);
+ level = vo_get_brightness(vo);
if (level < 0)
goto out;
break;
@@ -742,7 +746,7 @@
break;
}
if (new_level != level) {
- vo_set_brightness(handle, new_level);
+ vo_set_brightness(vo, new_level);
vo->vo_brightness = new_level;
}
@@ -807,7 +811,7 @@
if (level != -1 && (err = acpi_video_vo_check_level(vo, level)))
goto out;
vo->vo_brightness = level;
- vo_set_brightness(vo->handle, (level == -1) ? preset : level);
+ vo_set_brightness(vo, (level == -1) ? preset : level);
out:
ACPI_SERIAL_END(video_output);
@@ -848,7 +852,7 @@
goto out;
if (vo->vo_brightness == -1 && (power_profile_get_state() == arg2))
- vo_set_brightness(vo->handle, level);
+ vo_set_brightness(vo, level);
*preset = level;
out:
@@ -1018,15 +1022,39 @@
}
static int
-vo_get_brightness(ACPI_HANDLE handle)
+vo_get_bqc(struct acpi_video_output *vo, UINT32 *level)
{
+ ACPI_STATUS status;
+
+ switch (vo->vo_hasbqc) {
+ case 1:
+ case -1:
+ status = acpi_GetInteger(vo->handle, "_BQC", level);
+ if (vo->vo_hasbqc == 1)
+ break;
+ vo->vo_hasbqc = status != AE_NOT_FOUND;
+ if (vo->vo_hasbqc == 1)
+ break;
+ /* FALLTHROUGH */
+ default:
+ KASSERT(vo->vo_hasbqc == 0,
+ ("bad vo_hasbqc state %d", vo->vo_hasbqc));
+ *level = vo->vo_level;
+ status = AE_OK;
+ }
+ return (status);
+}
+
+static int
+vo_get_brightness(struct acpi_video_output *vo)
+{
UINT32 level;
ACPI_STATUS status;
ACPI_SERIAL_ASSERT(video_output);
- status = acpi_GetInteger(handle, "_BQC", &level);
+ status = vo_get_bqc(vo, &level);
if (ACPI_FAILURE(status)) {
- printf("can't evaluate %s._BQC - %s\n", acpi_name(handle),
+ printf("can't evaluate %s._BQC - %s\n", acpi_name(vo->handle),
AcpiFormatException(status));
return (-1);
}
@@ -1037,16 +1065,19 @@
}
static void
-vo_set_brightness(ACPI_HANDLE handle, int level)
+vo_set_brightness(struct acpi_video_output *vo, int level)
{
char notify_buf[16];
ACPI_STATUS status;
ACPI_SERIAL_ASSERT(video_output);
- status = acpi_SetInteger(handle, "_BCM", level);
- if (ACPI_FAILURE(status))
+ status = acpi_SetInteger(vo->handle, "_BCM", level);
+ if (ACPI_FAILURE(status)) {
printf("can't evaluate %s._BCM - %s\n",
- acpi_name(handle), AcpiFormatException(status));
+ acpi_name(vo->handle), AcpiFormatException(status));
+ } else {
+ vo->vo_level = level;
+ }
snprintf(notify_buf, sizeof(notify_buf), "notify=%d", level);
devctl_notify("ACPI", "Video", "brightness", notify_buf);
}
@@ -1060,9 +1091,18 @@
ACPI_SERIAL_ASSERT(video_output);
dcs = 0;
status = acpi_GetInteger(handle, "_DCS", &dcs);
- if (ACPI_FAILURE(status))
- printf("can't evaluate %s._DCS - %s\n",
- acpi_name(handle), AcpiFormatException(status));
+ if (ACPI_FAILURE(status)) {
+ /*
+ * If the method is missing, assume that the device is always
+ * operational.
+ */
+ if (status != AE_NOT_FOUND) {
+ printf("can't evaluate %s._DCS - %s\n",
+ acpi_name(handle), AcpiFormatException(status));
+ } else {
+ dcs = 0xff;
+ }
+ }
return (dcs);
}
@@ -1075,9 +1115,18 @@
dgs = 0;
status = acpi_GetInteger(handle, "_DGS", &dgs);
- if (ACPI_FAILURE(status))
- printf("can't evaluate %s._DGS - %s\n",
- acpi_name(handle), AcpiFormatException(status));
+ if (ACPI_FAILURE(status)) {
+ /*
+ * If the method is missing, assume that the device is always
+ * operational.
+ */
+ if (status != AE_NOT_FOUND) {
+ printf("can't evaluate %s._DGS - %s\n",
+ acpi_name(handle), AcpiFormatException(status));
+ } else {
+ dgs = 0xff;
+ }
+ }
return (dgs);
}
@@ -1089,7 +1138,7 @@
ACPI_SERIAL_ASSERT(video_output);
status = acpi_SetInteger(handle, "_DSS", state);
- if (ACPI_FAILURE(status))
+ if (ACPI_FAILURE(status) && status != AE_NOT_FOUND)
printf("can't evaluate %s._DSS - %s\n",
- acpi_name(handle), AcpiFormatException(status));
+ acpi_name(handle), AcpiFormatException(status));
}

File Metadata

Mime Type
text/plain
Expires
Fri, Feb 14, 8:39 AM (20 h, 19 m)
Storage Engine
blob
Storage Format
Raw Data
Storage Handle
16640121
Default Alt Text
D24653.diff (5 KB)

Event Timeline