From: Robert Foss <robert.f...@collabora.com>
Add support for configure flag --disable-pciaccess and the associated
automake define HAVE_PCIACCESS.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
configure.ac | 13 -
1 file changed, 12 insertions(+), 1 delet
On 2016-05-24 04:05 AM, Daniel Vetter wrote:
On Fri, May 20, 2016 at 06:59:31PM -0400, robert.f...@collabora.com wrote:
From: Robert Foss <robert.f...@collabora.com>
This patch provides stubs for functionality otherwise provided by libdrm_intel.
The stubbed functions all fail with
From: Robert Foss <robert.f...@collabora.com>
Use the HAS_INTEL automake flag to avoid building benchmarks that won't
compile unless libdrm_intel is available in the build system.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
benchmarks/Android.mk | 6 ++
From: Robert Foss <robert.f...@collabora.com>
Use the HAS_INTEL automake flag to avoid building tools that won't
compile unless libdrm_intel is available in the build system.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
demos/Android.mk | 5 -
demos/Makefile
From: Robert Foss <robert.f...@collabora.com>
Don't try to build overlay if libpciaccess is unavailable.
This is useful for platforms that typically don't have pci hardware,
like ARM platforms.
Robert Foss (2):
configure.ac: Add support for configure flag --disable-pciaccess.
o
From: Robert Foss <robert.f...@collabora.com>
Don't try to build overlay if libpciaccess is unavailable.
This is useful for platforms that typically don't have pci hardware, like ARM
platforms.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
overlay/Makefile.am | 5 +
On 2016-05-24 04:01 AM, Daniel Vetter wrote:
On Mon, May 23, 2016 at 03:04:16PM +0100, Emil Velikov wrote:
On Friday, May 20, 2016 23:59 BST, robert.f...@collabora.com wrote:
From: Robert Foss <robert.f...@collabora.com>
Use the HAS_INTEL automake flag to avoid building benc
On 2016-05-24 04:07 AM, Daniel Vetter wrote:
On Fri, May 20, 2016 at 06:59:32PM -0400, robert.f...@collabora.com wrote:
From: Robert Foss <robert.f...@collabora.com>
Replace intel specific header includes with intel_drm_stubs.h.
The stubbed functions will all call igt_require
Forward to ML.
On Wed, May 25, 2016 at 02:43:59PM -0400, robert.f...@collabora.com wrote:
From: Robert Foss <robert.f...@collabora.com>
Don't try to build overlay if libpciaccess is unavailable.
This is useful for platforms that typically don't have pci hardware, like ARM
platforms.
Forward to ML.
On Wednesday, May 25, 2016 19:18 BST, robert.f...@collabora.com wrote:
+if test "x$INTEL" = xyes; then
+ PKG_CHECK_MODULES(DRM_INTEL, [libdrm_intel >= 2.4.64])
+ AC_DEFINE(HAVE_LIBDRM_INTEL, 1, [Have intel support])
+fi
+AM_CONDITIONAL(HAVE_LIBDRM_INTEL, [test
Forward to ML.
On Wednesday, May 25, 2016 19:18 BST, robert.f...@collabora.com wrote:
From: Robert Foss <robert.f...@collabora.com>
This patch provides stubs for functionality otherwise provided by intel_bufmgr.
The stubbed functions all fail with a call to igt_require_f(false,"
On Wednesday, May 25, 2016 19:18 BST, robert.f...@collabora.com wrote:
From: Robert Foss <robert.f...@collabora.com>
Use the HAS_INTEL automake flag to avoid building tools that won't
compile unless libdrm_intel is available in the build system.
Signed-off-by: Robert Foss <
On Wednesday, May 25, 2016 19:18 BST, robert.f...@collabora.com wrote:
From: Robert Foss <robert.f...@collabora.com>
Harmonize tabs/spaces etc.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
tools/Makefile.sources | 57 +-
Forward to ML.
On 2016-05-25 03:55 PM, Emil Velikov wrote:
On Wednesday, May 25, 2016 19:18 BST, robert.f...@collabora.com wrote:
From: Robert Foss <robert.f...@collabora.com>
Use the HAS_INTEL automake flag to avoid building benchmarks that won't
compile unless libdrm_intel is ava
From: Robert Foss <robert.f...@collabora.com>
Use the HAS_INTEL automake flag to avoid building tools that won't
compile unless libdrm_intel is available in the build system.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
tools/Android.mk | 8
tools
Forward to ML.
On Wednesday, May 25, 2016 21:04 BST, Robert Foss
<robert.f...@collabora.com> wrote:
> And this is why ladies and gentlemen you don't align on the \ character. As
one adds a test/file with long name, you'll need to re-indent the whole lot. Then
again... pe
From: Robert Foss <robert.f...@collabora.com>
This patch provides stubs for functionality otherwise provided by intel_bufmgr.
The stubbed functions all fail with a call to igt_require_f(false,"").
Defines and enums have been copied from libdrm_intel.
Due to the stubb
From: Robert Foss <robert.f...@collabora.com>
Harmonize tabs/spaces etc.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
tools/Makefile.sources | 57 +-
1 file changed, 29 insertions(+), 28 deletions(-)
diff -
From: Robert Foss <robert.f...@collabora.com>
Changes since v1:
- Replaced the automake flags HAVE_VC4/NOUVEAU/INTEL with HAVE_LIBDRM_XXX.
- Move conditionals from Makefile.sources to Arduino.mk/Makefile.am.
- Removed duplicated i915_drm.h symbols from intel_drm_stubs.h.
- Replaced igt_r
From: Robert Foss <robert.f...@collabora.com>
Replace the automake flag HAVE_XXX for VC4/NOUVEAU with HAVE_LIBDRM_XXX in
order for the flags to be more descriptive and also follow the same convention
as HAVE_LIBDRM_INTEL.
---
configure.ac | 8
lib/Makefile.am | 2 +-
From: Robert Foss <robert.f...@collabora.com>
Test for libdrm_intel and build for it if present.
Also expose the HAVE_INTEL #define to allow code to be conditionally
compiled.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
configure.ac | 14 +-
1 file
From: Robert Foss <robert.f...@collabora.com>
Use the HAS_INTEL automake flag to avoid building tools that won't
compile unless libdrm_intel is available in the build system.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
tools/Android.mk | 8
tools
From: Robert Foss <robert.f...@collabora.com>
Harmonize tabs/spaces etc.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
tools/Makefile.sources | 57 +-
1 file changed, 29 insertions(+), 28 deletions(-)
diff -
From: Robert Foss <robert.f...@collabora.com>
Replace the automake specific name of listings in Makefile.sources
with something not automake specific.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
demos/Android.mk | 2 +-
demos/Makefile.am | 2 ++
demos/Make
From: Robert Foss <robert.f...@collabora.com>
Test for libdrm_intel and build for it if present.
Also expose the HAVE_INTEL #define to allow code to be conditionally
compiled.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
configure.ac | 17 -
1 file
From: Robert Foss <robert.f...@collabora.com>
Replace the automake specific name of listings in Makefile.sources
with something not automake specific.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
benchmarks/Android.mk | 2 +-
benchmarks/Makefile.am | 2 ++
From: Robert Foss <robert.f...@collabora.com>
Use the HAS_INTEL automake flag to avoid building tools that won't
compile unless libdrm_intel is available in the build system.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
tools/Android.mk | 5 +
tools
From: Robert Foss <robert.f...@collabora.com>
Replace the automake flag HAVE_XXX for VC4/NOUVEAU with HAVE_LIBDRM_XXX in
order for the flags to be more descriptive and also follow the same convention
as HAVE_LIBDRM_INTEL.
---
configure.ac | 8
lib/Makefile.am | 2 +-
From: Robert Foss <robert.f...@collabora.com>
Use the HAS_INTEL automake flag to avoid building tools that won't
compile unless libdrm_intel is available in the build system.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
demos/Android.mk | 5 -
demos/Makefile
From: Robert Foss <robert.f...@collabora.com>
Replace the automake specific names of listings in Makefile.sources with
something not automake specific.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
tools/Android.mk | 3 ++-
tools/Makefile.am |
From: Robert Foss <robert.f...@collabora.com>
This patch provides stubs for functionality otherwise provided by intel_bufmgr.
The stubbed functions all fail with a call to igt_require_f(false,"").
Defines and enums have been copied from libdrm_intel.
Due to the stubb
From: Robert Foss <robert.f...@collabora.com>
Hey,
I've been looking at the possibilty of removing the compile time depency on
libdrm_intel. There are two technical solutions to this problem as far as
I can see; stubs and conditional compilation.
This series uses the stubbing approach.
C
From: Robert Foss <robert.f...@collabora.com>
Replace the automake specific names of listings with something that isn't
automake specific.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
lib/tests/Android.mk | 2 +-
lib/tests/Makefile.am | 3 +++
lib/tests/Make
From: Robert Foss <robert.f...@collabora.com>
Always set HAVE_LIBDRM_INTEL to true for Android targets.
---
Android.mk | 2 ++
1 file changed, 2 insertions(+)
diff --git a/Android.mk b/Android.mk
index 681d114..3690fc5 100644
--- a/Android.mk
+++ b/Android.mk
@@ -1,2 +1,4 @@
+HAVE_LIBDRM
From: Robert Foss <robert.f...@collabora.com>
Use the HAS_INTEL automake flag to avoid building benchmarks that won't
compile unless libdrm_intel is available in the build system.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
benchmarks/Android.mk | 4
From: Robert Foss <robert.f...@collabora.com>
Replace the automake specific variable names for listings in Makefile.sources
with something not automake specific.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
lib/Android.mk | 2 +-
lib/Makefile.am
On 2016-05-25 03:55 PM, Emil Velikov wrote:
On Wednesday, May 25, 2016 19:18 BST, robert.f...@collabora.com wrote:
From: Robert Foss <robert.f...@collabora.com>
Use the HAS_INTEL automake flag to avoid building benchmarks that won't
compile unless libdrm_intel is available in the build
Verified working on rpi2.
Tested-by: Robert Foss <robert.f...@collabora.com>
On 2016-05-31 05:25 PM, Chris Wilson wrote:
Since the introduction of (struct_mutex) lockless GEM bo freeing, there
are a pair of driver vfuncs for freeing the GEM bo, of which the driver
may choose to only imp
From: Robert Foss <robert.f...@collabora.com>
Always set HAVE_LIBDRM_INTEL to true for Android targets.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
Android.mk | 2 ++
1 file changed, 2 insertions(+)
diff --git a/Android.mk b/Android.mk
index 681d114..36
From: Robert Foss <robert.f...@collabora.com>
Harmonize tabs/spaces etc.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
tools/Makefile.sources | 57 +-
1 file changed, 29 insertions(+), 28 deletions(-)
diff -
From: Robert Foss <robert.f...@collabora.com>
This patch provides stubs for functionality otherwise provided by intel_bufmgr.
The stubbed functions all fail with a call to igt_require_f(false,"").
Defines and enums have been copied from libdrm_intel.
Due to the stubb
From: Robert Foss <robert.f...@collabora.com>
Replace the automake specific name of listings in Makefile.sources
with something not automake specific.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
benchmarks/Android.mk | 2 +-
benchmarks/Makefile.am | 2 ++
From: Robert Foss <robert.f...@collabora.com>
Hey,
I've been looking at the possibilty of removing the compile time depency on
libdrm_intel. There are two technical solutions to this problem as far as
I can see; stubs and conditional compilation.
This series uses the stubbing ap
From: Robert Foss <robert.f...@collabora.com>
Test for libdrm_intel and build for it if present.
Also expose the HAVE_INTEL #define to allow code to be conditionally
compiled.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
configure.ac | 17 -
1 file
From: Robert Foss <robert.f...@collabora.com>
Use the HAS_INTEL automake flag to avoid building benchmarks that won't
compile unless libdrm_intel is available in the build system.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
benchmarks/Android.mk | 4
From: Robert Foss <robert.f...@collabora.com>
Replace the automake specific names of listings in Makefile.sources with
something not automake specific.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
tools/Android.mk | 1 +
tools/Makefile.am | 2 ++
tools/Make
From: Robert Foss <robert.f...@collabora.com>
Replace the automake specific names of listings with something that isn't
automake specific.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
lib/tests/Android.mk | 2 +-
lib/tests/Makefile.am | 3 +++
lib/tests/Make
From: Robert Foss <robert.f...@collabora.com>
Replace the automake specific name of listings in Makefile.sources
with something not automake specific.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
demos/Android.mk | 2 +-
demos/Makefile.am | 2 ++
demos/Make
From: Robert Foss <robert.f...@collabora.com>
Replace the automake flag HAVE_XXX for VC4/NOUVEAU with HAVE_LIBDRM_XXX in
order for the flags to be more descriptive and also follow the same convention
as HAVE_LIBDRM_INTEL.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
From: Robert Foss <robert.f...@collabora.com>
Replace the automake specific variable names for listings in Makefile.sources
with something not automake specific.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
lib/Android.mk | 2 +-
lib/Makefile.am
From: Robert Foss <robert.f...@collabora.com>
Use the HAS_INTEL automake flag to avoid building tools that won't
compile unless libdrm_intel is available in the build system.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
tools/Android.mk | 5 +
tools
From: Robert Foss <robert.f...@collabora.com>
Use the HAS_INTEL automake flag to avoid building tools that won't
compile unless libdrm_intel is available in the build system.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
demos/Android.mk | 5 -
demos/Makefile
From: Robert Foss <robert.f...@collabora.com>
Avoid running gem_quiescent_gpu() on non-Intel hardware.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
lib/drmtest.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/drmtest.c b/lib/drmtest.c
i
Thanks Chris!
On 2016-06-28 05:06 AM, Chris Wilson wrote:
On Mon, Jun 27, 2016 at 01:53:13PM -0400, robert.f...@collabora.com wrote:
From: Robert Foss <robert.f...@collabora.com>
Avoid running gem_quiescent_gpu() on non-Intel hardware.
Signed-off-by: Robert Foss <robert.f...@coll
From: Robert Foss <robert.f...@collabora.com>
ref_crc is never assigned or read, and can be safely
removed.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
tests/kms_panel_fitting.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/tests/kms_panel_fitti
From: Robert Foss <robert.f...@collabora.com>
Switched from DRIVER_INTEL to DRIVER_ANY to enable test
on all hardware.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
tests/kms_panel_fitting.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff -
From: Robert Foss <robert.f...@collabora.com>
Changed the DRM format to LOCAL_DRM_FORMAT_MOD_NONE since it
is hardware agnostic.
Also fixed formatting/tabs.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
tests/kms_panel_fitting.c | 10 +-
1 file changed, 5 inse
From: Robert Foss <robert.f...@collabora.com>
Changes since v1:
- Squash patches
- Based patch on
https://lists.freedesktop.org/archives/intel-gfx/2016-March/090617.html
Robert Foss (6):
kms_panel_fitting: Remove un-read variable.
kms_panel_fitting: Remove un-read variable pi
From: Robert Foss <robert.f...@collabora.com>
Add support for DRM_MODE_CONNECTOR_LVDS and DRM_MODE_CONNECTOR_DSI
connectors.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
tests/kms_panel_fitting.c | 13 ++---
1 file changed, 10 insertions(+), 3 deletions(-)
From: Robert Foss <robert.f...@collabora.com>
Remove devid from data_t since it is never read.
Also remove one assignment to devid.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
tests/kms_panel_fitting.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/tests/kms_pan
From: Robert Foss <robert.f...@collabora.com>
pipe_crc in data_t is assigned an allocated memory space and
then later free'd. But it is never used for any comparisons.
It should therefore be safe to remove pipe_crc and the crc
requirement.
Signed-off-by: Robert Foss <robert.f...@coll
From: Robert Foss <robert.f...@collabora.com>
Fix issue where the plane counting fails due to the number and
configuration of planes being unlike the intel configuration.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
lib/igt_kms.c | 6 --
1 file changed, 4 inse
From: Robert Foss <robert.f...@collabora.com>
Previously crtc0 was statically used for VBLANK tests, but
that assumption is not valid for the VC4 platform.
Instead we're now explicitly setting the mode.
Also add support for testing all connected connectors during
the same test.
Sign
From: Robert Foss <robert.f...@collabora.com>
Increase the number of planes supported to 10.
kmstest_plane_name only previously supported 4 planes,
this patch adds support for up to 10 planes.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
lib/igt_kms.c | 6 ++
l
From: Robert Foss <robert.f...@collabora.com>
Switch igt_assert to igt_assert_lte to provide more diagnostic
information.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
lib/igt_kms.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/igt_kms.c b/l
From: Robert Foss <robert.f...@collabora.com>
Changes since v2:
- Rebased onto trunk which already contains kms_flip_event_leak changes.
Changes since v1:
- kms_vblank: Removed un-used members of data_t struct.
- Rename plane_counter to n_planes.
- Removed un-needed handling CURSOR
On 04/25/2016 11:10 AM, Ville Syrjälä wrote:
On Mon, Apr 25, 2016 at 10:35:51AM -0400, robert.f...@collabora.com wrote:
From: Robert Foss <robert.f...@collabora.com>
Add support for DRM_MODE_CONNECTOR_LVDS and DRM_MODE_CONNECTOR_DSI
connectors.
Signed-off-by: Robert Foss <
From: Robert Foss <robert.f...@collabora.com>
Remove devid from data_t since it is never read.
Also remove one assignment to devid.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
tests/kms_panel_fitting.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/tests/kms_pan
From: Robert Foss <robert.f...@collabora.com>
Changed the DRM format to LOCAL_DRM_FORMAT_MOD_NONE since it
is hardware agnostic.
Also fixed formatting/tabs.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
tests/kms_panel_fitting.c | 10 +-
1 file changed, 5 inse
From: Robert Foss <robert.f...@collabora.com>
ref_crc is never assigned or read, and can be safely
removed.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
tests/kms_panel_fitting.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/tests/kms_panel_fitti
From: Robert Foss <robert.f...@collabora.com>
If an incompatible connector is found, don't skip the test,
but rather keep looking for valid connectors.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
tests/kms_panel_fitting.c | 5 +++--
1 file changed, 3 insertions(+),
From: Robert Foss <robert.f...@collabora.com>
This series removes some dead code from kms_panel_fitting and
enables support for LVDS and DSI connectors.
Robert Foss (7):
kms_panel_fitting: Remove un-read variable.
kms_panel_fitting: Remove un-read variable pipe_crc.
kms_panel_f
From: Robert Foss <robert.f...@collabora.com>
Add support for DRM_MODE_CONNECTOR_LVDS and DRM_MODE_CONNECTOR_DSI
connectors.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
tests/kms_panel_fitting.c | 12 +---
1 file changed, 9 insertions(+), 3 deletions(-)
From: Robert Foss <robert.f...@collabora.com>
Switched from DRIVER_INTEL to DRIVER_ANY to enable test
on all hardware.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
tests/kms_panel_fitting.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff -
From: Robert Foss <robert.f...@collabora.com>
pipe_crc in data_t is assigned an allocated memory space and
then later free'd. But it is never used for any comparisons.
It should therefore be safe to remove pipe_crc and the crc
requirement.
Signed-off-by: Robert Foss <robert.f...@coll
On 04/21/2016 10:50 AM, Daniel Vetter wrote:
On Wed, Apr 20, 2016 at 10:59:45AM -0400, robert.f...@collabora.com wrote:
From: Robert Foss <robert.f...@collabora.com>
Avoid overwriting planes with statically asigned indices
with planes that have dynamically assigned indices.
Sign
Makes sure we automatically extend that when adding more planes.
Inspired by a patch from Robert Foss who extended the max, but forgot
all about the enum.
While at it, also fix up the whitespace damage.
Cc: robert.f...@collabora.com
Signed-off-by: Daniel Vetter <daniel.vet...@intel.
On 04/21/2016 10:48 AM, Daniel Vetter wrote:
On Wed, Apr 20, 2016 at 10:59:46AM -0400, robert.f...@collabora.com wrote:
From: Robert Foss <robert.f...@collabora.com>
Avoid moving the cursor plane when on non-intel hardware.
Running the move block on hardware with more than IGT_PLANE_
From: Robert Foss <robert.f...@collabora.com>
Makes sure we automatically extend that when adding more planes.
Inspired by a patch from Robert Foss who extended the max, but forgot
all about the enum.
While at it, also fix up the whitespace damage.
Cc: robert.f...@collabora.com
Author:
From: Robert Foss <robert.f...@collabora.com>
Previously crtc0 was statically used for VBLANK tests, but
that assumption is not valid for the VC4 platform.
Instead we're now explicitly setting the mode.
Also add support for testing all connected connectors during
the same test.
Sign
From: Robert Foss <robert.f...@collabora.com>
Increase the number of planes supported to 10.
kmstest_plane_name only previously supported 4 planes,
this patch adds support for up to 10 planes.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
lib/igt_kms.c | 6 ++
l
From: Robert Foss <robert.f...@collabora.com>
Changes since v1:
- Removed un-used members of data_t struct in kms_vblank.c.
- Renamed plane_counter to n_planes.
- Removed un-needed handling CURSOR plane location.
- Added names for additional planes in update kmstest_plane_name.
Robert F
From: Robert Foss <robert.f...@collabora.com>
Fix issue where the plane counting fails due to the number and
configuration of planes being unlike the intel configuration.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
lib/igt_kms.c | 6 --
1 file changed, 4 inse
From: Robert Foss <robert.f...@collabora.com>
Switch igt_assert to igt_assert_lte to provide more diagnostic
information.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
lib/igt_kms.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/igt_kms.c b/l
From: Robert Foss <robert.f...@collabora.com>
Change DRIVER_INTEL to DRIVER_ANY to enable tests/kms_flip_event_leak
on any driver.
Switch the tiling format to something intel indenpendant.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
tests/kms_flip_event_leak.c | 8
On 04/22/2016 08:53 AM, Daniel Vetter wrote:
On Thu, Apr 21, 2016 at 01:31:48PM -0400, Robert Foss wrote:
On 04/21/2016 10:50 AM, Daniel Vetter wrote:
On Wed, Apr 20, 2016 at 10:59:45AM -0400, robert.f...@collabora.com wrote:
From: Robert Foss <robert.f...@collabora.com>
Avoid overw
From: Robert Foss <robert.f...@collabora.com>
Remove devid from data_t since it is never read.
Also remove one assignment to devid.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
Reviewed-by: Tomeu Vizoso <tomeu.viz...@collabora.com>
---
tests/kms_panel_fitting.c | 2
From: Robert Foss <robert.f...@collabora.com>
ref_crc is never assigned or read, and can be safely
removed.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
Reviewed-by: Tomeu Vizoso <tomeu.viz...@collabora.com>
---
tests/kms_panel_fitting.c | 1 -
1 file changed, 1
From: Robert Foss <robert.f...@collabora.com>
pipe_crc in data_t is assigned an allocated memory space and
then later free'd. But it is never used for any comparisons.
It should therefore be safe to remove pipe_crc and the crc
requirement.
Signed-off-by: Robert Foss <robert.f...@coll
From: Robert Foss <robert.f...@collabora.com>
Changes since v4:
- Added reviewed-by danvet to patch 6, all patches are now reviewed.
Changes since v3:
- kms_panel_fitting.c: Pass NULL arguments instead
- Added bugzilla link: https://bugs.freedesktop.org/show_bug.cgi?id=93012
Changes si
From: Robert Foss <robert.f...@collabora.com>
Switched from DRIVER_INTEL to DRIVER_ANY to enable test
on all hardware.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
Reviewed-by: Tomeu Vizoso <tomeu.viz...@collabora.com>
---
tests/kms_panel_fitting.c | 2 +-
1 file ch
On 04/28/2016 04:02 AM, Daniel Vetter wrote:
On Tue, Apr 26, 2016 at 01:08:35PM -0400, Robert Foss wrote:
On 04/26/2016 01:00 PM, Ville Syrjälä wrote:
On Tue, Apr 26, 2016 at 12:55:54PM -0400, Robert Foss wrote:
On 04/26/2016 10:11 AM, Daniel Vetter wrote:
On Fri, Apr 22, 2016 at 05:25
From: Robert Foss <robert.f...@collabora.com>
Changed the DRM format to LOCAL_DRM_FORMAT_MOD_NONE since it
is hardware agnostic.
Also fixed formatting/tabs.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
Reviewed-by: Tomeu Vizoso <tomeu.viz...@collabora.
From: Robert Foss <robert.f...@collabora.com>
Enable testing on all connectors that have the "scaling mode"
property set.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=93012
Signed-off-by: Robert Foss <robert.f...@collabora.com>
Reviewed-by: Daniel Vetter &
On 04/26/2016 10:10 AM, Daniel Vetter wrote:
On Fri, Apr 22, 2016 at 05:24:59PM -0400, robert.f...@collabora.com wrote:
From: Robert Foss <robert.f...@collabora.com>
If an incompatible connector is found, don't skip the test,
but rather keep looking for valid connectors.
Sign
On 04/26/2016 01:00 PM, Ville Syrjälä wrote:
On Tue, Apr 26, 2016 at 12:55:54PM -0400, Robert Foss wrote:
On 04/26/2016 10:11 AM, Daniel Vetter wrote:
On Fri, Apr 22, 2016 at 05:25:00PM -0400, robert.f...@collabora.com wrote:
From: Robert Foss <robert.f...@collabora.com>
Add s
On 04/26/2016 10:11 AM, Daniel Vetter wrote:
On Fri, Apr 22, 2016 at 05:25:00PM -0400, robert.f...@collabora.com wrote:
From: Robert Foss <robert.f...@collabora.com>
Add support for DRM_MODE_CONNECTOR_LVDS and DRM_MODE_CONNECTOR_DSI
connectors.
Signed-off-by: Robert Foss <
From: Robert Foss <robert.f...@collabora.com>
Changes since v3:
- kms_flip.c: Removed drm_fd from data_t, to decrease the number of unrelated
changes in the patch.
- kms_flip.c: Changed "int valid_tests" to "unsigned valid_tests".
- kms_flip.c: Changed cr
From: Robert Foss <robert.f...@collabora.com>
Increase the number of planes supported to 10.
kmstest_plane_name only previously supported 4 planes,
this patch adds support for up to 10 planes.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
lib/igt_kms.c | 6 ++
l
From: Robert Foss <robert.f...@collabora.com>
Fix issue where the plane counting fails due to the number and
configuration of planes being unlike the intel configuration.
Signed-off-by: Robert Foss <robert.f...@collabora.com>
---
lib/igt_kms.c | 6 --
1 file changed, 4 inse
1 - 100 of 699 matches
Mail list logo