[Intel-gfx] [PATCH i-g-t v5 5/7] kms_vblank: Switch from using crtc0 statically to explicitly setting mode.

2016-05-16 Thread robert . foss
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

[Intel-gfx] [PATCH i-g-t v5 1/7] lib/igt_kms: Add support for up to 10 planes.

2016-05-16 Thread robert . foss
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

[Intel-gfx] [PATCH i-g-t v5 0/7] kms_flip_event_leak and kms_vblank fixes for

2016-05-16 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> In addition to the changes made in v4, danvet suggested that the argument for the (newly renamed) helper function kmstest_get_vbl_flag should be changed. I investigated it and found that all of the places where the helper function currently i

[Intel-gfx] [PATCH i-g-t v5 4/7] lib/igt_kms: Added pipe_id_to_vbl_flag() to igt_kms.

2016-05-16 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> Added function pipe_id_to_vbl_flag() to igt_kms. This function formats a pipe_id into the format expected by DRM while processing DRM_IOCTL_WAIT_VBLANK. Signed-off-by: Robert Foss <robert.f...@collabora.com> --- lib/ig

[Intel-gfx] [PATCH i-g-t v5 2/7] lib/igt_kms: Fix plane counting in igt_display_init.

2016-05-16 Thread robert . foss
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

Re: [Intel-gfx] [PATCH i-g-t v4 5/5] lib/igt_kms: Added pipe_id_to_vbl_flag() to igt_kms.

2016-05-11 Thread Robert Foss
On 2016-05-11 09:45 AM, Robert Foss wrote: On 2016-05-10 03:44 AM, Daniel Vetter wrote: On Tue, Apr 26, 2016 at 11:54:22AM -0400, robert.f...@collabora.com wrote: From: Robert Foss <robert.f...@collabora.com> Added function pipe_id_to_vbl_flag() to igt_kms. This function formats a p

Re: [Intel-gfx] [PATCH i-g-t v4 5/5] lib/igt_kms: Added pipe_id_to_vbl_flag() to igt_kms.

2016-05-11 Thread Robert Foss
On 2016-05-10 01:54 PM, Ville Syrjälä wrote: On Tue, Apr 26, 2016 at 11:54:22AM -0400, robert.f...@collabora.com wrote: From: Robert Foss <robert.f...@collabora.com> Added function pipe_id_to_vbl_flag() to igt_kms. This function formats a pipe_id into the format expected by DRM

Re: [Intel-gfx] [PATCH i-g-t v4 5/5] lib/igt_kms: Added pipe_id_to_vbl_flag() to igt_kms.

2016-05-11 Thread Robert Foss
On 2016-05-10 03:44 AM, Daniel Vetter wrote: On Tue, Apr 26, 2016 at 11:54:22AM -0400, robert.f...@collabora.com wrote: From: Robert Foss <robert.f...@collabora.com> Added function pipe_id_to_vbl_flag() to igt_kms. This function formats a pipe_id into the format expected by DRM

Re: [Intel-gfx] [PATCH i-g-t v4 4/5] kms_vblank: Switch from using crtc0 statically to explicitly setting mode.

2016-05-09 Thread Robert Foss
On 2016-05-09 11:38 AM, Marius Vlad wrote: > On Tue, Apr 26, 2016 at 11:54:21AM -0400, robert.f...@collabora.com wrote: >> From: Robert Foss <robert.f...@collabora.com> >> >> Previously crtc0 was statically used for VBLANK tests, but >> that assumption

[Intel-gfx] [PATCH i-g-t v6 6/7] kms_panel_fitting: Enabled test on non-Intel hardware.

2016-05-03 Thread robert . foss
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

[Intel-gfx] [PATCH i-g-t v6 4/7] kms_panel_fitting: Remove un-used variable.

2016-05-03 Thread robert . foss
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

[Intel-gfx] [PATCH i-g-t v6 5/7] kms_panel_fitting: Switched DRM format to a hardware agnostic alternative.

2016-05-03 Thread robert . foss
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.

