From: Frank Huang <[email protected]>

*mode validation(lx_output_mode_valid) in geode driver should return
MODE_OK for all modes filtered out by previous process in this function.
Otherwise, new modelines(conf_modes) will be pruned by the Xserver function
Xf86PruneInvalidModes. The result is that the user can not set any resolution
they want. We comply with the code of ATI&&Intel mode_valid function to do this.
*For the mode that can not be supported in geode driver, it is better to give
the specific MODE_XXX(such as MODE_CLOCK_RANGE) instead of MODE_BAD.

Signed-off-by: Frank Huang <[email protected]>
---
 src/lx_output.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/src/lx_output.c b/src/lx_output.c
index 413d376..aa7687c 100644
--- a/src/lx_output.c
+++ b/src/lx_output.c
@@ -182,7 +182,7 @@ lx_output_mode_valid(xf86OutputPtr output, DisplayModePtr 
pMode)
     if (pMode->type & (M_T_DRIVER | M_T_PREFERRED))
         return MODE_OK;
 
-    return MODE_BAD;
+    return MODE_OK;
 }
 
 static Bool
-- 
1.7.1


_______________________________________________
Xorg-driver-geode mailing list
[email protected]
http://lists.x.org/mailman/listinfo/xorg-driver-geode

Reply via email to