Re: [igt-dev] [PATCH] tests/amdgpu: Add test description for amd_abm

From: Rodrigo Siqueira <Rodrigo.Siqueira@amd.com>
To: Aurabindo Pillai <mail@aurabindo.in>
Cc: igt-dev@lists.freedesktop.org
Subject: Re: [igt-dev] [PATCH] tests/amdgpu: Add test description for amd_abm
Date: Mon, 29 Mar 2021 09:44:10 -0400	[thread overview]
Message-ID: <20210329134410.o5jnxnh2d36y6tu4@outlook.office365.com> (raw)
In-Reply-To: <20210329130642.236359-1-mail@aurabindo.in>

[-- Attachment #1.1: Type: text/plain, Size: 1080 bytes --]

Hi Aurabindo,

On 03/29, Aurabindo Pillai wrote:
> From: Aurabindo Pillai <aurabindo.pillai@amd.com>
> 
> Add overall test description for amd_abm
> 
> Change-Id: I4baa52ea66189688b12a82befbbd842ba3d3a24d

We don't need this Change-id in IGT.

Aside from the above comment, everything LGTM.

Reviewed-by: Rodrigo Siqueira <Rodrigo.Siqueira@amd.com>

Let me know if you need help for applying this change in IGT.

Thanks
Siqueira

> Signed-off-by: Aurabindo Pillai <aurabindo.pillai@amd.com>
> ---
>  tests/amdgpu/amd_abm.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/tests/amdgpu/amd_abm.c b/tests/amdgpu/amd_abm.c
> index 0505f17e..a1e48533 100644
> --- a/tests/amdgpu/amd_abm.c
> +++ b/tests/amdgpu/amd_abm.c
> @@ -32,6 +32,8 @@
>  #include <fcntl.h>
>  #include <time.h>
>  
> +IGT_TEST_DESCRIPTION("Tests Adaptive Backlight Modulation on supported hardware");
> +
>  #define BACKLIGHT_PATH "/sys/class/backlight/amdgpu_bl0"
>  
>  typedef struct data {
> -- 
> 2.31.0
> 

-- 
Rodrigo Siqueira
https://siqueira.tech

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

[-- Attachment #2: Type: text/plain, Size: 154 bytes --]

_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev