Re: [PATCH i-g-t v1 4/5] tests/kms_plane_scaling: Add debug logs for output scaling attempts

["Sharma, Swati2" <swati2.sharma@intel.com>]

Hi Ramanaidu,

On 05-Sep-24 8:20 PM, Naladala Ramanaidu wrote:
> This will introduces additional debug information to the
> kms_plane_scaling test. The new logs provide clarity on which
> outputs are being tested for scaling operations and indicate
> when a required scaling operation is not supported.
> 
> Signed-off-by: Naladala Ramanaidu <ramanaidu.naladala@intel.com>
> ---
>   tests/kms_plane_scaling.c | 4 ++++
>   1 file changed, 4 insertions(+)
> 
> diff --git a/tests/kms_plane_scaling.c b/tests/kms_plane_scaling.c
> index 5d69eabf4..04264d35d 100644
> --- a/tests/kms_plane_scaling.c
> +++ b/tests/kms_plane_scaling.c
> @@ -1528,6 +1528,8 @@ igt_main_args("", long_opts, help_str, opt_handler, &data)
>   				for_each_pipe(&data.display, pipe) {
>   					igt_dynamic_f("pipe-%s", kmstest_pipe_name(pipe)) {
>   						for_each_valid_output_on_pipe(&data.display, pipe, output) {
> +							igt_info("Trying on %s\n",
> +								 igt_output_name(output));
>   							if (!pipe_output_combo_valid(&data.display, pipe, output))
>   								continue;
>   							if (get_num_scalers(&data.display, pipe) < 2)
> @@ -1539,6 +1541,8 @@ igt_main_args("", long_opts, help_str, opt_handler, &data)
>   								scaler_with_2_planes_tests[index].test_type);
>   							if (ret == 0)
>   								break;
> +							igt_info("Required Scaling operation not supported on %s trying on next output\n",
> +								 igt_output_name(output));

s/Scaling/scaling

With this fixed,
Reviewed-by: Swati Sharma <swati2.sharma@intel.com>

>   						}
>   						igt_skip_on_f(ret == -ERANGE || ret == -EINVAL,
>   							      "Unsupported scaling operation in driver with return value %s\n",