[igt-dev] [i-g-t 18/20] tests/xe/xe_debugfs: Use xe_config() helper to get the config

Kamil Konieczny kamil.konieczny at linux.intel.com
Wed May 24 08:47:43 UTC 2023


Hi Bhanuprakash,

On 2023-05-16 at 22:20:56 +0530, Bhanuprakash Modem wrote:
> Use xe_config() helper to get the config.
> 
> Signed-off-by: Bhanuprakash Modem <bhanuprakash.modem at intel.com>
> ---
>  tests/xe/xe_debugfs.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tests/xe/xe_debugfs.c b/tests/xe/xe_debugfs.c
> index 4006981c6..10dd05740 100644
> --- a/tests/xe/xe_debugfs.c
> +++ b/tests/xe/xe_debugfs.c
> @@ -252,7 +252,7 @@ igt_main_args("", long_options, help_str, opt_handler, NULL)
>  	}
>  
>  	igt_subtest("base") {
> -		test_base(fd, xe_dev->config);
> +		test_base(fd, xe_config(fd));

Is this needed ? Old code looks good, I see no point in change.
Maybe you should merge this with one removing xe_dev var ?

Regards,
Kamil

>  	}
>  
>  
> -- 
> 2.40.0
> 


More information about the igt-dev mailing list