Re: [PATCH i-g-t] tests/kms: fix pixel format functionality

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

On 18-Jan-24 3:15 PM, Kamil Konieczny wrote:
> Hi Swati,
> On 2024-01-17 at 13:44:16 +0530, Swati Sharma wrote:
> 
> in subject add "doc", like:
> 
> [PATCH i-g-t] tests/kms: fix doc pixel format functionality
> ---------------------------- ^^^ added
> 
>> Pixel format functionality is used in different ways. Align all.
> 
> Better:
> In tests documentation, pixel format ...
> 
> with that:
> Acked-by: Kamil Konieczny <kamil.konieczny@linux.intel.com>
> 
> Regards,
> Kamil

Thanks Kamil! Will fix while merging.

> 
>>
>> Signed-off-by: Swati Sharma <swati2.sharma@intel.com>
>> ---
>>   tests/intel/kms_psr2_su.c  | 2 +-
>>   tests/kms_pipe_crc_basic.c | 2 +-
>>   tests/kms_plane.c          | 4 ++--
>>   3 files changed, 4 insertions(+), 4 deletions(-)
>>
>> diff --git a/tests/intel/kms_psr2_su.c b/tests/intel/kms_psr2_su.c
>> index 96b4fd2b4..936b5beb3 100644
>> --- a/tests/intel/kms_psr2_su.c
>> +++ b/tests/intel/kms_psr2_su.c
>> @@ -47,7 +47,7 @@
>>    *
>>    * SUBTEST: page_flip-%s
>>    * Description: Test the selective update with %arg[1] when screen changes
>> - * Functionality: pixel-format, psr2, selective_update
>> + * Functionality: pixel_formats, psr2, selective_update
>>    *
>>    * arg[1]:
>>    *
>> diff --git a/tests/kms_pipe_crc_basic.c b/tests/kms_pipe_crc_basic.c
>> index 3c12d8098..0c19745bc 100644
>> --- a/tests/kms_pipe_crc_basic.c
>> +++ b/tests/kms_pipe_crc_basic.c
>> @@ -68,7 +68,7 @@
>>   /**
>>    * SUBTEST: compare-crc-sanitycheck-%s
>>    * Description: Basic sanity check for CRC mismatches with %arg[1]
>> - * Functionality: crc, pixel_format
>> + * Functionality: crc, pixel_formats
>>    *
>>    * arg[1]:
>>    *
>> diff --git a/tests/kms_plane.c b/tests/kms_plane.c
>> index 51ca082ae..406aecc04 100644
>> --- a/tests/kms_plane.c
>> +++ b/tests/kms_plane.c
>> @@ -44,7 +44,7 @@
>>   /**
>>    * SUBTEST: planar-pixel-format-settings
>>    * Description: verify planar settings for pixel format are handled correctly
>> - * Functionality: pixel_format, plane
>> + * Functionality: pixel_formats, plane
>>    *
>>    * SUBTEST: plane-position-%s
>>    * Description: Verify plane position using two planes to create a %arg[1]
>> @@ -80,7 +80,7 @@
>>   /**
>>    * SUBTEST: pixel-%s
>>    * Description: verify the pixel formats for given plane and pipe
>> - * Functionality: pixel_format, plane
>> + * Functionality: pixel_formats, plane
>>    *
>>    * arg[1]:
>>    *
>> -- 
>> 2.25.1
>>