[haiku-commits] haiku: hrev44341 - src/add-ons/accelerants/radeon_hd

  • From: kallisti5@xxxxxxxxxxx
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Sun, 15 Jul 2012 19:02:56 +0200 (CEST)

hrev44341 adds 1 changeset to branch 'master'
old head: 6d51de482c740d2cc722ea6a39f0c29c6a6fb64e
new head: e2339980052673d16a2457483ff7b92e8c4fd179

----------------------------------------------------------------------------

e233998: radeon_hd: Skip 9DIN connector.
  
  * Since we really don't support multiple
    heads well, skip 9DIN for the moment as
    it is a luxury.

                          [ Alexander von Gluck IV <kallisti5@xxxxxxxxxxx> ]

----------------------------------------------------------------------------

Revision:    hrev44341
Commit:      e2339980052673d16a2457483ff7b92e8c4fd179
URL:         http://cgit.haiku-os.org/haiku/commit/?id=e233998
Author:      Alexander von Gluck IV <kallisti5@xxxxxxxxxxx>
Date:        Sun Jul 15 16:58:19 2012 UTC

----------------------------------------------------------------------------

1 file changed, 11 insertions(+), 3 deletions(-)
src/add-ons/accelerants/radeon_hd/display.cpp |   14 +++++++++++---

----------------------------------------------------------------------------

diff --git a/src/add-ons/accelerants/radeon_hd/display.cpp 
b/src/add-ons/accelerants/radeon_hd/display.cpp
index ecd15a4..47e8a32 100644
--- a/src/add-ons/accelerants/radeon_hd/display.cpp
+++ b/src/add-ons/accelerants/radeon_hd/display.cpp
@@ -257,6 +257,12 @@ detect_displays()
                if (displayIndex >= MAX_DISPLAY)
                        continue;
 
+               if (gConnector[id]->type == VIDEO_CONNECTOR_9DIN) {
+                       TRACE("%s: Skipping 9DIN connector (not yet 
supported)\n",
+                               __func__);
+                       continue;
+               }
+
                // TODO: As DP aux transactions don't work yet, just use LVDS 
as a hack
                #if 0
                if (gConnector[id]->encoderExternal.isDPBridge == true) {
@@ -310,9 +316,11 @@ detect_displays()
                                        TRACE("%s: connector %" B_PRIu32 " has 
digital EDID "
                                                "and is not a analog 
encoder.\n", __func__, id);
                                } else {
-                                       // ???, shouldn't happen... I think.
-                                       TRACE("%s: Warning: connector %" 
B_PRIu32 " has neither "
-                                               "digital EDID nor is an analog 
encoder?\n",
+                                       // This generally means the monitor is 
of poor design
+                                       // Since we *know* there is no load on 
the analog encoder
+                                       // we assume that it is a digital 
display.
+                                       TRACE("%s: Warning: monitor on 
connector %" B_PRIu32 " has "
+                                               "false digital EDID flag and 
unloaded analog encoder!\n",
                                                __func__, id);
                                }
                        }


Other related posts:

  • » [haiku-commits] haiku: hrev44341 - src/add-ons/accelerants/radeon_hd - kallisti5