[Intel-gfx] [PATCH i-g-t 1/3] tests/kms_force_connector: Fixes

Daniel Vetter daniel.vetter at ffwll.ch
Tue Dec 1 02:24:18 PST 2015


Somehow the kernel's mode list changed with our EDID. No idea
whether that's the right thing here since I'm not really an EDID
expert. But then again the testcase wants to check that the
injection works, not validate the kernel's parser.

v2: Only check the preferred mode for more future-proofing (Thomas).

Cc: Thomas Wood <thomas.wood at intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter at intel.com>
---
 tests/kms_force_connector.c | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/tests/kms_force_connector.c b/tests/kms_force_connector.c
index 838a3b0ae2e6..99a5649ec0c3 100644
--- a/tests/kms_force_connector.c
+++ b/tests/kms_force_connector.c
@@ -150,10 +150,8 @@ int main(int argc, char **argv)
 					   vga_connector->connector_id);
 
 		CHECK_MODE(temp->modes[0], 1920, 1080, 60);
-		CHECK_MODE(temp->modes[1], 1280, 720, 60);
-		CHECK_MODE(temp->modes[2], 1024, 768, 60);
-		CHECK_MODE(temp->modes[3], 800, 600, 60);
-		CHECK_MODE(temp->modes[4], 640, 480, 60);
+		/* Don't check non-preferred modes to avoid to tight coupling
+		 * with the in-kernel EDID parser. */
 
 		drmModeFreeConnector(temp);
 
-- 
2.1.0



More information about the Intel-gfx mailing list