From: Daniel Vetter <daniel@ffwll.ch>
To: Paulo Zanoni <przanoni@gmail.com>
Cc: Takashi Iwai <tiwai@suse.de>,
Mengdong Lin <mengdong.lin@intel.com>,
intel-gfx@lists.freedesktop.org,
Paulo Zanoni <paulo.r.zanoni@intel.com>
Subject: Re: [PATCH 1/7] drm/i915: don't disable the power well yet
Date: Sat, 23 Mar 2013 12:34:54 +0100 [thread overview]
Message-ID: <20130323113454.GW9021@phenom.ffwll.local> (raw)
In-Reply-To: <1363972043-2602-1-git-send-email-przanoni@gmail.com>
On Fri, Mar 22, 2013 at 02:07:23PM -0300, Paulo Zanoni wrote:
> From: Paulo Zanoni <paulo.r.zanoni@intel.com>
>
> We're still not 100% ready to disable the power well, so don't disable
> it for now. When we disable it we break the audio driver (because some
> of the audio registers are on the power well) and machines with eDP on
> port D (because it doesn't use TRANSCODER_EDP).
>
> Also, instead of just reverting the code, add a Kernel option to let
> us disable it if we want. This will allow us to keep developing and
> testing the feature while it's not enabled.
>
> This fixes problems caused by the following commit:
> commit d6dd9eb1d96d2b7345fe4664066c2b7ed86da898
> Author: Daniel Vetter <daniel.vetter@ffwll.ch>
> Date: Tue Jan 29 16:35:20 2013 -0200
> drm/i915: dynamic Haswell display power well support
>
> References: http://www.mail-archive.com/intel-gfx@lists.freedesktop.org/msg18788.html
> Cc: Takashi Iwai <tiwai@suse.de>
> Cc: Mengdong Lin <mengdong.lin@intel.com>
> Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
Picked up for -fixes, thanks for the patch.
-Daniel
--
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
prev parent reply other threads:[~2013-03-23 11:32 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-03-22 9:53 [PATCH 1/3] Revert "drm/i915: dynamic Haswell display power well support" Daniel Vetter
2013-03-22 9:53 ` [PATCH 2/3] Revert "drm/i915: set TRANSCODER_EDP even earlier" Daniel Vetter
2013-03-22 12:30 ` Paulo Zanoni
2013-03-22 13:59 ` Daniel Vetter
2013-03-22 13:43 ` Daniel Vetter
2013-03-22 9:53 ` [PATCH 3/3] drm/i915: Revert backlight cargo-culting Daniel Vetter
2013-03-22 10:00 ` Takashi Iwai
2013-03-22 10:13 ` Daniel Vetter
2013-03-22 10:22 ` Daniel Vetter
2013-03-22 10:42 ` Takashi Iwai
2013-03-22 11:22 ` Daniel Vetter
2013-03-22 11:34 ` Takashi Iwai
2013-03-22 14:01 ` Daniel Vetter
2013-03-22 14:44 ` [PATCH] Revert "drm/i915: write backlight harder" Daniel Vetter
2013-03-22 15:06 ` Takashi Iwai
2013-03-23 11:36 ` Daniel Vetter
2013-03-22 14:45 ` [PATCH 1/7] drm/i915: don't disable the power well yet Paulo Zanoni
2013-03-22 15:22 ` Daniel Vetter
2013-03-22 17:07 ` Paulo Zanoni
2013-03-23 11:34 ` Daniel Vetter [this message]
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=20130323113454.GW9021@phenom.ffwll.local \
--to=daniel@ffwll.ch \
--cc=intel-gfx@lists.freedesktop.org \
--cc=mengdong.lin@intel.com \
--cc=paulo.r.zanoni@intel.com \
--cc=przanoni@gmail.com \
--cc=tiwai@suse.de \
/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.