[Intel-gfx] [PATCH i-g-t v6 3/7] kms_panel_fitting: Remove un-read variable pipe_crc.

2016-05-03 Thread robert . foss
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

[Intel-gfx] [PATCH i-g-t v6 7/7] kms_panel_fitting: Enable connectors with "scaling mode" property set.

2016-05-03 Thread robert . foss
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 &

[Intel-gfx] [PATCH i-g-t v6 1/7] kms_panel_fitting: skips if there's a non-eDP display connected. Fix it.

2016-05-03 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=93012 Signed-off-by: Jani Nikula <jani.nik...@intel.com> --- tests/kms_panel_fitting.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/tests/kms_panel_fitti

[Intel-gfx] [PATCH i-g-t v6 2/7] kms_panel_fitting: Remove un-read variable.

2016-05-03 Thread 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> Reviewed-by: Tomeu Vizoso <tomeu.viz...@collabora.com> --- tests/kms_panel_fitting.c | 2

[Intel-gfx] [PATCH i-g-t v6 0/7] Improve kms_panel_fitting.

2016-05-03 Thread robert . foss
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 Changes since v2: - Added reviewed-by tomeu to patch 1-5. - kms_panel_fitting.c: Replaced connector type ch

[Intel-gfx] [PATCH i-g-t v6 0/7] Improve kms_panel_fitting.

2016-05-03 Thread robert . foss
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 Changes since v2: - Added reviewed-by tomeu to patch 1-5. - kms_panel_fitting.c: Replaced connector type ch

[Intel-gfx] [PATCH i-g-t v6 0/7] Improve kms_panel_fitting.

2016-05-03 Thread robert . foss
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 Changes since v2: - Added reviewed-by tomeu to patch 1-5. - kms_panel_fitting.c: Replaced connector type ch

Re: [Intel-gfx] [PATCH i-g-t 7/7] kms_panel_fitting: Add support for LVDS and DSI connectors.

2016-05-02 Thread Robert Foss
On 05/02/2016 04:20 AM, Daniel Vetter wrote: On Thu, Apr 28, 2016 at 11:57:21AM -0400, Robert Foss wrote: On 04/28/2016 10:41 AM, Daniel Vetter wrote: On Thu, Apr 28, 2016 at 10:07:41AM -0400, Robert Foss wrote: On 04/28/2016 04:02 AM, Daniel Vetter wrote: On Tue, Apr 26, 2016 at 01:08

Re: [Intel-gfx] [PATCH i-g-t 7/7] kms_panel_fitting: Add support for LVDS and DSI connectors.

2016-04-28 Thread Robert Foss
On 04/28/2016 10:41 AM, Daniel Vetter wrote: On Thu, Apr 28, 2016 at 10:07:41AM -0400, Robert Foss wrote: 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

Re: [Intel-gfx] [PATCH i-g-t 7/7] kms_panel_fitting: Add support for LVDS and DSI connectors.

2016-04-28 Thread Robert Foss
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

[Intel-gfx] [PATCH i-g-t v5 5/6] kms_panel_fitting: Enabled test on non-Intel hardware.

2016-04-28 Thread robert . foss
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

[Intel-gfx] [PATCH i-g-t v5 4/6] kms_panel_fitting: Switched DRM format to a hardware agnostic alternative.

2016-04-28 Thread robert . foss
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.

[Intel-gfx] [PATCH i-g-t v5 6/6] kms_panel_fitting: Enable connectors with "scaling mode" property set.

2016-04-28 Thread robert . foss
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 &

[Intel-gfx] [PATCH i-g-t v5 3/6] kms_panel_fitting: Remove un-used variable.

2016-04-28 Thread robert . foss
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

[Intel-gfx] [PATCH i-g-t v5 2/6] kms_panel_fitting: Remove un-read variable pipe_crc.

2016-04-28 Thread robert . foss
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

[Intel-gfx] [PATCH i-g-t v5 0/6] Improve kms_panel_fitting.

2016-04-28 Thread robert . foss
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

