From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-ed1-x52d.google.com (mail-ed1-x52d.google.com [IPv6:2a00:1450:4864:20::52d]) by gabe.freedesktop.org (Postfix) with ESMTPS id CB8AC10E05C for ; Mon, 18 Sep 2023 10:51:35 +0000 (UTC) Received: by mail-ed1-x52d.google.com with SMTP id 4fb4d7f45d1cf-52bcd4db4c0so9563713a12.0 for ; Mon, 18 Sep 2023 03:51:35 -0700 (PDT) Message-ID: Date: Mon, 18 Sep 2023 13:51:28 +0300 MIME-Version: 1.0 Content-Language: en-US To: Swati Sharma , igt-dev@lists.freedesktop.org References: <20230918054926.238027-1-swati2.sharma@intel.com> <20230918054926.238027-6-swati2.sharma@intel.com> From: Juha-Pekka Heikkila In-Reply-To: <20230918054926.238027-6-swati2.sharma@intel.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [igt-dev] [v10 5/5] tests/intel/kms_dsc: remove redundant documentation List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: juhapekka.heikkila@gmail.com Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" List-ID: Reviewed-by: Juha-Pekka Heikkila On 18.9.2023 8.49, Swati Sharma wrote: > With the introduction of testplan documentation, test documentation > seems redundant. Remove it. > > Signed-off-by: Swati Sharma > --- > tests/intel/kms_dsc.c | 21 --------------------- > 1 file changed, 21 deletions(-) > > diff --git a/tests/intel/kms_dsc.c b/tests/intel/kms_dsc.c > index 9b137ef87..59a1e670b 100644 > --- a/tests/intel/kms_dsc.c > +++ b/tests/intel/kms_dsc.c > @@ -69,27 +69,6 @@ IGT_TEST_DESCRIPTION("Test to validate display stream compression"); > #define TEST_DSC_OUTPUT_FORMAT (1<<2) > #define TEST_DSC_FRACTIONAL_BPP (1<<3) > > -/* > - * Starting from gen11, intel driver supports DSC1.1. For validating > - * DSC, the first step is to verify if the sink supports DSC. > - * If the sink does support DSC, we will validate different > - * scenarios by forcing dsc. Outline of the tests is as follows: > - * (i) basic modeset (ii) input bpc (iii) pixel formats > - * (iv) output formats (v) fractional bpp > - * In the basic subtest, we perform modeset with default parameters. > - * Input bpc and pixel formats subtests, we perform modeset > - * with different input bpc (12/10/8) and pixel formats (YUV/RGB), > - * respectively. From MTL+, we can verify DSC YCBCR420 output format. > - * The tests are executed with the RGB444 output format by default. > - * However, in the output-format subtest, we verify different > - * output formats (RGB/YCBCR444/YCBCR420). Also, test is added to > - * validate output formats with different input bpc (12/10/8). > - * Lastly, fractional bpp is tested with default parameters. > - * In this, driver will ignore integer compressed bpp value and > - * will do modeset with fractional bpp only. Test is added to > - * validate fractional bpp with different input bpc (12/10/8). > - */ > - > typedef struct { > int drm_fd; > uint32_t devid;