[PATCH] Set initial value for backlight_active_level

[PATCH] Set initial value for backlight_active_level

[Mickaël THOMAS]

If the "Backlight" option is set, backlight_active_level is not set which
results in a default value of 0, causing a black screen upon starting Xorg.
---
 src/intel_display.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/intel_display.c b/src/intel_display.c
index 2c93c2d..5ee955e 100644
--- a/src/intel_display.c
+++ b/src/intel_display.c
@@ -271,6 +271,7 @@ intel_output_backlight_init(xf86OutputPtr output)
                        intel_output->backlight_iface = str;
                        intel_output->backlight_max =
intel_output_backlight_get_max(output);
                        if (intel_output->backlight_max > 0) {
+                               intel_output->backlight_active_level =
intel_output_backlight_get(output);
                                xf86DrvMsg(output->scrn->scrnIndex, X_CONFIG,
                                           "found backlight control
interface %s\n", path);
                                return;
-- 
1.8.0.3

[PATCH] Set initial value for backlight_active_level

[Mickaël THOMAS]

If the "Backlight" option is set, backlight_active_level is not set which
results in a default value of 0, causing a black screen upon starting Xorg.
---
 src/intel_display.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/intel_display.c b/src/intel_display.c
index 2c93c2d..5ee955e 100644
--- a/src/intel_display.c
+++ b/src/intel_display.c
@@ -271,6 +271,7 @@ intel_output_backlight_init(xf86OutputPtr output)
 			intel_output->backlight_iface = str;
 			intel_output->backlight_max = intel_output_backlight_get_max(output);
 			if (intel_output->backlight_max > 0) {
+				intel_output->backlight_active_level = intel_output_backlight_get(output);
 				xf86DrvMsg(output->scrn->scrnIndex, X_CONFIG,
 					   "found backlight control interface %s\n", path);
 				return;
-- 
1.8.0.3

Re: [PATCH] Set initial value for backlight_active_level

[Chris Wilson]

[-- Attachment #1: Type: text/plain, Size: 312 bytes --]

On Mon, 7 Jan 2013 20:47:51 +0100, Mickaël THOMAS <mickael9@gmail.com> wrote:
> If the "Backlight" option is set, backlight_active_level is not set which
> results in a default value of 0, causing a black screen upon starting Xorg.

Thanks, pushed.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre

[-- Attachment #2: Type: text/plain, Size: 159 bytes --]

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx