[Intel-gfx] [gpu-tools 1/7] testdisplay: fix compiler warnings

From: Paulo Zanoni testdisplay.c:117:1: warning: comparison between signed and unsigned integer expressions testdisplay.c:125:1: warning: comparison between signed and unsigned integer expressions testdisplay.c:145:1: warning: comparison between signed and unsigned integer expressions testdisplay

[Intel-gfx] [gpu-tools 2/7] Add missing files to .gitignore

From: Paulo Zanoni Signed-off-by: Paulo Zanoni --- .gitignore | 11 +++ 1 files changed, 11 insertions(+), 0 deletions(-) diff --git a/.gitignore b/.gitignore index 70736e2..592a1b8 100644 --- a/.gitignore +++ b/.gitignore @@ -34,6 +34,8 @@ tests/getclient tests/getstats tests/get

[Intel-gfx] [gpu-tools 3/7] testdisplay: test all modes if no option is provided

From: Paulo Zanoni Previously, when called without any arguments, the application required user input to finish. However, testdisplay is ran by "make check", and it is not a good idea to run a program that requires user input in "make check". So we change the default behavior to something that d

[Intel-gfx] [gpu-tools 4/7] testdisplay: don't enter the main loop if dump_info or test_all_modes

From: Paulo Zanoni If calls inside update_display fail, the function returns and we don't quit the program if dump_info or test_all_modes. So we enter the main loop and keep waiting for user input, even on cases where we are not supposed to require user input. To fix this, we move the check to ou

[Intel-gfx] [gpu-tools 5/7] testdisplay: remove wrong return statement

From: Paulo Zanoni It seems that the kernel patches required by the TEST_PLANES feature are still not upstream, so I didn't test this patch. Signed-off-by: Paulo Zanoni --- tests/testdisplay.c |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/tests/testdisplay.c b/tests/

[Intel-gfx] [gpu-tools 6/7] drm_open_any: explain why we are abort()ing

From: Paulo Zanoni Signed-off-by: Paulo Zanoni --- lib/drmtest.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/lib/drmtest.c b/lib/drmtest.c index 95df93f..7e2c3fe 100644 --- a/lib/drmtest.c +++ b/lib/drmtest.c @@ -64,6 +64,7 @@ int drm_open_any(void)

[Intel-gfx] [gpu-tools 7/7] getstats: remove useless assertion

From: Paulo Zanoni getstats.c:48:2: warning: comparison of unsigned expression >= 0 is always true Signed-off-by: Paulo Zanoni --- tests/getstats.c |2 -- 1 files changed, 0 insertions(+), 2 deletions(-) This patch can be applied to both intel-gpu-tools and mesa/drm. Whoever apply this to

[Intel-gfx] [gpu-tools 6/7 v2] drm_open_any: explain why we are abort()ing

From: Paulo Zanoni Signed-off-by: Paulo Zanoni --- lib/drmtest.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/lib/drmtest.c b/lib/drmtest.c index 95df93f..5d5d180 100644 --- a/lib/drmtest.c +++ b/lib/drmtest.c @@ -64,6 +64,7 @@ int drm_open_any(void)

[Intel-gfx] [PATCH 1/2] Fix "always false" conditionals

From: Paulo Zanoni Enums are unsigned by default in gcc and we can't rely on any specific signedess for the other compilers. i965_render.c: In function ‘i965_prepare_composite’: i965_render.c:2018:2: warning: comparison of unsigned expression < 0 is always false i965_render.c:2025:2: warning: c

[Intel-gfx] [PATCH 2/2] Remove useless assertion

From: Paulo Zanoni Removes 17 instances of: warning: comparison of unsigned expression >= 0 is always true Signed-off-by: Paulo Zanoni --- src/xvmc/intel_batchbuffer.h |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/src/xvmc/intel_batchbuffer.h b/src/xvmc/intel_batchbu

[Intel-gfx] [PATCH] drm/i915: add missing "break"

From: Paulo Zanoni It seems to be missing from this commit: "drm/i915: split out PCH refclk update code" Signed-off-by: Paulo Zanoni --- drivers/gpu/drm/i915/intel_display.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_display.c b/drive

[Intel-gfx] [PATCH] drm/i915: set the right SDVO transcoder for CPT

From: Paulo Zanoni Fixes fd.o #41272 Signed-off-by: Paulo Zanoni --- drivers/gpu/drm/i915/intel_sdvo.c | 10 +- 1 files changed, 9 insertions(+), 1 deletions(-) It would be nice to have this in Kernel 3.1 too diff --git a/drivers/gpu/drm/i915/intel_sdvo.c b/drivers/gpu/drm/i915/in

[Intel-gfx] [PATCH] drm/i915: set the right SDVO transcoder for CPT

From: Paulo Zanoni Fixes fd.o #41272 v2: add a CPT-specific macro, make code cleaner Signed-off-by: Paulo Zanoni --- drivers/gpu/drm/i915/i915_reg.h |8 drivers/gpu/drm/i915/intel_sdvo.c |8 ++-- 2 files changed, 10 insertions(+), 6 deletions(-) diff --git a/drivers/gp

[Intel-gfx] [PATCH 1/3] drm/i915: set the right SDVO transcoder for CPT

From: Paulo Zanoni v2: add a CPT-specific macro, make code cleaner v3: fix commit message Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=41272 Cc: sta...@kernel.org Signed-off-by: Paulo Zanoni --- drivers/gpu/drm/i915/i915_reg.h |8 drivers/gpu/drm/i915/intel_sdvo.c |

[Intel-gfx] [PATCH 2/3] drm/i915: Rename HDMI register field definitions

From: Paulo Zanoni Also making the differences between HDMI{B,C,D} and SDVO registers for non-PCH, IBX and CPT more explicit. Signed-off-by: Paulo Zanoni --- drivers/gpu/drm/i915/i915_reg.h | 49 -- drivers/gpu/drm/i915/intel_display.c | 16 ++-

[Intel-gfx] [PATCH 3/3] drm/i915: add PCH info to i915_capabilities

From: Paulo Zanoni Suggested-by: Chris Wilson Signed-off-by: Paulo Zanoni --- drivers/gpu/drm/i915/i915_debugfs.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) Chris actually wrote the patch, not just suggested it... diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gp

[Intel-gfx] [RFC] drm/i915: always set positive sync in the sdvo register

From: Paulo Zanoni We use struct intel_sdvo_dtd for that too. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=15766 Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=42174 Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=4 Signed-off-by: Paulo Zanoni --- drivers/gpu/drm/

[Intel-gfx] [PATCH] drm/i915/sdvo: always set positive sync polarity

From: Paulo Zanoni This is a revert of 81a14b46846fea0741902e8d8dfcc6c6c78154c8. We already set the mode polarity using the SDVO commands with struct intel_sdvo_dtd. We have at least 3 bugs that get fixed with this patch. The documentation, despite not clear, can also be interpreted in a way tha

[Intel-gfx] [RFC] drm: implement DRM_IOCTL_MODE_SETROTATION

From: Paulo Zanoni This ioctl is used to signal the drivers that the screen is rotated, not to make the drivers rotate the screen. - add a driver-specific "rotation_set" function - implement Intel's rotation_set by setting the right values to the PIPECONF registers. The idea is that when us

[Intel-gfx] [PATCH] drm/i915/sdvo: always set positive sync polarity

From: Paulo Zanoni This is a revert of 81a14b46846fea0741902e8d8dfcc6c6c78154c8. We already set the mode polarity using the SDVO commands with struct intel_sdvo_dtd. We have at least 3 bugs that get fixed with this patch. The documentation, despite not clear, can also be interpreted in a way tha

[Intel-gfx] [PATCH] Add tools/intel_infoframes

From: Paulo Zanoni This is a command-line tool that allows us to display and modify the InfoFrames we send. Signed-off-by: Paulo Zanoni --- Hi Before we add Kernel interfaces to deal with the infoframes, I decided to write this tool. It allows us to quickly test everything without requiring a