Re: [igt-dev] [i-g-t v8 10/51] tests/kms_content_protection: Add support for Bigjoiner

["Modem, Bhanuprakash" <bhanuprakash.modem@intel.com>]

HI Karthik,

On Fri-17-03-2023 12:01 pm, Karthik B S wrote:
> 
> On 2/23/2023 8:39 PM, Bhanuprakash Modem wrote:
>> This patch will add a check to Skip the subtest if a selected pipe/output
>> combo won't support Bigjoiner or 8K mode.
>>
>> Example:
>> * Pipe-D wont support a mode > 5K
>> * To use 8K mode on a pipe then consecutive pipe must be available & 
>> free.
> 
> Please add the revision history details.

Thanks for the review, will add those details while pushing the patch.

- Bhanu

> 
> With this added,
> 
> Reviewed-by: Karthik B S <karthik.b.s@intel.com>
> 
>>
>> Signed-off-by: Bhanuprakash Modem <bhanuprakash.modem@intel.com>
>> ---
>>   tests/kms_content_protection.c | 6 +++++-
>>   1 file changed, 5 insertions(+), 1 deletion(-)
>>
>> diff --git a/tests/kms_content_protection.c 
>> b/tests/kms_content_protection.c
>> index aa28b7bfd..638e1301a 100644
>> --- a/tests/kms_content_protection.c
>> +++ b/tests/kms_content_protection.c
>> @@ -174,7 +174,6 @@ static void modeset_with_fb(const enum pipe pipe, 
>> igt_output_t *output,
>>       igt_plane_t *primary;
>>       mode = igt_output_get_mode(output);
>> -    igt_output_set_pipe(output, pipe);
>>       primary = igt_output_get_plane_type(output, 
>> DRM_PLANE_TYPE_PRIMARY);
>>       igt_plane_set_fb(primary, &data.red);
>> @@ -502,6 +501,11 @@ test_content_protection(enum igt_commit_style s, 
>> int content_type)
>>                   continue;
>>               igt_display_reset(display);
>> +
>> +            igt_output_set_pipe(output, pipe);
>> +            if (!i915_pipe_output_combo_valid(display))
>> +                continue;
>> +
>>               modeset_with_fb(pipe, output, s);
>>               if (!output_hdcp_capable(output, content_type))