[PATCH] tests/kms_cursor_legacy: use 'enum pipe' type instead of 'int'

[PATCH] tests/kms_cursor_legacy: use 'enum pipe' type instead of 'int'

[Gustavo Padovan]

From: Gustavo Padovan <gustavo.padovan@collabora.com>

Signed-off-by: Gustavo Padovan <gustavo.padovan@collabora.com>
---
 tests/kms_cursor_legacy.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/tests/kms_cursor_legacy.c b/tests/kms_cursor_legacy.c
index 8180b04..15208f8 100644
--- a/tests/kms_cursor_legacy.c
+++ b/tests/kms_cursor_legacy.c
@@ -47,7 +47,7 @@
 IGT_TEST_DESCRIPTION("Stress legacy cursor ioctl");
 
 static void stress(igt_display_t *display,
-		   int pipe, int num_children, unsigned mode,
+		   enum pipe pipe, int num_children, unsigned mode,
 		   int timeout)
 {
 	struct drm_mode_cursor arg;
@@ -148,7 +148,7 @@ static void stress(igt_display_t *display,
 	munmap(results, 4096);
 }
 
-static igt_output_t *set_fb_on_crtc(igt_display_t *display, int pipe, struct igt_fb *fb_info)
+static igt_output_t *set_fb_on_crtc(igt_display_t *display, enum pipe pipe, struct igt_fb *fb_info)
 {
 	igt_output_t *output;
 
@@ -464,7 +464,7 @@ static void flip(igt_display_t *display,
 		igt_remove_fb(display->drm_fd, &cursor_fb2);
 }
 
-static inline uint32_t pipe_select(int pipe)
+static inline uint32_t pipe_select(enum pipe pipe)
 {
 	if (pipe > 1)
 		return pipe << DRM_VBLANK_HIGH_CRTC_SHIFT;
@@ -474,7 +474,7 @@ static inline uint32_t pipe_select(int pipe)
 		return 0;
 }
 
-static unsigned get_vblank(int fd, int pipe, unsigned flags)
+static unsigned get_vblank(int fd, enum pipe pipe, unsigned flags)
 {
 	union drm_wait_vblank vbl;
 
-- 
2.9.4

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [PATCH] tests/kms_cursor_legacy: use 'enum pipe' type instead of 'int'

[Arkadiusz Hiler]

On Wed, Aug 02, 2017 at 07:54:17PM -0300, Gustavo Padovan wrote:
> From: Gustavo Padovan <gustavo.padovan@collabora.com>
> 
> Signed-off-by: Gustavo Padovan <gustavo.padovan@collabora.com>
Reviewed-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

and pushed, thanks!

-- 
Cheers,
Arek
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx