Intel-GFX Archive on lore.kernel.org
 help / color / mirror / Atom feed
* back light conflict i915 vs dell-laptop
@ 2010-11-25  9:33 Dan Carpenter
  2010-11-25 14:41 ` Andrea Arcangeli
                   ` (3 more replies)
  0 siblings, 4 replies; 7+ messages in thread
From: Dan Carpenter @ 2010-11-25  9:33 UTC (permalink / raw)
  To: chris; +Cc: mjg, rpurdie, kernelorg, intel-gfx, linux-kernel

Patrick Schaaf has reported that the i915 driver is setting his
backlight very dim when he starts GDM.

https://bugzilla.kernel.org/show_bug.cgi?id=23472

It should be the dell-laptop platform driver controlling his backlight.

Gnome tries to restore the backlight settings through the dell_laptop
interface in sysfs but it fails.  (Because the platform hardware refuses
to set the settings to what it thinks they already are).

regards,
dan carpenter

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: back light conflict i915 vs dell-laptop
  2010-11-25  9:33 back light conflict i915 vs dell-laptop Dan Carpenter
@ 2010-11-25 14:41 ` Andrea Arcangeli
  2010-11-28 18:05 ` Patrick Schaaf
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 7+ messages in thread
From: Andrea Arcangeli @ 2010-11-25 14:41 UTC (permalink / raw)
  To: Dan Carpenter, chris, mjg, rpurdie, kernelorg, intel-gfx,
	linux-kernel

On Thu, Nov 25, 2010 at 12:33:01PM +0300, Dan Carpenter wrote:
> Patrick Schaaf has reported that the i915 driver is setting his
> backlight very dim when he starts GDM.
> 
> https://bugzilla.kernel.org/show_bug.cgi?id=23472
> 
> It should be the dell-laptop platform driver controlling his backlight.
> 
> Gnome tries to restore the backlight settings through the dell_laptop
> interface in sysfs but it fails.  (Because the platform hardware refuses
> to set the settings to what it thinks they already are).

I think I run into the same problem, xset dpms force standby is enough
to trigger it, after that the backlight will never come up again. I
need to suspend to ram to be able to see my screen again. The screen
is on, just the backlight is off. Not dimmed to low level: completely
off.

Ironically xset dpms force off will come up right... but all screen
powermanagement will start with standby, so I'm stuck for the last few
weeks in presentation mode to be able to work on 37-rc1... It was fine
in .36 and it broke in 37-rc1.

I reported it to chris@chris-wilson.co.uk on 8 Nov but got no reply.

If you need me to test something let me know. It's super annoying when
I unplug the laptop and it exits presentation mode...

CONFIG_DRM_I915=y
CONFIG_DRM_I915_KMS=y

Thanks,
Andrea

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: back light conflict i915 vs dell-laptop
  2010-11-25  9:33 back light conflict i915 vs dell-laptop Dan Carpenter
  2010-11-25 14:41 ` Andrea Arcangeli
@ 2010-11-28 18:05 ` Patrick Schaaf
  2010-11-28 19:36   ` Dan Carpenter
  2010-11-28 18:52 ` Patrick Schaaf
  2011-01-11 11:31 ` Patrick Schaaf
  3 siblings, 1 reply; 7+ messages in thread
From: Patrick Schaaf @ 2010-11-28 18:05 UTC (permalink / raw)
  To: Dan Carpenter; +Cc: chris, mjg, rpurdie, kernelorg, intel-gfx, linux-kernel

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

On Thu, 2010-11-25 at 12:33 +0300, Dan Carpenter wrote:
> Patrick Schaaf has reported that the i915 driver is setting his
> backlight very dim when he starts GDM.
> 
> https://bugzilla.kernel.org/show_bug.cgi?id=23472
> 
> It should be the dell-laptop platform driver controlling his backlight.
> 
> Gnome tries to restore the backlight settings through the dell_laptop
> interface in sysfs but it fails.  (Because the platform hardware refuses
> to set the settings to what it thinks they already are).

As far as my instrumentation showed, gnome did try no such thing - there
were no setting calls through the backlight interface when I switch
between the GDM screen and text console.

I have now run with drm.debug=0x07 and noticed get/set backlight PWM
messages from intel_panel_get/set_backlight which became lower and lower
in value for each successive switching between GDM login screen and text
console, until reaching a low point and overflowing to 255255.

I append the extract of these kernel messages, as well as putting them
in the bugzilla entry linked above.

Hope this helps clarify the problems I'm seeing.

best regards
  Patrick



[-- Attachment #2: intel_panel_..._backlight --]
[-- Type: text/plain, Size: 4471 bytes --]

Nov 28 18:43:17 lat1 kernel: [   61.454973] [drm:intel_panel_get_backlight], get backlight PWM = 127500
Nov 28 18:43:17 lat1 kernel: [   61.672115] [drm:intel_panel_set_backlight], set backlight PWM = 127500
Nov 28 18:43:25 lat1 kernel: [   69.659366] [drm:intel_panel_get_backlight], get backlight PWM = 63627
Nov 28 18:43:25 lat1 kernel: [   69.876223] [drm:intel_panel_set_backlight], set backlight PWM = 63627
Nov 28 18:45:02 lat1 kernel: [  166.642624] [drm:intel_panel_get_backlight], get backlight PWM = 126735
Nov 28 18:45:02 lat1 kernel: [  166.861225] [drm:intel_panel_set_backlight], set backlight PWM = 126735
Nov 28 18:45:06 lat1 kernel: [  169.869343] [drm:intel_panel_get_backlight], get backlight PWM = 63246
Nov 28 18:45:06 lat1 kernel: [  170.086225] [drm:intel_panel_set_backlight], set backlight PWM = 63246
Nov 28 18:50:24 lat1 kernel: [  488.561922] [drm:intel_panel_get_backlight], get backlight PWM = 31563
Nov 28 18:50:24 lat1 kernel: [  488.780228] [drm:intel_panel_set_backlight], set backlight PWM = 31563
Nov 28 18:50:29 lat1 kernel: [  492.905273] [drm:intel_panel_get_backlight], get backlight PWM = 15776
Nov 28 18:50:29 lat1 kernel: [  493.122220] [drm:intel_panel_set_backlight], set backlight PWM = 15776
Nov 28 18:50:45 lat1 kernel: [  509.449869] [drm:intel_panel_get_backlight], get backlight PWM = 7888
Nov 28 18:50:45 lat1 kernel: [  509.667228] [drm:intel_panel_set_backlight], set backlight PWM = 7888
Nov 28 18:50:48 lat1 kernel: [  512.061427] [drm:intel_panel_get_backlight], get backlight PWM = 3944
Nov 28 18:50:48 lat1 kernel: [  512.278220] [drm:intel_panel_set_backlight], set backlight PWM = 3944
Nov 28 18:50:54 lat1 kernel: [  518.938610] [drm:intel_panel_get_backlight], get backlight PWM = 1972
Nov 28 18:50:55 lat1 kernel: [  519.156232] [drm:intel_panel_set_backlight], set backlight PWM = 1972
Nov 28 18:50:58 lat1 kernel: [  522.665295] [drm:intel_panel_get_backlight], get backlight PWM = 986
Nov 28 18:50:58 lat1 kernel: [  522.883222] [drm:intel_panel_set_backlight], set backlight PWM = 986
Nov 28 18:51:04 lat1 kernel: [  528.338612] [drm:intel_panel_get_backlight], get backlight PWM = 493
Nov 28 18:51:04 lat1 kernel: [  528.557235] [drm:intel_panel_set_backlight], set backlight PWM = 493
Nov 28 18:51:07 lat1 kernel: [  531.697225] [drm:intel_panel_get_backlight], get backlight PWM = 246
Nov 28 18:51:07 lat1 kernel: [  531.915244] [drm:intel_panel_set_backlight], set backlight PWM = 246
Nov 28 18:51:20 lat1 kernel: [  544.747370] [drm:intel_panel_get_backlight], get backlight PWM = 123
Nov 28 18:51:20 lat1 kernel: [  544.965241] [drm:intel_panel_set_backlight], set backlight PWM = 123
Nov 28 18:51:24 lat1 kernel: [  548.220386] [drm:intel_panel_get_backlight], get backlight PWM = 61
Nov 28 18:51:24 lat1 kernel: [  548.437215] [drm:intel_panel_set_backlight], set backlight PWM = 61
Nov 28 18:51:31 lat1 kernel: [  555.916888] [drm:intel_panel_get_backlight], get backlight PWM = 30
Nov 28 18:51:32 lat1 kernel: [  556.134239] [drm:intel_panel_set_backlight], set backlight PWM = 30
Nov 28 18:51:35 lat1 kernel: [  559.138346] [drm:intel_panel_get_backlight], get backlight PWM = 15
Nov 28 18:51:35 lat1 kernel: [  559.355242] [drm:intel_panel_set_backlight], set backlight PWM = 15
Nov 28 18:51:40 lat1 kernel: [  564.929606] [drm:intel_panel_get_backlight], get backlight PWM = 7
Nov 28 18:51:41 lat1 kernel: [  565.148225] [drm:intel_panel_set_backlight], set backlight PWM = 7
Nov 28 18:51:45 lat1 kernel: [  569.509391] [drm:intel_panel_get_backlight], get backlight PWM = 3
Nov 28 18:51:45 lat1 kernel: [  569.726226] [drm:intel_panel_set_backlight], set backlight PWM = 3
Nov 28 18:51:52 lat1 kernel: [  576.103569] [drm:intel_panel_get_backlight], get backlight PWM = 1
Nov 28 18:51:52 lat1 kernel: [  576.322226] [drm:intel_panel_set_backlight], set backlight PWM = 1
Nov 28 18:51:56 lat1 kernel: [  579.760386] [drm:intel_panel_get_backlight], get backlight PWM = 0
Nov 28 18:51:56 lat1 kernel: [  579.977224] [drm:intel_panel_set_backlight], set backlight PWM = 255255
Nov 28 18:52:12 lat1 kernel: [  596.022674] [drm:intel_panel_get_backlight], get backlight PWM = 127500
Nov 28 18:52:12 lat1 kernel: [  596.241232] [drm:intel_panel_set_backlight], set backlight PWM = 127500
Nov 28 18:52:15 lat1 kernel: [  599.502386] [drm:intel_panel_get_backlight], get backlight PWM = 63627
Nov 28 18:52:15 lat1 kernel: [  599.719217] [drm:intel_panel_set_backlight], set backlight PWM = 63627

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: back light conflict i915 vs dell-laptop
  2010-11-25  9:33 back light conflict i915 vs dell-laptop Dan Carpenter
  2010-11-25 14:41 ` Andrea Arcangeli
  2010-11-28 18:05 ` Patrick Schaaf
@ 2010-11-28 18:52 ` Patrick Schaaf
  2011-01-11 11:31 ` Patrick Schaaf
  3 siblings, 0 replies; 7+ messages in thread
From: Patrick Schaaf @ 2010-11-28 18:52 UTC (permalink / raw)
  To: intel-gfx

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

On Thu, 2010-11-25 at 12:33 +0300, Dan Carpenter wrote:
> Patrick Schaaf has reported that the i915 driver is setting his
> backlight very dim when he starts GDM.
> 
> https://bugzilla.kernel.org/show_bug.cgi?id=23472
> 
> It should be the dell-laptop platform driver controlling his backlight.
> 
> Gnome tries to restore the backlight settings through the dell_laptop
> interface in sysfs but it fails.  (Because the platform hardware refuses
> to set the settings to what it thinks they already are).

As far as my instrumentation showed, gnome did try no such thing - there
were no setting calls through the backlight interface when I switch
between the GDM screen and text console.

I have now run with drm.debug=0x07 and noticed get/set backlight PWM
messages from intel_panel_get/set_backlight which became lower and lower
in value for each successive switching between GDM login screen and text
console, until reaching a low point and overflowing to 255255.

I append the extract of these kernel messages, as well as putting them
in the bugzilla entry linked above.

Hope this helps clarify the problems I'm seeing.

best regards
  Patrick




[-- Attachment #2: intel_panel_..._backlight --]
[-- Type: text/plain, Size: 4471 bytes --]

Nov 28 18:43:17 lat1 kernel: [   61.454973] [drm:intel_panel_get_backlight], get backlight PWM = 127500
Nov 28 18:43:17 lat1 kernel: [   61.672115] [drm:intel_panel_set_backlight], set backlight PWM = 127500
Nov 28 18:43:25 lat1 kernel: [   69.659366] [drm:intel_panel_get_backlight], get backlight PWM = 63627
Nov 28 18:43:25 lat1 kernel: [   69.876223] [drm:intel_panel_set_backlight], set backlight PWM = 63627
Nov 28 18:45:02 lat1 kernel: [  166.642624] [drm:intel_panel_get_backlight], get backlight PWM = 126735
Nov 28 18:45:02 lat1 kernel: [  166.861225] [drm:intel_panel_set_backlight], set backlight PWM = 126735
Nov 28 18:45:06 lat1 kernel: [  169.869343] [drm:intel_panel_get_backlight], get backlight PWM = 63246
Nov 28 18:45:06 lat1 kernel: [  170.086225] [drm:intel_panel_set_backlight], set backlight PWM = 63246
Nov 28 18:50:24 lat1 kernel: [  488.561922] [drm:intel_panel_get_backlight], get backlight PWM = 31563
Nov 28 18:50:24 lat1 kernel: [  488.780228] [drm:intel_panel_set_backlight], set backlight PWM = 31563
Nov 28 18:50:29 lat1 kernel: [  492.905273] [drm:intel_panel_get_backlight], get backlight PWM = 15776
Nov 28 18:50:29 lat1 kernel: [  493.122220] [drm:intel_panel_set_backlight], set backlight PWM = 15776
Nov 28 18:50:45 lat1 kernel: [  509.449869] [drm:intel_panel_get_backlight], get backlight PWM = 7888
Nov 28 18:50:45 lat1 kernel: [  509.667228] [drm:intel_panel_set_backlight], set backlight PWM = 7888
Nov 28 18:50:48 lat1 kernel: [  512.061427] [drm:intel_panel_get_backlight], get backlight PWM = 3944
Nov 28 18:50:48 lat1 kernel: [  512.278220] [drm:intel_panel_set_backlight], set backlight PWM = 3944
Nov 28 18:50:54 lat1 kernel: [  518.938610] [drm:intel_panel_get_backlight], get backlight PWM = 1972
Nov 28 18:50:55 lat1 kernel: [  519.156232] [drm:intel_panel_set_backlight], set backlight PWM = 1972
Nov 28 18:50:58 lat1 kernel: [  522.665295] [drm:intel_panel_get_backlight], get backlight PWM = 986
Nov 28 18:50:58 lat1 kernel: [  522.883222] [drm:intel_panel_set_backlight], set backlight PWM = 986
Nov 28 18:51:04 lat1 kernel: [  528.338612] [drm:intel_panel_get_backlight], get backlight PWM = 493
Nov 28 18:51:04 lat1 kernel: [  528.557235] [drm:intel_panel_set_backlight], set backlight PWM = 493
Nov 28 18:51:07 lat1 kernel: [  531.697225] [drm:intel_panel_get_backlight], get backlight PWM = 246
Nov 28 18:51:07 lat1 kernel: [  531.915244] [drm:intel_panel_set_backlight], set backlight PWM = 246
Nov 28 18:51:20 lat1 kernel: [  544.747370] [drm:intel_panel_get_backlight], get backlight PWM = 123
Nov 28 18:51:20 lat1 kernel: [  544.965241] [drm:intel_panel_set_backlight], set backlight PWM = 123
Nov 28 18:51:24 lat1 kernel: [  548.220386] [drm:intel_panel_get_backlight], get backlight PWM = 61
Nov 28 18:51:24 lat1 kernel: [  548.437215] [drm:intel_panel_set_backlight], set backlight PWM = 61
Nov 28 18:51:31 lat1 kernel: [  555.916888] [drm:intel_panel_get_backlight], get backlight PWM = 30
Nov 28 18:51:32 lat1 kernel: [  556.134239] [drm:intel_panel_set_backlight], set backlight PWM = 30
Nov 28 18:51:35 lat1 kernel: [  559.138346] [drm:intel_panel_get_backlight], get backlight PWM = 15
Nov 28 18:51:35 lat1 kernel: [  559.355242] [drm:intel_panel_set_backlight], set backlight PWM = 15
Nov 28 18:51:40 lat1 kernel: [  564.929606] [drm:intel_panel_get_backlight], get backlight PWM = 7
Nov 28 18:51:41 lat1 kernel: [  565.148225] [drm:intel_panel_set_backlight], set backlight PWM = 7
Nov 28 18:51:45 lat1 kernel: [  569.509391] [drm:intel_panel_get_backlight], get backlight PWM = 3
Nov 28 18:51:45 lat1 kernel: [  569.726226] [drm:intel_panel_set_backlight], set backlight PWM = 3
Nov 28 18:51:52 lat1 kernel: [  576.103569] [drm:intel_panel_get_backlight], get backlight PWM = 1
Nov 28 18:51:52 lat1 kernel: [  576.322226] [drm:intel_panel_set_backlight], set backlight PWM = 1
Nov 28 18:51:56 lat1 kernel: [  579.760386] [drm:intel_panel_get_backlight], get backlight PWM = 0
Nov 28 18:51:56 lat1 kernel: [  579.977224] [drm:intel_panel_set_backlight], set backlight PWM = 255255
Nov 28 18:52:12 lat1 kernel: [  596.022674] [drm:intel_panel_get_backlight], get backlight PWM = 127500
Nov 28 18:52:12 lat1 kernel: [  596.241232] [drm:intel_panel_set_backlight], set backlight PWM = 127500
Nov 28 18:52:15 lat1 kernel: [  599.502386] [drm:intel_panel_get_backlight], get backlight PWM = 63627
Nov 28 18:52:15 lat1 kernel: [  599.719217] [drm:intel_panel_set_backlight], set backlight PWM = 63627

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

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

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: back light conflict i915 vs dell-laptop
  2010-11-28 18:05 ` Patrick Schaaf
