public inbox for intel-gfx@lists.freedesktop.org
 help / color / mirror / Atom feed
From: Daniel Vetter <daniel@ffwll.ch>
To: Thomas Richter <richter@rus.uni-stuttgart.de>
Cc: intel-gfx <intel-gfx@lists.freedesktop.org>
Subject: Re: Breaking suspend/resume by the Pipe A quirk
Date: Mon, 2 Jun 2014 17:27:02 +0200	[thread overview]
Message-ID: <20140602152702.GU19050@phenom.ffwll.local> (raw)
In-Reply-To: <538C54E0.8090507@rus.uni-stuttgart.de>

On Mon, Jun 02, 2014 at 12:41:36PM +0200, Thomas Richter wrote:
> Am 02.06.2014 10:27, schrieb Daniel Vetter:
> >
> >
> >Can you go right ahead and please submit this as a patch?
> 
> Certainly, but I would prefer to get more information on this. Even though
> the R31 *also* works without the pipe A quirk, I am not sure it does work on
> all other hardware configurations.
> 
> There is, however, an important difference between the R31 and the S6010:
> The R31 uses two independent display pipes for the generating the display,
> LVDS for the internal and VGA for the external display. As a result, frame
> rates and resolutions can be different between the two outputs.
> 
> The S6010, however, seems to use a single pipe design, with the internal
> display connected via DVI (not LVDS!) and the external by VGA. This has the
> unfortunate side effect that I cannot set the resolutions of internal and
> external display independently. Any attempt to modify the external
> resolution while using the internal screen results in an "no crtc found for
> output VGA1" when using xrandr. (Not quite sure what this means, but I
> believe that the VGA output is simply a duplicate of the DVI output, and the
> two are probably connected through a bios-switchable bridge chip).
> 
> Thus, I would *prefer* to be conservative and only disable the pipe_A quirk
> only in situations where there is a single display pipe (as in the S6010)
> and, just to be on the safe side, keep it enabled in dual-pipe (as in R31)
> configurations.

We've put a crtc restriction on VGA (it needs to be crtc 0) to work around
some issues. DVI/LVDS should work on crtc 1. You can set this with the
--crtc knob for xrandr.

> Now I wonder how I could possibly distinguish between the two. Could you
> please provide some pointers?

You're probably the last real user of this hw left. You're needs win,
especially if you know that it fixes stuff on other platforms, too.

So holesale removal of the pipe quirk for i830M seems like the right thing
to do here. Especially since Chris also complained that it makes stuff
worse for his i845.
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch

  reply	other threads:[~2014-06-02 15:27 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-05-16 14:02 830GM still woes Thomas Richter
2014-05-16 14:41 ` Chris Wilson
2014-05-16 15:09   ` Daniel Vetter
2014-05-16 16:04     ` Ville Syrjälä
2014-05-16 16:50       ` Daniel Vetter
     [not found]       ` <23914_1400259040_537641E0_23914_9298_1_20140516165034.GT8790@phenom.ffwll.local>
2014-05-17 10:58         ` Thomas Richter
2014-05-29 14:10         ` Thomas Richter
2014-05-29 21:19         ` Breaking suspend/resume by the Pipe A quirk Thomas Richter
2014-06-02  8:27           ` Daniel Vetter
2014-06-02 10:34             ` [PATCH] Align i830 watermark to cache lines Thomas Richter
2014-06-02 10:41             ` Breaking suspend/resume by the Pipe A quirk Thomas Richter
2014-06-02 15:27               ` Daniel Vetter [this message]
2014-06-02 15:38                 ` [Patch] Disabling the pipe A quirk for the Fujitsu S6010 Thomas Richter
2014-06-02 15:56                   ` Daniel Vetter
2014-06-02 16:52                     ` Thomas Richter
2014-06-02 17:39                       ` Daniel Vetter
2014-06-02 18:44                         ` Thomas Richter
     [not found]               ` <1027_1401722832_538C97D0_1027_15897_1_20140602152702.GU19050@phenom.ffwll.local>
2014-06-03 14:38                 ` Breaking suspend/resume by the Pipe A quirk Thomas Richter
2014-06-03 14:45                   ` Daniel Vetter
2014-06-03 15:04                     ` Thomas Richter
2014-06-03 15:14                       ` Chris Wilson
2014-06-03 15:19                         ` Thomas Richter
2014-06-03 15:26                           ` Chris Wilson
2014-06-03 15:50                             ` Thomas Richter
2014-06-03 16:03                               ` Chris Wilson
     [not found]                               ` <17205_1401811442_538DF1F2_17205_16362_1_20140603160352.GA6129@nuc-i3427.alporthouse.com>
2014-06-04 22:43                                 ` Bug: Pipe A underrun on resolution switching on 830MG Thomas Richter
2014-06-08 21:20                                 ` Broken suspend/resume on i830 (with debug information) Thomas Richter

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=20140602152702.GU19050@phenom.ffwll.local \
    --to=daniel@ffwll.ch \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=richter@rus.uni-stuttgart.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox