Re: [igt-dev] [PATCH i-g-t 1/5] tests/i915/i915_pm_dc: Add test description

From: Imre Deak <imre.deak@intel.com>
To: Swati Sharma <swati2.sharma@intel.com>
Cc: igt-dev@lists.freedesktop.org
Subject: Re: [igt-dev] [PATCH i-g-t 1/5] tests/i915/i915_pm_dc: Add test description
Date: Mon, 12 Sep 2022 16:04:25 +0300	[thread overview]
Message-ID: <Yx8uWTr9PNP/M08v@ideak-desk.fi.intel.com> (raw)
In-Reply-To: <20220909114911.1852-2-swati2.sharma@intel.com>

On Fri, Sep 09, 2022 at 05:19:07PM +0530, Swati Sharma wrote:
> Signed-off-by: Swati Sharma <swati2.sharma@intel.com>

Commit message is missing, something to explain why moving the
description is required. With that:

Reviewed-by: Imre Deak <imre.deak@intel.com>

> ---
>  tests/i915/i915_pm_dc.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/tests/i915/i915_pm_dc.c b/tests/i915/i915_pm_dc.c
> index f664a2a2c..40811ae52 100644
> --- a/tests/i915/i915_pm_dc.c
> +++ b/tests/i915/i915_pm_dc.c
> @@ -44,6 +44,8 @@
>  #define KMS_HELPER "/sys/module/drm_kms_helper/parameters/"
>  #define KMS_POLL_DISABLE 0
>  
> +IGT_TEST_DESCRIPTION("Tests to validate display power DC states.");
> +
>  bool kms_poll_saved_state;
>  
>  typedef struct {
> @@ -473,7 +475,6 @@ static void kms_poll_state_restore(int sig)
>  	}
>  }
>  
> -IGT_TEST_DESCRIPTION("These tests validate Display Power DC states");
>  int main(int argc, char *argv[])
>  {
>  	data_t data = {};
> -- 
> 2.25.1
> 