Re: [PATCH i-g-t 2/3] tests/intel/xe_pm: Move mocs documentation to the right place

[Rodrigo Vivi <rodrigo.vivi@intel.com>]

On Mon, May 13, 2024 at 09:24:33AM -0700, Lucas De Marchi wrote:
> {s_state}-mocs is not tested by test_exec(). Move it on top of the right
> function.
> 
> Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>

Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>

> ---
>  tests/intel/xe_pm.c | 15 +++++++++++----
>  1 file changed, 11 insertions(+), 4 deletions(-)
> 
> diff --git a/tests/intel/xe_pm.c b/tests/intel/xe_pm.c
> index 1215b5c2a..584cb8548 100644
> --- a/tests/intel/xe_pm.c
> +++ b/tests/intel/xe_pm.c
> @@ -251,10 +251,6 @@ static void close_fw_handle(int sig)
>   * Description: suspend/autoresume on %arg[1] state and exec after RPM
>   * Functionality: pm - %arg[1]
>   *
> - * SUBTEST: %s-mocs
> - * Description: test checks for mocs state before and after %arg[1] state
> - * Functionality: pm - %arg[1]
> - *
>   * arg[1]:
>   *
>   * @s2idle:	s2idle
> @@ -624,6 +620,17 @@ static void test_mmap(device_t device, uint32_t placement, uint32_t flags)
>   * Functionality: mocs
>   * Run type: FULL
>   */
> +/**
> + * SUBTEST: %s-mocs
> + * Description: test checks for mocs state before and after %arg[1] state
> + * Functionality: pm - %arg[1]
> + *
> + * arg[1]:
> + *
> + * @s2idle:	s2idle
> + * @s3:		s3
> + * @s4:		s4
> + */
>  static void test_mocs_suspend_resume(device_t device, int s_state)
>  {
>  	int gt;
> -- 
> 2.43.0
>