[Intel-gfx] [PATCH i-g-t v5 1/6] kms_panel_fitting: Remove un-read variable.

2016-04-28 Thread 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> Reviewed-by: Tomeu Vizoso <tomeu.viz...@collabora.com> --- tests/kms_panel_fitting.c | 2

[Intel-gfx] [PATCH i-g-t v4 0/6] Improve kms_panel_fitting.

2016-04-27 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> Changes since v3: - kms_panel_fitting.c: Pass NULL arguments instead - Added bugzilla link: https://bugs.freedesktop.org/show_bug.cgi?id=93012 Changes since v2: - Added reviewed-by tomeu to patch 1-5. - kms_panel_fitting.c: Replaced connecto

[Intel-gfx] [PATCH i-g-t v4 1/6] kms_panel_fitting: Remove un-read variable.

2016-04-27 Thread 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> Reviewed-by: Tomeu Vizoso <tomeu.viz...@collabora.com> --- tests/kms_panel_fitting.c | 2

[Intel-gfx] [PATCH i-g-t v4 3/6] kms_panel_fitting: Remove un-used variable.

2016-04-27 Thread robert . foss
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

[Intel-gfx] [PATCH i-g-t v4 2/6] kms_panel_fitting: Remove un-read variable pipe_crc.

2016-04-27 Thread robert . foss
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

[Intel-gfx] [PATCH i-g-t v4 5/6] kms_panel_fitting: Enabled test on non-Intel hardware.

2016-04-27 Thread robert . foss
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

[Intel-gfx] [PATCH i-g-t v4 4/6] kms_panel_fitting: Switched DRM format to a hardware agnostic alternative.

2016-04-27 Thread robert . foss
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.

[Intel-gfx] [PATCH i-g-t v4 6/6] kms_panel_fitting: Enable connectors with "scaling mode" property set.

2016-04-27 Thread robert . foss
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> --- tests/kms_panel_fitting.c |

[Intel-gfx] [PATCH i-g-t v3 3/6] kms_panel_fitting: Remove un-used variable.

2016-04-26 Thread robert . foss
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

[Intel-gfx] [PATCH i-g-t v3 1/6] kms_panel_fitting: Remove un-read variable.

2016-04-26 Thread 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> Reviewed-by: Tomeu Vizoso <tomeu.viz...@collabora.com> --- tests/kms_panel_fitting.c | 2

[Intel-gfx] [PATCH i-g-t v3 4/6] kms_panel_fitting: Switched DRM format to a hardware agnostic alternative.

2016-04-26 Thread robert . foss
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.

[Intel-gfx] [PATCH i-g-t v3 6/6] kms_panel_fitting: Enable connectors with "scaling mode" property set.

2016-04-26 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> Enable testing on all connectors that have the "scaling mode" property set. Signed-off-by: Robert Foss <robert.f...@collabora.com> --- tests/kms_panel_fitting.c | 18 +++--- 1 file changed, 15 insertions(+), 3 de

[Intel-gfx] [PATCH i-g-t v3 5/6] kms_panel_fitting: Enabled test on non-Intel hardware.

2016-04-26 Thread robert . foss
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

[Intel-gfx] [PATCH i-g-t v3 0/6] Improve kms_panel_fitting.

2016-04-26 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> Changes since v2: - Added reviewed-by tomeu to patch 1-5. - kms_panel_fitting.c: Replaced connector type checking with "scaling mode" property check. Changes since v1: - Squash patches - Based patch on https://lists.freedesktop.or

[Intel-gfx] [PATCH i-g-t v3 2/6] kms_panel_fitting: Remove un-read variable pipe_crc.

2016-04-26 Thread robert . foss
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

Re: [Intel-gfx] [PATCH i-g-t 7/7] kms_panel_fitting: Add support for LVDS and DSI connectors.

2016-04-26 Thread Robert Foss
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

Re: [Intel-gfx] [PATCH i-g-t 7/7] kms_panel_fitting: Add support for LVDS and DSI connectors.

2016-04-26 Thread Robert Foss
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 <

