Re: [PATCH i-g-t 12/12] testdisplay/skl: Add command line options for Yb/Yf tiled fbs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Feb 23, 2015 at 03:57:55PM +0000, Tvrtko Ursulin wrote:
> From: Damien Lespiau <damien.lespiau@xxxxxxxxx>
> 
> Signed-off-by: Damien Lespiau <damien.lespiau@xxxxxxxxx>

testdisplay is a bit an awkward test, mostly used by QA for manual
testing. I think we also need some basic kms_setmode subtest to use the
new tiling modes on skl in one of the automated testcases. Otherwise I
fear this will bitrot fast.
-Daniel

> ---
>  tests/testdisplay.c | 16 ++++++++++++++--
>  1 file changed, 14 insertions(+), 2 deletions(-)
> 
> diff --git a/tests/testdisplay.c b/tests/testdisplay.c
> index 64ce4d7..dab9e12 100644
> --- a/tests/testdisplay.c
> +++ b/tests/testdisplay.c
> @@ -51,6 +51,7 @@
>  
>  #include <cairo.h>
>  #include <errno.h>
> +#include <getopt.h>
>  #include <math.h>
>  #include <stdint.h>
>  #include <stdbool.h>
> @@ -71,8 +72,10 @@
>  #include <stdlib.h>
>  #include <signal.h>
>  
> -#define SUBTEST_OPTS 1
> +#define SUBTEST_OPTS	 1
>  #define HELP_DESCRIPTION 2
> +#define Yb_OPT		 3
> +#define Yf_OPT		 4
>  
>  static int tio_fd;
>  struct termios saved_tio;
> @@ -544,7 +547,7 @@ int update_display(void)
>  	return 1;
>  }
>  
> -static char optstr[] = "3hiaf:s:d:p:mrto:j:";
> +static char optstr[] = "3hiaf:s:d:p:mrto:j:y";
>  
>  static void __attribute__((noreturn)) usage(char *name, char opt)
>  {
> @@ -645,6 +648,8 @@ int main(int argc, char **argv)
>  		{"run-subtest", 1, 0, SUBTEST_OPTS},
>  		{"help-description", 0, 0, HELP_DESCRIPTION},
>  		{"help", 0, 0, 'h'},
> +		{"yb", 0, 0, Yb_OPT},
> +		{"yf", 0, 0, Yf_OPT},
>  		{ 0, 0, 0, 0 }
>  	};
>  
> @@ -697,6 +702,13 @@ int main(int argc, char **argv)
>  		case 't':
>  			tiling = LOCAL_I915_FORMAT_MOD_X_TILED;
>  			break;
> +		case 'y':
> +		case Yb_OPT:
> +			tiling = LOCAL_I915_FORMAT_MOD_Y_TILED;
> +			break;
> +		case Yf_OPT:
> +			tiling = LOCAL_I915_FORMAT_MOD_Yf_TILED;
> +			break;
>  		case 'r':
>  			qr_code = 1;
>  			break;
> -- 
> 2.3.0
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx

-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx





[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux