All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jani Nikula <jani.nikula@intel.com>
To: "Ortwin Glück" <odi@odi.ch>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"intel-gfx@lists.freedesktop.org"
	<intel-gfx@lists.freedesktop.org>
Subject: Re: [drm] 3.18 regression: backlight dark after resume
Date: Mon, 03 Nov 2014 12:34:15 +0200	[thread overview]
Message-ID: <87wq7ch8t4.fsf@intel.com> (raw)
In-Reply-To: <5454E2F3.2060404@odi.ch>

On Sat, 01 Nov 2014, Ortwin Glück <odi@odi.ch> wrote:
> Hi,
>
> Since 3.18-rc1 backlight is dark after resume from suspend. This is on a Samsung 
> laptop with i5 IvyBridge
>
> 00:02.0 VGA compatible controller: Intel Corporation 3rd Gen Core processor 
> Graphics Controller (rev 09)
>
>
> I have bisected this problem to the following commit:
> 73580fb764c4213d305c0d36bd8f856ae631eb42 is the first bad commit
> commit 73580fb764c4213d305c0d36bd8f856ae631eb42
> Author: Jani Nikula <jani.nikula@intel.com>
> Date:   Tue Aug 12 17:11:41 2014 +0300
>
>      drm/i915/dp: make backlight bl_power control power sequencer backlight
>
>      This lets the userspace switch off the backlight using the backlight
>      class sysfs bl_power file. The switch is done using the power sequencer;
>      the backlight PWM, and everything else, remains enabled. The display
>      backlight won't draw power, but for maximum power savings the encoder
>      needs to be switched off.
>
>      Signed-off-by: Jani Nikula <jani.nikula@intel.com>
>      Reviewed_by: Clinton Taylor <Clinton.A.Taylor@intel.com>
>      Tested_by: Clinton Taylor <Clinton.A.Taylor@intel.com>
>      Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

See https://bugs.freedesktop.org/show_bug.cgi?id=80773 (please ignore
comments up to #10 or so).

Please attach dmesg with drm.debug=14 module parameter set, from boot to
suspend/resume and the problem, running the debug patch from comment
#18. Preferrably please attach the dmesg on the bug to keep everything
in one place.

Thanks,
Jani.


-- 
Jani Nikula, Intel Open Source Technology Center

WARNING: multiple messages have this Message-ID (diff)
From: Jani Nikula <jani.nikula@intel.com>
To: "Ortwin Glück" <odi@odi.ch>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"intel-gfx@lists.freedesktop.org"
	<intel-gfx@lists.freedesktop.org>
Subject: Re: [drm] 3.18 regression: backlight dark after resume
Date: Mon, 03 Nov 2014 12:34:15 +0200	[thread overview]
Message-ID: <87wq7ch8t4.fsf@intel.com> (raw)
In-Reply-To: <5454E2F3.2060404@odi.ch>

On Sat, 01 Nov 2014, Ortwin Glück <odi@odi.ch> wrote:
> Hi,
>
> Since 3.18-rc1 backlight is dark after resume from suspend. This is on a Samsung 
> laptop with i5 IvyBridge
>
> 00:02.0 VGA compatible controller: Intel Corporation 3rd Gen Core processor 
> Graphics Controller (rev 09)
>
>
> I have bisected this problem to the following commit:
> 73580fb764c4213d305c0d36bd8f856ae631eb42 is the first bad commit
> commit 73580fb764c4213d305c0d36bd8f856ae631eb42
> Author: Jani Nikula <jani.nikula@intel.com>
> Date:   Tue Aug 12 17:11:41 2014 +0300
>
>      drm/i915/dp: make backlight bl_power control power sequencer backlight
>
>      This lets the userspace switch off the backlight using the backlight
>      class sysfs bl_power file. The switch is done using the power sequencer;
>      the backlight PWM, and everything else, remains enabled. The display
>      backlight won't draw power, but for maximum power savings the encoder
>      needs to be switched off.
>
>      Signed-off-by: Jani Nikula <jani.nikula@intel.com>
>      Reviewed_by: Clinton Taylor <Clinton.A.Taylor@intel.com>
>      Tested_by: Clinton Taylor <Clinton.A.Taylor@intel.com>
>      Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

See https://bugs.freedesktop.org/show_bug.cgi?id=80773 (please ignore
comments up to #10 or so).

Please attach dmesg with drm.debug=14 module parameter set, from boot to
suspend/resume and the problem, running the debug patch from comment
#18. Preferrably please attach the dmesg on the bug to keep everything
in one place.

Thanks,
Jani.


-- 
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

  reply	other threads:[~2014-11-03 10:34 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-11-01 13:41 [drm] 3.18 regression: backlight dark after resume Ortwin Glück
2014-11-03 10:34 ` Jani Nikula [this message]
2014-11-03 10:34   ` Jani Nikula

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87wq7ch8t4.fsf@intel.com \
    --to=jani.nikula@intel.com \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=odi@odi.ch \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.