Re: [Intel-gfx] [PATCH i-g-t 6/7] kms_panel_fitting: Don't skip test after finding an incompatible connector.

2016-04-26 Thread Robert Foss
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

[Intel-gfx] [PATCH i-g-t v4 3/5] lib/igt_kms: Switch to verbose assert.

2016-04-26 Thread robert . foss
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

[Intel-gfx] [PATCH i-g-t v4 5/5] lib/igt_kms: Added pipe_id_to_vbl_flag() to igt_kms.

2016-04-26 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> Added function pipe_id_to_vbl_flag() to igt_kms. This function formats a pipe_id into the format expected by DRM while processing DRM_IOCTL_WAIT_VBLANK. Signed-off-by: Robert Foss <robert.f...@collabora.com> --- lib/ig

[Intel-gfx] [PATCH i-g-t v4 4/5] kms_vblank: Switch from using crtc0 statically to explicitly setting mode.

2016-04-26 Thread robert . foss
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

[Intel-gfx] [PATCH i-g-t v4 2/5] lib/igt_kms: Fix plane counting in igt_display_init.

2016-04-26 Thread robert . foss
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

[Intel-gfx] [PATCH i-g-t v4 0/5] kms_flip_event_leak and kms_vblank fixes for VC4.

2016-04-26 Thread 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

[Intel-gfx] [PATCH i-g-t v4 1/5] lib/igt_kms: Add support for up to 10 planes.

2016-04-26 Thread robert . foss
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

Re: [Intel-gfx] [PATCH i-g-t v2 0/6] Improve kms_panel_fitting.

2016-04-26 Thread Robert Foss
Changes since v1: - Squash patches - Based patch on https://lists.freedesktop.org/archives/intel-gfx/2016-March/090617.html Thanks for the patches, you can add my Reviewed-by tag to all except the 6th patch, about which I don't know enough. Tomeu Thank you, I will submit the next version

Re: [Intel-gfx] [PATCH i-g-t v3 4/4] kms_vblank: Switch from using crtc0 statically to explicitly setting mode.

2016-04-26 Thread Robert Foss
for the intermediate step would be somewhat convoluted I think. Signed-off-by: Robert Foss <robert.f...@collabora.com> --- tests/kms_vblank.c | 179 + 1 file changed, 140 insertions(+), 39 deletions(-) diff --git a/tests/kms_vblank.c b/tests/kms_vb

Re: [Intel-gfx] [PATCH i-g-t v2 6/6] kms_panel_fitting: Add support for LVDS and DSI connectors.

2016-04-25 Thread Robert Foss
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 <

[Intel-gfx] [PATCH i-g-t v3 2/4] lib/igt_kms: Fix plane counting in igt_display_init.

2016-04-25 Thread robert . foss
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

[Intel-gfx] [PATCH i-g-t v3 4/4] kms_vblank: Switch from using crtc0 statically to explicitly setting mode.

2016-04-25 Thread robert . foss
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

[Intel-gfx] [PATCH i-g-t v3 1/4] lib/igt_kms: Add support for up to 10 planes.

2016-04-25 Thread robert . foss
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

[Intel-gfx] [PATCH i-g-t v3 3/4] lib/igt_kms: Switch to verbose assert.

2016-04-25 Thread robert . foss
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

[Intel-gfx] [PATCH i-g-t v3 0/4] kms_flip_event_leak and kms_vblank fixes for VC4.

2016-04-25 Thread robert . foss
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

[Intel-gfx] [PATCH i-g-t v2 4/6] kms_panel_fitting: Switched DRM format to a hardware agnostic alternative.

2016-04-25 Thread robert . foss
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

[Intel-gfx] [PATCH i-g-t v2 3/6] kms_panel_fitting: Remove un-used variable.

2016-04-25 Thread robert . foss
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

[Intel-gfx] [PATCH i-g-t v2 5/6] kms_panel_fitting: Enabled test on non-Intel hardware.

2016-04-25 Thread robert . foss
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 -

[Intel-gfx] [PATCH i-g-t v2 6/6] kms_panel_fitting: Add support for LVDS and DSI connectors.

2016-04-25 Thread robert . foss
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(-)

[Intel-gfx] [PATCH i-g-t v2 0/6] Improve kms_panel_fitting.

2016-04-25 Thread robert . foss
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

[Intel-gfx] [PATCH i-g-t v2 1/6] kms_panel_fitting: Remove un-read variable.

2016-04-25 Thread 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

[Intel-gfx] [PATCH i-g-t v2 2/6] kms_panel_fitting: Remove un-read variable pipe_crc.

2016-04-25 Thread robert . foss
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

[Intel-gfx] [PATCH i-g-t 4/7] kms_panel_fitting: Switched DRM format to a hardware agnostic alternative.

2016-04-22 Thread robert . foss
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

[Intel-gfx] [PATCH i-g-t 3/7] kms_panel_fitting: Remove un-used variable.

2016-04-22 Thread robert . foss
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

[Intel-gfx] [PATCH i-g-t 1/7] kms_panel_fitting: Remove un-read variable.

2016-04-22 Thread 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

[Intel-gfx] [PATCH i-g-t 0/7] Improve kms_panel_fitting.

2016-04-22 Thread robert . foss
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

[Intel-gfx] [PATCH i-g-t 7/7] kms_panel_fitting: Add support for LVDS and DSI connectors.

2016-04-22 Thread robert . foss
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(-)

[Intel-gfx] [PATCH i-g-t 5/7] kms_panel_fitting: Enabled test on non-Intel hardware.

2016-04-22 Thread robert . foss
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 -

[Intel-gfx] [PATCH i-g-t 2/7] kms_panel_fitting: Remove un-read variable pipe_crc.

2016-04-22 Thread robert . foss
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

[Intel-gfx] [PATCH i-g-t 6/7] kms_panel_fitting: Don't skip test after finding an incompatible connector.

2016-04-22 Thread robert . foss
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(+),

[Intel-gfx] [PATCH i-g-t v2 3/6] lib/igt_kms: Fix plane counting in igt_display_init.

2016-04-22 Thread robert . foss
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

[Intel-gfx] [PATCH i-g-t v2 1/6] lib/igt_kms: Move IGT_MAX_PLANES into the igt_plane enum

2016-04-22 Thread robert . foss
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:

[Intel-gfx] [PATCH i-g-t v2 6/6] kms_vblank: Switch from using crtc0 statically to explicitly setting mode.

2016-04-22 Thread robert . foss
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

[Intel-gfx] [PATCH i-g-t v2 0/6] kms_flip_event_leak and kms_vblank fixes for VC4

2016-04-22 Thread robert . foss
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

[Intel-gfx] [PATCH i-g-t v2 2/6] lib/igt_kms: Add support for up to 10 planes.

2016-04-22 Thread robert . foss
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

[Intel-gfx] [PATCH i-g-t v2 4/6] lib/igt_kms: Switch to verbose assert.

2016-04-22 Thread robert . foss
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

[Intel-gfx] [PATCH i-g-t v2 5/6] kms_flip_event_leak: Enable test on DRIVER_ANY.

2016-04-22 Thread robert . foss
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

Re: [Intel-gfx] [PATCH i-g-t 3/7] lib/igt_kms: Make sure that default planes aren't overwritten.

2016-04-22 Thread Robert Foss
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

[Intel-gfx] [PATCH] kms_atomic: Fix crtc_id comparison failing.

2016-04-21 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> Fixed ctrc_id comparison failing due to bad initialization of crtc variable. Signed-off-by: Robert Foss <robert.f...@collabora.com> --- tests/kms_atomic.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/kms_atom

Re: [Intel-gfx] [PATCH i-g-t 3/7] lib/igt_kms: Make sure that default planes aren't overwritten.

2016-04-21 Thread Robert Foss
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

Re: [Intel-gfx] [PATCH i-g-t 4/7] lib/igt_kms: Only move the in cursor plane for Intel hw.

2016-04-21 Thread Robert Foss
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_

