Attachment 30757 Details for Bug 24765 – ACPI backlight support for FreeBSD (revised)

[patch] ACPI backlight support for FreeBSD (revised)

radeonhd-backlight.diff (text/plain), 2.01 KB, created by Jung-uk Kim on 2009-10-27 11:09:41 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jung-uk Kim

Created: 2009-10-27 11:09:41 UTC

Size: 2.01 KB

patch

obsolete

>--- src/rhd_acpi.c.orig	2009-10-06 07:36:07.000000000 -0400
>+++ src/rhd_acpi.c	2009-10-27 03:05:23.000000000 -0400
>@@ -47,6 +47,55 @@
> #include "rhd_acpi.h"
> 
> 
>+#if defined(__FreeBSD__)
>+
>+#include <stdlib.h>
>+#include <sys/sysctl.h>
>+
>+/*
>+ * Get/Set LCD backlight brightness via acpi_video(4).
>+ */
>+static Bool
>+rhdDoBacklight(struct rhdOutput *Output, Bool do_write, int *val)
>+{
>+    int *levels;
>+    size_t len;
>+    int i, num_levels;
>+    RHDFUNC(Output);
>+
>+    if (sysctlbyname("hw.acpi.video.lcd0.levels", NULL, &len, NULL, 0) != 0 || len == 0)
>+	return FALSE;
>+
>+    if (do_write) {
>+	levels = (int *)malloc(len);
>+	if (levels == NULL)
>+	    return FALSE;
>+	if (sysctlbyname("hw.acpi.video.lcd0.levels", levels, &len, NULL, 0) != 0) {
>+	    free(levels);
>+	    return FALSE;
>+	}
>+	num_levels = len / sizeof(*levels);
>+	for (i = 0; i < num_levels; i++)
>+	    if (levels[i] == *val)
>+		break;
>+	free(levels);
>+	if (i >= num_levels)
>+	    return FALSE;
>+	if (sysctlbyname("hw.acpi.video.lcd0.brightness", NULL, 0, val, sizeof(*val)) != 0)
>+	    return FALSE;
>+	    RHDDebug(Output->scrnIndex,"%s: Set brightness value %i\n", __func__, *val);
>+    } else {
>+	len = sizeof(*val);
>+	if (sysctlbyname("hw.acpi.video.lcd0.brightness", val, &len, NULL, 0) != 0)
>+	    return FALSE;
>+	RHDDebug(Output->scrnIndex,"%s: Get brightness value %i\n", __func__, *val);
>+    }
>+
>+    return TRUE;
>+}
>+
>+#elif defined(__linux__)
>+
> #define ACPI_PATH "/sys/class/backlight"
> 
> /*
>@@ -120,6 +169,7 @@
> 
>     return FALSE;
> }
>+#endif
> 
> /*
>  * RhdACPIGetBacklightControl(): return backlight value in range 0..255;
>@@ -128,7 +178,7 @@
> int
> RhdACPIGetBacklightControl(struct rhdOutput *Output)
> {
>-#ifdef __linux__
>+#if defined(__FreeBSD__) || defined(__linux__)
>     int ret;
>     RHDFUNC(Output);
>     if (rhdDoBacklight(Output, FALSE, &ret))
>@@ -144,7 +194,7 @@
> RhdACPISetBacklightControl(struct rhdOutput *Output, int val)
> {
>     RHDFUNC(Output);
>-#ifdef __linux__
>+#if defined(__FreeBSD__) || defined(__linux__)
>     rhdDoBacklight(Output, TRUE, &val);
> #endif
> }

Actions: View | Splinter Review

Attachments on bug 24765: 30756 | 30757 | 30780 | 30783 | 30784 | 30785 | 30812

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.