From 7408b8cc24c57230ef69243df6a0e38a1679f96e Mon Sep 17 00:00:00 2001
From: czeidler <haiku@clemens-zeidler.de>
Date: Thu, 2 Feb 2012 10:52:39 +1300
Subject: [PATCH] Add workaround for #8001. * check known mode list first
before sanitizing the target mode. However the optimal way is to fix the
constraints in sanitize_display_mode.
---
.../accelerants/intel_extreme/mode.cpp | 19 +++++++++++++++++++
1 file changed, 19 insertions(+)
diff --git a/src/add-ons/accelerants/intel_extreme/mode.cpp b/src/add-ons/accelerants/intel_extreme/mode.cpp
index fdbcfe5d71..eacf6a35a4 100644
--- a/src/add-ons/accelerants/intel_extreme/mode.cpp
+++ b/src/add-ons/accelerants/intel_extreme/mode.cpp
@@ -645,6 +645,25 @@ intel_propose_display_mode(display_mode* target, const display_mode* low,
{
TRACE(("intel_propose_display_mode()\n"));
+ // first search for the specified mode in the list, if no mode is found
+ // try to fix the target mode in sanitize_display_mode
+ // TODO: Only sanitize_display_mode should be used. However, at the moments
+ // the mode constraints are not optimal and do not work for all
+ // configurations.
+ for (uint32 i = 0; i < gInfo->shared_info->mode_count; i++) {
+ display_mode *mode = &gInfo->mode_list[i];
+
+ // TODO: improve this, ie. adapt pixel clock to allowed values!!!
+
+ if (target->virtual_width != mode->virtual_width
+ || target->virtual_height != mode->virtual_height
+ || target->space != mode->space)
+ continue;
+
+ *target = *mode;
+ return B_OK;
+ }
+
sanitize_display_mode(*target);
return is_display_mode_within_bounds(*target, *low, *high)