Re: [igt-dev] [PATCH v2] intel-ci: Add compare CRC sanity test to BAT

Sharma, Swati2
2020-08-06T10:45:23Z

LGTM
Reviewed-by: Swati Sharma <swati2.sharma@intel.com>

On 06-Aug-20 11:54 PM, Bhanuprakash Modem wrote:
> We need to sanity-check for the CRC mismatches, otherwise
> the display validation tests are flawed.
>
> This patch will add CRC compare mismatch subtests to the
> fast-feedback list to make sure the CRC mechanism is clean
> from the platform.
>
> V2:
> * Add pipe-d as well (Petri)
>
> Cc: Swati Sharma <swati2.sharma@intel.com>
> Cc: Karthik B S <karthik.b.s@intel.com>
> Cc: Jeevan B <jeevan.b@intel.com>
> Cc: Petri Latvala <petri.latvala@intel.com>
> Cc: Arkadiusz Hiler <arek@hiler.eu>
> Acked-by: Petri Latvala <petri.latvala@intel.com>
> Signed-off-by: Bhanuprakash Modem <bhanuprakash.modem@intel.com>
> ---
> tests/intel-ci/fast-feedback.testlist | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/tests/intel-ci/fast-feedback.testlist b/tests/intel-ci/fast-feedback.testlist
> index b796b264..b98cdb24 100644
> --- a/tests/intel-ci/fast-feedback.testlist
> +++ b/tests/intel-ci/fast-feedback.testlist
> @@ -110,6 +110,10 @@ igt@kms_force_connector_basic@force-edid
> igt@kms_force_connector_basic@force-load-detect
> igt@kms_force_connector_basic@prune-stale-modes
> igt@kms_frontbuffer_tracking@basic
> +igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-a
> +igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-b
> +igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-c
> +igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-d
> igt@kms_pipe_crc_basic@hang-read-crc-pipe-a
> igt@kms_pipe_crc_basic@nonblocking-crc-pipe-a
> igt@kms_pipe_crc_basic@nonblocking-crc-pipe-a-frame-sequence
>

--
~Swati Sharma