mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Linus Torvalds <torvalds@linux-foundation.org>
To: Dave Airlie <airlied@gmail.com>
Cc: Zhenyu Wang <zhenyuw@linux.intel.com>,
	Eric Anholt <eric@anholt.net>, mailing54 <mailing54@plzk.org>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	Dave Airlie <airlied@redhat.com>,
	dri-devel@lists.sourceforge.net, Ma Ling <ling.ma@intel.com>,
	Jesse Barnes <jbarnes@virtuousgeek.org>,
	"Zhao, Yakui" <yakui.zhao@intel.com>
Subject: Re: Linux 2.6.31-rc7
Date: Wed, 26 Aug 2009 10:12:14 -0700 (PDT)	[thread overview]
Message-ID: <alpine.LFD.2.01.0908260943590.3218@localhost.localdomain> (raw)
In-Reply-To: <21d7e9970908252158q30d7e4fay587458b79cd0c36d@mail.gmail.com>



On Wed, 26 Aug 2009, Dave Airlie wrote:
> 
> 3. Here's the thing, we do detect something has changed, we detect we no
> longer have a monitor connected on the mac mini because the routing
> for the DDC pins is insane and need special treatment in the driver.

That's the second thing - DDC in general seems to be _very_ flaky on intel 
chips.

And I don't mean that in a "Mac Mini" kind of sense. I'm seeing in on 
other machines. Here's my old EeePC 701:

	[drm] Initialized drm 1.1.0 20060810
	i915 0000:00:02.0: PCI INT A -> GSI 16 (level, low) -> IRQ 16
	i915 0000:00:02.0: setting latency timer to 64
	i2c-adapter i2c-1: unable to read EDID block.
	i915 0000:00:02.0: LVDS-1: no EDID data
	[drm] DAC-6: set mode 640x480 0
	i2c-adapter i2c-1: unable to read EDID block.
	i915 0000:00:02.0: LVDS-1: no EDID data
	[drm] TV-11: set mode NTSC 480i 0
	allocated 800x480 fb: 0x007df000, bo f735b540
	[drm] LVDS-8: set mode 800x480 15
	Console: switching to colour frame buffer device 100x30

and quite frankly, I suspect you've never ever actually googled for "no 
EDID data", because if you had, you'd have seen that this is quite common.

And yes, the response of KMS seems to be "ok, I failed at EDID, so I'll 
just assume something isn't there". And then usually it picks LVDS 
instead, if it finds a VBT table for it. Which happens to work fine on 
most laptops, but it's still just a random heuristic.

And yes, for all I know it's because the display really doesn't do any 
EDID.

And that really kind of is the point - it doesn't even matter whether the 
problem is the Intel driver doing something wrong on the EDID front (which 
it has done quite often - ranging from just looking at the wrong port, to 
not setting the right bits to make bit-banging i2c work AT ALL), or 
whether the problem is that the hardware is wired up oddly (Mac Mini) or 
whether it's the hardware simply not doign EDID at all (maybe my EeePC? Or 
a really old VGA monitor, or whatever).

In all three cases the end result is "no EDID", but regardless of that, 
the correct action is basically _never_ to say "ok, I'll just assume that 
the display is on connector XYZ regardless of what the state of the 
graphics chip is".

> so yes I think do better at failing is what is needed, its still
> failing but its more user friendly fail.

Yes. If something fails ("oops, I can't seem EDID for any connector"), I 
wish KMS would fail way better than just default to some crazy setup. The 
failure mode should be to at least drive whatever the BIOS enabled.

			Linus

  reply	other threads:[~2009-08-26 17:13 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-08-22  1:26 Linus Torvalds
2009-08-22  3:09 ` Regression: Linux 2.6.31-rc7 lost sensors on asus mobo Gene Heskett
2009-08-22  3:47   ` Linus Torvalds
2009-08-22 12:56     ` Gene Heskett
2009-08-22  6:12   ` Robert Hancock
2009-08-22 10:54     ` Stefan Richter
2009-08-22 13:48       ` Gene Heskett
2009-08-22 14:38         ` Stefan Richter
2009-08-22 19:55           ` Gene Heskett
2009-08-22 13:40     ` Gene Heskett
2009-08-23 10:56 ` Linux 2.6.31-rc7 Geert Uytterhoeven
2009-08-26  5:06   ` KOSAKI Motohiro
2009-08-25 17:25 ` mailing54
2009-08-25 18:11   ` Linus Torvalds
2009-08-25 21:37     ` mailing54
2009-08-25 22:07       ` Linus Torvalds
     [not found]         ` <1251239637.26348.20.camel@gaiman.anholt.net>
2009-08-26  1:51           ` Zhenyu Wang
2009-08-26  3:33             ` Linus Torvalds
2009-08-26  3:47               ` Dave Airlie
2009-08-26  4:13                 ` Linus Torvalds
2009-08-26  4:58                   ` Dave Airlie
2009-08-26 17:12                     ` Linus Torvalds [this message]
2009-08-26 17:18                       ` Jesse Barnes
2009-08-26  6:26                 ` Eric Anholt
2009-08-26  6:35                   ` Dave Airlie
2009-08-26  3:58               ` Zhenyu Wang
2009-08-26  4:20                 ` Linus Torvalds
2009-09-10  5:47                   ` Zhenyu Wang
2009-08-26 10:09                 ` ykzhao
2009-08-30 22:01         ` Tino Keitel

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=alpine.LFD.2.01.0908260943590.3218@localhost.localdomain \
    --to=torvalds@linux-foundation.org \
    --cc=airlied@gmail.com \
    --cc=airlied@redhat.com \
    --cc=dri-devel@lists.sourceforge.net \
    --cc=eric@anholt.net \
    --cc=jbarnes@virtuousgeek.org \
    --cc=ling.ma@intel.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mailing54@plzk.org \
    --cc=yakui.zhao@intel.com \
    --cc=zhenyuw@linux.intel.com \
    /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

Powered by JetHome