[i-g-t] kms_vrr: Add comment explaining PR and LOBF dependency

Kamil Konieczny kamil.konieczny at linux.intel.com
Fri Jun 27 16:45:24 UTC 2025


On 2025-06-23 at 10:50:56 +0530, Jeevan B wrote:

Small nit about subject, add "tests/" prefix, so it will be:

[i-g-t] tests/kms_vrr: Add comment explaining PR and LOBF dependency

> Add a comment to explain that LOBF requires PR support,
> which is only available on eDP 1.5 panels.
> 
> Signed-off-by: Jeevan B <jeevan.b at intel.com>
> ---
>  tests/kms_vrr.c | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/tests/kms_vrr.c b/tests/kms_vrr.c
> index 4919b7cb7..b7600ae2c 100644
> --- a/tests/kms_vrr.c
> +++ b/tests/kms_vrr.c
> @@ -1049,6 +1049,10 @@ run_vrr_test(data_t *data, test_t test, uint32_t flags)
>  				continue;
>  			}
>  
> +			/* Panel Replay is supported only with eDP 1.5 panels.
> +			 * LOBF and PR are mutually exclusive features.

I do not understand this, did you mean:

			 * LOBF is possible only when PR is present.

Regards,
Kamil

> +			 * Skip the test if PR is not supported, as LOBF cannot be enabled.
> +			 */
>  			if (flags == TEST_LINK_OFF)
>  				igt_require_f(psr_sink_support(data->drm_fd,
>  					      data->debugfs_fd, PR_MODE, output),


More information about the igt-dev mailing list