From: "Gustavo F. Padovan" <padovan@profusion.mobi>
To: Chris Wilson <chris@chris-wilson.co.uk>
Cc: dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org
Subject: Re: Wavering Arrandale output
Date: Fri, 24 Jun 2011 17:38:52 -0300 [thread overview]
Message-ID: <20110624203851.GA22619@joana> (raw)
In-Reply-To: <013811$1gi55@fmsmga002.fm.intel.com>
* Chris Wilson <chris@chris-wilson.co.uk> [2011-05-14 08:55:15 +0100]:
> On Fri, 13 May 2011 17:45:11 -0300, "Gustavo F. Padovan" <padovan@profusion.mobi> wrote:
> > Hi Chris,
> >
> > Bug #28306 is going to complete his first birthday at the end of this month,
> > it is nagging around for a long time and I have many interest in fix this.
> >
> > So I tested your patch in duplicated bug #36599 and it doesn't work, but I
> > managed to add some printks(see diff) there to help debug.
> >
> > [ 1.033866] [drm:intel_crtc_mode_set] *ERROR* 1. use_ssc 1
> > [ 1.033868] [drm:intel_crtc_mode_set] *ERROR* encoder->type 4
> > [ 1.033869] [drm:intel_crtc_mode_set] *ERROR* 2. use_ssc 1
> > [ 1.034120] [drm:intel_crtc_mode_set] *ERROR* is_lvds 1
> > [ 1.034121] [drm:intel_crtc_mode_set] *ERROR* has_edp_encoder
> > (null)
> > [ 1.191465] [drm:intel_crtc_mode_set] *ERROR* 1. use_ssc 1
> > [ 1.191466] [drm:intel_crtc_mode_set] *ERROR* encoder->type 4
> > [ 1.191468] [drm:intel_crtc_mode_set] *ERROR* encoder->type 1
> > [ 1.191469] [drm:intel_crtc_mode_set] *ERROR* 2. use_ssc 0
> > [ 1.191720] [drm:intel_crtc_mode_set] *ERROR* is_lvds 0
> > [ 1.191721] [drm:intel_crtc_mode_set] *ERROR* has_edp_encoder
> > (null)
> >
> > It seems your patch has no effect once has_edp_encoder is null. Can you please
> > look into this? I can provide any test you may want, just send me the patches.
> > ;)
>
> With no eDP on the system, we wouldn't expect to apply any of the eDP
> settings.
>
> Even after your fix to the patch, nothing? Can you try just setting
> use_ssc=0 (and keep your fix).
>
> In 633f2ea26665d37bb3c8ae30799aa14988622653 (drm/i915: Disable SSC for
> outputs other than LVDS or DP) I tried to implement another aspect of
> source-clock control, that is the patch that appeared to work if you only
> had VGA connected. So the next step would be to combine the two patches.
No fixes for this yet. There are many devices out there with this bug.
Can someone please take a look on this? Just send me patches, I can test them.
Gustavo
prev parent reply other threads:[~2011-06-24 20:38 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-05-13 20:45 Wavering Arrandale output Gustavo F. Padovan
2011-05-14 7:55 ` Chris Wilson
2011-05-14 19:31 ` Gustavo F. Padovan
2011-06-24 20:38 ` Gustavo F. Padovan [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=20110624203851.GA22619@joana \
--to=padovan@profusion.mobi \
--cc=chris@chris-wilson.co.uk \
--cc=dri-devel@lists.freedesktop.org \
--cc=linux-kernel@vger.kernel.org \
/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.