@ 2010-11-28 19:36   ` Dan Carpenter
  0 siblings, 0 replies; 7+ messages in thread
From: Dan Carpenter @ 2010-11-28 19:36 UTC (permalink / raw)
  To: Patrick Schaaf
  Cc: chris, mjg, rpurdie, kernelorg, intel-gfx, linux-kernel, keithp

On Sun, Nov 28, 2010 at 07:05:27PM +0100, Patrick Schaaf wrote:
> On Thu, 2010-11-25 at 12:33 +0300, Dan Carpenter wrote:
> > Patrick Schaaf has reported that the i915 driver is setting his
> > backlight very dim when he starts GDM.
> > 
> > https://bugzilla.kernel.org/show_bug.cgi?id=23472
> > 
> > It should be the dell-laptop platform driver controlling his backlight.
> > 
> > Gnome tries to restore the backlight settings through the dell_laptop
> > interface in sysfs but it fails.  (Because the platform hardware refuses
> > to set the settings to what it thinks they already are).
> 

Keith Packard wrote in another thread that there were some patches
to address this in his git repo:

git://people.freedesktop.org/~keithp/linux-2.6 drm-intel-next

regards,
dan carpenter

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: back light conflict i915 vs dell-laptop
  2010-11-25  9:33 back light conflict i915 vs dell-laptop Dan Carpenter
                   ` (2 preceding siblings ...)
  2010-11-28 18:52 ` Patrick Schaaf
@ 2011-01-11 11:31 ` Patrick Schaaf
  2011-01-11 13:33   ` Chris Wilson
  3 siblings, 1 reply; 7+ messages in thread
From: Patrick Schaaf @ 2011-01-11 11:31 UTC (permalink / raw)
  To: intel-gfx

The dell laptop backlight problems I reported previously, are fixed with Indan's
patch found here:

https://bugzilla.kernel.org/show_bug.cgi?id=23472#c32

which disables the is_backlight_combination_mode() function by making it
return 0 immediately.

I'm running 2.6.37 with that patch applied, and the laptop is symptom
free now.

best regards
  Patrick

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: back light conflict i915 vs dell-laptop
  2011-01-11 11:31 ` Patrick Schaaf
@ 2011-01-11 13:33   ` Chris Wilson
  0 siblings, 0 replies; 7+ messages in thread
From: Chris Wilson @ 2011-01-11 13:33 UTC (permalink / raw)
  To: Patrick Schaaf, intel-gfx

On Tue, 11 Jan 2011 12:31:24 +0100, Patrick Schaaf <bof@bof.de> wrote:
> The dell laptop backlight problems I reported previously, are fixed with Indan's
> patch found here:
> 
> https://bugzilla.kernel.org/show_bug.cgi?id=23472#c32
> 
> which disables the is_backlight_combination_mode() function by making it
> return 0 immediately.

I'd be happier in this case to disable the combination mode rather than
ignore a value being used by the hardware to determine the backlight level.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2011-01-11 13:33 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-11-25  9:33 back light conflict i915 vs dell-laptop Dan Carpenter
2010-11-25 14:41 ` Andrea Arcangeli
2010-11-28 18:05 ` Patrick Schaaf
2010-11-28 19:36   ` Dan Carpenter
2010-11-28 18:52 ` Patrick Schaaf
2011-01-11 11:31 ` Patrick Schaaf
2011-01-11 13:33   ` Chris Wilson

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox