public inbox for intel-gfx@lists.freedesktop.org
 help / color / mirror / Atom feed
From: Daniel Vetter <daniel@ffwll.ch>
To: Thomas Richter <thor@math.tu-berlin.de>
Cc: intel-gfx <intel-gfx@lists.freedesktop.org>
Subject: Re: Questions on display pipes on 835GM
Date: Sat, 5 Oct 2013 21:39:04 +0200	[thread overview]
Message-ID: <20131005193904.GX31334@phenom.ffwll.local> (raw)
In-Reply-To: <CAKMK7uG_QMajktV-6TEPR+ci3KjEP8Mq8=-w-fYJe=JyoqY21A@mail.gmail.com>

Hi Thomas,

btw I've just read through your dvo code again and I think we can fix this
easier. If I read your enable hack correctly then we need to have the dpll
running and the DVO port on. The problem now is that in the dvo ->modeset
callback this is explicitly _not_ the case.

This is not a big issue when there's only one pipe in use, but it wreaks
havoc when more than one pipe is in use. So we need to move all that code
somewhere else.

Now if you follow the callchains around the dvo->dpms callbacks the DVO
port and DPLL are always enabled at that point in time, so I think we
should be able to fix this all by moving the modeset code around to that
place.

Let me work a bit on a patch and see whether it helps my X30, too.

Cheers, Daniel
On Sat, Oct 5, 2013 at 9:24 PM, Daniel Vetter <daniel@ffwll.ch> wrote:
> On Sat, Oct 5, 2013 at 6:47 PM, Thomas Richter <thor@math.tu-berlin.de> wrote:
>>
>>> The CRT port can only be driven by pipe A, so I think that explains why
>>> your hacks may go bad when a CRT monitor is used.
>>
>>
>> Understood, this makes sense. Thus, DVO would then be at pipe B. How is this
>> switch made, and where? There need to be a register somewhere that tells the
>> i835 which signal goes where.
>
> If the intel_connector->base.encoder pointer is non-NULL, then you can
> get at the pipe throught
> to_intel_crtc(intel_connector->base.encoder->crtc)->pipe. If this
> pointer doesn't exist then the dvo chip shouldn't be in use at the
> moment.
>
> For intel_encoder you can chase
> to_intel_crtc(intel_encoder->base.crtc)->pipe, but you first need to
> check the crtc pointer for non-NULL.
>
> btw you can check whether pipe B works without a 2nd display
> potentially wreaking havoc (so should be simpler to debug) with:
>
> xrandr --output LVDS --crtc 1 --auto
>
> Cheers, Daniel
> --
> Daniel Vetter
> Software Engineer, Intel Corporation
> +41 (0) 79 365 57 48 - http://blog.ffwll.ch



-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch

  reply	other threads:[~2013-10-05 19:38 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-08-23 22:51 [PATCH] drm/i915: enable trickle feed on Haswell Paulo Zanoni
2013-08-30 17:25 ` Ville Syrjälä
2013-09-02  6:01   ` Daniel Vetter
     [not found]   ` <29427_1378101775_52242A0E_29427_747_1_20130902060151.GE9374@phenom.ffwll.local>
2013-10-05  8:33     ` Questions on display pipes on 835GM Thomas Richter
2013-10-05 11:44       ` Ville Syrjälä
     [not found]       ` <29761_1380973482_524FFBA9_29761_4042_1_20131005114432.GE9395@intel.com>
2013-10-05 16:47         ` Thomas Richter
2013-10-05 19:24           ` Daniel Vetter
2013-10-05 19:39             ` Daniel Vetter [this message]
     [not found]             ` <29761_1381001928_52506AC8_29761_5606_1_20131005193904.GX31334@phenom.ffwll.local>
2013-10-05 23:09               ` Thomas Richter
2013-10-05 23:37                 ` Daniel Vetter
     [not found]                 ` <5132_1381149082_5252A99A_5132_80_1_CAKMK7uHkOJ=urFKCacocf_i+6YL9Azf5vK=Au-kRJMi=sNBP1Q@mail.gmail.com>
2013-10-07 21:58                   ` Broken in 3.10.10 (was: Questions on display pipes on 835GM) Thomas Richter
2013-10-08  7:24                     ` Daniel Vetter
2013-10-08  8:39                       ` Daniel Vetter
2013-10-08  9:14                         ` Chris Wilson
     [not found]                       ` <26192_1381221546_5253C4A9_26192_7222_1_CAKMK7uGvk9vuNXunutyHjkGjvBWsv7ObN02EfbG2679MtQ5M=A@mail.gmail.com>
2013-10-08 17:06                         ` Broken in 3.10.10 Thomas Richter
2013-10-05 23:04         ` [PATCH] 835GM DLL setup Thomas Richter
2013-10-05 23:46           ` Daniel Vetter

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=20131005193904.GX31334@phenom.ffwll.local \
    --to=daniel@ffwll.ch \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=thor@math.tu-berlin.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