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

2012-01-09 Thread przanoni
From: Paulo Zanoni paulo.r.zan...@intel.com This is a command-line tool that allows us to display and modify the InfoFrames we send. Signed-off-by: Paulo Zanoni paulo.r.zan...@intel.com --- Hi Before we add Kernel interfaces to deal with the infoframes, I decided to write this tool. It allows

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

2012-01-06 Thread przanoni
From: Paulo Zanoni paulo.r.zan...@intel.com 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

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

2012-01-05 Thread przanoni
From: Paulo Zanoni paulo.r.zan...@intel.com 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.

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

2011-12-19 Thread przanoni
From: Paulo Zanoni paulo.r.zan...@intel.com 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

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

2011-12-08 Thread przanoni
From: Paulo Zanoni paulo.r.zan...@intel.com 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

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

2011-10-14 Thread przanoni
From: Paulo Zanoni paulo.r.zan...@intel.com 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 paulo.r.zan...@intel.com --- drivers/gpu/drm/i915/i915_reg.h |8

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

2011-10-14 Thread przanoni
From: Paulo Zanoni paulo.r.zan...@intel.com 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 paulo.r.zan...@intel.com --- drivers/gpu/drm/i915/i915_reg.h | 49 --

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

2011-10-13 Thread przanoni
From: Paulo Zanoni paulo.r.zan...@intel.com Fixes fd.o #41272 Signed-off-by: Paulo Zanoni paulo.r.zan...@intel.com --- 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

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

2011-09-22 Thread przanoni
From: Paulo Zanoni paulo.r.zan...@intel.com It seems to be missing from this commit: drm/i915: split out PCH refclk update code Signed-off-by: Paulo Zanoni paulo.r.zan...@intel.com --- drivers/gpu/drm/i915/intel_display.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git

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

2011-09-16 Thread przanoni
From: Paulo Zanoni paulo.r.zan...@intel.com 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

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

2011-09-16 Thread przanoni
From: Paulo Zanoni paulo.r.zan...@intel.com Removes 17 instances of: warning: comparison of unsigned expression = 0 is always true Signed-off-by: Paulo Zanoni paulo.r.zan...@intel.com --- src/xvmc/intel_batchbuffer.h |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git

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

2011-09-15 Thread przanoni
From: Paulo Zanoni paulo.r.zan...@intel.com 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

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

2011-09-15 Thread przanoni
From: Paulo Zanoni paulo.r.zan...@intel.com Signed-off-by: Paulo Zanoni paulo.r.zan...@intel.com --- .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

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

2011-09-15 Thread przanoni
From: Paulo Zanoni paulo.r.zan...@intel.com 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

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

2011-09-15 Thread przanoni
From: Paulo Zanoni paulo.r.zan...@intel.com 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

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

2011-09-15 Thread przanoni
From: Paulo Zanoni paulo.r.zan...@intel.com 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 paulo.r.zan...@intel.com --- tests/testdisplay.c |1 - 1 files changed, 0 insertions(+), 1

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

2011-09-15 Thread przanoni
From: Paulo Zanoni paulo.r.zan...@intel.com Signed-off-by: Paulo Zanoni paulo.r.zan...@intel.com --- 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 @@

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

2011-09-15 Thread przanoni
From: Paulo Zanoni paulo.r.zan...@intel.com getstats.c:48:2: warning: comparison of unsigned expression = 0 is always true Signed-off-by: Paulo Zanoni paulo.r.zan...@intel.com --- tests/getstats.c |2 -- 1 files changed, 0 insertions(+), 2 deletions(-) This patch can be applied to both

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

2011-09-15 Thread przanoni
From: Paulo Zanoni paulo.r.zan...@intel.com Signed-off-by: Paulo Zanoni paulo.r.zan...@intel.com --- 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 @@