2023-09-12 21:10:40

by Helen Koike

[permalink] [raw]
Subject: Re: [PATCH v3 1/9] drm: ci: igt_runner: Remove todo



On 08/09/2023 12:22, Vignesh Raman wrote:
> /sys/kernel/debug/dri/*/state exist for every atomic KMS driver.
> We do not test non-atomic drivers, so remove the todo.
>
> Signed-off-by: Vignesh Raman <[email protected]>

Acked-by: Helen Koike <[email protected]>

> ---
>
> v2:
> - No changes
>
> v3:
> - No changes
>
> ---
> drivers/gpu/drm/ci/igt_runner.sh | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/ci/igt_runner.sh b/drivers/gpu/drm/ci/igt_runner.sh
> index 2bb759165063..5bf130ac57c9 100755
> --- a/drivers/gpu/drm/ci/igt_runner.sh
> +++ b/drivers/gpu/drm/ci/igt_runner.sh
> @@ -15,7 +15,6 @@ cat /sys/kernel/debug/device_component/*
> '
>
> # Dump drm state to confirm that kernel was able to find a connected display:
> -# TODO this path might not exist for all drivers.. maybe run modetest instead?
> set +e
> cat /sys/kernel/debug/dri/*/state
> set -e