Intel-GFX Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Takashi Iwai <tiwai@suse.de>
To: Adam Jackson <ajax@redhat.com>
Cc: mmarek@suse.cz, intel-gfx@lists.freedesktop.org
Subject: Re: Valid DP connection without EDID?
Date: Mon, 17 Sep 2012 10:16:49 +0200	[thread overview]
Message-ID: <s5hd31lysge.wl%tiwai@suse.de> (raw)
In-Reply-To: <50534C4F.6020607@redhat.com>

Hi Adam,

At Fri, 14 Sep 2012 11:25:03 -0400,
Adam Jackson wrote:
> 
> On 9/14/12 10:19 AM, Takashi Iwai wrote:
> > Hi,
> >
> > we've got a machine showing a ghost DP2 output on a docking station.
> > The docking station has only one DP port and it's connected to DP1.
> > As a result, we get an DP2 active output containing the bogus VESA
> > standard modes 1024x768, 800x600, 640x480 although it's not connected
> > at all.
> >
> > Looking a bit deeply on it, it seems that the connector gives actually
> > the valid DPCD.  So intel_dp_detect() returns
> > connector_status_connected.  But since there is no real connection,
> > EDID isn't obtained.  Thus of course no valid modes set.
> 
> Can you be more specific here?  What DPCD does it return?

It shows "DPCD: 110a820100030181"

> Does it claim 
> to be a branch device?  We don't currently get that case right, try for 
> instance plugging in a DP->VGA pigtail.
 
Well, the port doesn't exist physically at all.  The docking station
has only one physical DP output although it exposes two.  (And the
machine has one another built-in DP port as DP3, but it works well.)

So it's definitely a hardware problem.  But we can still work around
it.

> > A quick patch below adds (moves) a check of EDID and returns the
> > disconnected state when no valid EDID is returned.  An open question
> > is whether this is really safe.  Is there any case where no valid EDID
> > is returned but the connection is still valid?
> 
> DisplayPort requires EDID.  There may be a platform method for obtaining 
> it for eDP, but for normal DP it has to exist over AUXCH.
> 
> DVI and HDMI require it too, in fact.  The only one that doesn't is VGA.

OK, so a workaround like my patch (just checking the validity of EDID)
won't bring regressions in theory?


thanks,

Takashi

  reply	other threads:[~2012-09-17  8:16 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-09-14 14:19 Valid DP connection without EDID? Takashi Iwai
2012-09-14 15:25 ` Adam Jackson
2012-09-17  8:16   ` Takashi Iwai [this message]
2012-09-17  8:21     ` Dave Airlie
2012-09-17  8:28       ` Takashi Iwai
2012-09-17  9:11         ` Dave Airlie
2012-09-17  9:32           ` Takashi Iwai
2012-09-17 16:24             ` Adam Jackson
2012-09-18 11:01               ` Takashi Iwai
2012-09-18 14:04                 ` Adam Jackson
2012-09-18 14:32                   ` Takashi Iwai
2012-09-18 14:48                     ` Adam Jackson
2012-09-18 15:11                       ` Takashi Iwai

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=s5hd31lysge.wl%tiwai@suse.de \
    --to=tiwai@suse.de \
    --cc=ajax@redhat.com \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=mmarek@suse.cz \
    /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