Re: [igt-dev] [PATCH i-g-t v2 0/6] Enable VDSC output formats validation

Swati Sharma
2023-02-03T10:53:38Z

Thanks for the review. Pushed 4/6 patches.

On 02-Feb-23 4:22 PM, Hogander, Jouni wrote:
> On Thu, 2023-02-02 at 12:51 +0530, Swati Sharma wrote:
>> This series is split from
>> https://patchwork.freedesktop.org/series/111342/
>>
>> Initial dsc lib related changes got merged. Received review
>> comments on driver debugfs YCBCR420 changes, to make debugfs
>> entry generic so that all supported output formats can be
>> validated.
>>
>> In this series, review comments from above PW are addressed.
>> Along, with that changes are done to make test generic to
>> validate all supported dsc formats.
>
> For the whole set:
>
> Reviewed-by: Jouni Högander <jouni.hogander@intel.com>
>
>>
>> Swati Sharma (6):
>>   lib/dsc: Fix return value
>>   tests/i915/kms_dsc: Add plane_format as struct data_t member
>>   tests/i915/kms_dsc: Remove pointless struct
>>   tests/i915/kms_dsc: Make bpc_list static
>>   lib/dsc: Add helpers for VDSC output format debugfs entry
>>   tests/i915/kms_dsc: Enable validation for VDSC output formats
>>
>>  lib/igt_dsc.c               |  56 +++++++++++++++-
>>  lib/igt_dsc.h               |   5 ++
>>  lib/igt_kms.c               |  18 +++++
>>  lib/igt_kms.h               |   7 ++
>>  tests/i915/kms_dsc.c        | 128 ++++++++++++++++++++++++++--------
>> --
>>  tests/i915/kms_dsc_helper.c |  40 ++++++++++-
>>  tests/i915/kms_dsc_helper.h |   4 ++
>>  7 files changed, 218 insertions(+), 40 deletions(-)
>>
>

--
~Swati Sharma