Re: [Intel-gfx] [PATCH i-g-t] lib/igt_kms: Move IGT_MAX_PLANES into the igt_plane enum

2016-04-21 Thread Robert Foss
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.

[Intel-gfx] [PATCH i-g-t v2 0/2] core_getversion: Only verify major version for i915.

2016-04-20 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> This series enables tests/core_getversion to test vc4. It's been tested on rpi2 hardware and kernel 4.6.0-rc1. Changes since v1: - Improved commit message - Switched to is_915_device() helper function Robert Foss (2): core_getversion: Sw

[Intel-gfx] [PATCH i-g-t v2 2/2] core_getversion: Only verify major version for i915.

2016-04-20 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> This change mirrors the change in drm made by k...@redhat.com on "Mon Apr 6 17:18:17 2009" on the drm branch intel_on_all_hw. The assert(major < 1) is only needed for the legacy intel driver. Signed-off-by: Robert Foss <rob

[Intel-gfx] [PATCH i-g-t v2 1/2] core_getversion: Switched from igt_assert to assert helpers.

2016-04-20 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> Switched to assert helpers to enable better error output. Signed-off-by: Robert Foss <robert.f...@collabora.com> --- tests/core_getversion.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/tests/core_getversi

[Intel-gfx] [PATCH i-g-t 1/7] lib/igt_kms: Add support for up to 10 planes per pipe.

2016-04-20 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> The VC4 DRM currently uses 10 planes which is more than any other DRM, let's allocate space for the worst case scenario. Signed-off-by: Robert Foss <robert.f...@collabora.com> --- lib/igt_kms.h | 2 +- 1 file changed, 1 insertion(+)

[Intel-gfx] [PATCH i-g-t 7/7] kms_flip_event_leak: Enable test on DRIVER_ANY.

2016-04-20 Thread robert . foss
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

[Intel-gfx] [PATCH i-g-t 3/7] lib/igt_kms: Make sure that default planes aren't overwritten.

2016-04-20 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> Avoid overwriting planes with statically asigned indices with planes that have dynamically assigned indices. Signed-off-by: Robert Foss <robert.f...@collabora.com> --- lib/igt_kms.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/l

[Intel-gfx] [PATCH i-g-t 5/7] lib/igt_kms: Switch to verbose assert.

2016-04-20 Thread robert . foss
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

[Intel-gfx] [PATCH i-g-t 6/7] kms_vblank: Switch from using crtc0 statically to explicitly setting mode.

2016-04-20 Thread robert . foss
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

[Intel-gfx] [PATCH i-g-t 4/7] lib/igt_kms: Only move the in cursor plane for Intel hw.

2016-04-20 Thread robert . foss
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_CURSOR number of planes causes planes do be zeroed out. Signed-off-by: Robert Foss <robert.f...@collabora.com> --- l

[Intel-gfx] [PATCH i-g-t 2/7] lib/igt_kms: Fix plane counting in igt_display_init.

2016-04-20 Thread robert . foss
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

[Intel-gfx] [PATCH i-g-t 0/7] Enable kms_flip_event_leak and kms_vblank on vc4.

2016-04-20 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> This series enables tests/kms_flip_event_leak and tests/kms_vblank on non-intel hardware. It's been tested with vc4 on an rpi2 running kernel 4.6.0-rc1. Robert Foss (7): lib/igt_kms: Add support for up to 10 planes per pipe. lib/igt_km

[Intel-gfx] [PATCH i-g-t] kms_atomic: Fix crtc_id comparison failing.

2016-04-20 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> Fixed ctrc_id comparison failing due to bad initialization of crtc variable. Signed-off-by: Robert Foss <robert.f...@collabora.com> --- tests/kms_atomic.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/kms_atom

[Intel-gfx] [PATCH i-g-t 1/2] core_getversion: Switched from igt_assert to assert helpers.

2016-04-20 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> Switched to assert helpers to enable better error output. Signed-off-by: Robert Foss <robert.f...@collabora.com> --- tests/core_getversion.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/tests/core_getversi

<    2   3   4   5   6   7   8   >